Hi,

Just installed the latest beta 2. Very cool concept
During testing, I notice the Digg module doesn't appear to pull back any content and wondered whether there's a problem.
Further, enabling the Twitter module, produces a white screen of death for some reason. Disabling the Twitter module makes it all good again.

Delicous works great. Haven't tested the feed option yet.

Comments

tayknight’s picture

tayknight commented

As far as the twitter problem goes...

the error comes from the activitystream_twitter.module file in the activitystream_twitter_link function. I can't figure out what is causing the error.

rastarr’s picture

rastarr commented

I also don't think it's dragging back (or at least displaying) the correct number of characters either.

tayknight’s picture

tayknight commented

Yeah, I think there is something wrong with the $text going into that function.

I did a print_r($text) in the function, and got "node" after both the preg_replaces. That can't be right.

tayknight’s picture

tayknight commented

I think the issue here is the name of the function. activitystream_twitter_link gets registered as a hook_link. I don't think this is what he module author meant to happen.

if you change the name of the function to something like activitystream_twitter_makelink and then each of the references to the same name, the module works.

akalsey’s picture

akalsey commented
Title: Twitter and Digg » Twitter module causes white screen
Assigned: Unassigned » akalsey
Priority: Normal » Critical

Splitting this into two separate issues. One for Twitter and one for Digg.

Try beta 3 and let me know if Twitter looks better now.

rastarr’s picture

rastarr commented

Ahh cool. Twitter works (and looks) really well now. Great work.
One addition you might like to think about for a future release is to toggle off the display of person-to-person Tweets. i.e. where you specifically send an individual a private Tweet using the syntax of @username

rastarr’s picture

rastarr commented

Still not displaying the correct amount of characters. From the looks of it, we're still about 12 characters short of the maximum allowance under Twitter

akalsey’s picture

akalsey commented
Status: Active » Closed (fixed)

Rastarr,

Please create a separate issue for things not specifically related to the white screen. Marking this issue as closed since Beta 3 fixes it.