CRM-15315 CiviGroup Roles Sync rules listing has blank groups

          Details

          • Documentation Required?:
            None

            Description

            See http://forum.civicrm.org/index.php?topic=29404.0, "CiviGroup Roles Sync not picking up Group".

            Symptom:
            Missing group names in the CiviGroup Roles Sync list of association rules, /admin/config/civicrm/civicrm_group_roles .

            To replicate:
            1. Have more than 25 groups in Civi.
            2. Enable CiviGroup Roles Sync.
            3. Create some association rules.
            4. View the association rules list.

            Result:
            Names are missing for groups that are not in the 25 returned by an API v3 Group get.

            Fix:
            Ensure that the API v3 Group get retrieves all groups.

            Patch attached, against 4.4.6 Drupal 7 version.

            If a pull request is preferred to a patch, which branch(es) should it be against, for the D6 & D7 versions?

              Attachments

                Activity

                [CRM-15315] CiviGroup Roles Sync rules listing has blank groups
                Donald A. Lobo added a comment -

                hey dave:

                can u please submit the PR against the 4.5 branch for D6 and D7. If you can do it today/tomorrow, we can include it in the 4.5.0 release

                thaxn

                lobo

                Dave Jenkins added a comment -

                Hi Lobo,

                PR for D7: https://github.com/civicrm/civicrm-drupal/pull/162
                PR for D6: https://github.com/civicrm/civicrm-drupal/pull/163

                Hope those are OK.

                Cheers,

                Dave

                Kurund Jalmi added a comment -

                thanks

                  People

                  • Assignee:
                    Kurund Jalmi
                    Reporter:
                    Dave Jenkins

                    Dates

                    • Created:
                      Updated:
                      Resolved: