Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

can function is already defined in laravel #200

Open
rhadoo200 opened this issue Feb 11, 2017 · 4 comments
Open

can function is already defined in laravel #200

rhadoo200 opened this issue Feb 11, 2017 · 4 comments

Comments

@rhadoo200
Copy link

Hello,

The can function is already defined on the Authorizable trait. Could you rename your function to something else. Personally I prefer the laravel native authorization because I can implement my logic based on the user settings but I like your implementation of permission and roles. I've tried a custom implementation over your package but I need to extend a lot of classes to reach all the methods.

Thanks in advanced

@konovalov-nk
Copy link

@rhadoo200
Copy link
Author

Hi,
I know how to fix the problem. I thought it would be better to fix it in the package because it's a conflict with the framework itself. The main reason for using a package like this is to avoid extra configuration or local fixes.

@kodeine
Copy link
Owner

kodeine commented Feb 28, 2017

@rhadoo200 propose a pr? :)

@kodeine
Copy link
Owner

kodeine commented Mar 24, 2017

being worked on #208

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

No branches or pull requests

3 participants