<br><br><div class="gmail_quote">On Sat, May 12, 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-05-12 08:28, Chanaka Dharmarathna a écrit :<br>
<div class="im">> On Sat, May 12, 2012 at 4:58 PM, Marc Delisle <<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>> wrote:<br>
><br>
>> Le 2012-05-12 00:51, Chanaka Dharmarathna a écrit :<br>
>>> Hi All,<br>
>>><br>
>>> While I'm refactoring some files relating to display query results<br>
>>> (currently display_tbl.lib.php, common.lib.php), I can method names are<br>
>>> used regardless of the inside functionality.<br>
>>><br>
>>> Ex :- "PMA_displayHtmlRadio" in common.lib.php method return a string of<br>
>>> HTML content.<br>
>> It does not return anything.<br>
>><br>
>>> But the name tells the function displays HTML itself.<br>
>> This is exactly what this function is doing.<br>
><br>
</div><div class="im">> Hi Marc,<br>
><br>
> It was a mistake by me. Due to I have push some of modifications to my<br>
> repository, I thought they were returning string.<br>
> However still there are some functions need to modify. ( PMA_buttonOrImage<br>
> should be PMA_displayButtonOrImage as you pointed)<br>
><br>
> But what I actually need to suggest is, instead of render them inside<br>
> functions, render them in files like sql.php<br>
> Above function can be PMA_getButtonOrImage and if it need to render<br>
> somewhere we can just do "echo PMA_getButtonOrImage"<br>
> So that function only contains some logics (manipulations, calculations<br>
> etc) and it gives a separate layer for PMA.<br>
<br>
</div>I agree that these functions should only return a string.<br>
<div class="im HOEnZb"><br>
><br>
> I did some modifications in my repository [0].<br>
> I'm stuck with these things, because when I merge PMA to my repository,<br>
> there were lots of conflicts in files.<br>
> Please guide me to do the right thing.<br>
><br>
> [0] :<br>
> <a href="https://github.com/Chanaka/phpmyadmin/commit/bf15800089d11e83dc7998e5064dafcf2280c118" target="_blank">https://github.com/Chanaka/phpmyadmin/commit/bf15800089d11e83dc7998e5064dafcf2280c118</a><br>
><br>
> Regards !<br>
<br>
<br>
<br>
--<br>
</div><div class="HOEnZb"><div class="h5">Marc Delisle<br>
<a href="http://infomarc.info" target="_blank">http://infomarc.info</a><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>
</div></div></blockquote></div><br>Hi Marc,<br><br>I'll do the necessary changes.<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>