Project

General

Profile

Actions
Copy link

Bug #92699

closed

variables in ContentObjectRenderer::checkIf

Added by David Bruchmann about 4 years ago. Updated 6 months ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Frontend
Target version:
-
Start date:
2020-10-24
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
10
PHP Version:
7.3
Tags:
ContentObjectRenderer rendering TypoScript
Complexity:
no-brainer
Is Regression:
Sprint Focus:

Description

Some variable are not checked properly before usage:

For many variables is checked this:

    if (isset($conf['a']) || isset($conf['a.'])) {

but used are both.
Concrete example:
        if (isset($conf['isTrue']) || isset($conf['isTrue.'])) {
            $isTrue = isset($conf['isTrue.']) ? trim($this->stdWrap($conf['isTrue'], $conf['isTrue.'])) : trim($conf['isTrue']);
            if (!$isTrue) {
                $flag = false;
            }
        }

This example is throwing an error if $conf['isTrue.'] is set but $conf['isTrue'] not.

Actions
Copy link

Also available in: Atom PDF