JLoader : files located at the root path, where the prefix is registered... #1436

Closed
wants to merge 3 commits into
from

Conversation

Projects
None yet
2 participants
Contributor

florianv commented Aug 6, 2012

Files located at the root path, where the prefix is registered, cannot be autoloaded.

Ex : Prefix PN registered in /path/to/pn.
A class PNVariable located in path/to/pn/variable.php won't be autoloaded, because the auto loader try to find it in path/to/pn/variable/variable.php

Contributor

pasamio commented Aug 10, 2012

Do you mind rebasing the commits to remove the extra two commits that don't really add much value.

I'm not sure I entirely agree with the direction as this would encourage people to place files in the root and I don't feel that this is generally what we'd be after. I can see a few use cases (my own included) where this would be useful however. I'll wait for feedback from others.

florianv closed this Aug 10, 2012

Contributor

florianv commented Aug 10, 2012

Hello, I rebased it here : #1445.

For Joomla itself, it is not so useful but it can allow to autoload the factory.

For Platform applications, if a separate library come with it, you might want to have some files in the root path of the library, or the front controller of the application located in the root path of the application, for example.

Contributor

pasamio commented Aug 12, 2012

For future reference if you rebase and do a force push to the original branch, the original pull request will be updated with the rebase'd commits.

Contributor

florianv commented Aug 12, 2012

Ok, thanks for the tip.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment