Skip to content

Don't blow up in tree view on empty repo

username-removed-443319 requested to merge 19928-nomethoderror-undefined-method-id-for-nil-nilclass into master

What does this MR do?

Stop the empty repo page blowing up with a 500 error when a user's default view is tree view.

Are there points in the code the reviewer needs to double check?

Don't think so.

Why was this MR needed?

https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/4997 made sure that the tree filter was only called if the user's default project view is tree, but left out the empty repo check. When users visit an empty repo with the tree view as default, they now get a 500.

What are the relevant issue numbers?

Closes #19928 (closed).

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

Merge request reports