Skip to content
Snippets Groups Projects

Handled user account failure and sender acknowledgement

Merged Monish Deb requested to merge fix-username into master

Created by: Edzelopez

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Monish Deb
  • Monish Deb
  • Monish Deb
    Monish Deb @monish started a thread on commit ec6bc715
  • 354 'return' => 'contact_id_a',
    355 ]);
    356 if ($relationship['count'] > 0 && !empty($relationship['values'][$relationship['id']]['contact_id_a'])) {
    357 $primaryCid = $relationship['values'][$relationship['id']]['contact_id_a'];
    358 }
    356 359 // Create drupal account if not exists.
    357 self::createUserAccount($cid);
    360 if (!self::createUserAccount($primaryCid)) {
    361 // Set status message indicating that user account creation was unsuccessful.
    362 $userUrl = CRM_Utils_System::url('civicrm/contact/view/useradd', "reset=1&action=add&cid=$primaryCid");
    363 CRM_Core_Session::setStatus(ts('There was an error creating the user account. Please proceed to add one here: %1', [1 => $userUrl]), ts('Warning'), 'alert');
    364 }
    358 365
    359 366 // Send Mail
    360 self::sendMessage($cid, APPROVED_MESSAGE);
    367 self::sendMessage($primaryCid, APPROVED_MESSAGE);
    • Author Owner

      Created by: seamuslee001

      Can confirm that this change causes the approval email to go to the primary contact now correctly

  • Author Owner

    Created by: seamuslee001

    Changes look good to me

  • Author Owner

    Created by: seamuslee001

    Review: Approved

  • Author Owner

    Merged by: Edzelopez at 2020-05-12 08:46:19 UTC

  • Please register or sign in to reply
    Loading