<p dir="ltr"><br>
</p>
<br><div class="gmail_quote"><div dir="ltr">Le sam. 5 déc. 2015 à 13:57, Marc Delisle <<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
We have some PSR1 Scrutinizer warnings in many files. For example, in<br>
js/whitelist.php:<br>
<br>
"For compatibility and reusability of your code, PSR1 recommends that a<br>
file should either new symbols (like classes, functions, etc.) or have<br>
side-effects (like outputting something, or including other files), but<br>
not both at the same time. The first symbol is defined on line 20 and<br>
the first side effect is on line 9."<br>
<br>
Is it true that PSR1 objects to our having a define() statement and<br>
ordinary code like chdir() in the same file?<br>
<br>
--<br>
Marc Delisle | phpMyAdmin<br>
<br>
_______________________________________________<br>
Developers mailing list<br>
<a href="mailto:Developers@phpmyadmin.net" target="_blank">Developers@phpmyadmin.net</a><br>
<a href="https://lists.phpmyadmin.net/mailman/listinfo/developers" rel="noreferrer" target="_blank">https://lists.phpmyadmin.net/mailman/listinfo/developers</a><br>
</blockquote></div><p dir="ltr">Hi,</p>
<p dir="ltr">Yes, that's true because one of the rules is not to have declaration and logic code in the same file. Define is declaration, chdir is logic.</p>
<p dir="ltr">H.</p>