<br><br><div class="gmail_quote">On Wed, May 23, 2012 at 12:14 PM, Michal Èihaø <span dir="ltr"><<a href="mailto:michal@cihar.com" target="_blank">michal@cihar.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi<br>
<br>
Dne Tue, 22 May 2012 22:37:47 +0530<br>
Chanaka Dharmarathna <<a href="mailto:pe.chanaka.ck@gmail.com">pe.chanaka.ck@gmail.com</a>> napsal(a):<br>
<div class="im"><br>
> On Tue, May 22, 2012 at 6:02 PM, Rouslan Placella <<a href="mailto:rouslan@placella.com">rouslan@placella.com</a>>wrote:<br>
</div><div class="im">> > BTW, what about extending our existing PMA_Table class? I'm not sure how<br>
> > good of an idea it is, so I'm just throwing it out there...<br>
<br>
</div>I don't think this would be useful, this class should care less about<br>
table (actually should be table independent) and more oriented on<br>
displaying results.<br>
<div class="im"><br>
> Thanks for participating to this discussion and share your valuable<br>
> suggestions.<br>
><br>
> As I remember Michal has already suggested to not to use an object of this<br>
> case.<br>
> And use PMA_Display::someFunction where needed.<br>
> Thats why I focused on static fields and behaviors.<br>
<br>
</div>Well it was just a naming condition, you should really make that<br>
object, get parameters in constructor. So basically Rouslans example is<br>
what I would expect.<br>
<div class="im"><br>
> The constructor of the function only executed, if create an object of a<br>
> class. (I think your code trying to say create an object, new keyword is<br>
> missing there).<br>
> So I don't think to get the use of a constructor. That's why I'm using a<br>
> function for initialize instance variables.<br>
><br>
> I'm agreed to use more meaningful name. It is better if we can name it as<br>
> PMA_DisplayTable .<br>
<br>
</div>PMA_DisplayResults might be better. Please remember this should not be<br>
about displaying tables but results of any SQL query - results of<br>
stored procedure, browsing SQL processes or displaying binary log.<br>
<div class="HOEnZb"><div class="h5"><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>------------------------------------------------------------------------------<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>Hi Michal,<br><br>I was thinking of get the use of this class by using an object as in my proposal [0].<br>But, I had misunderstood your comment about creating an object.<br>That's why I looked into this way.<br>
Any way it's a mistake by me.<br><br>Thanks for getting me into the right track soon.<br>I'll introduce the new design of the class as soon as possible.<br><br>[0] : <a href="http://www.google-melange.com/gsoc/proposal/review/google/gsoc2012/chanaka777/1">http://www.google-melange.com/gsoc/proposal/review/google/gsoc2012/chanaka777/1</a><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>