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

Sensio Kernel Manipulator false return value not handled #22

Open
rrajkomar opened this issue Jun 10, 2015 · 3 comments
Open

Sensio Kernel Manipulator false return value not handled #22

rrajkomar opened this issue Jun 10, 2015 · 3 comments
Labels

Comments

@rrajkomar
Copy link

In your bundle (which is great BTW), the following method return value is not checked :
$kernelManipulator->addBundle($bundle->fullyQualifiedClassname);

This method can return false, in which case the bundle is not registered and yet your bundle would return a success !

@rrajkomar rrajkomar changed the title Sension Kernel Manipulator false return value not handled Sensio Kernel Manipulator false return value not handled Jun 10, 2015
@gnugat gnugat added the bug label Jun 10, 2015
@gnugat
Copy link
Collaborator

gnugat commented Jun 10, 2015

Hello @rrajkomar thanks for your comment, you're right we should check it.

However this bundle is kind of doomed as I still haven't found a way to make it work with PSR-4 (it only works with PSR-0 which is now deprecated), so unless we find a solution for this I don't intend to further maintain it... But I'd love to review and merge Pull requests!

Cheers,

@rrajkomar
Copy link
Author

What's the issue with psr-4? Your code should be working (haven't tested it though yet).

@gnugat
Copy link
Collaborator

gnugat commented Jun 11, 2015

While trying to write an answer for you, I've found it difficult to remember the issue. And by looking at the code it really looks like it should be working!
This needs more care I guess, when I have time I'll have a closer look at it and if it turns out everything works I'll improve it :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants