[issue] article time stamp and incorrect date

The-Exit · 2006-12-13 12:45:20

Today I wanted to display an article in the future, but instead of altering the publication time, i altered the publication date. i inserted into the field for the month the value “18” which is not valid.

textpattern didn’t show a message, as i would expect. instead, the publication date was changed to 1970-01-01 which is really not the behaviour i expect.

could there an error message inserted?

Mary · 2006-12-13 15:42:05

I don’t know for certain, but yes, I think there’s no error checking done there, and there should be.

The question is how to handle it properly. I’m thinking force dates into the correct structure (so 18 becomes 12, etc), but give you a warning (like we warn about duplicate url titles, for instance).

The-Exit · 2006-12-13 15:49:13

or probably just no change if the inserted value is not correct, i.e. if the inserted date is not valid, e.g. the 30th february :)

Mary · 2006-12-13 16:00:56

Unless you’re referring to some kind of JavaScript early warning, which I can see, that wouldn’t be a good idea: the result would be completely losing all your changes. That’d be the quickest way to send most people into orbit. ;p

The-Exit · 2006-12-13 16:25:53

well, for that, i’m not deep enough in the code.

i just thought:

when user clicks “save”
1. check if entered date is valid
2. if date is not valid, use the value, which was stored in the db.

problem: if user changes date to invalid value before 1st save. then you could use the current date.

or you can go for:
if entered date is not valid, use current date

in both cases the user should get a error message.

ruud · 2006-12-13 17:49:05

Hmm… if you don’t mind letting MySQL do the error checking, you can do this:

store the old date (or current date for new articles) in a hidden form field when displaying the article edit page.
after saving the article, fetch the stored date from the database, compare it with the date entered by the user. If they differ, restore the old date (stored in hidden field), otherwise it’s okay.
It costs an extra, but simple query on each article save, but avoids having to add complex date validation code in TXP

Last edited by ruud (2006-12-13 17:50:18)

hongster · 2008-12-01 14:59:38

The-Exit wrote:

well, for that, i’m not deep enough in the code.

i just thought:

when user clicks “save”
1. check if entered date is valid
2. if date is not valid, use the value, which was stored in the db.

problem: if user changes date to invalid value before 1st save. then you could use the current date.

or you can go for:
if entered date is not valid, use current date

in both cases the user should get a error message.

I agree with those suggestions. By default, the input fields for article expiry date is empty, when new article is created. When I edit the article and click “Save”, SQL error message appeared on top of the page!

The error start with the following string:
Incorrect datetime value: '0' for column 'Expires' at row 1 update txp_textpattern set Title = ...

ruud · 2008-12-01 15:13:21

hongster, which MySQL version are you using?

hongster · 2008-12-01 15:39:03

MySQL 5.0.45

ruud · 2008-12-01 18:10:41

hongster, can you open a new topic for this specific error and include exact steps to reproduce. This appears to be something that is dependent on MySQL version and may be a bug in TXP.
This topic is quite old and deals with a problem that’s not related to the error message you’re getting.

rooon · 2008-12-01 22:32:08

I’m also getting this error… was the new topic created?

ruud · 2008-12-01 23:02:18

No, but feel free to do so yourself.

Textpattern CMS

Textpattern CMS support forum

#1 2006-12-13 12:45:20

[issue] article time stamp and incorrect date

#2 2006-12-13 15:42:05

Re: [issue] article time stamp and incorrect date

#3 2006-12-13 15:49:13

Re: [issue] article time stamp and incorrect date

#4 2006-12-13 16:00:56

Re: [issue] article time stamp and incorrect date

#5 2006-12-13 16:25:53

Re: [issue] article time stamp and incorrect date

#6 2006-12-13 17:49:05

Re: [issue] article time stamp and incorrect date

#7 2008-12-01 14:59:38

Re: [issue] article time stamp and incorrect date

#8 2008-12-01 15:13:21

Re: [issue] article time stamp and incorrect date

#9 2008-12-01 15:39:03

Re: [issue] article time stamp and incorrect date

#10 2008-12-01 18:10:41

Re: [issue] article time stamp and incorrect date

#11 2008-12-01 22:32:08

Re: [issue] article time stamp and incorrect date

#12 2008-12-01 23:02:18

Re: [issue] article time stamp and incorrect date

Board footer