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

civicrm_member_roles_sync_user breaks with Drupal membership-based integration

    Details

    • Type: Bug
    • Status: Done/Fixed
    • Priority: Minor
    • Resolution: Fixed/Completed
    • Affects Version/s: 4.1.2
    • Fix Version/s: 4.3.0
    • Labels:
      None

      Description

      The following function references the variable $op which is undefined, and causes an E_NOTICE warning to be shown to end users. My fix was to remove the if condition entirely, but not knowing the intended functionality, I haven't attached a patch.

      File: sites/all/modules/civicrm/drupal/modules/civicrm_member_roles/civicrm_member_roles.module

      function civicrm_member_roles_sync_user( $account)
      {
      if ( ! civicrm_initialize( ) )

      { return; }

      if ( variable_get('civicrm_member_roles_sync_method', 0) == 0)

      { _civicrm_member_roles_sync($account->uid); }

      if (variable_get('civicrm_member_roles_sync_method', 0) == 3) {
      if ($op == 'insert')

      { _civicrm_member_roles_sync($account->uid); }

      }
      }

        Attachments

          Activity

            People

            • Assignee:
              lobo Donald A. Lobo
              Reporter:
              stefan Stefan
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - 5 minutes
                5m
                Remaining:
                Remaining Estimate - 5 minutes
                5m
                Logged:
                Time Spent - Not Specified
                Not Specified