mongodb / laravel-mongodb

A MongoDB based Eloquent model and Query builder for Laravel (Moloquent)
https://www.mongodb.com/docs/drivers/php/laravel-mongodb/
MIT License
7.02k stars 1.43k forks source link

getDateFormat() must be public for laravel 5.6 #1432

Closed slouma2000 closed 4 years ago

slouma2000 commented 6 years ago

Access level to Jenssegers\Mongodb\Eloquent\Model::getDateFormat() must be public (as in class Illuminate\Database\Eloquent\Model)

Sohit18595 commented 6 years ago

I am also getting same issue with Jenssegers\Mongodb package...

jwrunge commented 6 years ago

I'm having the same problem, but installed and used with no problem a few days ago. I'm noticing the MongoDB PHP driver was updated on 2/9. Perhaps that has something to do with it?

EDIT: Never mind. Just now noticing that 'laravel new' command is installing Laravel 5.6, and that doesn't seem to be supported yet. Pays to read more carefully...

RemiCollin commented 6 years ago

A PR for supporting Laravel 5.6 is underway : https://github.com/jenssegers/laravel-mongodb/pull/1428

dev-erem commented 6 years ago

After changing function's access modifier from protected to public, another Error occurs with lumen 5.6 (1/1) ErrorException Declaration of Jenssegers\Mongodb\Eloquent\HybridRelations::morphTo($name = NULL, $type = NULL, $id = NULL) should be compatible with Illuminate\Database\Eloquent\Model::morphTo($name = NULL, $type = NULL, $id = NULL, $ownerKey = NULL)

vad-naitik commented 6 years ago

I am also getting the same issue with Jenssegers\Mongodb package... any idea about that? how can I solve and how can I use a passport in Laravel with MongoDB

heismehrab commented 5 years ago

it might be help https://github.com/moloquent/moloquent/issues/97#issuecomment-480710970