Bug #82970
closed
Page tree reload is triggered to early
100%
Description
Given:
- a page with many records and references to sys_categories
- delete that page by using the context menu
Expectation:
- delete process is started
- after everything is done (can take longer because of relations und sys_reference changes), the tree should be reloaded, the page should be removed
Actual:
- the page tree is reloaded immediately, as the page record is not yet deleted the page is still visible
Updated by Tymoteusz Motylewski over 6 years ago
this is caused probably by the
.defer(500) in the ContextMenuActions.js
currently is (in v8):
top.TYPO3.Backend.ContentContainer.setUrl(
top.TYPO3.settings.RecordCommit.moduleUrl + ....
);
if (table === 'pages' && top.TYPO3.Backend.NavigationContainer.PageTree) {
top.TYPO3.Backend.NavigationContainer.PageTree.refreshTree.defer(500);
}
But should be (this is already fixed in 9)
top.TYPO3.Backend.ContentContainer.setUrl(
top.TYPO3.settings.RecordCommit.moduleUrl + '&redirect=' + ContextMenuActions.getReturnUrl() ...
).done(function () {
if (table === 'pages' && top.TYPO3.Backend.NavigationContainer.PageTree) {
top.TYPO3.Backend.NavigationContainer.PageTree.refreshTree();
}
});
It was changed in 9 in here:
https://review.typo3.org/#/c/51594/113/typo3/sysext/backend/Resources/Public/JavaScript/ContextMenuActions.js
Updated by Gerrit Code Review over 6 years ago
- Status changed from New to Under Review
Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/55233
Updated by Gerrit Code Review over 6 years ago
Patch set 2 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/55233
Updated by Tymoteusz Motylewski over 6 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 17136ee9f2bfc29268b1676a56f96eb6d97b59aa.
Updated by Benjamin Kluge about 6 years ago
Tymoteusz Motylewski wrote:
Applied in changeset 17136ee9f2bfc29268b1676a56f96eb6d97b59aa.
This patch reloads the page tree on every load event. To reproduce disable a page and then edit a content element. The Page tree reloads on every button click on the main frame. The load event should be removed after execution.
Updated by Benjamin Kluge about 6 years ago
i opened a new bug #84294 and committing now a patch
Updated by Andreas Kienast about 6 years ago
- Related to Bug #84294: Pagetree reload every time after delete, pasteInto, disable and enable added
Updated by