<br><br><div class="gmail_quote">On Fri, May 25, 2012 at 4:40 PM, Dieter Adriaenssens <span dir="ltr"><<a href="mailto:dieter.adriaenssens@gmail.com" target="_blank">dieter.adriaenssens@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">2012/5/24 Chanaka Dharmarathna <<a href="mailto:pe.chanaka.ck@gmail.com">pe.chanaka.ck@gmail.com</a>>:<br>
<div><div class="h5">><br>
><br>
> On Wed, May 23, 2012 at 11:11 PM, Chanaka Dharmarathna<br>
> <<a href="mailto:pe.chanaka.ck@gmail.com">pe.chanaka.ck@gmail.com</a>> wrote:<br>
>><br>
>><br>
>><br>
>> On Wed, May 23, 2012 at 8:15 PM, Michal Čihař <<a href="mailto:michal@cihar.com">michal@cihar.com</a>> wrote:<br>
>>><br>
>>> Hi<br>
>>><br>
>>> Dne Wed, 23 May 2012 18:47:18 +0530<br>
>>> Chanaka Dharmarathna <<a href="mailto:pe.chanaka.ck@gmail.com">pe.chanaka.ck@gmail.com</a>> napsal(a):<br>
>>><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<br>
>>> > ones.<br>
>>><br>
>>> 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>
>>><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>
>>><br>
>>><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>
>><br>
>> Hi Michal,<br>
>><br>
>> Thanks for helping more in that case.<br>
>> I'll follow that procedure.<br>
>><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>
><br>
> Hi Michal,<br>
><br>
> Its indeed, you have fixed large number of coding violation issues in my<br>
> 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<br>
> the changes for any other files like language files (es.po, en_GB.po) ?<br>
<br>
</div></div>There is no need to change the .po files manually. There is a script<br>
to update the .po files. But you don't need to run it in your branch,<br>
the .po files will be updated when your changes are merged with<br>
master.<br>
<br>
Anyway, make sure to use the __() method for translatable strings.<br>
More info can be found on our wiki [0]<br>
<br>
On a related note, if you output variables (as part of a translatable<br>
string, or seperately), make sure to escape them properly, please<br>
check our wiki regarding this, especially the part about escaping html<br>
output : [1]<br>
<div class="im"><br>
> As well I feel its better to have more clarification on your word on the<br>
> design of PMA_DisplayResults class.<br>
><br>
> "Yes it looks okay. Maybe just the method should be called something like<br>
> getHtml. "<br>
> Can you please explain me about the getHtml thing ?<br>
<br>
</div>I think Michal means that method getHtml() should generate and return<br>
a html string.<br>
When calling the method you would get something like this :<br>
<br>
$class_results = new PMA_DisplayResults();<br>
echo $class_results->getHtml();<br>
<br>
<br>
[0] <a href="http://wiki.phpmyadmin.net/pma/Gettext_for_developers" target="_blank">http://wiki.phpmyadmin.net/pma/Gettext_for_developers</a><br>
[1] <a href="http://wiki.phpmyadmin.net/pma/Security_pitfalls" target="_blank">http://wiki.phpmyadmin.net/pma/Security_pitfalls</a><br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Kind regards,<br>
<br>
Dieter Adriaenssens<br>
</font></span><div class="HOEnZb"><div class="h5"><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>
</div></div></blockquote></div><br>Hi Dieter,<br><br>Thanks for your valuable information and pointing those links.<br>That will be very helpful for me.<br>I'll follow that procedure.<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>