search

duxet / laravel-rethinkdb

RethinkDB adapter for Laravel (with Eloquent support)
MIT License
148 stars 37 forks source link

RethinkDB\Schema\Blueprint@index should compatible with Illuminate\Schema@index #44

Closed ux-engineer closed 6 years ago

ux-engineer commented 7 years ago

Just installed Laravel 5.4.25 (Laravel Spark) and and set up 'homestead' table on RethinkDB.

Changed use duxet\Rethinkdb\Schema\Blueprint; to each of Spark's migration file. Ran php artisan migrate and got this error:

[ErrorException] Declaration of duxet\RethinkDB\Schema\Blueprint::index($column, $options = NULL) should be compatible with Illuminate\Database\Schema\Blueprint::index($columns, $name = NULL, $algorithm = NULL)

randuck-dev commented 7 years ago

Just piggybacking on this. Having the same issue

vzool commented 7 years ago

Same problem here at Laravel 5.5.17 :(

bhavikji commented 7 years ago

getting the same issue have you find any hack for this? I'm using Laravel 5.5.20

randuck-dev commented 7 years ago

I have not. We have since gone away from using Rethink for our project, but if someone one day finds a fix I am still interested.

bhavikji commented 6 years ago

issue is now fixed in latest commit pls check and close this issue :)