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

PHP Notice: Undefined variable: ace in /var/www/camdram/20151223/src/Acts/CamdramSecurityBundle/Security/Acl/AclProvider.php on line 191, referer: https://www.camdram.net/shows/2016-bugsy-malone/admin/edit #364

Closed
ajf58 opened this Issue Apr 27, 2016 · 1 comment

Comments

Projects
None yet
3 participants
@ajf58
Copy link
Contributor

ajf58 commented Apr 27, 2016

This needs investigating.

@GKFX

This comment has been minimized.

Copy link
Member

GKFX commented Mar 13, 2019

I haven't attempted to test the 2016 code, but by inspection of the code this was fixed in 4ab5a80, and I've used this approve button successfully since then.
The (former) line 191 is inside an if ($persist) loop, and $persist is now true if and only if $ace is non-null.

@GKFX GKFX closed this Mar 13, 2019

@CHTJonas CHTJonas added the invalid label Mar 13, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.