braintree_php
braintree_php copied to clipboard
Update IsNode.php
👋 @ElonSmithee can we get some context on these changes? Why do we need them? What is the problem that this PR solves? Just trying to get some additional info behind these changes. thanks!
This object type screams allover IDE trying to pass simple string, It’s redundant to inform that object must be passed and then convert it to string. Let user decide which way data will be converted. Will it be magical _toString, custom toString, simple cast or straight string.
upd. If string not what you wish, add dochints with multiple choices
@params mixed|object|string $var
@ElonSmithee would this change prevent an IDE from allowing the dev to pass, say, a number and have it automatically be coerced to a string?
@ElonSmithee would this change prevent an IDE from allowing the dev to pass, say, a number and have it automatically be coerced to a string?
They will notice type inconsistency and will cast it manualy
for internal use, ticket 2003