View Issue Details

IDProjectCategoryView StatusLast Update
0015671phplist applicationUser Managementpublic23-05-12 02:00
Reporterduncanc 
PriorityhighSeveritymajorReproducibilityalways
Status resolvedResolutionfixed 
Product Version2.10.16 
Target Version4.0.xFixed in Version2.11.7 
Summary0015671: Function parseDate() does not allow valid dates
DescriptionThere are some serious errors in the function parseDate() in file admin/connect.php.

It refers to a non-existant variable $value that means the only allowed date format is dd-mm-yyyy.
Additionally the processing for the date format yyyy-mm-dd uses incorrect regex matches (it should be matches 2,3,1 not 3,1,1).

This is actually in version 2.10.17, but the drop down list for Product Version does not include that.

See attached file for a corrected version of the function.
TagsNo tags attached.

Activities

duncanc

30-01-12 18:53

developer  

parseDate.php (866 bytes)

support

01-02-12 12:20

administrator   ~0051440


marking it for 2.11, unless you think it's breaking something crucial in 2.10, in which case it can go there.

duncanc

01-02-12 16:09

developer   ~0051448

Well, it is stopping the import of date fields in YYYY-MM-DD format, which is probably the most common date format people use. That sounds like a serious break to me. As I have provided a fix I think that it should go into the next release, 2.10.18.

In general, shouldn't fixes be applied to the next release? If not, then that's a discouragement to people reporting problems.

michiel

23-05-12 02:00

manager   ~0051586

applied to trunk as well as 2.10 branch