We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
I was testing new PHP linter and it catches this always-false condition:
WARNING badCond: always true condition at /matomo/tests/PHPUnit/Fixtures/UITestFixture.php:590 if ($pluginName == 'ExampleAPI' && ($apiAction != 'xssReportforTwig' || $apiAction != 'xssReportforAngular')) { ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
matomo/tests/PHPUnit/Fixtures/UITestFixture.php
Line 590 in 458f0fa
Probably should be fixed. Presumably, && is intended instead of ||.
&&
||
The text was updated successfully, but these errors were encountered:
ping @diosmosis you've added that piece of code a few weeks ago, maybe you can have a look and fix it.
Sorry, something went wrong.
diosmosis
Successfully merging a pull request may close this issue.
I was testing new PHP linter and it catches this always-false condition:
matomo/tests/PHPUnit/Fixtures/UITestFixture.php
Line 590 in 458f0fa
Probably should be fixed.
Presumably,
&&
is intended instead of||
.The text was updated successfully, but these errors were encountered: