search

mitchellvanw / laravel-doctrine

NO LONGER MAINTAINED! A Doctrine 2 implementation that melts with Laravel
MIT License
187 stars 74 forks source link

Timestamp TrashedFilter issue #34

Closed glennjacobs closed 10 years ago

glennjacobs commented 10 years ago

Been using the Timestamps trait and I get an error when running a query.

The following line appears to be the problem

return "{$table}.deleted_at IS NULL || NOW() < {$table}.deleted_at";

When I change it to this, all seems fine (changed || to OR)

return "{$table}.deleted_at IS NULL OR NOW() < {$table}.deleted_at";
kirkbushell commented 10 years ago

Have added to fix/PR here: https://github.com/mitchellvanw/laravel-doctrine/pull/31

glennjacobs commented 10 years ago

I've also added it in my pull request here #35

mitchellvanw commented 10 years ago

This has been added to the filter. Thanks for the suggestion!