JavaScript breaks if teaser length is too long

On my site, the teaser length is the same as the post, and some posts can be fairly long. It turns out that the JavaScript variable "digg_bodytext" contains the teaser, and in the case of my site, the entire post. When the post is long, the JavaScript breaks.

I have not tested the boundary case that causes this to break. I just discovered it on my site and had to patch it quickly, so I have not done much investigation.

Here is the patch code for the diggthis.module file (around line 107)

From:

  $skin = variable_get('diggthis_button_skin', 'standard');
  $title = drupal_to_js($node->title);
  $teaser = strip_tags($node->teaser);
  if (strlen($teaser) > 400) {$teaser = substr($teaser, 0, 400);}
  $teaser = drupal_to_js($teaser);

To:

  $skin = variable_get('diggthis_button_skin', 'standard');
  $title = drupal_to_js($node->title);
  $teaser = strip_tags($node->teaser);
  if (strlen($teaser) > 400) {$teaser = substr($teaser, 0, 400);}
  $teaser = drupal_to_js($teaser);

The code is a quick fix. I should use a number other than 400, but I didn't know how to come up with something meaningful. This code needs to be reviewed before it's used in production sites (although I'm most definitely using it.) =)

cheers,
sebs