ACL 2.0

[sergeyklay]

I have some ideas on ACL...
I think I could improve it.
Let me move it in a separate module (may be Gleez ACL) to improve it

[sandeepone]

Can you explain your ideas if possible?

[sergeyklay]

First, I think it is wrong that in the absence of access, triggered exception or throws (403, 404, etc.).
But I think we need to to log such events.
Maybe, if the user is not logged in, it makes sense to redirect the authentication page

[sandeepone]

Many cases we need to throw 404 or 403 and in few cases we need to redirect to login form if not logged in.

[sergeyklay]

In general, the fundamental idea is to add ability a callback method as an argument to ACL::required()

[sandeepone]

I think we can add ACL::redirect cases where we need redirect to another page or login

[sergeyklay]

I looked similar implementation in Zend and Symfony and I found that we could borrow a something

[sergeyklay]

Maybe worth paying attention to the ready-made solutions for similar problems:

• Wouterrr / ACL — ACL for Kohana (based on Zend_ACL)
• synapsestudios / kohana-acl — Request-based ACL Module for Ko3, Auth, and ORM
• NeZanyat / kohanaACL — More powerful ACL system for Kohana
• smgladkovskiy / ACL Kohana — Access Controll List module
• morgan / kohana-deputy — Powerful Kohana ACL designed with the minimalist in mind
• anroots / kohana-db-acl — Database-based ACL module for Kohana 3.3
• banks / aacl — Another ACL for Kohana 3
• ener / Kohana-ACL — ACL for Kohana 3.0.8
• StefanRHRO / KohanaACL — A little Access Control List (ACL) Module for Kohana

[sergeyklay]

Hi all!

@sandeepone

I have some ideas about ACL 2.0 that you can look at this url: https://gist.github.com/sergeyklay/5075595

Here's one way how we can use this: https://gist.github.com/sergeyklay/5075608
But this is not the only change in the ACL class, and this is beta version

I seriously reworked the code and add new methods.
And I'm interested in your opinion

@dishuostec

If you have anything to add - it would be good

Best regards,

[sergeyklay]

#128

[sergeyklay]

also see:
68b60b7
5da99fe
a4b8b03
0abe2a3
d35b326

[sandeepone]

Sorry, I didn't follow this issue, but we've covered most of the things.

[sergeyklay]

yep!