When changing the label of a field, all corresponding permissions are being dropped.
This kind of update is not understood as reason to check access permissions again and fields got hidden accidentally...

Why not use the field naming for permission dataset?
This is a major issue in productivity. I see no reason to use Label in this context and therefore file this as a bugreport (due to loss of information in case of a minor update).

Thanks for your opinion.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

halstead’s picture

Status: Active » Needs review
FileSize
2.69 KB

This fixes the issue by storing the field_name in the permissions table and form_altering the access page to display the field label.

webchick’s picture

Status: Needs review » Reviewed & tested by the community

Confirmed that this patch fixed the problem for me.

To reproduce, I created a text field called "Moose" and assigned view permissions to authenticated users. Before the patch, renaming this field to "Moosesd87s8sd45" caused the checkbox at admin/user/access to clear. After the patch, on subsequent rename, it stayed checked. :)

webchick’s picture

Priority: Normal » Critical

Also, I would call this critical. Taking away your permissions you worked so hard to setup is not nice.

halstead’s picture

Priority: Critical » Normal
Status: Reviewed & tested by the community » Needs review
FileSize
3.24 KB

Here is a pre-patched module file for people not familiar with CVS and patch.
(The patch above was done against CVS. This module file is the version in the 5.x-1.10 download.)

webchick’s picture

Status: Needs review » Reviewed & tested by the community

Oops. I'm pretty sure halstead didn't mean to reset the status here, so switching it back.

halstead’s picture

Assigned: Unassigned » halstead
Priority: Normal » Critical
Status: Reviewed & tested by the community » Fixed

This is in 5.x-1.x-dev which should appear soon.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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