Bug #55923
intval() to (int) conversion regressions
| Status: | Resolved | Start date: | 2014-02-12 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assigned To: | - | % Done: | 100% |
|
| Category: | - | Spent time: | - | |
| Target version: | 6.2.0 | |||
| TYPO3 Version: | 6.2 | Is Regression: | Yes | |
| PHP Version: | 5.3 | Sprint Focus: | ||
| Complexity: | no-brainer |
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
Associated revisions
[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 over 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/27576
#2 Updated by Gerrit Code Review over 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/27576
#3 Updated by Gerrit Code Review over 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/27576
#4 Updated by Gerrit Code Review over 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/27576
#5 Updated by Michiel Roos over 1 year ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset d12b2003fb628864f7f323199f97001110d3baab.