Bug #60405
closedPageTree cannot select/expand hidden nodes
100%
Description
TYPO3.Backend.NavigationContainer.PageTree.select(64) does not select the correct page, but tries to determine the correct node levels, fakes the local node states and reloads the page tree. One thing that might not be considered there is the fact that pages are represented in hex-values with a leading "p", thus the integer "64" with be converted to "p40" as node id.
To overcome this scenario and enable custom handling there, a new generic method is introduced:
TYPO3.Backend.NavigationContainer.PageTree.invokePageId(pageId, callback, allResults);
The methods "selectPageId" and "expandPageId" show how this is actually used.
Updated by Gerrit Code Review over 10 years ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32155
Updated by Gerrit Code Review over 10 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32155
Updated by Gerrit Code Review about 10 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32155
Updated by Gerrit Code Review over 9 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32155
Updated by Gerrit Code Review over 9 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32155
Updated by Gerrit Code Review over 9 years ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32155
Updated by Gerrit Code Review about 9 years ago
Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/32155
Updated by Gerrit Code Review about 9 years ago
Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/32155
Updated by Gerrit Code Review about 9 years ago
Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/32155
Updated by Oliver Hader almost 9 years ago
How to test?
- Apply the patch
- Clear all caches using the Install Tool (ExtDirect cache must be flushed)
- Find a nested page somewhere in the page tree and notice the page ID
- collapse all page branches
- reload the whole backend
- open the browser console and e.g. type (replace the number "20" with the page ID you've chosen)
TYPO3.Backend.NavigationContainer.PageTree.selectPageId(20);
or
TYPO3.Backend.NavigationContainer.PageTree.expandPageId(20);
Updated by Gerrit Code Review almost 9 years ago
Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/32155
Updated by Gerrit Code Review almost 9 years ago
Patch set 1 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47443
Updated by Oliver Hader almost 9 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 9aaab9b3e657131f536cbaa212227e983cf8e486.