Suggested that the Controller.* should be a task

tingobol / yii-rights

Automatically exported from code.google.com/p/yii-rights

BSD 3-Clause "New" or "Revised" License

0 stars 0 forks source link

Suggested that the Controller.* should be a task #34

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

I always think that the Controller.* AuthItem should be a task, rather than a 
operation.

Original issue reported on code.google.com by wangyong...@gmail.com on 14 Jan 2011 at 3:48

GoogleCodeExporter commented 9 years ago

You're probably correct, I'll look into this when I have time.

Original comment by Christof...@gmail.com on 15 Jan 2011 at 10:37

Changed state: Accepted
Added labels: Priority-Low
Removed labels: Priority-Medium

GoogleCodeExporter commented 9 years ago

Hello wangyong,

I tried this and it seems to be a good idea. This will be included in the 
upcoming version 1.3.0.

Original comment by Christof...@gmail.com on 31 Mar 2011 at 9:42

Changed state: Fixed

GoogleCodeExporter commented 9 years ago

Hi, I have found another problem, and I think it is much more better to do like 
this:
1. The Controller.* has been generated as Task, that's good.
2. And the Controller's actions of Operations should be added as the 
Controller.*'s child auth items.
3. So It's not necessary to check the rights of 'Controller.*' in 
RightsFilter.php 

What do you think about my suggest?

Original comment by wangyong...@gmail.com on 12 Aug 2011 at 10:03