I tried out your module on my site and thought it was great but found there were a few things I wanted it to do which it didn't do already...

  1. The Digg link was just prefixed to the body. Personally I wanted it in the links section. This was achieved using the link hook (hook_link).
  2. The Digg was fixed to the page it was displayed on. I use digg in a number of blog pages and wanted to display the digg count for target pages which were external to my site (eg the Drupal 5.0 beta announcement). I achieved this using a combination of hook_nodeapi and hook_form_alter. You can now specify either the target URL of the digg or, alternatively, the digg ID (which you can find in the form on the digg page). If you leave the box blank it defaults to using the current page
  3. Tweaked the XML callback function to limit the results returned by digg for digg pages for a target URL to just 1 (we're only interested in 1, so anything else is excessive). Some people sometimes duplicate diggs target URL's... Silly people!
  4. Tweaked the settings - you can enabled/disable the digg link for body and teaser individually
  5. Rewrote the callback to return the whole array from digg rather than just one piece of info. This allows you to include information in the link like dig count and if its been promoted or not

I've implemented it on my site and overridden the themed output to produce a slimline dig count... See www.thingy-ma-jig.co.uk

CommentFileSizeAuthor
diggthis.patch10.5 KBnicholasthompson

Comments

nicholasthompson’s picture

Status: Reviewed & tested by the community » Closed (fixed)

Patch + more updates submitted...