When a node is created, no settings of "Printer, e-mail and PDF versions" group are saved. It happens because all nodeapi hooks do not contain "insert" mode branch. Patch is trivial: just add "case 'insert':" before "case 'update':" in all three modules.

print\print_mail\print_mail.module
@@ -213,12 +213,13 @@
         }
       }
       break;
     case 'load':
       _print_mail_set_node_fields($node);
       break;
+    case 'insert':
     case 'update':
       if (user_access('administer print') || user_access('node-specific print configuration')) {
       _print_mail_node_conf_modify($node->nid, $node->print_mail_display, $node->print_mail_display_comment, $node->print_mail_display_urllist);
       }
       break;
     case 'delete':

- and the same in print_pdf.module and print.module. I've tested it, and it works fine.

Comments

neclimdul’s picture

StatusFileSize
new2.27 KB

+1 seems pretty much a no brainer.

Went ahead and made a patch that applies this to all the modules.

jcnventura’s picture

Status: Needs review » Fixed

Thanks for the patch. I've just committed it to CVS.

Status: Fixed » Closed (fixed)

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