hiqdev /
yii2-merchant
This project does not seem to handle request data directly as such no vulnerable execution paths were found.
include, or for example
via PHP's auto-loading mechanism.
These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more
| 1 | <?php |
||
| 2 | /** |
||
| 3 | * Yii2 extension for payment processing with Omnipay, Payum and more later. |
||
| 4 | * |
||
| 5 | * @link https://github.com/hiqdev/yii2-merchant |
||
| 6 | * @package yii2-merchant |
||
| 7 | * @license BSD-3-Clause |
||
| 8 | * @copyright Copyright (c) 2015-2017, HiQDev (http://hiqdev.com/) |
||
| 9 | */ |
||
| 10 | |||
| 11 | namespace hiqdev\yii2\merchant\controllers; |
||
| 12 | |||
| 13 | use hiqdev\yii2\merchant\actions\RequestAction; |
||
| 14 | use hiqdev\yii2\merchant\models\DepositForm; |
||
| 15 | use hiqdev\yii2\merchant\models\DepositRequest; |
||
| 16 | use hiqdev\yii2\merchant\Module; |
||
| 17 | use hiqdev\yii2\merchant\transactions\Transaction; |
||
| 18 | use Yii; |
||
| 19 | use yii\base\InvalidConfigException; |
||
| 20 | use yii\web\BadRequestHttpException; |
||
| 21 | use yii\web\NotFoundHttpException; |
||
| 22 | use yii\web\Response; |
||
| 23 | |||
| 24 | class PayController extends \yii\web\Controller |
||
| 25 | { |
||
| 26 | public function actions() |
||
| 27 | { |
||
| 28 | return array_merge(parent::actions(), [ |
||
| 29 | 'request' => [ |
||
| 30 | 'class' => RequestAction::class |
||
| 31 | ] |
||
| 32 | ]); |
||
| 33 | } |
||
| 34 | |||
| 35 | /** |
||
| 36 | * @return Module|\yii\base\Module |
||
| 37 | */ |
||
| 38 | public function getMerchantModule() |
||
| 39 | { |
||
| 40 | return $this->module; |
||
| 41 | } |
||
| 42 | |||
| 43 | /** |
||
| 44 | * Disable CSRF validation for POST requests we receive from outside |
||
| 45 | * {@inheritdoc} |
||
| 46 | */ |
||
| 47 | public function beforeAction($action) |
||
| 48 | { |
||
| 49 | if (in_array($this->action->id, ['notify', 'return', 'cancel', 'proxy-notification'], true)) { |
||
| 50 | Yii::$app->controller->enableCsrfValidation = false; |
||
| 51 | } |
||
| 52 | |||
| 53 | return parent::beforeAction($action); |
||
| 54 | } |
||
| 55 | |||
| 56 | /** |
||
| 57 | * @return Response |
||
| 58 | */ |
||
| 59 | public function actionCancel() |
||
| 60 | { |
||
| 61 | Yii::$app->session->addFlash('error', Yii::t('merchant', 'Payment failed or cancelled')); |
||
| 62 | |||
| 63 | return $this->redirect($this->getMerchantModule()->previousUrl() ?: ['deposit']); |
||
| 64 | } |
||
| 65 | |||
| 66 | /** |
||
| 67 | * @param string|null $transactionId |
||
| 68 | * |
||
| 69 | * Parameters are NOT required because some merchants may NOT send them, or send in POST or JSON bode. |
||
| 70 | * The main purpose of these parameters is handling of special routes using UrlManager |
||
| 71 | * |
||
| 72 | * @return string |
||
| 73 | * @throws InvalidConfigException |
||
| 74 | */ |
||
| 75 | public function actionReturn(string $transactionId = null) |
||
| 76 | { |
||
| 77 | $transaction = $this->checkNotify($transactionId); |
||
| 78 | if ($transaction === null) { |
||
| 79 | return $this->actionCancel(); |
||
| 80 | } |
||
| 81 | |||
| 82 | return $this->render('return', [ |
||
| 83 | 'transactionId' => $transaction->getId(), |
||
| 84 | ]); |
||
| 85 | } |
||
| 86 | |||
| 87 | /** |
||
| 88 | * @param string $transactionId |
||
| 89 | * @throws BadRequestHttpException |
||
| 90 | * @return array |
||
| 91 | */ |
||
| 92 | public function actionCheckReturn($transactionId) |
||
| 93 | { |
||
| 94 | Yii::$app->response->format = Response::FORMAT_JSON; |
||
| 95 | $transaction = $this->getMerchantModule()->findTransaction($transactionId); |
||
| 96 | |||
| 97 | if ($transaction === null) { |
||
| 98 | throw new NotFoundHttpException('Transaction does not exist'); |
||
| 99 | } |
||
| 100 | |||
| 101 | if ($transaction->getParameter('username') !== $this->getMerchantModule()->getUsername()) { |
||
| 102 | throw new BadRequestHttpException('Access denied', 403); |
||
| 103 | } |
||
| 104 | |||
| 105 | return [ |
||
| 106 | 'status' => $transaction->getSuccess(), |
||
| 107 | 'url' => $transaction->isConfirmed() |
||
| 108 | ? $transaction->getParameter('finishUrl') |
||
| 109 | : $transaction->getParameter('cancelUrl'), |
||
| 110 | ]; |
||
| 111 | } |
||
| 112 | |||
| 113 | /** |
||
| 114 | * Action handles notifications from payment systems, |
||
| 115 | * processes them and report success or error for the payment system. |
||
| 116 | * |
||
| 117 | * @param string|null $transactionId Parameters is NOT required because some merchants may NOT send it, or send in POST or JSON body. |
||
| 118 | * The main purpose of these parameters is handling of special routes using UrlManager |
||
| 119 | * |
||
| 120 | * @return null|string |
||
| 121 | * @throws InvalidConfigException |
||
| 122 | */ |
||
| 123 | public function actionNotify(string $transactionId = null) |
||
| 124 | { |
||
| 125 | $transaction = $this->checkNotify($transactionId); |
||
| 126 | if ($transaction === null) { |
||
| 127 | return 'Unknown transaction'; |
||
| 128 | } |
||
| 129 | |||
| 130 | Yii::$app->response->format = Response::FORMAT_RAW; |
||
| 131 | |||
| 132 | return $transaction->isConfirmed() ? 'OK' : $transaction->getParameter('error'); |
||
| 133 | } |
||
| 134 | |||
| 135 | /** |
||
| 136 | * Check notifications. |
||
| 137 | * TODO: implement actual request check and proper handling. |
||
| 138 | * |
||
| 139 | * @param string|null $transactionId Parameters is NOT required because some merchants may NOT send it, or send in POST or JSON body. |
||
| 140 | * The main purpose of these parameters is handling of special routes using UrlManager |
||
| 141 | * |
||
| 142 | * @return Transaction|null |
||
| 143 | * @throws InvalidConfigException |
||
| 144 | */ |
||
| 145 | public function checkNotify(string $transactionId = null): ?Transaction |
||
|
0 ignored issues
–
show
|
|||
| 146 | { |
||
| 147 | throw new InvalidConfigException('Method checkNotify must be implemented'); |
||
| 148 | } |
||
| 149 | |||
| 150 | public function actionProxyNotification() |
||
| 151 | { |
||
| 152 | throw new InvalidConfigException('Method actionProxyNotification must be implemented'); |
||
| 153 | } |
||
| 154 | |||
| 155 | public function actionDeposit() |
||
| 156 | { |
||
| 157 | $merchantModule = $this->getMerchantModule(); |
||
| 158 | |||
| 159 | $model = Yii::createObject($merchantModule->depositFromClass); |
||
| 160 | $request = Yii::$app->request; |
||
| 161 | if ($model->load($request->isPost ? $request->post() : $request->get()) && $model->validate()) { |
||
| 162 | return $this->renderDeposit($model); |
||
| 163 | } |
||
| 164 | |||
| 165 | return $this->render('deposit-form', [ |
||
| 166 | 'model' => $model, |
||
| 167 | 'availableMerchants' => $this->getMerchantModule()->getPurchaseRequestCollection()->getItems(), |
||
| 168 | 'availableCurrencies' => $this->getMerchantModule()->getAvailableCurrenciesCollection()->getList(), |
||
| 169 | ]); |
||
| 170 | } |
||
| 171 | |||
| 172 | /** |
||
| 173 | * Renders depositing buttons for given request data. |
||
| 174 | * |
||
| 175 | * @param DepositForm $form request data |
||
| 176 | * @return \yii\web\Response |
||
| 177 | */ |
||
| 178 | public function renderDeposit($form) |
||
| 179 | { |
||
| 180 | $request = new DepositRequest(); |
||
| 181 | $request->amount = $form->amount; |
||
|
0 ignored issues
–
show
The property
$amount was declared of type string, but $form->amount is of type double. Maybe add a type cast?
This check looks for assignments to scalar types that may be of the wrong type. To ensure the code behaves as expected, it may be a good idea to add an explicit type cast. $answer = 42;
$correct = false;
$correct = (bool) $answer;
Loading history...
|
|||
| 182 | $request->currency = $form->currency; |
||
| 183 | $request->finishUrl = $form->finishUrl; |
||
| 184 | |||
| 185 | $requests = $this->getMerchantModule()->getPurchaseRequestCollection($request)->getItems(); |
||
| 186 | |||
| 187 | return $this->render('deposit', [ |
||
| 188 | 'requests' => $requests, |
||
| 189 | 'depositForm' => $form |
||
| 190 | ]); |
||
| 191 | } |
||
| 192 | } |
||
| 193 |
This check looks from parameters that have been defined for a function or method, but which are not used in the method body.