Skip to content

Commit

Permalink
Sales funnel related changes to access_token usage
Browse files Browse the repository at this point in the history
- Creation of new user within sales funnel does not create
new access token. It was never used as sales funnel never
actually logs user in before the payment is confirmed.
- `ReturnPresenter` now always refreshes all user payment
user tokens to contain fresh info about latest subscriptions.

#10
  • Loading branch information
rootpd committed Jun 3, 2020
1 parent a4f66b0 commit c74b898
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions src/presenters/ReturnPresenter.php
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
namespace Crm\PaymentsModule\Presenters;

use Crm\ApplicationModule\Presenters\FrontendPresenter;
use Crm\ApplicationModule\User\UserData;
use Crm\PaymentsModule\Gateways\GatewayAbstract;
use Crm\PaymentsModule\Model\PaymentCompleteRedirectManager;
use Crm\PaymentsModule\Model\PaymentCompleteRedirectResolver;
Expand All @@ -13,6 +12,7 @@
use Crm\PaymentsModule\Repository\PaymentsRepository;
use Crm\UsersModule\Repository\UserMetaRepository;
use Crm\UsersModule\Repository\UsersRepository;
use Crm\UsersModule\User\UserData;

class ReturnPresenter extends FrontendPresenter
{
Expand Down Expand Up @@ -137,9 +137,8 @@ private function processPayment($payment)
}

// update all user tokens with new access data
if ($presenter->getUser()->isLoggedIn()) {
$presenter->userData->refreshUserTokens($presenter->getUser()->getId());
}
$presenter->userData->refreshUserTokens($payment->user_id);

$presenter->paymentLogsRepository->add(
'OK',
"Redirecting to success url with vs '{$payment->variable_symbol}'",
Expand Down

0 comments on commit c74b898

Please sign in to comment.