<div dir="ltr"><div class="gmail_quote"><div dir="ltr">Le sam. 3 oct. 2015 à 21:13, Hugues Peccatte <<a href="mailto:hugues.peccatte@gmail.com" target="_blank">hugues.peccatte@gmail.com</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div dir="ltr">Le sam. 3 oct. 2015 à 12:46, Remi Collet <<a href="mailto:remi@fedoraproject.org" target="_blank">remi@fedoraproject.org</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">-----BEGIN PGP SIGNED MESSAGE-----<br>
Hash: SHA1<br>
<br>
Le 03/10/2015 09:14, Hugues Peccatte a écrit :<br>
> Dear developers,<br>
><br>
> The following PR is trying to use namespaces and autoloading for<br>
> pMA: <a href="https://github.com/phpmyadmin/phpmyadmin/pull/11428" rel="noreferrer" target="_blank">https://github.com/phpmyadmin/phpmyadmin/pull/11428</a> This PR<br>
> seems to be quite stable now. As almost all PHP files had been<br>
> updated, this is a huge work to test all cases, all configurations,<br>
> all features. Would it be possible to have some testers please?<br>
<br>
At a first (and quick) look<br>
<br>
PMA\libraries namespace seems a bit uggly...<br>
<br>
'libraries' is a directory, only a layout artefact, why should it be<br>
part of the namespace.<br>
<br>
Per PSR-4 definition, namespace should be<br>
<br>
        "The fully qualified class name MUST have a top-level<br>
        namespace name, also known as a "vendor namespace".<br>
<br>
So, is PMA the "vendor" ? it seems.<br>
<br>
        "The fully qualified class name MAY have one or<br>
        more sub-namespace names."<br>
<br>
I think a second level is needed (but again, "libraries" seems strange<br>
to me... but perhaps only to me)<br>
<br>
Notice: it will be nice to move all 3rd party lib to a separate<br>
directory (will make downstream work much easier)<br>
<br>
Remi.<br>
<br>
> Thanks to Marc and Isaac who already did a lot of tests. And thanks<br>
> to new testers.<br>
><br>
> Have a nice day,<br>
><br>
> H.<br>
><br>
><br>
> _______________________________________________ Developers mailing<br>
> list <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><br>
><br>
<br>
-----BEGIN PGP SIGNATURE-----<br>
Version: GnuPG v2<br>
Comment: Using GnuPG with Thunderbird - <a href="http://www.enigmail.net/" rel="noreferrer" target="_blank">http://www.enigmail.net/</a><br>
<br>
iEYEARECAAYFAlYPhnsACgkQYUppBSnxahgeqQCgrqY4W1TuTzxvpiwSLcVfiVKq<br>
lUYAoIopoJ4hHCeYEdGD7YFGsPUg6htx<br>
=d2Wp<br>
-----END PGP SIGNATURE-----<br></blockquote><div><br></div></div></div><div dir="ltr"><div class="gmail_quote"><div>Hi Remi,</div><div><br></div><div>Thanks for your feedback. </div><div><br></div><div>Why does PMA\libraries seem ugly?</div><div>OK, "libraries" is a directory, but all classes are not in libraries, that's why I decided to set the vendor namespace (PMA) at project root. If needed, we can imagine to have a specific vendor namespace for libraries, but why and which one?</div><div><br></div><div>I saw your notice about moving 3rd party libraries. But I think this is not the subject here.</div><div>In fact, this is a really huge work that had been done (at least, I think…) and I think that a lot of subjects could be discussed here. That's why I'ld like to focus on test and errors.</div><div><br></div><div>Had you some time to test the PR?</div><div><br></div><div>Thanks,</div><div>H.</div></div></div></blockquote><div><br></div><div>Hi,</div><div><br></div><div>Any other feedback please? Or maybe we can merge…</div><div><br></div><div>H. </div></div></div>