I do not have time today to write a patch on this as GIT is not my strong suit ( working on that), but here are the changes I made to make the #URL# work again.
Google sends the event link back under the $events->htmlLink

STARTING ON 690

    $start = $event->getStart();
    $end = $event->getEnd();
    $allday = !empty($start->date);
    $date = !empty($start->date) ? $start->date : $start->dateTime;
    $gcal_time = !empty($start->dateTime) ? format_date($start->dateTime, 'custom', $timeformat) : '';
    $gcal_endtime = !empty($end->dateTime) ? format_date($end->dateTime, 'custom', $timeformat) : '';
    $unixdate = strtotime($date);

	//Changed by tomseeber
	$link = $event->htmlLink;

    if (strlen($title) > 1 && $status != 'canceled' && strlen($date) > 0) {
      $temp[] = array(
        'title' => $title,
        'description' => $description,
        'allday' => $allday,
        'where' => $location,
        'date' => format_date($unixdate, 'custom', $dateformat),
        'timestamp' => $unixdate,
        'time' => $gcal_time,
        'endtime' => $gcal_endtime,
        'link' => $link, // TODO
        'today' => date("Y-m-d") == date("Y-m-d", $unixdate),
        'tomorrow' => (date("Y-m-d",strtotime("+1 day")) == date("Y-m-d", $unixdate)),
        'thisweek' => (date("Y-W") == date("Y-W", $unixdate)),
        'thismonth' => (date("Y-m") == date("Y-m", $unixdate)),
      );
    }
    $count++;
  }

Hope this helps someone else, I will be on vacation for the next few days ( but I will try to get the time to create this as a more formal patch if someone does not get to this first).

Happy Holidays all.

Comments

stefan freudenberg’s picture

StatusFileSize
new897 bytes

I have created a patch, the only difference to the above suggestion is that I opted for using Google_Service_Calendar_Event::getHtmlLink instead of accessing the property directly. That is more in line with the rest of the code.

stefan freudenberg’s picture

Version: 6.x-1.5 » 7.x-1.x-dev

Change version, because I rolled my patch against 7.x-1.x. I assume the patch applies to 6.x-1.x as well.

jackalope’s picture

StatusFileSize
new1.12 KB

I ran into the same problem with the Drupal 6 dev version of this module. I tried Stefan's patch but it didn't apply, so I've created my own patch for 6.x-1.x-dev.

p.s. Hi Stefan! :)

stefan freudenberg’s picture

Status: Active » Needs review

Please review.

chrisfromredfin’s picture

StatusFileSize
new1.14 KB

Patch in 3 wouldn't apply - I think it was generated from the wrong folder. I re-rolled it here.

chrisfromredfin’s picture

StatusFileSize
new1.14 KB

OOPS I typo'd event as "events" which barfed. this is correct. This is a D6 patch, to be clear, based on #3 above.

  • jdwfly committed 610b0ab on 6.x-1.x
    Issue #2382039 by cwells, Stefan Freudenberg, jackalope: #URL# not...

  • jdwfly committed ff56ec3 on
    Issue #2382039 by cwells, Stefan Freudenberg, jackalope: #URL# not...
jdwfly’s picture

Status: Needs review » Fixed

For some odd reason the patch wouldn't apply cleanly, but its a simple change. Thanks for the work everyone. Fixes have been committed to both 7.x and 6.x branches.

Status: Fixed » Closed (fixed)

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