PHP Error : Unparenthesized `a ? b : c ? d : e` is deprecated. Use either `(a ? b : c) ? d : e` or `a ? b : (c ? d : e)` [duplicate]

This question already has answers here: Stacking Multiple Ternary Operators in PHP (10 answers) How to avoid undefined index warning for chains of 'default' values? (4 answers) Closed 3 years ago. I am using PHP 7.4 for a laravel application and I am getting this exception very frequently. ErrorException (E_DEPRECATED) Unparenthesized `a ? b : c ? d : e` is deprecated. Use either `(a ? b : c) ? d : e` or `a ? b : (c ? d : e)` The code which triggers this exception is : foreach ($allLanguages as $languageKey) { $original[$languageKey] = isset($values[$languageKey]) ? $values[$languageKey] : isset($filesContent[$fileName][$languageKey][$key]) ? $filesContent[$fileName][$languageKey][$key] : ''; } Can any help me to resolve this issue? Found that this is E_DEPRECATED error due to some upgrade in PHP, but is there any way to resolve this exception by converting the deprecated code to latest?

Comment (3)

Jese Leos

August 28, 2024

Verified user

You need to add parentheses around your code: Before: $reference->frotel_vitrine = empty($item->special) ? null : $item->special == 2 || $item->special == 3 ? 'active' : 'deactivate'; After : $reference->frotel_vitrine = empty($item->special) ? null : (($item->special == 2 || $item->special == 3 )? 'active' : 'deactivate'); That should solve the issue.

Jese Leos

August 28, 2024

Verified user

This change in php has been done to remove ambiguity in the decision tree so that there is an explicit order of condition execution. The deprecation warning is reproduced here: Code: $allLanguages = ['en', 'es', 'fr']; $values = ['es' => 'Spanish1']; $filesContent = [ 'foo' => [ 'es' => ['bar' => 'Spanish2'], 'fr' => ['bar' => 'French'] ] ]; $fileName = 'foo'; $key = 'bar'; $original = []; foreach ($allLanguages as $languageKey) { $original[$languageKey] = isset($values[$languageKey]) ? $values[$languageKey] : isset($filesContent[$fileName][$languageKey][$key]) ? $filesContent[$fileName][$languageKey][$key] : ''; } var_export($original); Output: Deprecated: Unparenthesized `a ? b : c ? d : e` is deprecated. Use either `(a ? b : c) ? d : e` or `a ? b : (c ? d : e)` in /in/TG4g2 on line 17 array ( 'en' => '', 'es' => 'Spanish2', 'fr' => 'French', ) As a human-reader of your script, I would assume the reading of your condition as left to right -- but this would place Spanish1 as the output value. Even before php7.4, the output isSpanish2 because the latter fork in the decision tree is given precedence. To avoid this, you must wrap your conditions in parentheses to dictate exactly how the order of execution should be handled. Also, I agree with @Laurel that in php7 it is time for you to embrace the syntactic sugary sweetness that is the null coalescing operator. This will avoid precedence issues and the need to use parentheses, but depending on your desired results, you may need to reorder your conditions. Priority to $values: (Demo) $original[$languageKey] = $values[$languageKey] ?? $filesContent[$fileName][$languageKey][$key] ?? ''; Priority to $filesContent: (Demo) $original[$languageKey] = $filesContent[$fileName][$languageKey][$key] ?? $values[$languageKey] ?? ''; P.s. IIRC, the php manual advises against the use of nested ternaries/conditionals like this on the basis of code clarity. I don't mind this scenario and I like the avoidance of code bloat, but other devs may take a more purist stance.

Jese Leos

August 28, 2024

Verified user

In your case, you should use ?? instead of isset and ternaries: foreach ($allLanguages as $languageKey) { $original[$languageKey] = $values[$languageKey] ?? $filesContent[$fileName][$languageKey][$key] ?? ''; } Note: The ?? operator was added in 7.0. The warning you're getting was added in 7.4, and was upgraded to an error in 8.0 and higher.

You’ll be in good company