Project

General

Profile

Actions

Bug #2244

closed

Date of PT/PI/CR/Group archives is not valid

Added by Jonathan CLARKE almost 13 years ago. Updated almost 13 years ago.

Status:
Released
Priority:
2
Category:
Web - Maintenance
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

Description

Unless February 36th is a new kind of leap year (in the future)...


Files

Screenshot.png (12.1 KB) Screenshot.png Jonathan CLARKE, 2012-02-07 12:06
Actions #1

Updated by François ARMAND almost 13 years ago

Ah, silly you, of course that should be read "2012, February, 36th days of the year".

Or perhaps a developer misused "DD" in place of "dd" in a date format string...

Actions #2

Updated by François ARMAND almost 13 years ago

  • Status changed from 2 to In progress
Actions #3

Updated by François ARMAND almost 13 years ago

  • Assignee set to François ARMAND
Actions #4

Updated by François ARMAND almost 13 years ago

  • Status changed from In progress to Pending technical review
  • % Done changed from 0 to 100
Actions #5

Updated by Jonathan CLARKE almost 13 years ago

  • Status changed from Pending technical review to Discussion

Looks OK to me, but didn't we decide to store the date format in one central place to avoid this kind of problem?

Actions #6

Updated by François ARMAND almost 13 years ago

Well, here we have a date and time format, not only a date - but ok, its a really weak argument.
I believe I add a far better one when I choose to not use the standard date formatter, I just can't remember it now. Perhaps something to do with the ordering of the dates, but I'm not sure.

I let it open, and will come back to it if I don't find any good reason to not do so.

Actions #7

Updated by François ARMAND almost 13 years ago

  • Status changed from Discussion to In progress
Actions #8

Updated by François ARMAND almost 13 years ago

  • Status changed from In progress to Pending technical review
Actions #9

Updated by Nicolas CHARLES almost 13 years ago

  • Status changed from Pending technical review to Released

This loooks valid, thank you Francois !

Actions

Also available in: Atom PDF