<br><br><div class="gmail_quote">On Sun, Jul 8, 2012 at 10:29 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">

<div class="im"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_quote"><div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">




<div><br>
<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>> Hi Marc,<br>
><br>
> Hi,<br>
>> can someone explain the goal of this method? AFAIK it's never called.<br>
>><br>
>><br>
> I introduce this method inside PMA_DisplayResults class.<br>
> As you mentioned, this method has used nowhere.<br>
> I thought that this will useful for further improvements as well generally<br>
> getter setter methods use inside classes.<br>
> But not sure it will definitely use in future.<br>
><br>
> It can be removed since it is not using at the moment.<br>
> What should be better ?<br>
><br>
> Regards !<br>
<br>
</div>Chanaka,<br>
I'm not sure, let's see opinions of others on this.<br></blockquote></div></div></blockquote></div><div>I think, as to set the local PMA_CommonFunctions object of any class using setCommonFunctions(), we would always need the existing single object using PMA_CommonFunctions::getInstance().</div>




<div>And when we use local getCommonFunctions(), it already sets the local PMA_CommonFunctions object to PMA_CommonFunctions::getInstance(). So I see no particular utility of setCommonFunctions().</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div>

<div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"></blockquote></div><br></div>Hi Marc,<br><br>I introduce common setter method for set any property of that class.<br>




So I'll remove<span> setCommonFunctions()</span> function since the expected behavior can obtain from the common setter.</div></blockquote></div></blockquote></div><br clear="all"></div>Hi Atul,<br><br>Seems that is the point.<br>


Since PMA_CommonFunctions is a singleton class, that method makes no sense.<br>Anyway I'm going to remove it. :)</blockquote><div>Hi Chanaka,</div><div>Ok :). Apart from PMA_DisplayResults(), similar setCommonFunctions() is used in many other classes too, please remove it from there too.</div>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><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>
</div></div><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><br clear="all"><div><br></div>-- <br>Atul Pratap Singh<div>Senior Undergraduate Student</div><div>Institute Of Technology,<br>Banaras Hindu University, Varanasi,<br>India.</div><br>