Skip to content
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

Added force string type to 3rd argument of preg_replace in util::log() #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

RickKukiela
Copy link

Added force string type to 3rd argument of preg_replace in util::log() which would otherwise throw a deprecated error for passing null in certain instances. For example, attempting to omit the cardCode value on "Stored Card" transaction where the card code is not known, while SDK logging is enabled, would cause the affected line to attempt to perform preg_replace on a null value when parsing the card code value.

…()` which would otherwise throw a deprecated error for passing `null` in certain instnaces. For example, attempting to omit the `cardCode` value on "Stored Card" transaction where the card code is not known, while SDK logging is enabled, would cause the affected line to attempt to perform `preg_replace` on a null value when parsing the card code value.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant