Great module, but small 'bug' though.

In line 91 you have code:

$vars['link'] = l($vars['day'], $vars['url']) . $bt_text;

Which outputs a link to a day's page (like calendar-eventdate/month/2011-12-01) , even though there's no day view for the day with events.

I HAD to hack this string to change output. 

$vars['link'] = l($vars['day'], '#',  array('fragment' => '', 'external' => TRUE)) . $bt_text;

But please, fix it to consider output when day view is disabled.

Comments

rjb’s picture

rjb commented
Assigned: Unassigned » rjb
Status: Active » Needs work

Thanks! Will look into it.

rjb’s picture

rjb commented
Version: 7.x-2.0 » 7.x-2.1
Status: Needs work » Fixed

A fix is included in 7.x-2.1. If the day view is disabled, then a span element is used instead of a link. The span is used because we need a placeholder for the div containing the balloon data.

skolesnyk’s picture

skolesnyk commented

Thank you!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.