CiviCRM Community Forums (archive)

I changed the long date format setting here:
http://drupal.demo.civicrm.org/civicrm/admin/setting/date?reset=1

to "%A %B %E%f, %Y %l:%M %P"

following the POSIX standard I found at:
http://www.opengroup.org/onlinepubs/009695399/functions/strftime.html

But it came up with a capital AM (which is what you would expect if you were using PHP date).

Yes, it seems we don’t handle %a/%A in CiviCRM.

I see this bug: http://issues.civicrm.org/jira/browse/CRM-4893 and this related forum post.

Have you made a choice about what you want to settle on?

We’re switching the calendar widget in CiviCRM 3.1, so we’ll make a call based on the new calendar choice.

If so, you think this bug is simple enough for me to fix?

You can try tweaking the CRM_Utils_Date::customFormat() function so that it maps %A to the appropriate weekday (the mapping is provided by the CRM_Utils_Date::getFullWeekdayNames() method, so you need to extract the weekday from the date and map %A to the weekday-number → weekday-name in the $date array near the end of the function).

Hm, it seems we map %A to AM/PM (right above where you patched the file), so I’d rather not break the existing behaviour in a stable release. I’ll see what we do for the calendar stuff in v3.1.

On that version Date.php, line 378 should be changed to:

'%A' => date('l',strtotime($dateString)),

And then you can use %A to stand for the day of the week.

However, Piotr is right - it's not a great solution because anyone who is using %A now and expecting to get AM/PM will be surprised.

As far as I can tell, CiviCRM is still using a mixture of POSIX standards and PHP standards (PHP says %A should be AM/PM while POSIX says it should be the day of the week). 

CRM-4893 was originally opened to solve that problem, but the scope seems to have changed. Now this issue has been opened: http://issues.civicrm.org/jira/browse/CRM-10367 which directly addresses the problem (but is still unresolved).

jamie