-
Notifications
You must be signed in to change notification settings - Fork 335
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
Show error when deleting parent account #4809
Show error when deleting parent account #4809
Conversation
IMO there is currently only one issue: When the error message is shown, deletion cannot be continued, but the dialog still shows two buttons: "Ok" and "Cancel". I think one of both should be gone, but I am not completely sure which one. @chirimoya Do you have any idea? |
|
||
return $query->getResult(); | ||
} catch (NoResultException $ex) { | ||
return null; |
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.
i think i would expect an empty array in this case
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.
You are right, that's better 🙂 Done.
|
||
### DeleteToolbarAction with conflict | ||
|
||
The `DeleteToolbarAction` asks for confirmation if e.g. a page that is being tried to deleted is linked on other pages. |
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.
not sure if being tried to deleted
is correct. but dont have a better suggestion and i guess people will understand it anyway 🙂
fc897d9
to
9edd378
Compare
What's in this PR?
This PR introduces an error message when an account which is the parent of another account is trying to be deleted.
Why?
Because we don't want to delete the child accounts when a parent account is deleted. What should happen instead is not completely clear, and this way the user can decide how to resolve this conflict.
And it is the same behavior as in Sulu 1.6.
To Do