access profiles not work

Hello
Still same issue here with RC3...
I made some Access group...
Then I add an Access profil
set checkbox some attributes view, edit and delete for an Access group and submit...
If I re-edit Access profil all previous settings checkbox attributes are undone...

I missing something...

Thank

I confirm.
rc3 still have this bug - marking as critical

Try this with rc4 as I fixed these bugs there.

hey all...i have one site running 6.x-2.0-rc1 and another running 6.x-2.0 (the new release) and i have just found out that Access Profiles don't work on either of them (though the rest of the module does, and it's AWESOME! big thanks to those who wrote this!)

i just upgraded one of them from 6.x-2.0-rc6, before i checked the Access Profiles, maybe it works on that version? i should reinstall the old one and find out...

any chance we could get this fixed? I am guessing that the bug got re-introduced before the new release, due to some kind of version control accident?

cheers
Peter 'Fish' Fisera
GoatVirus Consulting

i have compared most of the files between 6.x-2.0 (the new one) and 6.x-2.0-rc4 (where this bug was fixed, according to #3) and can't find a difference that would account for this not working...

I have fixed it in commit http://drupal.org/cvs?commit=303384 so if you can check it and let me know I will release 2.1 to fix it.

I just checked the D7 version and I had fixed this. I just had not backported it. Grrrr!!!

hey Gordon, I applied the diff described in #6 http://drupalcode.org/viewvc/drupal/contributions/modules/simple_access/... to an install of 6.x-2.0 (the new release)... and it worked! thanks so much for fixing this, it is much appreciated! nice module, sir.

cheers
Fish
GoatVirus Technologies
http://goatvirus.com

ok, found ANOTHER bug, maybe one that was fixed before and then accidentally re-introduced

re: latest dev release 6.x-2.x-dev, Dec 17, 2009

see line 283 of simple_access.module (in function simple_access_node_grants)

$grants['simple_access_profiles'] = $pids;

should be:

$grants['simple_access_profile'] = $pids;

trust me on this... i made this fix to my local copy, and now profiles are working as expected :-)

cheers
Peter "Fish" Fisera
GoatVirus Technologies

Updated a few days ago from CVS :
I found 'assign profiles to profiles' instead of 'assign access to profiles'
:-(

/**
 * Implementation of hook_perm().
 */
function simple_access_perm() {
  return array('manage simple access', 'assign access to nodes', 'assign access to profiles', 'assign owner permissions');
}

Please update on CVS

thanks bloggybusiness, i found that one too, just hadn't posted it yet!

and one more... in function simple_access_form

//BUG FIX - following line changed Feb 1, 2009 - Peter Fisera, GoatVirus Technologies
//(because checkboxes control expects an array of defaults, then if node->simple_access_profiles is null or empty string
//instead of an empty array, an error will result in function form_type_checkboxes_value in includes/form.inc )
//'#default_value' => $node->simple_access_profiles,
'#default_value' => (is_array($node->simple_access_profiles) ? $node->simple_access_profiles : array()),

seems to me that Simple Access is a great and useful module except that the Access Profiles feature (which I am guessing hardly anyone uses) was never tested to the same degree as the rest of it :-)

still, i'd rather chase down the odd bug than have written this one from scratch

cheers
Fish