ive attached a patch to prevent 1970-1-1 coming up when expiry date has been left blank.
(quite annoying having to delete 1970-1-1 every time you amend a node that shouldnt expire)

ive not tested it with nodes that will expire yet! (i dont have cron access on my server yet)

CommentFileSizeAuthor
#2 node_expire.nodeapi.inc_.diff547 bytesvannus
node_expire.nodeapi.inc_.diff428 bytesvannus

Comments

vannus’s picture

vannus commented

actually, #405608: Empty Expiration Date Field is filled with bad information! - default Expiration Date set to '1970-01-01' is similar.

vannus’s picture

vannus commented
StatusFileSize
new node_expire.nodeapi.inc_.diff547 bytes

improvement - now handles empty default value when creating new node as well as node edits.
(disregard earlier patch/apply this patch to fresh 2.03 version)

bomarmonk’s picture

bomarmonk commented

Does yours work more effectively? Maybe you can help with this larger patch to get this module working with Drupal 6: http://drupal.org/node/546494

Thank you for your help.

vannus’s picture

vannus commented
Status: Needs review » Closed (fixed)

it works for me :)

thanks for mentioning that other one - ill close this now, and make any contributions to #546494: Disable node_expire completely for certain node types.