Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Do not change autoloaders' order on AopComposerLoader initialization #93

Merged
merged 1 commit into from
Aug 24, 2013
Merged

Do not change autoloaders' order on AopComposerLoader initialization #93

merged 1 commit into from
Aug 24, 2013

Conversation

andrey-yantsen
Copy link
Contributor

Actually I didn't fixed #91 by #92 - it's just changed behavior of this error. And this PR makes my test-code working completely correct.

@andrey-yantsen
Copy link
Contributor Author

Arghr... It's fixed one issue and made few our tests fails :(
Ignore this PR for a while, please.

@andrey-yantsen
Copy link
Contributor Author

@lisachenko ok, fixed. Can you take a look?

lisachenko added a commit that referenced this pull request Aug 24, 2013
Do not change autoloaders' order on AopComposerLoader initialization
@lisachenko lisachenko merged commit 1299a85 into goaop:master Aug 24, 2013
@andrey-yantsen andrey-yantsen deleted the patch-1 branch August 24, 2013 23:59
genkiroid pushed a commit to genkiroid/framework that referenced this pull request Jun 23, 2016
Replacing strpos check with regex
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

GO! AOP breaks composer autoloader
2 participants