<br><br><div class="gmail_quote">On Sat, May 12, 2012 at 4:58 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 00:51, Chanaka Dharmarathna a écrit :<br>
<div class="im">> 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>
</div>It does not return anything.<br>
<div class="im"><br>
> But the name tells the function displays HTML itself.<br>
</div>This is exactly what this function is doing.<br>
<br>
--<br>
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>
</blockquote></div><br>Hi Marc,<br><br>It was a mistake by me. Due to I have push some of modifications to my repository, I thought they were returning string.<br>However still there are some functions need to modify. ( PMA_buttonOrImage should be PMA_displayButtonOrImage as you pointed)<span class="nf"><br>
<br>But what I actually need to suggest is, instead of render them inside functions, render them in files like sql.php<br>Above function can be </span>PMA_getButtonOrImage and if it need to render somewhere we can just do "echo <span class="nf"></span>PMA_getButtonOrImage"<br>
<span class="nf">So that function only contains some logics (manipulations, calculations etc) and it gives a separate layer for PMA.<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, there were lots of conflicts in files.<br>
Please guide me to do the right thing.<br><br>[0] : <a href="https://github.com/Chanaka/phpmyadmin/commit/bf15800089d11e83dc7998e5064dafcf2280c118">https://github.com/Chanaka/phpmyadmin/commit/bf15800089d11e83dc7998e5064dafcf2280c118</a><br>
<br></span>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>