I have encountered an issue on our staging server (doesn't occur on dev) where the suffix portion of a date format is not handled correctly. This occurs when picking a date from the popup during creation or editing of a node. When I pick a date, it shows up as "March 24S, 2010" instead of "March 24th, 2010". If I leave it as is, when the date is displayed in the node it shows up as "March 2nd, 2010".
Any ideas? Again, the exact same date format configuration works correctly on my dev site. Same version of jQuery on both machines. The staging server is running php version 5.2.6 and the dev site is running 4.4.8 which is the only difference I can find, but I can't understand why 4.4.8 would work and 5.2.6 wouldn't.
Comment | File | Size | Author |
---|---|---|---|
#8 | 751696-date-popup-not-handling-suffix-correctly-8.patch | 405 bytes | daggerhart |
|
Comments
Comment #1
spxero CreditAttribution: spxero commentedThis has happened to me, too. I have the jQuery UI module enabled.
Comment #2
spxero CreditAttribution: spxero commentedThis issue is with the suffix handling only. The same options without the suffix work correctly, i.e. "June 21, 2010" works, whereas "June 21st, 2010" will not (picker changes date to "June 21S, 2010")
Minor annoyance only, I consider this a reasonable workaround.
Comment #3
jiv_e CreditAttribution: jiv_e as a volunteer commentedThis issue is getting old with no activity. I'm cleaning the issue queue, so I'll close this. Please feel free to reopen if needed!
Comment #4
Chris CharltonThis is still an issue in Date-7.x-2.9 :(
Comment #5
Chris CharltonComment #6
Chris CharltonIs there a way to not have the suffix? This bug is killing our UX.
Comment #7
Chris CharltonChanging version affected since D6 isn't supported, and this issue is appearing for D7.
Comment #8
daggerhart CreditAttribution: daggerhart as a volunteer commentedjQuery UI's datepicker doesn't allow for suffixes.
Here is a patch that fixes the data-saving bug by stripping out capital S from date formats before they are presented on a date field's setting form.
Comment #9
DamienMcKennaThanks for the patch, we'll try to review it soon.
BTW when you upload a patch it helps to set the status to "needs review", that triggers the testbots and lets others know that there's something review.
Comment #10
DamienMcKennaComment #11
DamienMcKennaWhat if a string has "\S" in the string? This change would break such a string.