Fix GetContents(): Dont't ignore Executables (#11192) (#11209)

6543 · web-flow · commit bfc25fcf40ee · 2020-04-25T01:54:38.000-03:00
diff --git a/modules/repofiles/content.go b/modules/repofiles/content.go
@@ -159,7 +159,7 @@ func GetContents(repo *models.Repository, treePath, ref string, forList bool) (*
 	}
 
 	// Now populate the rest of the ContentsResponse based on entry type
-	if entry.IsRegular() {
+	if entry.IsRegular() || entry.IsExecutable() {
 		contentsResponse.Type = string(ContentTypeRegular)
 		if blobResponse, err := GetBlobBySHA(repo, entry.ID.String()); err != nil {
 			return nil, err

Original file line number	Diff line number	Diff line change
`@@ -159,7 +159,7 @@ func GetContents(repo models.Repository, treePath, ref string, forList bool) (`
`159`	`159`	`}`
`160`	`160`
`161`	`161`	`// Now populate the rest of the ContentsResponse based on entry type`
`162`		`- if entry.IsRegular() {`
	`162`	`+ if entry.IsRegular() \|\| entry.IsExecutable() {`
`163`	`163`	`contentsResponse.Type = string(ContentTypeRegular)`
`164`	`164`	`if blobResponse, err := GetBlobBySHA(repo, entry.ID.String()); err != nil {`
`165`	`165`	`return nil, err`