Uploaded image for project: 'CiviCRM'
  1. CiviCRM
  2. CRM-15111

"Available for Dashboard" doesn't respect ACL Group/Role

    Details

    • Type: Bug
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 4.4.6, 4.5
    • Fix Version/s: Unscheduled
    • Component/s: CiviReport
    • Labels:
      None
    • Versioning Impact:
      Patch (backwards-compatible bug fixes)
    • Documentation Required?:
      None

      Description

      Reports whose access is restricted via the "permission" field are hidden both in the "All Reports" list (/civicrm/report/list?reset=1), and also are not available for the dashboard. However, reports who access is restricted via "ACL Group/Role" show up as available for the dashboard, a potential information leakage.

      I did some digging, and it seems that the issue is at the schema level - reports have a "permission" and "grouprole" field, but dashboard records only have a "permission" field. To implement correct behavior, we'd need to add "grouprole" to the dashboard entity, store it when saving the report, and modify Core_BAO_Dashboard::CheckPermissions.

      I'd take a first crack at it, but there's a lot here I've never done! However, I'm available for testing, and I could implement the CheckPermissions portion if the rest happened.

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              palantejon Jon K Goldberg
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated: