- Issue created by @paucala
- Status changed to Needs review
over 1 year ago 1:55pm 15 June 2023 - last update
over 1 year ago 785 pass - @paucala opened merge request.
Running sonarqube we found this issue on ln 118 of commerce_promotion.post_update.php: Make sure that the referenced value variable is unset after the loop.
Why is this a problem?
When a reference is used in a foreach loop instead of using a simple variable, the reference remains assigned and keeps its "value" which is a reference, even after the foreach execution. Most of the time, this is not what the developer is expecting and the reference may be used wrongly in the rest of the code. For this reason, it is recommended to always unset a reference that is used in a foreach to avoid any unexpected side effects.
Steps to reproduce
The proposed solution is unset the referenced variable after the loop