Skip to content
This repository has been archived by the owner on Jul 16, 2023. It is now read-only.

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

Open
george1974 opened this issue Apr 14, 2016 · 4 comments
Open

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

george1974 opened this issue Apr 14, 2016 · 4 comments

Comments

@george1974
Copy link

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.

@igorsantos07
Copy link
Member

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" notifications@github.com 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
#296

@george1974
Copy link
Author

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.

@igorsantos07
Copy link
Member

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" notifications@github.com 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
#296 (comment)

@george1974
Copy link
Author

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.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants