I know I know...

Now there is the problem, I install the AddThis module, everybody's happy, and then some day my customer clicks on it and book marks their admin/whatever page on their public Delicious bookmark recorder. Then all the people who go by click on it and it gets them to a page saying that they are not authorized to see the page. These "people" will include all the robots such as Google Bot.

Trying to teach all the customers in the world not to do that, or having to edit the block each time, is quite tedious. Now if there is a way to pre-fill the block info instead of doing what I offer here, it could be a lot better! But I do not know that it is possible. From what I can see in the block.module, there isn't a way to do that otherwise in D6.

Anyway, I'm offering a patch 8-)

Those who think this is a good idea, just vote for it! 9-)

CommentFileSizeAuthor
addthis-autohide-block-6.x-1.x.patch825 bytesAlexisWilke

Comments

vesapalmu’s picture

vesapalmu commented

I can see that this could be an issue for some clients, but isn't this the reason block visibility rules exist in Drupal?

AlexisWilke’s picture

AlexisWilke commented

The block visibility is neat, but when I get 10 customers a day from my advertising, it will be a lot of work to edit them all. When if it is done in code, it works automatically! 8-)

And it saves anyone from having to do it in the first place. I already have 100's of things to fix up like that all over... So I thought this patch could be useful.

Note that it could be ameliorated a bit since it still appears in the main admin page at this time.

Thank you for your consideration.
Alexis

vesapalmu’s picture

vesapalmu commented
Status: Needs review » Closed (won't fix)

If you want to do this kind of visibility settings for a number of sites you can work around it by theming the module at your themes template.php. I'm not accepting this patch because there are other easy ways to do this and your case is quite special. Some people could even want to share something under admin, I just don't want to create hard coded special cases like this one. Hopefully theming will do the trick for you.