Skip to content

Commit a7f89d8

Browse files
add exception on not set required array keys at getInvoicePayResultFromRequestArray method
1 parent 1332ffc commit a7f89d8

1 file changed

Lines changed: 10 additions & 3 deletions

File tree

src/RobokassaApi.php

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,10 @@
88

99
use Http\Discovery\Psr18Client;
1010
use JsonException;
11-
use netFantom\RobokassaApi\Exceptions\{MissingRequestFactory, MissingStreamFactory, TooLongSmsMessageException};
11+
use netFantom\RobokassaApi\Exceptions\{InvalidArgumentException,
12+
MissingRequestFactory,
13+
MissingStreamFactory,
14+
TooLongSmsMessageException};
1215
use netFantom\RobokassaApi\Options\{InvoiceOptions, ReceiptStatusOptions, SecondReceiptOptions};
1316
use netFantom\RobokassaApi\Params\Receipt\Vat;
1417
use netFantom\RobokassaApi\Results\{InvoicePayResult, ReceiptAttachResult, ReceiptStatusResult, SmsSendResult};
@@ -74,9 +77,13 @@ private function checkPsr18Client(): void
7477
public static function getInvoicePayResultFromRequestArray(array $requestParameters): InvoicePayResult
7578
{
7679
return new InvoicePayResult(
77-
outSum: $requestParameters['OutSum'],
80+
outSum: $requestParameters['OutSum'] ?? throw new InvalidArgumentException(
81+
'OutSum request parameter required'
82+
),
7883
invId: isset($requestParameters['InvId']) ? (int)$requestParameters['InvId'] : null,
79-
signatureValue: $requestParameters['SignatureValue'],
84+
signatureValue: $requestParameters['SignatureValue'] ?? throw new InvalidArgumentException(
85+
'SignatureValue request parameter required'
86+
),
8087
userParameters: self::getUserParametersFromRequestArray($requestParameters),
8188
);
8289
}

0 commit comments

Comments
 (0)