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

Search menu visible even when a user has not access to any contacts.

          Details

          • Type: Improvement
          • Status: Done/Fixed
          • Priority: Trivial
          • Resolution: Fixed/Completed
          • Affects Version/s: 4.7
          • Fix Version/s: None
          • Component/s: Core CiviCRM
          • Labels:
          • Versioning Impact:
            Patch (backwards-compatible bug fixes)
          • Documentation Required?:
            None
          • Funding Source:
            Contributed Code
          • Verified?:
            No

            Description

            Issue CRM-19934 deals with performance improvements however the suggested fix is huge and therefor I am trying to break it up smaller chunks.

            In [#CRM-19934) I made the search menu visible even when the user has not access to any contact records. The reason for doing is that I did a refactor of CRM_Core_Permission class and discovered that the function giveMeAllAcls is used to determine whether the search menu should be visible or not.
            I would suggest that is based on permission and not whether the user is allowed to see any contacts.

              Attachments

                Issue Links

                supplements

                Improvement - An improvement or enhancement to an existing feature or task. CRM-19934 Performance improvement on civicrm_acl_contact_cache

                • Major - Major loss of function.
                • Open
                links to

                Web Link PR: civicrm-core#10040: Fixed CRM-20333: search menu visible even when a user does not has access to any contacts.

                  Activity

                    People

                    • Assignee:
                      Unassigned
                      Reporter:
                      grapio Jaap Jansma
                    • Votes:
                      0 Vote for this issue
                      Watchers:
                      3 Start watching this issue

                      Dates

                      • Created:
                        Updated:
                        Resolved: