Bug #69642
closedCrash on Workspace Preview if page in menu is hidden on LiveWS
100%
Description
After the changes in #33436 we crash, if you preview a page with menu which have a page which is hidden on LiveWS.
See my comment in the change for the position: https://review.typo3.org/#/c/42553/3/typo3/sysext/frontend/Classes/ContentObject/Menu/AbstractMenuContentObject.php
Updated by Daniel Goerz about 9 years ago
- Status changed from New to Closed
HI Alex, I close this as duplicate of #67001. I copied your comment to that ticket to keep all information together.
Updated by Alexander Opitz about 9 years ago
- Status changed from Closed to New
Reopen:
The issue, which I mention exists since 6.2.15 and is inside the menu.
This doesn't have to do with the issue #67001, which is about viewing the page which is hidden in Live (which doesn't crash TYPO3 this only do not work).
Updated by Gerrit Code Review about 9 years ago
- Status changed from New to Under Review
Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43188
Updated by Gerrit Code Review about 9 years ago
Patch set 2 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43188
Updated by Gerrit Code Review about 9 years ago
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/43189
Updated by Alexander Opitz about 9 years ago
Testing:
- You need a menu configured with the special "directory"
- On Live set a page in the menu directory to hidden
- Open the page as Workspace preview
- You will get SQL statements like "SELECT FROM 'pages'" (See the missing field names in the select). This comes from the versionOL with an empty row.
Updated by Alexander Opitz about 9 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 25a69735e69e79de0acbc450026f2d5f0ad656ee.
Updated by Alexander Opitz about 9 years ago
- Status changed from Resolved to Under Review
Fix for master is still open.
Updated by Gerrit Code Review about 9 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/43189
Updated by Gerrit Code Review about 9 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/43189
Updated by Alexander Opitz about 9 years ago
- Status changed from Under Review to Resolved
Applied in changeset 4012b988b9f6905429eabd1f1a44adc57b1c95f6.
Updated by Gerrit Code Review about 9 years ago
- Status changed from Resolved 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 https://review.typo3.org/44639
Updated by Alexander Opitz almost 9 years ago
- Status changed from Under Review to Closed
Reclose as Review https://review.typo3.org/44639 wasn't for this issue.