Previous method didn't work, use new method
This commit is contained in:
@@ -1057,10 +1057,17 @@ class OrganisationMembersController extends SimpleController
|
||||
/** @var \UserFrosting\Sprinkle\Core\Alert\AlertStream $ms */
|
||||
$ms = $this->ci->alerts;
|
||||
|
||||
/** @var \UserFrosting\Sprinkle\Organisations\Repository\OrganisationMembershipApprovalRepository */
|
||||
$tokenRepo = $this->ci->repoOrganisationMembershipApproval;
|
||||
|
||||
|
||||
if ($manual && !$tokenRepo->exists($tokenOwner)) {
|
||||
$tokenRepo->create($tokenOwner, 1);
|
||||
}
|
||||
|
||||
// Try and complete the token, bail if not found
|
||||
$verification = $this->ci->repoOrganisationMembershipApproval->completeForOwner($tokenOwner, ['approved' => true, 'approver_id' => $currentUser->id]);
|
||||
if (!$verification && !$manual) {
|
||||
$verification = $tokenRepo->completeForOwner($tokenOwner, ['approved' => true, 'approver_id' => $currentUser->id]);
|
||||
if (!$verification) {
|
||||
$ms->addMessageTranslated('danger', 'ORGANISATION.JOIN_REQUEST.TOKEN_NOT_FOUND');
|
||||
return false;
|
||||
}
|
||||
@@ -1091,10 +1098,17 @@ class OrganisationMembersController extends SimpleController
|
||||
/** @var \UserFrosting\Sprinkle\Core\Alert\AlertStream $ms */
|
||||
$ms = $this->ci->alerts;
|
||||
|
||||
/** @var \UserFrosting\Sprinkle\Organisations\Repository\OrganisationMembershipApprovalRepository */
|
||||
$tokenRepo = $this->ci->repoOrganisationMembershipApproval;
|
||||
|
||||
|
||||
if ($manual && !$tokenRepo->exists($tokenOwner)) {
|
||||
$tokenRepo->create($tokenOwner, 1);
|
||||
}
|
||||
|
||||
// Try and complete the token, bail if not found
|
||||
$verification = $this->ci->repoOrganisationMembershipApproval->completeForOwner($tokenOwner, ['approved' => false, 'approver_id' => $currentUser->id]);
|
||||
if (!$verification && !$manual) {
|
||||
$verification = $tokenRepo->completeForOwner($tokenOwner, ['approved' => false, 'approver_id' => $currentUser->id]);
|
||||
if (!$verification) {
|
||||
$ms->addMessageTranslated('danger', 'ORGANISATION.JOIN_REQUEST.TOKEN_NOT_FOUND');
|
||||
return false;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user