<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2014-11-03 18:46 GMT+01:00 Marc Delisle <span dir="ltr"><<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hugues Peccatte a écrit :<br>
<div><div class="h5">> Hi,<br>
><br>
> While fixing some PHPCS issues, I found something strange here [1] (you<br>
> need to get the last master version).<br>
> As you can see, there is a mb_strpos !== true… but, this should always be<br>
> true… mb_strpos can't be strictly true, it could be false are an int.<br>
><br>
> It seems that this condition is here since the file is created (commit<br>
> 7248bf87fad6f371943c7b26c3a597bc04265dc7 – from the log, I can imagine that<br>
> the code wasn't in a class before).<br>
> Did anyone already use this import please? Because I can't believe that the<br>
> returned value is not the initial $cell…<br>
><br>
> [1] libraries/plugins/import/ImportMediawiki.class.php:561 (method _getCell)<br>
><br>
> Thanks for your feedbacks,<br>
> Hugues.<br>
<br>
</div></div>Hi Hugues,<br>
I have not used this import; I assume that the intention was to use this<br>
condition:<br>
<br>
=== false<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Marc Delisle (phpMyAdmin)<br></font></span></blockquote></div><br></div><div class="gmail_extra">Hi,</div><div class="gmail_extra"><br></div><div class="gmail_extra">Thanks, I think so…</div><div class="gmail_extra">I'll fix the code in this way.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Hugues.</div></div>