pub-solar/forgejo
5
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

Remember to attach the parent tree when converting TreeEntry() -> Tree() (#22902)

Browse source 
!fixup https://github.com/go-gitea/gitea/pull/22177

The only place this function is used so far is in
findReadmeFileInEntries(), so the only visible effect of this oversight
was in an obscure README-related corner: if the README was in a
subfolder and was a symlink that pointed up, as in .github/README.md ->
../docs/old/setup.md, the README would fail to render when FollowLinks()
hit the nil ptree. This makes the ptree non-nil and thus repairs it.
This commit is contained in:
Nick 2023-02-14 16:23:04 -05:00 committed by GitHub
parent 71ee488098
commit 618c9118c1
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
modules/git/tree_entry.go
View file

@ -101,12 +101,13 @@ func (te *TreeEntry) FollowLinks() (*TreeEntry, error) {
return entry, nil return entry, nil
} }
// returns the subtree, or nil if this is not a tree // returns the Tree pointed to by this TreeEntry, or nil if this is not a tree
func (te *TreeEntry) Tree() *Tree { func (te *TreeEntry) Tree() *Tree {
t, err := te.ptree.repo.getTree(te.ID) t, err := te.ptree.repo.getTree(te.ID)
if err != nil { if err != nil {
return nil return nil
} }
t.ptree = te.ptree
return t return t
} }