Skip to content

[WIP] Refactor some ApplicationController methods

Nuttanart Pornprasitsakul requested to merge visibletrap/gitlab-ce:refactor-issuable-finder into master

This is a refactoring MR which the idea came up when I was working on !2854 (merged). The idea is some methods in ApplicationController are being used only by IssuableFinder. So this MR cleans up some of those unnecessary codes and move the remaining codes to IssuableFinder where they are actually used.

I intent not to squash the commits yet because I want to leave trace for reviewers in case they want to follow the refactoring path during the review. They will be eventually squashed to a single refactoring commit before merging to master.

Merge request reports