<br><br><div class="gmail_quote">On Wed, Jun 13, 2012 at 8:04 PM, Marc Delisle <span dir="ltr"><<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Le 2012-06-12 13:29, Chanaka Dharmarathna a écrit :<br>
<div class="im">> On Tue, Jun 12, 2012 at 6:18 PM, Marc Delisle <<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>> wrote:<br>
><br>
>> Chanaka,<br>
>><br>
>> about your question on long lines, I agree with you that this could be<br>
>> put on two lines. Suggestion:<br>
>><br>
>> if ((PMA_strlen($wktval) > $GLOBALS['cfg']['LimitChars'])<br>
>>    && ($_SESSION['tmp_user_values']['display_text'] ==<br>
>>        self::DISPLAY_PARTIAL_TEXT)<br>
>> ) {<br>
>><br>
>> --<br>
>> Marc Delisle<br>
>> <a href="http://infomarc.info" target="_blank">http://infomarc.info</a><br>
>><br>
>><br>
</div><div class="im">> Hi Marc,<br>
><br>
> Thanks for your suggestion.<br>
> I check whether it works or not before also.<br>
><br>
> But code sniffer gives following errors,<br>
> | ERROR   | Multi-line IF statement not indented correctly; expected 20<br>
> spaces but found 24<br>
<br>
</div>The code sniffer does not like the additional indenting of "self", so<br>
align it under "&&".<br>
<div class="im"><br>
> | ERROR   | Each line in a multi-line IF statement must begin with a<br>
> boolean operator<br>
<br>
</div>Maybe move the "==" under the "&&".<br>
<div class="HOEnZb"><div class="h5"><br>
><br>
> Sorry if I'm doing any wrong. Can you please tell any guesses why I'm<br>
> getting these errors ?<br>
><br>
> Regards !<br>
<br>
<br>
<br>
--<br>
</div></div><span class="HOEnZb"><font color="#888888">Marc Delisle<br>
<a href="http://infomarc.info" target="_blank">http://infomarc.info</a><br>
<br>
</font></span><br>------------------------------------------------------------------------------<br>
Live Security Virtual Conference<br>
Exclusive live event will cover all the ways today's security and<br>
threat landscape has changed and how IT managers can respond. Discussions<br>
will include endpoint security, mobile security and the latest in malware<br>
threats. <a href="http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/" target="_blank">http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/</a><br>_______________________________________________<br>
Phpmyadmin-devel mailing list<br>
<a href="mailto:Phpmyadmin-devel@lists.sourceforge.net">Phpmyadmin-devel@lists.sourceforge.net</a><br>
<a href="https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel" target="_blank">https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel</a><br>
<br></blockquote></div><br>Hi Marc,<br><br>Yes your first suggestion works fine.<br>But second one is not working for me.<br>I thought that code sniffer doesn't support multi line comparisons.<br>Since you again suggest me same way I may wrong.<br>
<br>But I couldn't find any way to use comparisons in two lines without errors.<br><br clear="all">Regards !<br>-- <br>____________________________________<br><br>Chanaka Indrajith<br>Bsc.Computer Engineering Undergraduate<br>
Faculty of Engineering<br>University of Peradeniya<br>Sri Lanka<br>____________________________________<br><br>