Task #91657

Use stdWrapValue consitently instead of manually checking existing config with ternary operator

Added by Nikita Hovratov 16 days ago. Updated 16 days ago.

Status:
Under Review
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2020-06-16
Due date:
% Done:

0%

TYPO3 Version:
11
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

A common way to apply stdWrap is: $value = isset($config['value.']) ? $cObj->stdWrap($config['value'], $config['value.']) : $config['value'];
This can be replaced by $value = $cObj->stdWrapValue('value', $config);
Not only is it shorter, it is also more robust and less error-prone.

History

#1 Updated by Gerrit Code Review 16 days 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/c/Packages/TYPO3.CMS/+/64898

#2 Updated by Gerrit Code Review 16 days 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/c/Packages/TYPO3.CMS/+/64898

#3 Updated by Gerrit Code Review 16 days 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/c/Packages/TYPO3.CMS/+/64898

#4 Updated by Gerrit Code Review 16 days 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/c/Packages/TYPO3.CMS/+/64898

#5 Updated by Gerrit Code Review 16 days ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/64898

#6 Updated by Gerrit Code Review 16 days ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/64898

#7 Updated by Gerrit Code Review 16 days ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/64898

Also available in: Atom PDF