Activitiy stream generates warnings

wentsch - October 29, 2009 - 23:21
Project:Activity Stream
Version:6.x-2.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

Hey there,

great module, hope it'll soon be able to replace sweetcron, so thanks at first.

Unfortunately I'm experiencing a little problem. All pages that display activitystream output (profile pages, '/stream/' and '/stream/{uid}' show up with the following warning, always 8 times the same message:

warning: Parameter 1 to theme_activitystream_header() expected to be a reference, value given in D:\xampp\htdocs_wentsch\includes\theme.inc on line 617.

Same issue on different versions, I tried 6.x-1.0-rc2 as well as 6.x-2.x-dev.
Switching themes didn't have an effect, neither (tried Garland, Framework, Five, Ninesixty and BlueTrip)

I'm running Drupal 6.14 on Windows 7 RC with XAMPP 1.7.2 and PHP 5.3.

I almost forgot to mention: the aggregation itself does work, the infos of twitter, flickr and delicious do get pulled into Drupal.

Regards,
Daniel

AttachmentSize
activitystream-error.gif67.12 KB

#1

akalsey - October 29, 2009 - 23:46

It's likely that changing the line in activitystream.module from

function theme_activitystream_header(&$action) {

to

function theme_activitystream_header($action) {

(Remove the & from the line)

Would fix this. PHP5.3 changed how functions and references work in a way that causes issues with how Drupal's theme functions and hooks are implemented.

The other option would be to step back to PHP 5.2.

#2

wentsch - October 30, 2009 - 10:54

Thanks a bunch, that helped :)

#3

akalsey - November 7, 2009 - 01:39
Status:active» fixed

Fixed in CVS

#4

System Message - November 21, 2009 - 01:40
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.