Reduce nested #capture_git_error calls on Wiki
-
Please check this box if this contribution uses AI-generated content (including content generated by GitLab Duo features) as outlined in the GitLab DCO & CLA
What does this MR do and why?
This MR refactors the wiki model to make #capture_git_error private in order to solve the problem raised in the issue #468185 (closed):
As far as I could see in the code base, I still think that we can make the
#capture_git_errorprivate, as it being used outside of theWikiPageseems like a leak of internals IMHO. This should be quite straightforward because:
- Currently,
WikiPages::CreateServicecallspage.createfrom within acapture_git_error, but thepage.createalready uses thecapture_git_errorinternally, so I don't see a reason why we're using it twice.- Same thing happens on
WikiPages::UpdateService, it's callingpage.updatefrom within acapture_git_error, but thepage.updatealready calls thecapture_git_errorinternally, therefore I think we can remove the external call as well.
MR acceptance checklist
Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
Edited by Dannyel Cardoso da Fonseca