Duplicate functionality?

mcrittenden - April 13, 2009 - 02:48
Project:DiggBar Blocker
Version:6.x-1.0-beta1
Component:Miscellaneous
Category:task
Priority:minor
Assigned:henrrrik
Status:needs review
Description

This module may be seen as a duplicate of http://drupal.org/project/diggbar although this module seems to redirect the user to the actual URL and that module displays a message. Might be good to combine forces and have one module which gives the option to do both/either. Just a suggestion :).

#1

henrrrik - April 13, 2009 - 12:07
Assigned to:Anonymous» henrrrik

diggbar_blocker.module currently has a couple of advantages.

  1. Removes the DiggBar
  2. Redirects to the original URL or a special page
  3. Since it's JavaScript it works with any level of caching, including Boost module or a reverse proxy cache (which might be useful if you're dugg)

#2

mcrittenden - April 13, 2009 - 14:21

Hmm, I missed the part about redirecting the user to a specified URL. That pretty much replaces the need for a special message ala /project/diggbar, so never mind :).

#3

mannkind - April 14, 2009 - 23:23

EDIT: Lame, it didn't save my comment!

In the spirit of collaboration and consolidation, I submit a patch (against CVS) to you sir!

AttachmentSize
diggbar_blocker-431902.patch 3.87 KB

#4

mannkind - April 14, 2009 - 23:23
Status:active» needs review

#5

mcrittenden - April 15, 2009 - 00:30

Mannkind, awesome. If I could send you a cookie right now, I totally would. Hopefully henrrrick will pick it up and get the best of both worlds :).

#6

Dave Reid - April 16, 2009 - 04:40

This module should actually be marked as a duplicate of the *original* one, Frameprevention.

#7

henrrrik - April 16, 2009 - 13:18

The main difference is that this module only targets digg.com as referrer (+ the redirect functionality). It's more of a temporary campaign module against the DiggBar and with the latest developments it seems it might not be necessary for much longer.

@mannkind: Cheers for the patch, though I don't think either of us needs to spend more time on this.

 
 

Drupal is a registered trademark of Dries Buytaert.