-
-
Notifications
You must be signed in to change notification settings - Fork 224
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
BUGFIX: Change workspace runs into endless while-loop #5184
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
doh :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oof yeah, thanks for finding this!
I think this needs a rebase to after the DBAL changes?! That probably fixes the two CI problems. |
Before merging we should definitely come up with a test for this. The method is used in change base workspace which is apparently untested. |
All dependencies fixed, tests green, as discussed today in our meeting, we merge this before the test as it's a nasty bug. |
Review instructions
Currently when switching the workspace in the UI it just runs endlessly and the docker container complains if you switch workspaces too often.
Apparently the bug was introduced here: PR
The line used to be
$nextBaseWorkspace = $contentRepository->getWorkspaceFinder()->findOneByName($nextBaseWorkspace->baseWorkspaceName);
and was changed to
$nextBaseWorkspace = $this->requireBaseWorkspace($workspace, $workspaceFinder);
since we always check for $workspace, the $nextBaseWorkspace always stays the same
Checklist
FEATURE|TASK|BUGFIX
!!!
and have upgrade-instructions