After upgrading to calendar 2.4 from calendar 2.2 my week display only shows all day events. These events displayed correctly under calendar 2.2. I updated to the latest dev version and the problem still exists.

These events with times display correctly on month and day views.

Looking at the calendar-week.tpl.php template file I un-commented the dsm($items) line and found the $items variable was an empty array.

What things can I do to debug this?

Drupal 6.19
Calendar 2.x dev (Jan 18)
CCK 2.9
Date 2.7

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kristiaanvandeneynde’s picture

Version: 6.x-2.x-dev » 7.x-3.x-dev
Priority: Normal » Major

Same goes for the Drupal 7 variant.

The docblock at the top of calendar-week.tpl.php seems to have been copied directly from calendar-day.tpl.php, even though the passed variables are totally different.

In calendar-day.tpl.php, the $rows array has the following keys:

  • date
  • datebox
  • empty
  • link (Undocumented)
  • all_day
  • items

In calendar-week.tpl.php, the $rows array has the following keys:

  • 0 (Undocumented)
  • multiday_buckets (Undocumented)
  • singleday_buckets (Undocumented)
  • total_rows (Undocumented)

The calendar-week.tpl.php uses an $item variable in a crucial place, even though the variable is empty and undocumented.

Bumping to major as this breaks functionality.

KarenS’s picture

Priority: Major » Normal
Status: Active » Postponed (maintainer needs more info)

You took a D6 report and switched it to D7, where the code is a total rewrite, so you've polluted the original report. Lack of documentation is not a major bug. Beyond that I have no idea what bug you are actually trying to report.

kristiaanvandeneynde’s picture

Version: 7.x-3.x-dev » 6.x-2.x-dev

Ok, apologies.

Tried not to create a new issue and instead posted here.
However, it now turns out the D7 bug is, in fact, a different bug.

I have created a new ticket here: #1454546: Week display for custom hour grouping is broken.

Neslee Canil Pinto’s picture

Issue summary: View changes
Status: Postponed (maintainer needs more info) » Closed (outdated)

The D6 branch is no longer supported so we're closing this issue. If you think this issue still exists in D7 you can open a new one.