Closed (duplicate)
Project:
Drupal core
Version:
x.y.z
Component:
comment.module
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
25 Nov 2005 at 19:19 UTC
Updated:
17 Dec 2005 at 11:45 UTC
Jump to comment: Most recent file
Comments
Comment #1
Wesley Tanaka commentedI can reproduce the bug when editing the comment as the uid=1 account, as well as the comment owner (non-privileged). The date shows up as correct in the preview, but saves into the database incorrectly (as 0)
Comment #2
Wesley Tanaka commentedSeems to be another case of "it wasn't in the form," so an unset value starts getting around.
The culprit seems to be this line:
In this case, I'm guessing that !issset($edit['timestamp']) holds true.
Comment #3
Wesley Tanaka commentedOut of all the $edit indices referred to in the update statement, 'status', 'timestamp', 'format' and 'name' aren't set by default.
Comment #4
Wesley Tanaka commentedPatch attached. This seems like a reasonable way to deal with the issue.
Comment #5
asimmonds commentedI've reopened http://drupal.org/node/35726, as this bug is a result of that issue, and I screwed up.
Can followups be directed to http://drupal.org/node/35726
Comment #6
chx commentedComment #7
Wesley Tanaka commentedReopening. See http://drupal.org/node/35726#comment-56652 for reasoning.
Comment #8
Wesley Tanaka commentedsince been rolled into the patch 35726_1.patch on http://drupal.org/node/35726
Comment #9
Wesley Tanaka commentedand since re-split into a separate issue.
http://drupal.org/node/40762