<div dir="ltr"><div>I opened a PR that moves some classes to the libraries/classes directory under the PhpMyAdmin namespace.</div><div><br></div><a href="https://github.com/phpmyadmin/phpmyadmin/pull/13411">https://github.com/phpmyadmin/phpmyadmin/pull/13411</a><br></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Maurício Meneghini Fauth</div></div></div>
<br><div class="gmail_quote">On Tue, Jun 20, 2017 at 1:28 PM, Maurício Meneghini Fauth <span dir="ltr"><<a href="mailto:mauriciofauth@gmail.com" target="_blank">mauriciofauth@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 dir="ltr"><br><div class="gmail_extra"><br clear="all"><div><div class="m_8976250396461985278gmail_signature"><div dir="ltr">Maurício Meneghini Fauth</div></div></div>
<br><div class="gmail_quote"><span class="">On Fri, Jun 16, 2017 at 11:43 AM, 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:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hello<br>
<br>
Maurício Meneghini Fauth píše v Út 13. 06. 2017 v 21:35 -0300:<br>
<span class="m_8976250396461985278gmail-">> Currently, classes do not have a single location in the directory<br>
> structure. The class files are mixed with other kinds of files in the<br>
> libraries directory, and the PMA namespace points to the root path<br>
> which causes all classes to have a namespace that begins with<br>
> PMA\libraries. This affects the performance of the autoloader.<br>
<br>
</span>Is this something noticeable or just hypothetic?<br></blockquote><div><br></div></span><div>It's not something I would worry about, because there are certainly things that hurt performance more than that.<br></div><div>I care more about how the files are organized.<br></div><span class=""><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<span class="m_8976250396461985278gmail-"><br>
> One solution to solve this problem would be to move the classes to a<br>
> new directory that contains only class files and create a new<br>
> namespace that points to that directory.<br>
> The directory can have any name that describes it, like lib, classes,<br>
> includes or src, but what I usually see in open source projects is<br>
> the src directory.<br>
<br>
</span>I don't think src is good fit there, it's good for libraries, but I<br>
don't think it makes sense for us. Also we've spent quite some time to<br>
educate people to block access to the libraries folder, so I'd stick<br>
with that.<br></blockquote><div><br></div></span><div>As I commented with Isaac, I also think src would not be a good idea at the moment. I prefer libraries/classes.<br></div><span class=""><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<span class="m_8976250396461985278gmail-"><br>
> Then we can use the namespace PhpMyAdmin, which is already used in<br>
> SqlParser, in the src directory, and after all classes are moved to<br>
> that directory, we remove the PMA namespace.<br>
<br>
</span>I was thinking about namespace unification as well, so this certainly<br>
makes sense.<br>
<br>
How about just registering namespace on the libraries folder (eg.<br>
PhpMyAdmin\Core)?<br></blockquote><div><br></div></span><div>I thought about using PhpMyAdmin only on libraries/classes. What do you think?<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">
<span class="m_8976250396461985278gmail-HOEnZb"><font color="#888888"><br>
--<br>
        Michal Čihař | <a href="https://cihar.com/" rel="noreferrer" target="_blank">https://cihar.com/</a> | <a href="https://weblate.org/" rel="noreferrer" target="_blank">https://weblate.org/</a><br>
</font></span><br></span><span class="">______________________________<wbr>_________________<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/m<wbr>ailman/listinfo/developers</a><br>
<br></span></blockquote></div><br></div></div>
</blockquote></div><br></div>