Skip to content

Removed php @ warning suppression#1

Open
jkwakman wants to merge 2 commits intomasterfrom
jkwakman-anonymous-function-test-patch
Open

Removed php @ warning suppression#1
jkwakman wants to merge 2 commits intomasterfrom
jkwakman-anonymous-function-test-patch

Conversation

@jkwakman
Copy link
Copy Markdown
Owner

Anonymous function handling was causing massive warnings in our API logs:
Severity: Warning --> get_class() expects parameter 1 to be object, string given

get_class can only be used on objects so it is quite safe to first check if we have an object. Otherwise E_WARNINGS are thrown.

Anonymous function handling was causing massive warnings in our API logs:
Severity: Warning  --> get_class() expects parameter 1 to be object, string given 

get_class can only be used on objects so it is quite safe to first check if we have an object. Otherwise E_WARNINGS are thrown.
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