The check for $node->op on line 124 doesn't account for translations

  if ($node->op == 'Save' && $node->published) {
    $node->published = strtotime($node->published);
  }

On my site the $node->op == 'Indsend' - so an easy fix could be to check for $node->op == t('Save')

But I maybe a better solution could be to check for $node->form_id == $node->type.'_node_form' instead

CommentFileSizeAuthor
#1 877702-2.patch985 bytesnaxoc
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

naxoc’s picture

FileSize
985 bytes

Here is a patch that just puts the 'Save' in a t() function.

naxoc’s picture

Status: Active » Needs review

Setting status

liquidcms’s picture

yea, this is wrong way to do this.. no need to check where the submit is coming from; simply need to check if the string is an int already... committing to dev ver.

liquidcms’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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