Bug #66520

Fatal when rendering categories that was removed

Added by Marc Bastian Heinrichs almost 6 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Categorization API
Target version:
-
Start date:
2015-04-21
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

PHP Catchable Fatal Error: Argument 2 passed to TYPO3\CMS\Backend\Utility\IconUtility::mapRecordTypeToSpriteIconClass() must be
of the type array, null given, called in typo3/sysext/core/Classes/Tree/TableConfiguration/DatabaseTreeDataProvider.php on line 254 

In DatabaseTreeDataProvider calling mapRecordTypeToSpriteIconClass $row have to be an array.

Patch will follow.


Files

66520.patch (943 Bytes) 66520.patch Marc Bastian Heinrichs, 2015-11-10 10:56
#1

Updated by Marc Bastian Heinrichs almost 6 years ago

  • Description updated (diff)
#2

Updated by Susanne Moog over 5 years ago

  • Status changed from New to Needs Feedback

Is this still a problem and will a patch still follow? ;)

#3

Updated by Marc Bastian Heinrichs about 5 years ago

still valid - also for IconFactory in v7.

#4

Updated by Alexander Opitz about 5 years ago

  • Status changed from Needs Feedback to Accepted
  • Target version set to 7 LTS
#5

Updated by Marc Bastian Heinrichs about 5 years ago

For the moment no idea for a nice patch. attached the fast and unclean patch (6.2)

If the row, that is the representation of the node, was deleted, the node must not be created, or?

#6

Updated by Alexander Opitz about 5 years ago

  • Target version changed from 7 LTS to 6.2.16
#7

Updated by Marc Bastian Heinrichs about 5 years ago

  • Status changed from Accepted to Needs Feedback
  • Assignee set to Marc Bastian Heinrichs
  • Target version deleted (6.2.16)
#8

Updated by Alexander Opitz about 5 years ago

I couldn't validate this issue. Marc will report back if it happens again on his system or if there is any dependency to reproduce this.

#9

Updated by Marc Bastian Heinrichs about 5 years ago

  • Status changed from Needs Feedback to Closed
  • Assignee deleted (Marc Bastian Heinrichs)
  • Priority changed from Should have to -- undefined --

can not reproduce it again

#10

Updated by Dirk Wenzel about 5 years ago

This error can be reproduced when configuring a category field with a missing rootId

    'my_field' => [
        'config' => [
            'type' => 'select',
            'renderMode' => 'tree',
            'treeConfig' => [
                'parentField' => 'parent',
                'rootUid' => <NON EXISTING CATEGORY UID>,
                'appearance' => [
                    'showHeader' => TRUE,
                    'allowRecursiveMode' => TRUE,
                    'expandAll' => TRUE,
                    'maxLevels' => 99,
                    'width' => 700,
                ],
            ],
            'MM' => 'sys_category_record_mm',
            'MM_match_fields' => [
                'fieldname' => 'my_field',
                'tablenames' => 'tx_myext_my_table',
            ],
            'MM_opposite_field' => 'items',
            'foreign_table' => 'sys_category',
            'foreign_table_where' => ' AND (sys_category.sys_language_uid = 0 OR sys_category.l10n_parent = 0) ORDER BY sys_category.sorting',
            'size' => 10,
            'autoSizeMax' => 20,
            'minitems' => 0,
            'maxitems' => 99,
        ]
    ],

#11

Updated by Marc Bastian Heinrichs about 5 years ago

  • Status changed from Closed to New
  • Priority changed from -- undefined -- to Should have

Ah, thx. If I remember right, I tried this too, when trying to reproduce.

Alex, do you want to check this again?

#12

Updated by Riccardo De Contardi over 3 years ago

  • Status changed from New to Needs Feedback

Sorry for the so late feedback

AFAIK DatabaseTreeDataProvider should not call mapRecordTypeToSpriteIconClass anymore... is it still an issue? Thank you

#13

Updated by Riccardo De Contardi about 3 years ago

  • Status changed from Needs Feedback to Closed

90 days without feedback -> closing it.

If you think that this is the wrong decision or experience the issue again or have more information about how to reproduce it on recent TYPO3 versions like 8.7.x or 9.0.0, please reopen it or open a new issue with a reference to this one.

Thank you.

Also available in: Atom PDF