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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

spxero’s picture

This has happened to me, too. I have the jQuery UI module enabled.

spxero’s picture

This 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.

jiv_e’s picture

Issue summary: View changes
Status: Active » Closed (works as designed)

This 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!

Chris Charlton’s picture

This is still an issue in Date-7.x-2.9 :(

Chris Charlton’s picture

Status: Closed (works as designed) » Active
Chris Charlton’s picture

Is there a way to not have the suffix? This bug is killing our UX.

Chris Charlton’s picture

Version: 6.x-2.4 » 7.x-2.x-dev
Issue summary: View changes

Changing version affected since D6 isn't supported, and this issue is appearing for D7.

daggerhart’s picture

jQuery 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.

DamienMcKenna’s picture

Thanks 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.

DamienMcKenna’s picture

Status: Active » Needs review
DamienMcKenna’s picture

Status: Needs review » Needs work

What if a string has "\S" in the string? This change would break such a string.