Skip to content

Comments

Readme fix related to PSR-0 class loader#71

Merged
rmccue merged 1 commit intoWordPress:masterfrom
fulopattila122:master
Sep 10, 2013
Merged

Readme fix related to PSR-0 class loader#71
rmccue merged 1 commit intoWordPress:masterfrom
fulopattila122:master

Conversation

@fulopattila122
Copy link
Contributor

Hello,

I've added today Requests to an existing app that already uses the Symfony Class Loader.

According to the documentation I have to use:

$loader->registerNamespace('Requests', 'path/to/vendor/Requests/library');

However this doesn't work, as Requests doesn't use namespaces. I figured out that the proper way to initialize Requests with the class loader is to use the registerPrefix method instead:

$loader->registerPrefix('Requests', 'path/to/vendor/Requests/library');

I fixed it in the README

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.14%) when pulling bbc1ab6 on fulopattila122:master into dec780a on rmccue:master.

rmccue added a commit that referenced this pull request Sep 10, 2013
Readme fix related to PSR-0 class loader
@rmccue rmccue merged commit d11e02b into WordPress:master Sep 10, 2013
@rmccue
Copy link
Collaborator

rmccue commented Sep 10, 2013

Thanks! 🍰

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants