<br><br><div class="gmail_quote">On Fri, Apr 6, 2012 at 9:37 PM, Adnan <span dir="ltr"><<a href="mailto:hiddenpearls@gmail.com">hiddenpearls@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br><br><div class="gmail_quote"><div class="im">On Fri, Apr 6, 2012 at 9:35 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">
Op 4 april 2012 11:07 heeft Adnan <<a href="mailto:hiddenpearls@gmail.com" target="_blank">hiddenpearls@gmail.com</a>> het<br>
volgende geschreven:<br>
<div>><br>
><br>
> On Tue, Apr 3, 2012 at 9:15 PM, Adnan <<a href="mailto:hiddenpearls@gmail.com" target="_blank">hiddenpearls@gmail.com</a>> wrote:<br>
>><br>
>> hello Michal,<br>
>><br>
>> I have submitted my proposal, please have a look at it. Give your<br>
>> important input So I could edit my proposal then.<br>
>> Thanks<br>
><br>
> Any input please ?<br>
<br>
</div>You proposal will be reviewed by the mentors in due time. Your<br>
proposal is not the only one to be reviewed, so please be patient.<br>
<br></blockquote></div><div>Thanks, I got it.  </div></div></blockquote><div>I wanted to discuss about the bad design patterns used in schema generation classes. According to my knowledge, </div><div><br></div><p style="margin-top:0px;margin-right:0px;margin-bottom:8px;margin-left:0px;padding-top:0px;padding-right:0px;padding-bottom:0px;padding-left:0px;border-top-width:0px;border-right-width:0px;border-bottom-width:0px;border-left-width:0px;border-style:initial;border-color:initial;outline-width:0px;outline-style:initial;outline-color:initial;vertical-align:baseline;background-image:initial;line-height:19px;font-family:Arial,'Helvetica Neue',Helvetica,sans-serif">
1. In Class PMA_Schema_PDF and other schema classes, properties are used with var which is deprecated from PHP 5.0 to 5.1.3 as declaring properties like var was standard in php4. So, in php5 standards are to use public,protected or private keywords. Though after 5.1.3 var is not deprecated and doesn't issue E_STRICT warning. so var used in current code is considered as public. Best pattren is to use protected keyword for properties in Schema classes.</p>
<p style="margin-top:0px;margin-right:0px;margin-bottom:8px;margin-left:0px;padding-top:0px;padding-right:0px;padding-bottom:0px;padding-left:0px;border-top-width:0px;border-right-width:0px;border-bottom-width:0px;border-left-width:0px;border-style:initial;border-color:initial;outline-width:0px;outline-style:initial;outline-color:initial;vertical-align:baseline;background-image:initial;line-height:19px;font-family:Arial,'Helvetica Neue',Helvetica,sans-serif">
2. Singleton/factory  pattern is not used in schema classes. I guess, we should use it instead of instantiating objects on running scripts. </p><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="gmail_quote"><div class="im"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
--<br>
Kind regards,<br>
<br>
Dieter Adriaenssens<br>
<br>
------------------------------------------------------------------------------<br>
For Developers, A Lot Can Happen In A Second.<br>
Boundary is the first to Know...and Tell You.<br>
Monitor Your Applications in Ultra-Fine Resolution. Try it FREE!<br>
<a href="http://p.sf.net/sfu/Boundary-d2dvs2" target="_blank">http://p.sf.net/sfu/Boundary-d2dvs2</a><br>
_______________________________________________<br>
Phpmyadmin-devel mailing list<br>
<a href="mailto:Phpmyadmin-devel@lists.sourceforge.net" target="_blank">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></div><br><br clear="all"><div class="HOEnZb"><div class="h5"><div><br></div>-- <br>Muhammad Adnan<br>Twitter: @hiddenpearls<br>I blog too do u know ? http:// <a href="http://www.imblog.info" target="_blank">www.imblog.info</a><br>
Failure takes you higher than success ..!<div>
you sell code, I sell my brain.</div><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Muhammad Adnan<br>Twitter: @hiddenpearls<br>I blog too do u know ? http:// <a href="http://www.imblog.info" target="_blank">www.imblog.info</a><br>
Failure takes you higher than success ..!<div>you sell code, I sell my brain.</div><br>