Bug #54192
Menus don't respect deleted pages
| Status: | Closed | Start date: | 2013-12-04 | |
|---|---|---|---|---|
| Priority: | Must have | Due date: | ||
| Assigned To: | Bastian Waidelich | % Done: | 0% |
|
| Category: | - | |||
| Target version: | Base Distribution - 1.0 beta 2 |
Description
The menu render object seems not to ignore deleted pages. It's really easy to render single pages unusable:
1) Create a site "foo"
2) Create a site "bar"
3) Delete site "foo"
4) Rename URL name of site "bar" to "foo"
This crashes horribly as it makes the entire site unusable.
If you try to delete the malicous page "bar", you'll get an exception for "non-unique" paths. "/yoursite/foo" still exists but now twice.
The FE doesn't render any more. At least in my case. I used the demo site package. Publishing button doesn't work anymore.
The site itself can not be deleted nor can the name be changed as the backend will not load anything.
This is very critical.
Related issues
History
#1 Updated by Thomas Hempel over 1 year ago
Can somebody please change the title to "Deleting pages possibly crashes entire site"?
#2 Updated by Thomas Hempel over 1 year ago
Short update. It looks like this destroys the entire site, neither the site nor the backend is rendered anymore.
#3 Updated by Bastian Waidelich over 1 year ago
- Status changed from New to Closed
- Assigned To set to Bastian Waidelich
Hey Thomas,
thanks for reporting. This is very critical indeed! We know what the issue is and how to solve it and we're upset that the fix didn't make it into beta2, but this will be fixed for the final! see #54080 for more details
#4 Updated by Thomas Hempel over 1 year ago
Hi Basti,
okay, good to know. I would have very surprised if that wasn't a know issue. ;)
Greets,
Thomas
#5 Updated by Bastian Waidelich over 1 year ago
Thomas Hempel wrote:
okay, good to know. I would have very surprised if that wasn't a know issue. ;)
Yes, the CR is a complex beast with its overlays.. At least the server side should be fairly easy to fix. But we want to write behat tests first covering all of those scenarios, so that we don't have to test this manually all the time. Stay tuned ;)
#6 Updated by Bastian Waidelich over 1 year ago
- Target version set to 1.0 beta 2