i've found that checking skip checkbox does not prevent saving new nodes
(moreover, the fields are all empty but the node gets saved the same).

what about this fix:

  function is_empty(&$form, &$form_state) {
    if ($form_state['values']['remove']) {
      return TRUE;
    }
   ...

i think this can improve performance as well.

Comments

Leo.dil@gmail.com’s picture

Leo.dil@gmail.com commented

I have the same problem! I'm using this module in my production server.
How can I safely apply the patch?

Leo.dil@gmail.com’s picture

Leo.dil@gmail.com commented
Priority: Normal » Critical

Does anyone any idea's on this

rv0’s picture

rv0 commented

I was having problems too with empty nodes
Changed the function in node_widget.class.inc

EDIT: code i posted here was totally wrong

@gunzip: thanks

gunzip’s picture

gunzip commented

is incorrect as it won't save the node in case you have just _one_ record empty.

rv0’s picture

rv0 commented

@gunzip
what do you mean by "one record empty"?
i

gunzip’s picture

gunzip commented

i mean that there your solution returns "this node is empty" even if there's only one cck field empty (while the others are not)