<div dir="ltr"><div class="gmail_quote"><div dir="ltr">Le lun. 31 août 2015 à 10:06, Michal Čihař <<a href="mailto:michal@cihar.com">michal@cihar.com</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Dne Sun, 30 Aug 2015 13:45:14 +0000<br>
Hugues Peccatte <<a href="mailto:hugues.peccatte@gmail.com" target="_blank">hugues.peccatte@gmail.com</a>> napsal(a):<br>
<br>
> Le dim. 30 août 2015 à 15:11, Marc Delisle <<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a>> a écrit :<br>
><br>
> > Hi,<br>
> > please have a look at<br>
> ><br>
> > <a href="https://github.com/phpmyadmin/phpmyadmin/commit/993134cacd61add9f049eee96a53943a87c99342" rel="noreferrer" target="_blank">https://github.com/phpmyadmin/phpmyadmin/commit/993134cacd61add9f049eee96a53943a87c99342</a><br>
> ><br>
> > Can we have a validation when a template is rendered, to ensure we don't<br>
> > forget passing some of the parameters?<br>
> ><br>
> > Probably too late for 4.5, but at least for 4.6?<br>
> ><br>
> > --<br>
> > Marc Delisle | phpMyAdmin<br>
> ><br>
><br>
> Hi,<br>
><br>
> What do you mean by "validation"? Is it a code review or a message by<br>
> Template class that tells you that a variable / parameter is missing?<br>
<br>
The best would be if we can make the template layer fail on missing<br>
variable and test all templates in testsuite.<br>
<br>
<br>
--<br>
        Michal Čihař | <a href="http://cihar.com" rel="noreferrer" target="_blank">http://cihar.com</a> | <a href="http://phpmyadmin.net" rel="noreferrer" target="_blank">http://phpmyadmin.net</a><br>
<br>
_______________________________________________<br>
Developers mailing list<br>
<a href="mailto:Developers@phpmyadmin.net" target="_blank">Developers@phpmyadmin.net</a><br>
<a href="https://lists.phpmyadmin.net/mailman/listinfo/developers" rel="noreferrer" target="_blank">https://lists.phpmyadmin.net/mailman/listinfo/developers</a></blockquote><div><br></div><div>Hi,</div><div><br></div><div>I agree with this last comment.</div><div>Should we review our position regarding template unit tests?</div><div><br></div><div>H.</div></div></div>