Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
It would be good to use shorten URLs if shorten module is enabled. Here is a patch that does this.
Comment | File | Size | Author |
---|---|---|---|
#2 | buzzthis_use_shorten_url.patch | 655 bytes | zyxware |
Comments
Comment #1
chia CreditAttribution: chia commentedI think you forgot to attach the patch.
Initially when i developed the module i thought of including the bit.ly api. So that we can track click statistics and display like mashable. But it turns out that bit.ly service is not that good(unless you have pro account).
I didnt get much time to upload that code. I will try to upload the code.
Comment #2
zyxware CreditAttribution: zyxware commentedSorry, here is the patch :-)
Comment #3
chia CreditAttribution: chia commentedAdded, bit.ly url shorten service and click tracking
Comment #4
anoopjohn CreditAttribution: anoopjohn commentedWhy is it that buzzthis is using a custom shorten service? Wouldn't it be better to use shorten module which is already being used by other similar modules like tweetmeme, twitter, sexybookmarklets etc and would very likely have shorten already on the site. So even if bitly is a preferred service you would have to configure bitly settings twice in a case where you have bitly via shorten and also via this module.
Comment #5
chia CreditAttribution: chia commentedBit.ly was used to track the click count. Now, the tracking API is out and there is no need for any url shorten service required. Checkout the latest release.
Comment #7
IceCreamYou CreditAttribution: IceCreamYou commentedHi, I'm the maintainer of the Shorten URLs module.
It would be great if integration with the Shorten URLs module was added to this module. That way people can use services other than bit.ly if they choose to do so -- there's no downside, and the benefit is that people have a greater choice of which service to use.
Thanks
Comment #8
STINGER_LP CreditAttribution: STINGER_LP commented+1, I want use Shorten URLs with this module too!
Comment #9
IceCreamYou CreditAttribution: IceCreamYou commentedBy the way, since the patch in #2 doesn't apply, I think all that's required for integration is adding these 3 lines after line 67 of buzzthis.module:
Comment #10
chia CreditAttribution: chia commentedThe node url is needed to internally remember the rebuzz count for a post
If we use node/%nid thats better IMO
The 'data-url' attribute internally fetches the buzz counter. The url service or any other service might not work properly in case of url change or new alias and the buzz count will be lost.
Comment #11
IceCreamYou CreditAttribution: IceCreamYou commentedOkay I have to be honest, I've never used a Google Buzz button before today, but I went and tried it just now and I have a few thoughts.
First of all, in principle, protecting users from themselves is not in itself a valid reason to decline a feature. So in theory, if people want to use shortened URLs with Buzz, they should have that ability if they choose, even if it's not a great idea in general.
However, I don't really get the point. I thought posting to Buzz would be like posting to Twitter where the URL is part of your post. But from what I tried, it isn't -- so character limits aren't an issue, so there's not really any reason to use a shortened URL.
On the other hand, you can customize the default text in your post using the
data-message
parameter, which this module currently does not use. If this module starts using that parameter, then some people may want to pre-fill the post text with a shortened URL.So basically, my now-more-informed opinion is that this feature probably isn't needed right now, but could be useful in the future.
Comment #12
chia CreditAttribution: chia commentedBuzz is a little different than twitter.
The buzzthis module allows user to share content. When you click on a buzz button. It will link the title to your post, and no url is posted in message and It automatically fetches the teaser. Optionally we can set data-message parameter to use drupal teaser(which is not used currently). IMO i still don't think its needed, using node id will provide consistent counter regardless of changes to url.
In Buzz you can attach images, videos and links to your buzz post. I do plan to use shorten_url token for gbuzz_post module. Where user can post to Buzz from node add / edit form and optionally set a custom message.
Comment #13
anonymous07 CreditAttribution: anonymous07 commentedSubscribe
Comment #14
anoopjohn CreditAttribution: anoopjohn at Zyxware Technologies commentedCan we please mark this as Closed (Wont Fix) as the module is no longer relevant.
Comment #15
anoopjohn CreditAttribution: anoopjohn at Zyxware Technologies commented