Skip to content

Refactor common snippet controller methods

Vijay Hawoldar requested to merge vij-snippet-controller-refactor into master

What does this MR do?

As discussed in #208786 (closed), Project and Personal snippet controllers have duplicated code. This MR refactors common snippet controller functionality into new snippet application controllers and concerns in order to reduce the duplication for the existing controllers, as well as new ones that are due to be created (e.g. a SnippetBlobController in #217775 (closed))

I've not added a changelog entry as this is just a developer facing refactor, and I've also not added any additional test coverage for existing functionality because the existing tests should be sufficient (I have added coverage for the new route helper, though).

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Refs #208786 (closed)

Refs #217775 (closed)

Edited by 🤖 GitLab Bot 🤖

Merge request reports

Loading