Tweetmeme not compatible with AHAH content

We are seeing this exact issue, but when previewing a view. The JS that tweetmeme provide doesn't seem compatible with drupal AHAH stuff.

So it turns out that the js that you have to include from tweetmeme:
http://tweetmeme.com/i/scripts/button.js
Includes a nasty document.write() causing the page to be replaced with just the tweetmeme button when run from any other scope than the initial page load.

Our (not ideal) solution is to have a local copy of the tweetmeme script and tweak it to use jQuery.

So, you'll want to override the theme_tweetmeme_button() with this:

/**
 * Displays the TweetMeme button.
 *
 * This fixes issue: #571938 and means that tweetmeme
 * buttons can work when AHAH'd onto a page.
 */
function theme_tweetmeme_button($url, $style = 'normal') {
  $id = md5(uniqid(mt_rand(), TRUE));
  drupal_add_css(drupal_get_path('module', 'tweetmeme') . '/tweetmeme.css');
  $output = '<div class="tweetmeme tweetmeme-button" id="tweetmeme-widget-' . $id . '"><script type="text/javascript">tweetmeme_url = "' . $url . '";';
  if ($style != 'normal') {
    $output .= "tweetmeme_style = '$style';";
  }
  $output .= "tweetmeme_element_id = '#tweetmeme-widget-$id';";
  $output .= '</script>';
  //$output .= '</script><script type="text/javascript" src="http://tweetmeme.com/i/scripts/button.js"></script></div>';
  $output .= '<script type="text/javascript" src="' . base_path() . drupal_get_path('theme', 'theme_name') . '/js/button.js' . '"></script></div>';
  
  return $output;
}

And then copy the button.js from the tweetmeme server to your local server, and change the document.write line (might want to JS beautify the code so you can see what you're doing) to this:

$(tweetmeme_element_id).append('<iframe src="' + src + '" height="' + h + '" width="' + w + '" frameborder="0" scrolling="no"></iframe>');

I brought this up to them a while ago and they suggested using the iFrame directly:
http://help.tweetmeme.com/forum/topic/64#post-240

I think what you suggest is a better solution/hack though. It also means we could stick the JavaScript at the bottom of the page.

I have this problem too when trying to update, preview, or save a view - page refreshes to completely whitescreen. I don't see the tweetmeme image, but I do see that it's trying to contact tweetmeme in the browser's status bar. I also seem to have the same or similar problem with the addthis module.

Where possible, remote resources should be stored locally - this is much better from a performance point of view.

Attached is a patch that incorporates the above fix AND includes the js file in the module directory, so no need to download it and put it in your theme directory. Not sure of the licence of the tweetmeme script though, so that may be a no go.

Oops. Bad patch - IGNORE THE PATCH ABOVE. forgot to comment out the original code! Use this one instead.

Damn it. Really sorry for the multiple posts... forgot to alter the js that time!

I'm facing this issue as well.

Using TweetMeme w/calendar views causes a white screen, or when creating a view w/a node type that uses tweetmeme also causes a white screen.

Does this patch still work?

If so - how would I install it?

I have never used a patch before.

Thank you.

Sub. I'm having this issue and would love to test but this patch is old and don't have time to update it (deadline Sunday!). If someone redoes this patch based on dev, I'll happy report back.

I like this, but am concerned what would happen if they change their service or their JavaScript changes. It's nice to be able to aggregate the JavaScript, but I'm not sure if we want to assume their JavaScript will never change.

There's also #751374: Use TweetMeme API instead of JavaScript which might be a better solution.

I know this is not the preferred solution, but we needed an updated patch to apply to 1.3 to get things working, and we needed it now, so it was quick and easy for us to just update this patch./ Revised version attached.