Bug #85431

Using System > Access on a page and then deleting this page leads to exceptions

Added by Wolfgang Klinger about 1 year ago. Updated 12 months ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Backend User Interface
Target version:
-
Start date:
2018-06-29
Due date:
% Done:

100%

TYPO3 Version:
9
PHP Version:
7.0
Tags:
Complexity:
Is Regression:
Sprint Focus:
On Location Sprint

Description

Uncaught TYPO3 Exception

The first exception is:

Argument 1 passed to TYPO3\CMS\Backend\Template\Components\DocHeaderComponent::setMetaInformation() must be of the type array, boolean given, called in /var/www/core/typo3/sysext/beuser/Classes/Controller/PermissionController.php on line 134 

TypeError thrown in file
/var/www/core/typo3/sysext/backend/Classes/Template/Components/DocHeaderComponent.php in line 65.

if you fix this in \TYPO3\CMS\Beuser\Controller\PermissionController::initializeAction
(readPageAccess returns a boolean and not an array)

        $pageInfo = BackendUtility::readPageAccess($this->id, ' 1=1');
        if ($pageInfo !== false) {
            $this->pageInfo = $pageInfo;
        }

you get the next one:

Argument 2 passed to TYPO3\CMS\Core\Imaging\IconFactory::getIconForRecord() must be of the type array, null given, called in /var/www/core/typo3/sysext/backend/Classes/Tree/View/AbstractTreeView.php on line 690 

TypeError thrown in file
/var/www/core/typo3/sysext/core/Classes/Imaging/IconFactory.php in line 113.

34 TYPO3\CMS\Core\Imaging\IconFactory::getIconForRecord("pages", NULL, "small")

Schermata.png View (70 KB) Riccardo De Contardi, 2018-08-03 21:55

Associated revisions

Revision f308e662 (diff)
Added by Paweł Rogowicz about 1 year ago

[BUGFIX] Select first element in tree when deleting selected node

This patch fixes a bug which occured when deleting a currently selected
node. After deleting this node the first element of the tree is
displayed.

Releases: master
Resolves: #85431
Change-Id: I64cd7c4ebd8415a2c261da3f62f8e412575b0075
Reviewed-on: https://review.typo3.org/57791
Tested-by: TYPO3com <>
Reviewed-by: Łukasz Uznański <>
Tested-by: Łukasz Uznański <>
Reviewed-by: Joerg Kummer <>
Tested-by: Joerg Kummer <>
Reviewed-by: Andreas Fernandez <>
Tested-by: Andreas Fernandez <>
Reviewed-by: Tymoteusz Motylewski <>
Tested-by: Tymoteusz Motylewski <>

History

#1 Updated by Riccardo De Contardi about 1 year ago

  • Category set to Backend User Interface

Steps to reproduce

1) create some pages, eg. as subpages of a known page, click on the page
2) go in Access module (and use it to change something, but I don't think it is important
3) use pagetree to delete the page (see the screenshot)

Results

- The list of page in the access module is not refreshed
- If you try to edit the page that has been just deleted, you got the exception

Suggest:

- remove the pagetree while in access module (but that would make the module less usable)
- or force a refresh of the module when a page is deleted.

#2 Updated by Tymoteusz Motylewski about 1 year ago

  • Sprint Focus set to On Location Sprint

#3 Updated by Łukasz Uznański about 1 year ago

  • Assignee set to Łukasz Uznański

#4 Updated by Łukasz Uznański about 1 year ago

  • Assignee deleted (Łukasz Uznański)

#5 Updated by Gerrit Code Review about 1 year 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 https://review.typo3.org/57791

#6 Updated by Gerrit Code Review about 1 year ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57791

#8 Updated by Gerrit Code Review about 1 year ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57791

#9 Updated by Gerrit Code Review about 1 year ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57791

#10 Updated by Gerrit Code Review about 1 year ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57791

#11 Updated by Joerg Kummer about 1 year ago

Version 8.7 also affected

#12 Updated by Gerrit Code Review about 1 year ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57791

#13 Updated by Gerrit Code Review about 1 year 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/57791

#14 Updated by Gerrit Code Review about 1 year 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/57791

#15 Updated by Gerrit Code Review about 1 year 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/57791

#16 Updated by Gerrit Code Review about 1 year 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/57791

#17 Updated by Gerrit Code Review about 1 year ago

Patch set 12 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57791

#18 Updated by Gerrit Code Review about 1 year ago

Patch set 13 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57791

#19 Updated by Paweł Rogowicz about 1 year ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#20 Updated by Benni Mack 12 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF