<br><br><div class="gmail_quote">On Wed, May 23, 2012 at 11:11 PM, Chanaka Dharmarathna <span dir="ltr"><<a href="mailto:pe.chanaka.ck@gmail.com" target="_blank">pe.chanaka.ck@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br><br><div class="gmail_quote"><div><div class="h5">On Wed, May 23, 2012 at 8:15 PM, Michal Èihaø <span dir="ltr"><<a href="mailto:michal@cihar.com" target="_blank">michal@cihar.com</a>></span> wrote:<br></div></div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">
Hi<br>
<br>
Dne Wed, 23 May 2012 18:47:18 +0530<br>
Chanaka Dharmarathna <<a href="mailto:pe.chanaka.ck@gmail.com" target="_blank">pe.chanaka.ck@gmail.com</a>> napsal(a):<br>
<div><br>
> Its great to heard that you have merge my changes to the master branch.<br>
> I'll consider more on coding style in the future work and fix existing ones.<br>
<br>
</div>You should run phpcs locally to see possible violations when you're<br>
making changes, it's documented on the wiki:<br>
<br>
<a href="http://wiki.phpmyadmin.net/pma/CodeSniffer" target="_blank">http://wiki.phpmyadmin.net/pma/CodeSniffer</a><br>
<div><div><br>
--<br>
        Michal Èihaø | <a href="http://cihar.com" target="_blank">http://cihar.com</a> | <a href="http://blog.cihar.com" target="_blank">http://blog.cihar.com</a><br>
</div></div><br></div></div><div class="im">------------------------------------------------------------------------------<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" target="_blank">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></div></blockquote></div><br>Hi Michal,<br><br>Thanks for helping more in that case.<br>I'll follow that procedure.<div class="HOEnZb"><div class="h5"><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>
</div></div></blockquote></div><br>Hi Michal,<br><br>Its indeed, you have fixed large number of coding violation issues in my code.<br>Thanks for doing it for me.<br>And now I'm fixing existing coding violations.<br>
<br>I have a doubt on introducing or changing a string.<br>If we newly introduce a string or change existing one, do we need to make the changes for any other files like language files (es.po, en_GB.po) ?<br clear="all"><br>
As well I feel its better to have more clarification on your word on the design of PMA_DisplayResults class.<br>"Yes it looks okay. Maybe just the method should be called something
like getHtml.
"<br>Can you please explain me about the getHtml thing ?<br><br>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>