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

Group ACL update breaks if the group ACL hasn't been set correctly (Trac #2412) #2412

Closed
elgg-gitbot opened this issue Feb 16, 2013 · 2 comments

Comments

@elgg-gitbot
Copy link

Original ticket http://trac.elgg.org/ticket/2412 on 40639976-01-16 by brettp, assigned to unknown.

Elgg version: 1.7

If the group ACL hasn't been set, code upgrade 2010062301 throws a DB exception. I ran into this when testing 1.7.2 with community's database. The easiest solution is to check that the group ACL is valid before assigning the owner. If it doesn't exist, create it and then let 2010062302 handle assigning group owners.

@elgg-gitbot
Copy link
Author

cash wrote on 40646011-08-09

[svn:6857] should fix this. I going ahead and running the query and catching the exception if it doesn't work. In the catch code, I create the acl and reset the group_acl metadata. I'll leave it for Brett to test and close on the community site clone.

@elgg-gitbot
Copy link
Author

brettp wrote on 40649502-01-10

Cash - Thanks for taking care of this. Confirmed that it's working on the community DB.

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

No branches or pull requests

1 participant