If you have " Enabled and Public by default " for a field, the user cannot change his privacy settings for this field.

Its keeps green.

So while debugging I got to hook_nodeapi case 'insert'.
- $private_fields is filled out correctly
- So I checked the value for " Enabled and Public by default" its 0, or CCK_FIELDS_PRIVACY_STATUS_PUBLIC.
But empty() returns TRUE for 0.
=> change empty to !== FALSE
=> Only real false(disabled privacy settings) is filtered out.

Comments

markus_petrux’s picture

Priority: Normal » Critical
Status: Needs review » Fixed

Good catch! Thank you!

Committed to CVS. I'll release a new beta as it seems to me this issue is quite critical.

Status: Fixed » Closed (fixed)

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