Userreference and email fields should be supported.
Patch is attached.
I restructured the surrounding lines to avoid couple of almost identical lines.

CommentFileSizeAuthor
#1 905214_conditional_fields_empty_check.patch1.08 KBAron Novak
conditional_fields_empty_check.patch1.1 KBAron Novak
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Aron Novak’s picture

Aron Novak
Primary language Hungarian
Location Hungary, Budapest
CreditAttribution: Aron Novak commented
FileSize
905214_conditional_fields_empty_check.patch1.08 KB

Forgot to add a break;

Aron Novak’s picture

Aron Novak
Primary language Hungarian
Location Hungary, Budapest
CreditAttribution: Aron Novak commented

I saw #648934: Plugin system for compatibility with non-core CCK modules , but maybe it's still worth to make that check more frugal, so at least controlling of such fields will be possible (it's especially about required fields) with not much bloat, in fact, the patch removes 7 lines and adds only 7 :)

peterpoe’s picture

peterpoe CreditAttribution: peterpoe as a volunteer commented
Issue summary: View changes
Status: Needs review » Closed (outdated)