I would suggest to set the default value to '' instead 0 see line 37:

31 function refresh_form_node_form_alter(&$form, $form_state) {
32 if (isset($form['#node']->refresh)) {
33 $refresh = $form['#node']->refresh;
34 }
35 else {
36 $refresh = '';
37 }

Reason: if it is 0 during creating new pages the value must be set to a number or to '' otherwise it is impossible to save the page. The User do not know why the page will not be saved (also if the error is shown) if the module will be installed on a working page.

Knut

Comments

valderama’s picture

I fully agree! Would be great to see this change comitted..

Thanks!

dwalker51’s picture

Agreed...Its getting tiresome having to change the 0 to "".

mo6’s picture

Assigned: Unassigned » mo6
Status: Active » Fixed

Makes sense. Comitted.

mo6’s picture

Fixed in the 7.x-1.1 release.

Status: Fixed » Closed (fixed)

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