Duplicate code is one of the most pungent code smells. A rule that is often used is to re-structure code once it is duplicated in three or more places.
Common duplication problems, and corresponding solutions are:
Complex classes like PaymentManager often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes. You can also have a look at the cohesion graph to spot any un-connected, or weakly-connected components.
Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.
While breaking up the class, it is a good idea to analyze how other classes use PaymentManager, and based on these observations, apply Extract Interface, too.
1 | <?php |
||
69 | class PaymentManager implements PaymentManagerInterface |
||
70 | { |
||
71 | const LOG_TYPE_API_LOG = 'SpyPaymentPayoneApiLog'; |
||
72 | const LOG_TYPE_TRANSACTION_STATUS_LOG = 'SpyPaymentPayoneTransactionStatusLog'; |
||
73 | const ERROR_ACCESS_DENIED_MESSAGE = 'Access denied'; |
||
74 | |||
75 | /** |
||
76 | * @var \SprykerEco\Zed\Payone\Business\Api\Adapter\AdapterInterface |
||
77 | */ |
||
78 | protected $executionAdapter; |
||
79 | |||
80 | /** |
||
81 | * @var \SprykerEco\Zed\Payone\Persistence\PayoneQueryContainerInterface |
||
82 | */ |
||
83 | protected $queryContainer; |
||
84 | |||
85 | /** |
||
86 | * @var \Generated\Shared\Transfer\PayoneStandardParameterTransfer |
||
87 | */ |
||
88 | protected $standardParameter; |
||
89 | |||
90 | /** |
||
91 | * @var \SprykerEco\Zed\Payone\Business\SequenceNumber\SequenceNumberProviderInterface |
||
92 | */ |
||
93 | protected $sequenceNumberProvider; |
||
94 | |||
95 | /** |
||
96 | * @var \SprykerEco\Shared\Payone\Dependency\ModeDetectorInterface |
||
97 | */ |
||
98 | protected $modeDetector; |
||
99 | |||
100 | /** |
||
101 | * @var \SprykerEco\Zed\Payone\Business\Key\HmacGeneratorInterface |
||
102 | */ |
||
103 | protected $hashGenerator; |
||
104 | |||
105 | /** |
||
106 | * @var \SprykerEco\Zed\Payone\Business\Key\UrlHmacGenerator |
||
107 | */ |
||
108 | protected $urlHmacGenerator; |
||
109 | |||
110 | /** |
||
111 | * @var \SprykerEco\Zed\Payone\Business\Payment\PaymentMethodMapperInterface[] |
||
112 | */ |
||
113 | protected $registeredMethodMappers; |
||
114 | |||
115 | /** |
||
116 | * @param \SprykerEco\Zed\Payone\Business\Api\Adapter\AdapterInterface $executionAdapter |
||
117 | * @param \SprykerEco\Zed\Payone\Persistence\PayoneQueryContainerInterface $queryContainer |
||
118 | * @param \Generated\Shared\Transfer\PayoneStandardParameterTransfer $standardParameter |
||
119 | * @param \SprykerEco\Zed\Payone\Business\Key\HashGenerator $hashGenerator |
||
120 | * @param \SprykerEco\Zed\Payone\Business\SequenceNumber\SequenceNumberProviderInterface $sequenceNumberProvider |
||
121 | * @param \SprykerEco\Shared\Payone\Dependency\ModeDetectorInterface $modeDetector |
||
122 | * @param \SprykerEco\Zed\Payone\Business\Key\HmacGeneratorInterface $urlHmacGenerator |
||
123 | */ |
||
124 | public function __construct( |
||
125 | AdapterInterface $executionAdapter, |
||
126 | PayoneQueryContainerInterface $queryContainer, |
||
127 | PayoneStandardParameterTransfer $standardParameter, |
||
128 | HashGenerator $hashGenerator, |
||
129 | SequenceNumberProviderInterface $sequenceNumberProvider, |
||
130 | ModeDetectorInterface $modeDetector, |
||
131 | HmacGeneratorInterface $urlHmacGenerator |
||
132 | ) { |
||
133 | |||
134 | $this->executionAdapter = $executionAdapter; |
||
135 | $this->queryContainer = $queryContainer; |
||
136 | $this->standardParameter = $standardParameter; |
||
137 | $this->hashGenerator = $hashGenerator; |
||
|
|||
138 | $this->sequenceNumberProvider = $sequenceNumberProvider; |
||
139 | $this->modeDetector = $modeDetector; |
||
140 | $this->urlHmacGenerator = $urlHmacGenerator; |
||
141 | } |
||
142 | |||
143 | /** |
||
144 | * @param \SprykerEco\Zed\Payone\Business\Payment\PaymentMethodMapperInterface $paymentMethodMapper |
||
145 | * |
||
146 | * @return void |
||
147 | */ |
||
148 | public function registerPaymentMethodMapper(PaymentMethodMapperInterface $paymentMethodMapper) |
||
149 | { |
||
150 | $paymentMethodMapper->setStandardParameter($this->standardParameter); |
||
151 | $paymentMethodMapper->setSequenceNumberProvider($this->sequenceNumberProvider); |
||
152 | $paymentMethodMapper->setUrlHmacGenerator($this->urlHmacGenerator); |
||
153 | $this->registeredMethodMappers[$paymentMethodMapper->getName()] = $paymentMethodMapper; |
||
154 | } |
||
155 | |||
156 | /** |
||
157 | * @param string $name |
||
158 | * |
||
159 | * @return \SprykerEco\Zed\Payone\Business\Payment\PaymentMethodMapperInterface|null |
||
160 | */ |
||
161 | protected function findPaymentMethodMapperByName($name) |
||
162 | { |
||
163 | if (array_key_exists($name, $this->registeredMethodMappers)) { |
||
164 | return $this->registeredMethodMappers[$name]; |
||
165 | } |
||
166 | |||
167 | return null; |
||
168 | } |
||
169 | |||
170 | /** |
||
171 | * @param string $paymentMethodName |
||
172 | * |
||
173 | * @throws \SprykerEco\Zed\Payone\Business\Exception\InvalidPaymentMethodException |
||
174 | * |
||
175 | * @return \SprykerEco\Zed\Payone\Business\Payment\PaymentMethodMapperInterface |
||
176 | */ |
||
177 | protected function getRegisteredPaymentMethodMapper($paymentMethodName) |
||
178 | { |
||
179 | $paymentMethodMapper = $this->findPaymentMethodMapperByName($paymentMethodName); |
||
180 | if ($paymentMethodMapper === null) { |
||
181 | throw new InvalidPaymentMethodException( |
||
182 | sprintf('No registered payment method mapper found for given method name %s', $paymentMethodName) |
||
183 | ); |
||
184 | } |
||
185 | |||
186 | return $paymentMethodMapper; |
||
187 | } |
||
188 | |||
189 | /** |
||
190 | * @param \Generated\Shared\Transfer\OrderTransfer $orderTransfer |
||
191 | * |
||
192 | * @return \Generated\Shared\Transfer\AuthorizationResponseTransfer |
||
193 | */ |
||
194 | public function authorizePayment(OrderTransfer $orderTransfer) |
||
195 | { |
||
196 | $paymentEntity = $this->getPaymentEntity($orderTransfer->getIdSalesOrder()); |
||
197 | $paymentMethodMapper = $this->getPaymentMethodMapper($paymentEntity); |
||
198 | $requestContainer = $paymentMethodMapper->mapPaymentToAuthorization($paymentEntity, $orderTransfer); |
||
199 | $requestContainer = $this->prepareOrderItems($orderTransfer, $requestContainer); |
||
200 | $responseContainer = $this->performAuthorizationRequest($paymentEntity, $requestContainer); |
||
201 | |||
202 | $responseMapper = new AuthorizationResponseMapper(); |
||
203 | $responseTransfer = $responseMapper->getAuthorizationResponseTransfer($responseContainer); |
||
204 | |||
205 | return $responseTransfer; |
||
206 | } |
||
207 | |||
208 | /** |
||
209 | * @param int $idSalesOrder |
||
210 | * |
||
211 | * @return \Generated\Shared\Transfer\AuthorizationResponseTransfer |
||
212 | */ |
||
213 | public function preAuthorizePayment($idSalesOrder) |
||
214 | { |
||
215 | $paymentEntity = $this->getPaymentEntity($idSalesOrder); |
||
216 | $paymentMethodMapper = $this->getPaymentMethodMapper($paymentEntity); |
||
217 | $requestContainer = $paymentMethodMapper->mapPaymentToPreAuthorization($paymentEntity); |
||
218 | $responseContainer = $this->performAuthorizationRequest($paymentEntity, $requestContainer); |
||
219 | |||
220 | $responseMapper = new AuthorizationResponseMapper(); |
||
221 | $responseTransfer = $responseMapper->getAuthorizationResponseTransfer($responseContainer); |
||
222 | |||
223 | return $responseTransfer; |
||
224 | } |
||
225 | |||
226 | /** |
||
227 | * @param \Orm\Zed\Payone\Persistence\SpyPaymentPayone $paymentEntity |
||
228 | * @param \SprykerEco\Zed\Payone\Business\Api\Request\Container\AuthorizationContainerInterface $requestContainer |
||
229 | * |
||
230 | * @return \SprykerEco\Zed\Payone\Business\Api\Response\Container\AuthorizationResponseContainer |
||
231 | */ |
||
232 | protected function performAuthorizationRequest(SpyPaymentPayone $paymentEntity, AuthorizationContainerInterface $requestContainer) |
||
233 | { |
||
234 | $this->setStandardParameter($requestContainer); |
||
235 | |||
236 | $apiLogEntity = $this->initializeApiLog($paymentEntity, $requestContainer); |
||
237 | $rawResponse = $this->executionAdapter->sendRequest($requestContainer); |
||
238 | $responseContainer = new AuthorizationResponseContainer($rawResponse); |
||
239 | $this->updatePaymentAfterAuthorization($paymentEntity, $responseContainer); |
||
240 | $this->updateApiLogAfterAuthorization($apiLogEntity, $responseContainer); |
||
241 | $this->updatePaymentDetailAfterAuthorization($paymentEntity, $responseContainer); |
||
242 | |||
243 | return $responseContainer; |
||
244 | } |
||
245 | |||
246 | /** |
||
247 | * @param \Orm\Zed\Payone\Persistence\SpyPaymentPayone $paymentEntity |
||
248 | * |
||
249 | * @return \SprykerEco\Zed\Payone\Business\Payment\PaymentMethodMapperInterface |
||
250 | */ |
||
251 | protected function getPaymentMethodMapper(SpyPaymentPayone $paymentEntity) |
||
252 | { |
||
253 | return $this->getRegisteredPaymentMethodMapper($paymentEntity->getPaymentMethod()); |
||
254 | } |
||
255 | |||
256 | /** |
||
257 | * @param int $orderId |
||
258 | * |
||
259 | * @return \Orm\Zed\Payone\Persistence\SpyPaymentPayone |
||
260 | */ |
||
261 | protected function getPaymentEntity($orderId) |
||
262 | { |
||
263 | return $this->queryContainer->createPaymentById($orderId)->findOne(); |
||
264 | } |
||
265 | |||
266 | /** |
||
267 | * @param \Generated\Shared\Transfer\PayoneCaptureTransfer $captureTransfer |
||
268 | * |
||
269 | * @return \Generated\Shared\Transfer\CaptureResponseTransfer |
||
270 | */ |
||
271 | public function capturePayment(PayoneCaptureTransfer $captureTransfer): CaptureResponseTransfer |
||
272 | { |
||
273 | $paymentEntity = $this->getPaymentEntity($captureTransfer->getPayment()->getFkSalesOrder()); |
||
274 | $paymentMethodMapper = $this->getPaymentMethodMapper($paymentEntity); |
||
275 | |||
276 | $requestContainer = $paymentMethodMapper->mapPaymentToCapture($paymentEntity); |
||
277 | $requestContainer = $this->prepareOrderItems($captureTransfer->getOrder(), $requestContainer); |
||
278 | |||
279 | if (!empty($captureTransfer->getSettleaccount())) { |
||
280 | $businnessContainer = new BusinessContainer(); |
||
281 | $businnessContainer->setSettleAccount($captureTransfer->getSettleaccount()); |
||
282 | $requestContainer->setBusiness($businnessContainer); |
||
283 | } |
||
284 | |||
285 | $this->setStandardParameter($requestContainer); |
||
286 | |||
287 | $apiLogEntity = $this->initializeApiLog($paymentEntity, $requestContainer); |
||
288 | |||
289 | $rawResponse = $this->executionAdapter->sendRequest($requestContainer); |
||
290 | $responseContainer = new CaptureResponseContainer($rawResponse); |
||
291 | |||
292 | $this->updateApiLogAfterCapture($apiLogEntity, $responseContainer); |
||
293 | |||
294 | $responseMapper = new CaptureResponseMapper(); |
||
295 | $responseTransfer = $responseMapper->getCaptureResponseTransfer($responseContainer); |
||
296 | |||
297 | return $responseTransfer; |
||
298 | } |
||
299 | |||
300 | /** |
||
301 | * @param int $idPayment |
||
302 | * |
||
303 | * @return \Generated\Shared\Transfer\DebitResponseTransfer |
||
304 | */ |
||
305 | public function debitPayment($idPayment) |
||
326 | |||
327 | /** |
||
328 | * @param \Generated\Shared\Transfer\PayoneCreditCardTransfer $creditCardData |
||
329 | * |
||
330 | * @return \Generated\Shared\Transfer\CreditCardCheckResponseTransfer |
||
331 | */ |
||
332 | public function creditCardCheck(PayoneCreditCardTransfer $creditCardData) |
||
348 | |||
349 | /** |
||
350 | * @param \Generated\Shared\Transfer\PayoneBankAccountCheckTransfer $bankAccountCheckTransfer |
||
351 | * |
||
352 | * @return \Generated\Shared\Transfer\PayoneBankAccountCheckTransfer |
||
353 | */ |
||
354 | public function bankAccountCheck(PayoneBankAccountCheckTransfer $bankAccountCheckTransfer) |
||
371 | |||
372 | /** |
||
373 | * @param \Generated\Shared\Transfer\PayoneManageMandateTransfer $manageMandateTransfer |
||
374 | * |
||
375 | * @return \Generated\Shared\Transfer\PayoneManageMandateTransfer |
||
376 | */ |
||
377 | public function manageMandate(PayoneManageMandateTransfer $manageMandateTransfer) |
||
398 | |||
399 | /** |
||
400 | * @param \Generated\Shared\Transfer\PayoneGetFileTransfer $getFileTransfer |
||
401 | * |
||
402 | * @return \Generated\Shared\Transfer\PayoneGetFileTransfer |
||
403 | */ |
||
404 | View Code Duplication | public function getFile(PayoneGetFileTransfer $getFileTransfer) |
|
431 | |||
432 | /** |
||
433 | * @param \Generated\Shared\Transfer\PayoneGetInvoiceTransfer $getInvoiceTransfer |
||
434 | * |
||
435 | * @return \Generated\Shared\Transfer\PayoneGetInvoiceTransfer |
||
436 | */ |
||
437 | View Code Duplication | public function getInvoice(PayoneGetInvoiceTransfer $getInvoiceTransfer) |
|
463 | |||
464 | /** |
||
465 | * @param \Generated\Shared\Transfer\PayoneGetSecurityInvoiceTransfer $getSecurityInvoiceTransfer |
||
466 | * |
||
467 | * @return \Generated\Shared\Transfer\PayoneGetSecurityInvoiceTransfer |
||
468 | */ |
||
469 | View Code Duplication | public function getSecurityInvoice(PayoneGetSecurityInvoiceTransfer $getSecurityInvoiceTransfer): PayoneGetSecurityInvoiceTransfer |
|
495 | |||
496 | /** |
||
497 | * @param int $transactionId |
||
498 | * |
||
499 | * @return \SprykerEco\Zed\Payone\Business\Api\Response\Container\GetInvoiceResponseContainer |
||
500 | */ |
||
501 | public function getInvoiceTitle($transactionId) |
||
509 | |||
510 | /** |
||
511 | * @param \Generated\Shared\Transfer\PayoneRefundTransfer $refundTransfer |
||
512 | * |
||
513 | * @return \Generated\Shared\Transfer\RefundResponseTransfer |
||
514 | */ |
||
515 | public function refundPayment(PayoneRefundTransfer $refundTransfer) |
||
516 | { |
||
517 | $payonePaymentTransfer = $refundTransfer->getPayment(); |
||
518 | |||
519 | $paymentEntity = $this->getPaymentEntity($payonePaymentTransfer->getFkSalesOrder()); |
||
538 | |||
539 | /** |
||
540 | * @param \Generated\Shared\Transfer\OrderTransfer $orderTransfer |
||
541 | * |
||
542 | * @return \Generated\Shared\Transfer\PayonePaymentTransfer |
||
543 | */ |
||
544 | protected function getPayment(OrderTransfer $orderTransfer) |
||
558 | |||
559 | /** |
||
560 | * @param \Orm\Zed\Payone\Persistence\SpyPaymentPayone $paymentEntity |
||
561 | * @param \SprykerEco\Zed\Payone\Business\Api\Response\Container\AuthorizationResponseContainer $responseContainer |
||
562 | * |
||
563 | * @return void |
||
564 | */ |
||
565 | protected function updatePaymentAfterAuthorization(SpyPaymentPayone $paymentEntity, AuthorizationResponseContainer $responseContainer) |
||
570 | |||
571 | /** |
||
572 | * @param string $transactionId |
||
573 | * |
||
574 | * @return \Orm\Zed\Payone\Persistence\SpyPaymentPayone |
||
575 | */ |
||
576 | protected function findPaymentByTransactionId($transactionId) |
||
580 | |||
581 | /** |
||
582 | * @param string $invoiceTitle |
||
583 | * @param int $customerId |
||
584 | * |
||
585 | * @return \Orm\Zed\Payone\Persistence\SpyPaymentPayoneQuery |
||
586 | */ |
||
587 | protected function findPaymentByInvoiceTitleAndCustomerId($invoiceTitle, $customerId) |
||
591 | |||
592 | /** |
||
593 | * @param string $fileReference |
||
594 | * @param int $customerId |
||
595 | * |
||
596 | * @return \Orm\Zed\Payone\Persistence\SpyPaymentPayoneQuery |
||
597 | */ |
||
598 | protected function findPaymentByFileReferenceAndCustomerId($fileReference, $customerId) |
||
602 | |||
603 | /** |
||
604 | * @param \Orm\Zed\Payone\Persistence\SpyPaymentPayone $paymentEntity |
||
605 | * @param \SprykerEco\Zed\Payone\Business\Api\Request\Container\AbstractRequestContainer $container |
||
606 | * |
||
607 | * @return \Orm\Zed\Payone\Persistence\SpyPaymentPayoneApiLog |
||
608 | */ |
||
609 | protected function initializeApiLog(SpyPaymentPayone $paymentEntity, AbstractRequestContainer $container) |
||
626 | |||
627 | /** |
||
628 | * @param \Orm\Zed\Payone\Persistence\SpyPaymentPayoneApiLog $apiLogEntity |
||
629 | * @param \SprykerEco\Zed\Payone\Business\Api\Response\Container\AuthorizationResponseContainer $responseContainer |
||
630 | * |
||
631 | * @return void |
||
632 | */ |
||
633 | protected function updateApiLogAfterAuthorization(SpyPaymentPayoneApiLog $apiLogEntity, AuthorizationResponseContainer $responseContainer) |
||
647 | |||
648 | /** |
||
649 | * @param \Orm\Zed\Payone\Persistence\SpyPaymentPayoneApiLog $apiLogEntity |
||
650 | * @param \SprykerEco\Zed\Payone\Business\Api\Response\Container\CaptureResponseContainer $responseContainer |
||
651 | * |
||
652 | * @return void |
||
653 | */ |
||
654 | View Code Duplication | protected function updateApiLogAfterCapture(SpyPaymentPayoneApiLog $apiLogEntity, CaptureResponseContainer $responseContainer) |
|
665 | |||
666 | /** |
||
667 | * @param \Orm\Zed\Payone\Persistence\SpyPaymentPayoneApiLog $apiLogEntity |
||
668 | * @param \SprykerEco\Zed\Payone\Business\Api\Response\Container\DebitResponseContainer $responseContainer |
||
669 | * |
||
670 | * @return void |
||
671 | */ |
||
672 | View Code Duplication | protected function updateApiLogAfterDebit(SpyPaymentPayoneApiLog $apiLogEntity, DebitResponseContainer $responseContainer) |
|
683 | |||
684 | /** |
||
685 | * @param \Orm\Zed\Payone\Persistence\SpyPaymentPayoneApiLog $apiLogEntity |
||
686 | * @param \SprykerEco\Zed\Payone\Business\Api\Response\Container\RefundResponseContainer $responseContainer |
||
687 | * |
||
688 | * @return void |
||
689 | */ |
||
690 | View Code Duplication | protected function updateApiLogAfterRefund(SpyPaymentPayoneApiLog $apiLogEntity, RefundResponseContainer $responseContainer) |
|
701 | |||
702 | /** |
||
703 | * @param \SprykerEco\Zed\Payone\Business\Api\Request\Container\AbstractRequestContainer $container |
||
704 | * |
||
705 | * @return void |
||
706 | */ |
||
707 | protected function setStandardParameter(AbstractRequestContainer $container) |
||
720 | |||
721 | /** |
||
722 | * @param \SprykerEco\Zed\Payone\Business\Api\Response\Container\AbstractResponseContainer $container |
||
723 | * |
||
724 | * @return void |
||
725 | */ |
||
726 | protected function setAccessDeniedError(AbstractResponseContainer $container) |
||
732 | |||
733 | /** |
||
734 | * @param int $idOrder |
||
735 | * |
||
736 | * @return \Generated\Shared\Transfer\PaymentDetailTransfer |
||
737 | */ |
||
738 | View Code Duplication | public function getPaymentDetail($idOrder) |
|
747 | |||
748 | /** |
||
749 | * Gets payment logs (both api and transaction status) for specific orders in chronological order. |
||
750 | * |
||
751 | * @param \Propel\Runtime\Collection\ObjectCollection|\ArrayObject $orders |
||
752 | * |
||
753 | * @return \Generated\Shared\Transfer\PayonePaymentLogCollectionTransfer |
||
754 | */ |
||
755 | public function getPaymentLogs($orders) |
||
789 | |||
790 | /** |
||
791 | * @param \Generated\Shared\Transfer\PayoneCreditCardCheckRequestDataTransfer $creditCardCheckRequestDataTransfer |
||
792 | * |
||
793 | * @return array |
||
794 | */ |
||
795 | public function getCreditCardCheckRequestData(PayoneCreditCardCheckRequestDataTransfer $creditCardCheckRequestDataTransfer) |
||
805 | |||
806 | /** |
||
807 | * @param \Generated\Shared\Transfer\OrderTransfer $orderTransfer |
||
808 | * |
||
809 | * @return bool |
||
810 | */ |
||
811 | public function isRefundPossible(OrderTransfer $orderTransfer) |
||
823 | |||
824 | /** |
||
825 | * @param \Generated\Shared\Transfer\OrderTransfer $orderTransfer |
||
826 | * |
||
827 | * @return bool |
||
828 | */ |
||
829 | public function isPaymentDataRequired(OrderTransfer $orderTransfer) |
||
847 | |||
848 | /** |
||
849 | * @param \Generated\Shared\Transfer\QuoteTransfer $quoteTransfer |
||
850 | * @param \Generated\Shared\Transfer\CheckoutResponseTransfer $checkoutResponse |
||
851 | * |
||
852 | * @return \Generated\Shared\Transfer\CheckoutResponseTransfer |
||
853 | */ |
||
854 | public function postSaveHook(QuoteTransfer $quoteTransfer, CheckoutResponseTransfer $checkoutResponse) |
||
880 | |||
881 | /** |
||
882 | * @param \Generated\Shared\Transfer\PaymentDetailTransfer $paymentDataTransfer |
||
883 | * @param int $idOrder |
||
884 | * |
||
885 | * @return void |
||
886 | */ |
||
887 | View Code Duplication | public function updatePaymentDetail(PaymentDetailTransfer $paymentDataTransfer, $idOrder) |
|
896 | |||
897 | /** |
||
898 | * @param \Orm\Zed\Payone\Persistence\SpyPaymentPayone $paymentEntity |
||
899 | * @param \SprykerEco\Zed\Payone\Business\Api\Response\Container\AuthorizationResponseContainer $responseContainer |
||
900 | * |
||
901 | * @return void |
||
902 | */ |
||
903 | protected function updatePaymentDetailAfterAuthorization(SpyPaymentPayone $paymentEntity, AuthorizationResponseContainer $responseContainer) |
||
926 | |||
927 | /** |
||
928 | * @param \Generated\Shared\Transfer\PayoneInitPaypalExpressCheckoutRequestTransfer $requestTransfer |
||
929 | * |
||
930 | * @return \Generated\Shared\Transfer\PayonePaypalExpressCheckoutGenericPaymentResponseTransfer |
||
931 | */ |
||
932 | View Code Duplication | public function initPaypalExpressCheckout(PayoneInitPaypalExpressCheckoutRequestTransfer $requestTransfer) |
|
947 | |||
948 | /** |
||
949 | * @param \Generated\Shared\Transfer\QuoteTransfer $quoteTransfer |
||
950 | * |
||
951 | * @return \Generated\Shared\Transfer\PayonePaypalExpressCheckoutGenericPaymentResponseTransfer |
||
952 | */ |
||
953 | View Code Duplication | public function getPaypalExpressCheckoutDetails(QuoteTransfer $quoteTransfer) |
|
971 | |||
972 | /** |
||
973 | * @param \SprykerEco\Zed\Payone\Business\Api\Request\Container\GenericPaymentContainer $requestContainer |
||
974 | * |
||
975 | * @return \Generated\Shared\Transfer\PayonePaypalExpressCheckoutGenericPaymentResponseTransfer |
||
976 | */ |
||
977 | protected function performGenericRequest(GenericPaymentContainer $requestContainer) |
||
1004 | |||
1005 | /** |
||
1006 | * @param \Generated\Shared\Transfer\OrderTransfer $orderTransfer |
||
1007 | * @param \SprykerEco\Zed\Payone\Business\Api\Request\Container\AbstractRequestContainer $container |
||
1008 | * |
||
1009 | * @return \SprykerEco\Zed\Payone\Business\Api\Request\Container\AbstractRequestContainer |
||
1010 | */ |
||
1011 | public function prepareOrderItems(OrderTransfer $orderTransfer, AbstractRequestContainer $container): AbstractRequestContainer |
||
1057 | |||
1058 | /** |
||
1059 | * @param \ArrayObject $expenses |
||
1060 | * |
||
1061 | * @return string |
||
1062 | */ |
||
1063 | protected function getDeliveryCosts(ArrayObject $expenses): string |
||
1072 | } |
||
1073 |
Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property.
Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property..