Through code study (rather than by finding the error through use) I've found a minor error within the eventrepeat_event_load function.

The case for table and list displays pulls in an optional parameter (optional from the perspective of the event module) to describe the display duration. That parameter is zero based position number 7 (eighth component of the URL) from the event module. Eventrepeat is pulling parameter 6, which is typically the text "all", which sets duration to zero, which then results in the default site duration being used. It is subtle and in a feature of event module that I don't figure too many people use, so this is a very low "minor" priority issue.

I elected to bypass submitting a patch for a one character change. If having a patch will make your workflow go better, just ask for one and I'll make it and post it.

function eventrepeat_event_load()....

      case ('table'):
      case ('list'):

        //table/list end is simply the start day of the viewing period plus the event_table_duration.  if duration
        //is specified as an argument, then make sure it's not longer than a year.  if not, then just use the
        //table duration default or 30 days
        $duration = arg(6);
        $numberofdays = $duration && $duration < 366 ? $duration : variable_get('event_table_duration', '30');
        $endtime = gmmktime(23, 59, 59, $month, $day + $numberofdays, $year);
        break;
      default:

The $duration = arg(6); statement should read $duration = arg(7);

g

Comments

seanbfuller’s picture

Version: 4.7.x-1.x-dev » master
Status: Active » Fixed

Changed to arg(7). Committed to HEAD.

Anonymous’s picture

Status: Fixed » Closed (fixed)