ardent icon indicating copy to clipboard operation
ardent copied to clipboard

Conflict between Entrust::save() and Ardent::save()

Open george1974 opened this issue 9 years ago • 4 comments

I am using the latest Ardent version with entrust 5.2.x-dev on Laravel 5.2 framework

Entrust with Laravel Ardent is having a conflict and the following error message is displayed:

Declaration of Zizaco\Entrust\Traits\EntrustUserTrait::save() should be compatible with LaravelArdent\Ardent\Ardent::save(array $rules = Array, array $customMessages = Array, array $options = Array, Closure $beforeSave = NULL, Closure $afterSave = NULL)

Kindly provide a fix for this issue.

george1974 avatar Apr 14 '16 11:04 george1974

Sorry, have you tried opening this issue on THEIR side? As the message clearly states, their declaration extending our code is incompatible. I'm not sure I should change an important method signature just because of this. Try downgrading, their last release might be faulty.

Igor Santos -- Desenvolvedor Web [enviado do meu celular] On 14 Apr 2016 08:03, "george1974" [email protected] wrote:

I am using the latest Ardent version with entrust 5.2.x-dev on Laravel 5.2 framework

Entrust with Laravel Ardent is having a conflict and the following error message is displayed:

Declaration of Zizaco\Entrust\Traits\EntrustUserTrait::save() should be compatible with LaravelArdent\Ardent\Ardent::save(array $rules = Array, array $customMessages = Array, array $options = Array, Closure $beforeSave = NULL, Closure $afterSave = NULL)

Kindly provide a fix for this issue.

— You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub https://github.com/laravel-ardent/ardent/issues/296

igorsantos07 avatar Apr 14 '16 12:04 igorsantos07

I tried opening this issue on their side but no response. Not sure if there exists any other solution before making a localized change in either code at my end.

george1974 avatar Apr 16 '16 08:04 george1974

Have you tried downgrading? Was that motivated by an update you made or is it just your first install of Entrust?

Igor Santos -- Desenvolvedor Web [enviado do meu celular] On 16 Apr 2016 05:37, "george1974" [email protected] wrote:

I tried opening this issue on their side but no response. Not sure if there exists any other solution before making a localized change in either code at my end.

— You are receiving this because you commented. Reply to this email directly or view it on GitHub https://github.com/laravel-ardent/ardent/issues/296#issuecomment-210770692

igorsantos07 avatar Apr 16 '16 09:04 igorsantos07

I tried opening this issue on their side but no response. Not sure if there exists any other solution before making a localized change in either code at my end.

george1974 avatar Apr 20 '16 10:04 george1974