User Taxonomy-based Access does not remember settings

bryan.scott@ffa.int - November 19, 2008 - 23:20
Project:Taxonomy Access Control Lite
Version:6.x-1.x-dev
Component:User interface
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

If I set permissions by user
/user/XX/edit/tac_lite
and save, they are not displayed / remembered

#1

thomjjames - November 24, 2008 - 14:22

I can confirm this is happening for me too.
Edit permissions seem to hold the value of the view permission.

I'm going to investigate a fix and hopefully commit a patch if i can fix it.

Thanks
Tom

#2

Dave Cohen - November 24, 2008 - 18:47

Just to be clear, you're saying the settings are saved and privacy is correct, but when you return to the form the values are not shown there?

Or are you saying user-based privacy is not working at all?

#3

thomjjames - November 24, 2008 - 23:10

Hi,

The user-based privacy system is working fine it just will not save the permissions for any realms other than the first realm listed.
ie. if you have a view realm and an edit realm it will only save the view realm per user setting and the edit one will follow the view realm settings.

Tom

#4

Chadwick Wood - November 25, 2008 - 06:03

I'm getting this exact behavior as well. Whatever individual user settings I set (in the user edit tac_lite tab) only get remembered for my Scheme 1, and the setting for Scheme 1 is also applied to Scheme 2.

And actually, it looks like the carryover from Scheme 1 to Scheme 2 is not "real". I.e., the form for Scheme 2 mirrors the settings for Scheme 1, but in practice, the permissions for Scheme 2 are not being applied at all.

#5

thomjjames - November 26, 2008 - 14:49
Status:active» needs review

Hi,

I think i've fixed this and i've created a patch for it (attached). Its working fine for me so please review.

Thanks
Tom

AttachmentSize
tac_lite_patch.patch 799 bytes

#6

Dave Cohen - November 26, 2008 - 23:02

Tom, thanks so much. the logic of that fix looks right. I'll try to find some time to test this weekend, before I check it in.

#7

Dave Cohen - December 9, 2008 - 20:00
Status:needs review» fixed

Just checked into the DRUPAL-6--1 branch. Thanks again.

#8

thomjjames - December 10, 2008 - 08:11

No problem, thanks for this great little module!
Tom

#9

System Message - December 24, 2008 - 08:13
Status:fixed» closed

Automatically closed -- issue fixed for two weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.