Bug #55923

intval() to (int) conversion regressions

Added by Michiel Roos almost 6 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
Start date:
2014-02-12
Due date:
% Done:

100%

TYPO3 Version:
6.2
PHP Version:
5.3
Tags:
Complexity:
no-brainer
Is Regression:
Yes
Sprint Focus:

Description

Another set of errors after #54265:

intval around ternary expression was replaced with a cast, omitting the braces around the ternary expression.

inval($this->mconf['minItems'] ? $this->mconf['minItems'] : $this->conf['minItems']);

Was replaced with:
(int)$this->mconf['minItems'] ? $this->mconf['minItems'] : $this->conf['minItems'];

Instead of:
(int)($this->mconf['minItems'] ? $this->mconf['minItems'] : $this->conf['minItems']);

Related issues

Related to TYPO3 Core - Bug #55706: Different behavior of levelmedia in TYPO3 6.2 Closed 2014-02-05

Associated revisions

Revision d12b2003 (diff)
Added by Michiel Roos almost 6 years ago

[BUGFIX] inval() -> (int) regressions in ternaries

Change-Id: I0c105134ca9de463eed7c6a3760003b8694958ce
Resolves: #55923
Releases: 6.2
Reviewed-on: https://review.typo3.org/27576
Reviewed-by: Marcin SÄ…gol
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel

History

#1 Updated by Gerrit Code Review almost 6 years 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/27576

#2 Updated by Gerrit Code Review almost 6 years 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/27576

#3 Updated by Gerrit Code Review almost 6 years 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/27576

#4 Updated by Gerrit Code Review almost 6 years 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/27576

#5 Updated by Michiel Roos almost 6 years ago

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

#6 Updated by Riccardo De Contardi about 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF