Updates that fail because of a mapping issue appear as successful in Laravel

jonbwhite / elkuent

Elasticsearch ORM for Laravel

7 stars 0 forks source link

Updates that fail because of a mapping issue appear as successful in Laravel #26

Closed jonbwhite closed 9 years ago

jonbwhite commented 9 years ago

In onPerformUpdate in the Query Builder we currently return 0 on a failure, but this is not taken into account when laravel updates. It looks like the solution to this is to fail harder in the event of an error.