Opened 3 years ago
Closed 3 years ago
#2412 closed Defect (fixed)
Group ACL update breaks if the group ACL hasn't been set correctly
| Reported by: | brettp | Owned by: | |
|---|---|---|---|
| Priority: | high | Milestone: | Elgg 1.7.2 |
| Component: | Core | Version: | 1.7 |
| Severity: | critical | Keywords: | |
| Cc: | brettp | Difficulty: |
Description
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.
Change History (3)
comment:1 Changed 3 years ago by brettp
- Severity changed from minor to critical
comment:2 Changed 3 years ago by cash
- Type changed from unconfirmed defect to confirmed defect
comment:3 Changed 3 years ago by brettp
- Resolution set to fixed
- Status changed from new to closed
@Cash - Thanks for taking care of this. Confirmed that it's working on the community DB.
Note: See
TracTickets for help on using
tickets.
[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.