Closed (fixed)
Project:
Links Package
Version:
master
Component:
Code: links_weblink.module
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
1 Aug 2006 at 09:02 UTC
Updated:
18 May 2007 at 04:00 UTC
Jump to comment: Most recent file
Comments
Comment #1
syscrusher commentedI concur that this is a useful feature, and will consider it for the future, but it's not release-critical. :-)
Comment #2
enky commentedfor me, its very important to have that permission, otherwise, the workspace module
doesnt show EDIT operation for the weblink nodes.
hope it will be committed soon,
Comment #3
yched commentedbump.
I agree that the absence of an 'edit' permission is a big drawback.
Attached patch should do that.
Comment #4
summit commentedThanks for this!
This was just what I needed for my development aid website www.best-information.eu and it works!
greetings,
Martijn
Comment #5
Caleb G2 commented+1 - this patch from yched works like a charm on the latest release of links for Drupal 5.
Comment #6
ajayg commentedcan someone please post the entire file?
Comment #7
boris mann commented+1 as well
Comment #8
ajayg commentedAny reason this patch was left out from recent release?
Comment #9
syscrusher commentedajayg:
Yes, there is. Quite simply, it was after midnight and I was out of time and energy. :-) The last release included some significant bug fixes, and I didn't want to delay getting it out the door. Also, there was one very large contributed patch that added a lot of new features, and I wanted to get that one committed sooner because if I waited too long the code would change to the point where it wouldn't work any more. If I have to do manual merges, I want them to be small rather than large.
This feature is definitely on my short list, and should be in the CVS main trunk within the next couple of days, possibly as early as tonight if my schedule goes well today. Sorry for the delay.
Syscrusher
Comment #10
syscrusher commentedAs promised, I'm committing this patch tonight. It will be in version 5--1-7.
Thanks again for the contrib, and sorry it took so long for me to get around to reviewing it. The patch works flawlessly for me.
Syscrusher
Comment #11
syscrusher commentedComment #12
(not verified) commented