forgejo/routers/api
Gergely Nagy be8d16438a
Fix /api/v1/{owner}/{repo}/issue_templates
When issue templates were moved into services in
def4956122, the code was also refactored
and simplified. Unfortunately, that simplification broke the
`/api/v1/{owner}/{repo}/issue_templates` route, because it was
previously using a helper function that ignored invalid templates, and
after the refactor, the function it called *always* returned non-nil as
the second return value. This, in turn, results in the aforementioned
end point always returning an internal server error.

This change restores the previous behaviour of ignoring invalid files
returned by `issue.GetTemplatesFromDefaultBranch`, and adds a few test
cases to exercise the endpoint.

Other users of `GetTemplatesFromDefaultBranch` already ignore the second
return value, or handle it correctly, so no changes are necessary there.

Signed-off-by: Gergely Nagy <forgejo@gergo.csillger.hu>
2024-02-05 20:58:41 +01:00
..
actions Add artifacts v4 jwt to job message and accept it (#28885) 2024-02-02 14:25:59 +00:00
forgejo/v1 [CLEANUP] Reuse ForgejoVersion variable 2024-02-05 14:45:11 +01:00
packages [GITEA] Drop sha256-simd in favor of stdlib 2024-02-05 16:09:40 +01:00
v1 Fix /api/v1/{owner}/{repo}/issue_templates 2024-02-05 20:58:41 +01:00