search

danielmewes / php-rql

A PHP client driver for the RethinkDB query language (ReQL).
http://php-rql.dnsalias.net/
339 stars 60 forks source link

DatumConverter invokes autoload #162

Open ForbiddenEra opened 5 years ago

ForbiddenEra commented 5 years ago

rdb/DatumConverter.php:37 should be: if (is_subclass_of($val, "\\r\\Query", FALSE) && !is_subclass_of($val, '\r\Datum\Datum', FALSE)) {

unless I'm doing something wrong, trying to insert anything but a true/false value results in autoloader invoking and causing an error.

adding this fixed it.

Geekimo commented 5 years ago

https://github.com/danielmewes/php-rql/pull/148 solved this behavior. It may now need some update since the owner hasn't been there for long and the pull hasn't been merged.

c01nd01r commented 4 years ago

I discovered similar behavior. @danielmewes May be possible to review and merge PR #148 ?