-
Notifications
You must be signed in to change notification settings - Fork 54
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix NotificationFilter, Model issues and implement PSD2 #179
Conversation
README.md
Outdated
@@ -84,6 +84,31 @@ BunqContext::loadApiContext($apiContext); | |||
|
|||
**Tip:** both saving and restoring the context can be done without any arguments. In this case the context will be saved to/restored from the `bunq.conf` file in the same folder with your script. | |||
|
|||
##### PSD2 | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
empty line
src/Context/ApiContext.php
Outdated
* @param BunqEnumApiEnvironmentType $environmentType | ||
* @param Certificate $publicCertificate | ||
* @param PrivateKey $privateKey | ||
* @param array $allChainCertificate |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
array of strings? or Certificate?
src/Context/ApiContext.php
Outdated
* @param PrivateKey $privateKey | ||
* @param array $allChainCertificate | ||
* @param string $description | ||
* @param array $allPermittedIp |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same. is this an array of objects?
*/ | ||
private function initializeAllDefinitionIfNeeded() | ||
{ | ||
if (defined(self::CURL_FIELD_KEY_PINNING)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
put a comment why (and for which php version we do this
src/Model/Core/BunqModel.php
Outdated
@@ -349,6 +354,21 @@ protected static function fromJson(BunqResponseRaw $responseRaw, string $wrapper | |||
return new BunqResponse($value[self::INDEX_FIRST], $responseRaw->getHeaders()); | |||
} | |||
|
|||
/** | |||
* @param $path |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
string?
* @return string | ||
* @throws BunqException | ||
*/ | ||
private static function determineTokenUriFormat() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
return type
*/ | ||
class PaymentServiceProviderCredentialInternal extends PaymentServiceProviderCredential | ||
{ | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove empty line
* with the private key belonging to the QSEAL | ||
* certificate. | ||
* @param ApiContext $apiContext | ||
* @param array $allCustomHeader |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
define the type
string $clientPublicKeySignature, | ||
ApiContext $apiContext, | ||
array $allCustomHeader = [] | ||
) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
return type
protected static function fromRawResponse(BunqResponseRaw $responseRaw): BunqResponse | ||
{ | ||
$allCredential = UserCredentialPasswordIp::fromJson($responseRaw, self::FIELD_OBJECT_TYPE); | ||
return new BunqResponseUserCredentialPasswordIp($allCredential->getValue(), $responseRaw->getHeaders()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
always empty line above return
This PR closes/fixes the following issues:
merchant_category_code
in thecounterparty_alias
#171 - Models regenerated