Small patch attached

CommentFileSizeAuthor
user-op.patch617 bytesalex andrascu

Comments

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Version: master » 6.x-1.x-dev
Status: Active » Needs review

Thanks for the patch, please always set the status to needs review if you provide a patch and choose the correct version so that it will be tested by the testbot.

FYI: The D7 version makes use of the new entity_link() function so it works for all entities in core automatically, without custom code.

alex andrascu’s picture

alex andrascu commented

Nice feature on D7 but until it gets a little bit of a warm-up we're still on D6 with production servers :)

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Fixed

Commited.

Remember to only use 2 spaces instead of tabs and avoid trailing spaces. You can easily catch them when reviewing patches with dreditor.

alex andrascu’s picture

alex andrascu commented

Yeah i just set up my eclipse code formatter accordingly a couple of days ago. Shame on me but i didn't payed much attention to the "beautify" part of standards. All sorted out now.

Thanks for the commit and the good pointers.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.