Hi,
Thanks for your great module. Trying to get it to work in a new (yet to be released) add-on module. Just found out that while we have an create and a delete function for ACL, we don't have an update function yet.

My module creates ACL's automatically and assigns them a name. It seems I can't find a function to update that name again?

Simple patch below

CommentFileSizeAuthor
provice-acl-update-function.patch564 bytessplash112
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

salvis’s picture

Title: Provide ACL update function » Provide ACL update function: acl_update_acl()
Status: Active » Needs work

I'm willing to add this new function. However...

http://api.drupal.org/api/drupal/includes--common.inc/function/drupal_wr... seems to indicate that you need to pass in the name of the primary key. Have you tested this new function?

Returning the $acl_id doesn't make much sense — the caller already has that. Return the result of drupal_write_record() instead.

We've started to write some tests. Please add a test for acl_update_acl().

salvis’s picture

Issue summary: View changes
Status: Needs work » Closed (outdated)