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
Removing a page doesn't work from the edit route (works using batch delete) #1708
Comments
Hello On postRemove() we call a method who create a page. This is normal ? i have a similar issue on my project |
To fix the bug i think we can do this : public function postRemove(AdminInterface $admin, object $object): void
{
if ($object instanceof PageInterface) {
return;
}
$this->createByPage($object);
} case 1 : |
Can you try opening a PR with this change then ? And we will need an aditionnal test for your use case |
the patch is solving my issue on my side. it would be great if it was accepted :) kudo @c-Rolland for this patch ! |
Please let me know how to better MR if needed. I would be very pleased to make this merge request accepted and contribute in the future. thanks |
Environment
Sonata packages
show
Symfony packages
show
PHP version
Subject
Removing a page doesn't work from the edit route, it works only using batch delete:
Minimal repository with the bug
https://github.com/gremo/sonata-page-delete-bug
Steps to reproduce
See the minimal repository with the bug.
Expected results
The page can be deleted from the edit route.
Actual results
The page cannot be deleted.
The text was updated successfully, but these errors were encountered: