Closed (fixed)
Project:
Office Hours
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
25 Mar 2009 at 20:39 UTC
Updated:
20 Apr 2012 at 22:31 UTC
In Drupal's date and time configuration (admin/settings/date-time), you can select what day of the week it's to be considered the first.
Office hours does currently not respect this setting, so when the field is displayed, Sunday is always listed first.
Comments
Comment #1
Ozeuss commentedTrue, and i think this is a good idea. I've added this to today's snapshot.
Comment #3
matias commentedI found this problem in 6.x-1.0.
Here is an easy fix:
On office_hours.theme.inc in the function "theme_office_hours_formatter_default"
Al line 29
change
if ($keys[0] != $days[$first] && in_array(t($days[$first]), $items) ) {
for
if ($keys[0] != $days[$first] && array_key_exists(t($days[$first]), $items) ) {
Comment #4
Ozeuss commentedThanks, committed to head
Comment #6
jdanthinne commentedI had the same problem with grouped values.
Here's my fix:
In office_hours.theme.inc, changed
to
Comment #7
johnvPS. group formatter is not supported in D7 (yet).
Comment #8
johnvComment #9
johnvI have added a partial backport from the D7-version in commit da3654e.