We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
sdk_php/src/Context/ApiContext.php
Line 332 in 652e650
Instead of giving the default value a proper value, it should be null. Instead the method body we will then check if the parameter is null or not.
This way, in other places we should not check if we are going to pass null or not. e.g.
sdk_php/src/Util/InstallationUtil.php
Lines 169 to 173 in 652e650
The text was updated successfully, but these errors were encountered:
Use null as default parameter value. (#137)
e304704
Merge pull request #138 from bunq/remove_string_as_default_value_for_…
48f4b4a
…function_parameter_bunq/sdk_php#137 Use null as default parameter value. (#137)
OGKevin
No branches or pull requests
sdk_php/src/Context/ApiContext.php
Line 332 in 652e650
Instead of giving the default value a proper value, it should be null. Instead the method body we will then check if the parameter is null or not.
This way, in other places we should not check if we are going to pass null or not. e.g.
sdk_php/src/Util/InstallationUtil.php
Lines 169 to 173 in 652e650
The text was updated successfully, but these errors were encountered: