the latest 'links_weblink' module has just 2 permissions "create weblinks" and "view weblinks".

it will be great to have "edit own weblinks" permission too,

CommentFileSizeAuthor
#3 links_weblink.module_4.patch1003 bytesyched

Comments

syscrusher’s picture

Assigned: Unassigned » syscrusher
Category: task » feature
Priority: Critical » Normal

I concur that this is a useful feature, and will consider it for the future, but it's not release-critical. :-)

enky’s picture

for 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,

yched’s picture

Status: Active » Needs review
StatusFileSize
new1003 bytes

bump.
I agree that the absence of an 'edit' permission is a big drawback.
Attached patch should do that.

summit’s picture

Thanks for this!
This was just what I needed for my development aid website www.best-information.eu and it works!

greetings,
Martijn

Caleb G2’s picture

Status: Needs review » Reviewed & tested by the community

+1 - this patch from yched works like a charm on the latest release of links for Drupal 5.

ajayg’s picture

can someone please post the entire file?

boris mann’s picture

+1 as well

ajayg’s picture

Any reason this patch was left out from recent release?

syscrusher’s picture

ajayg:

Any reason this patch was left out from recent release?

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

syscrusher’s picture

As 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

syscrusher’s picture

Status: Reviewed & tested by the community » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)