Bug #68503: check if array key exists before use them - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #68503

closed

check if array key exists before use them

Added by Martin Tepper over 9 years ago. Updated about 6 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Stephan Großberndt

Category:

Target version:

Start date:

2015-07-24

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

PHP Version:

Tags:

Complexity:

Is Regression:

Sprint Focus:

Description

Hello,

in my case i call TYPO3\CMS\Core\Database\ReferenceIndex::updateRefIndexTable() and this runs fetchTableRelationFields().
This method uses the TCA. But when the given table hasn't a TCA configuration a PHP warning will thrown.

Actions

Copy link

Updated by Mathias Schreiber over 9 years ago

Status changed from New to Needs Feedback
Assignee set to Mathias Schreiber

Why would you call that method for a table that has no rendering definition?

Actions

Copy link

Updated by Martin Tepper over 9 years ago

I use the extension update_refindex which has a scheduler task where you can choose tables.
There was a wrong configuration in my case and i got the warning. In generell i think that should not happen.

Actions

Copy link

Updated by Mathias Schreiber over 9 years ago

How do you spot the false configuration then?
Personally, I'd rather throw an exception making clear that the configuration is corrupt making usage of the extension impossible until the error has been resolved.

Actions

Copy link

Updated by Martin Tepper over 9 years ago

Sure, because of the warning :)

I agree.

Issue can be closed.

Actions

Copy link

Updated by Mathias Schreiber over 9 years ago

Assignee changed from Mathias Schreiber to Stephan Großberndt

nono, no need to close it.
The core needs to throw the exception.

I'll assign this to Dr. RefIndex (currently on holiday)

Actions

Copy link

Updated by Gerrit Code Review over 9 years ago

Status changed from Needs Feedback 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/42599

Actions

Copy link

Updated by Gerrit Code Review over 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/42599

Actions

Copy link

Updated by Gerrit Code Review over 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/42599

Actions

Copy link

Updated by Gerrit Code Review over 9 years ago

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/42640

Actions

Copy link

#10