SimpleTest claims another new bug: When creating a field, if the "List" field is enabled and defaults to "List files by default", then the checkbox is impossible to uncheck on the node form. Editing a node and unchecking the "List" box is not saved, the file is still visible, and re-editing the node the checkbox will be checked again.

This is because an empty checkbox does not pass any value to CCK, so we must check for the lack of a value at all, rather than expecting "0". This patch checks if a "List" field is in a file field, then sets the value to "0" if the list value is missing entirely.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

quicksketch’s picture

Status: Needs review » Fixed
FileSize
1.5 KB

The top of the provided patch contains some changes not relevant to this post. Committed this patch containing only the bottom part.

Status: Fixed » Closed (fixed)

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