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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Stefan Freudenberg’s picture

FileSize
897 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

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

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

chrisfromredfin’s picture

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.