Closed (fixed)
Project:
Service links
Version:
6.x-2.x-dev
Component:
User interface
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
24 Jul 2008 at 23:41 UTC
Updated:
2 Jan 2014 at 23:45 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
AlexisWilke commentedI like this one! I'll attach a new patch since the module changed and the old one does not work anymore...
Thank you.
Alexis Wilke
Comment #2
AlexisWilke commentedOkay! There is new patch with the fixes to this patch. Note that the patch by Rob did not work properly. It would go at the right place on Ping.fm, but it would not send the correct URL and would transform the data (teaser) in a rather ugly way!
There are problems with the default l() and hook_links() feature so I had to generate the href by hand.
Now, there are still two issues:
1) The teaser may include local links, we would need to transform those into global links.
2) The teaser may include "ugly" things such as some javascript that would certainly not run after sent to ping.fm...
Other than that, it works great. 8-)
Thank you.
Alexis Wilke
Comment #3
robloachVery nicely done! Thanks a lot for the patch. Any thoughts on using the Service Links 6.x-2.x-dev for this? They have a hook_service_links that makes it very easy to add links with an icon.
Comment #4
AlexisWilke commentedI think you should have this patch for ping.fm, then ask Service Links to add Ping This! to their list. There is no reason for them not to add Ping.fm to their list, it is big already!
What do you think?
Alexis Wilke
Comment #5
robloachGreat idea!
Comment #6
robloachWhoops, as documented, it should be "body", not summary.
Comment #7
robloachhttp://drupal.org/cvs?commit=250946