Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: 5.8.0
-
Fix Version/s: 5.8.1
-
Component/s: None
-
Labels:None
Description
The way contact permission are set and evaluated seems inconsistent. For example Contacts::hasPermission() expects that the contact's client_id be set in the contact_permissions table, while delete() expects the column to be null. And the upgrade script does not set any client_id at all. Also ManagedAccounts::getAccounts seems to expect the column to be null for non account managers.
I think the correct thing to do is to make a script to set the column equal to the contact's client id if the column is null and change the column to be non-null. Then update all other area's to expect the column to be non-null
Activity
| Field | Original Value | New Value |
|---|---|---|
| Sprint | 5.9.0 Sprint 1 [ 182 ] |
| Rank | Ranked higher |
| Assignee | Jonathan Reissmueller [ jonathan ] |
| Status | Open [ 1 ] | In Progress [ 3 ] |
| Security | Private [ 10000 ] |
| Status | In Progress [ 3 ] | In Review [ 5 ] |
| Resolution | Fixed [ 1 ] |
| Remaining Estimate | 0 minutes [ 0 ] | |
| Time Spent | 25 minutes [ 1500 ] | |
| Worklog Id | 16544 [ 16544 ] |
| Status | In Review [ 5 ] | Closed [ 6 ] |
