Comments on: ACL with Groups http://lemoncake.wordpress.com/2007/07/19/acl-with-groups/ My forays into the Kitchen Wed, 04 Nov 2009 22:02:31 +0000 http://wordpress.com/ hourly 1 By: coskunlar vinc http://lemoncake.wordpress.com/2007/07/19/acl-with-groups/#comment-2299 coskunlar vinc Tue, 28 Apr 2009 11:07:51 +0000 http://lemoncake.wordpress.com/2007/07/19/acl-with-groups/#comment-2299 I think zero will work but it is better to pass null as the parent id, as seen with this condition I think zero will work but it is better to pass null as the parent id, as seen with this condition

]]> By: JM http://lemoncake.wordpress.com/2007/07/19/acl-with-groups/#comment-2172 JM Wed, 21 Jan 2009 01:22:22 +0000 http://lemoncake.wordpress.com/2007/07/19/acl-with-groups/#comment-2172 I guess that a many to many relation between users and groups will break the hierarchical rule of "only one parent", and that is not possible to implement such a solution with aclbehavior (i supose that from a strictly acl point of view it's possible to have a user in more than one group). Any input on subject will be welcome. By the way, congrats for the artcile, it's nice explained. I guess that a many to many relation between users and groups will break the hierarchical rule of “only one parent”, and that is not possible to implement such a solution with aclbehavior (i supose that from a strictly acl point of view it’s possible to have a user in more than one group).

Any input on subject will be welcome.

By the way, congrats for the artcile, it’s nice explained.

]]> By: CakePHP 1.2.0.6311b: Group + ACL + AuthComponent « वीर http://lemoncake.wordpress.com/2007/07/19/acl-with-groups/#comment-1946 CakePHP 1.2.0.6311b: Group + ACL + AuthComponent « वीर Thu, 24 Apr 2008 17:51:46 +0000 http://lemoncake.wordpress.com/2007/07/19/acl-with-groups/#comment-1946 [...] http://lemoncake.wordpress.com/2007/07/19/acl-with-groups/ [...] [...] http://lemoncake.wordpress.com/2007/07/19/acl-with-groups/ [...]

]]> By: Brade http://lemoncake.wordpress.com/2007/07/19/acl-with-groups/#comment-1888 Brade Fri, 15 Feb 2008 17:16:21 +0000 http://lemoncake.wordpress.com/2007/07/19/acl-with-groups/#comment-1888 Indeed, this is the revised approach: <code>if ($this->Group->save($this->data['Group'])) { $this->Group->Aro->set($this->Group->Aro->find("foreign_key = $id")); $this->Group->Aro->saveField('alias', $this->data['Group']['name']); $this->Group->Aro->setParent($this->Group->Aro->field('id', 'foreign_key = ' . $this->data['Group']['parent_id']));</code> ...where $id is the parameter in the "edit" action. The "saveField" line isn't really necessary unless (like me) you want a copy of the group name as the alias, so it makes it easier to use "generatetreelist" on the index page, and have the names of your groups readily available. Also in my "delete" action I do a check like this to see if a group has children: <code>} elseif ($children = $this->Group->Aro->children($this->Group->Aro->field('id', "foreign_key = $id"), true)) {</code> and set a flash message/redirect without deleting. Indeed, this is the revised approach:
if ($this->Group->save($this->data['Group'])) {
$this->Group->Aro->set($this->Group->Aro->find("foreign_key = $id"));
$this->Group->Aro->saveField('alias', $this->data['Group']['name']);
$this->Group->Aro->setParent($this->Group->Aro->field('id', 'foreign_key = ' . $this->data['Group']['parent_id']));
…where $id is the parameter in the “edit” action.
The “saveField” line isn’t really necessary unless (like me) you want a copy of the group name as the alias, so it makes it easier to use “generatetreelist” on the index page, and have the names of your groups readily available.

Also in my “delete” action I do a check like this to see if a group has children:
} elseif ($children = $this->Group->Aro->children($this->Group->Aro->field('id', "foreign_key = $id"), true)) {
and set a flash message/redirect without deleting.

]]> By: Brade http://lemoncake.wordpress.com/2007/07/19/acl-with-groups/#comment-1887 Brade Thu, 14 Feb 2008 23:52:22 +0000 http://lemoncake.wordpress.com/2007/07/19/acl-with-groups/#comment-1887 Okay, so don't use my previous code when editing an item with children, because it will delete the children. >=[ I can probably make use of the fact that the Aro object in Group already "actsAs" the TreeBehavior, so I can use some TreeBehavior function to move the node instead of deleting and re-creating. Let's find out... Okay, so don’t use my previous code when editing an item with children, because it will delete the children. >=[
I can probably make use of the fact that the Aro object in Group already “actsAs” the TreeBehavior, so I can use some TreeBehavior function to move the node instead of deleting and re-creating. Let’s find out…

]]> By: Brade http://lemoncake.wordpress.com/2007/07/19/acl-with-groups/#comment-1886 Brade Thu, 14 Feb 2008 23:31:20 +0000 http://lemoncake.wordpress.com/2007/07/19/acl-with-groups/#comment-1886 Mmmkay, so I figured a workaround. Building on Aran's solution (which is the only way I could get the parentNode to actually save properly), in my "edit" action, I substituted this code: <code> if ($this->Group->save($this->data['Group'])) { </code> with this: <code> if ($this->Group->save($this->data['Group'])) { $this->Group->behaviors['Acl']->afterDelete($this->Group); $this->Group->behaviors['Acl']->afterSave($this->Group, true); </code> I guess this is cheating a bit. I still don't know why the Cake auths decided to run the Acl behavior's afterSave logic only when adding a new record. But this solution seems to work for me right now. Mmmkay, so I figured a workaround. Building on Aran’s solution (which is the only way I could get the parentNode to actually save properly), in my “edit” action, I substituted this code:

if ($this->Group->save($this->data['Group'])) {

with this:

if ($this->Group->save($this->data['Group'])) {
$this->Group->behaviors['Acl']->afterDelete($this->Group);
$this->Group->behaviors['Acl']->afterSave($this->Group, true);

I guess this is cheating a bit. I still don’t know why the Cake auths decided to run the Acl behavior’s afterSave logic only when adding a new record. But this solution seems to work for me right now.

]]> By: Brade http://lemoncake.wordpress.com/2007/07/19/acl-with-groups/#comment-1885 Brade Thu, 14 Feb 2008 20:47:37 +0000 http://lemoncake.wordpress.com/2007/07/19/acl-with-groups/#comment-1885 As lemp points out, all of this is useless if you can't edit a user to change groups. The source code indicates parentNode only runs if you're adding a new record, which to me seems totally retarded. That's not your fault, of course, but obviously this Behavior stuff is not going to be nearly as easy as it could (and should) have been. As lemp points out, all of this is useless if you can’t edit a user to change groups. The source code indicates parentNode only runs if you’re adding a new record, which to me seems totally retarded. That’s not your fault, of course, but obviously this Behavior stuff is not going to be nearly as easy as it could (and should) have been.

]]> By: Aran http://lemoncake.wordpress.com/2007/07/19/acl-with-groups/#comment-1240 Aran Thu, 27 Dec 2007 01:13:41 +0000 http://lemoncake.wordpress.com/2007/07/19/acl-with-groups/#comment-1240 I had a similar problem as darkangel. My solution was to modify the Group::parentNode() method from this tutorial. Instead of: <code>return $this->data['Group']['parent_id'];</code> I had to borrow the code from the User model and make it: <code>return array('model' => 'Group', 'foreign_key' => $data['Group']['parent_id']);</code> The Acl behavior's afterSave() method does a call to Group::parentNode(), and the result is then fed to the node() method. The node() method needs the array with 'model' and 'foreign_key' in it in order to work. Now ... when I add a new group and specify as the parent id, the id of another group in the Groups table, the corresponding id for the Group in the Aros table, will be set as the parent_id of the new Group in the Aros table. I had a similar problem as darkangel. My solution was to modify the Group::parentNode() method from this tutorial.

Instead of:
return $this->data['Group']['parent_id'];

I had to borrow the code from the User model and make it:
return array('model' => 'Group', 'foreign_key' => $data['Group']['parent_id']);

The Acl behavior’s afterSave() method does a call to Group::parentNode(), and the result is then fed to the node() method. The node() method needs the array with ‘model’ and ‘foreign_key’ in it in order to work.

Now … when I add a new group and specify as the parent id, the id of another group in the Groups table, the corresponding id for the Group in the Aros table, will be set as the parent_id of the new Group in the Aros table.

]]> By: darkangel http://lemoncake.wordpress.com/2007/07/19/acl-with-groups/#comment-682 darkangel Sun, 25 Nov 2007 12:18:08 +0000 http://lemoncake.wordpress.com/2007/07/19/acl-with-groups/#comment-682 (ignore previous comment -- solution found) (ignore previous comment — solution found)

]]> By: darkangel http://lemoncake.wordpress.com/2007/07/19/acl-with-groups/#comment-660 darkangel Fri, 23 Nov 2007 22:14:38 +0000 http://lemoncake.wordpress.com/2007/07/19/acl-with-groups/#comment-660 Hi Geoff, Thanks for these articles. Is the result of parentNode() supposed to be entered into the 'parent_id' field of the ACL table? It seems NULL is inserted into the table regardless of whether or not I specify a parent. My function: <code> public function parentNode() { if (isset($this->data['Group']['parent_id'])) { return $this->data['Group']['parent_id']; } else { return null; } } </code> Any ideas? _da. Hi Geoff,

Thanks for these articles.

Is the result of parentNode() supposed to be entered into the ‘parent_id’ field of the ACL table? It seems NULL is inserted into the table regardless of whether or not I specify a parent.

My function:


public function parentNode()
{
if (isset($this->data['Group']['parent_id'])) {
return $this->data['Group']['parent_id'];
} else {
return null;
}
}

Any ideas?

_da.

]]>