search

youngguns-nl / moneybird_php_api

A PHP Library for the MoneyBird API
http://www.moneybird.nl
MIT License
37 stars 14 forks source link

Deprecated: is_a() #3

Closed jhogervorst closed 14 years ago

jhogervorst commented 14 years ago

Error:

Strict Standards: is_a(): Deprecated. Please use the instanceof operator in Object.php on line 140

Source: <?php if (is_a($this, 'iMoneybirdContact')) { $root = 'contact'; } elseif (is_a($this, 'iMoneybirdInvoice')) { $root = 'invoice'; } elseif (is_a($this, 'iMoneybirdRecurringTemplate')) { $root = 'recurringtemplate'; } ?>

Fix: <?php if ($this instanceof iMoneybirdContact) { $root = 'contact'; } elseif ($this instanceof iMoneybirdInvoice) { $root = 'invoice'; } elseif ($this instanceof iMoneybirdRecurringTemplate) { $root = 'recurringtemplate'; } ?>

pluijm commented 14 years ago

Vanaf PHP 5.3 is is_a niet meer depricated, daarom is de functie erin gekomen en wordt hij nog steeds gebruikt. Mocht men PHP5.2 gebruiken dan werkt bovenstaande fix.

jhogervorst commented 14 years ago

Oké, dat wist ik niet, thanks.

Het is echter wel zo dat op meerdere plekken wél instanceof gebruikt wordt, is het niet slim om consistent te zijn en overal hetzelfde te gebruiken (dus is_a óf instanceof)?

pluijm commented 14 years ago

is inderdaad netter. Heb het verwerkt