Turn these urls into links...

CommentFileSizeAuthor
#7 form_links_1.patch975 bytesChrisKennedy
#4 form_links_0.patch1.33 KBChrisKennedy
#1 form_links.patch804 bytesChrisKennedy
group.form.doc.patch1.06 KBdopry
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ChrisKennedy’s picture

ChrisKennedy CreditAttribution: ChrisKennedy commented
Version: x.y.z » 5.x-dev
Status: Active » Needs review
FileSize
form_links.patch804 bytes

Sync with HEAD. I'm not sure if the line-breaks I have in between @link and @endlink are allowed or not.

RobRoy’s picture

RobRoy CreditAttribution: RobRoy commented

Will need someone more familiar with Doxygen to comment.

drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm commented
Status: Needs review » Needs work

The linebreaks confuse api.module. Please make the @links not break or change this issue's status back after api.module is more accepting.

ChrisKennedy’s picture

ChrisKennedy CreditAttribution: ChrisKennedy commented
Status: Needs work » Needs review
FileSize
form_links_0.patch1.33 KB

Non-breaking @links and another minor comment tweak.

drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm commented
Version: 5.x-dev » 6.x-dev
Status: Needs review » Reviewed & tested by the community

Committed to 5 with adding an extra linebreak on that extremely long line, but not within the link.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Needs work

This patch doesn't seem to apply against CVS HEAD. Needs to be re-rolled, it seems.

ChrisKennedy’s picture

ChrisKennedy CreditAttribution: ChrisKennedy commented
Status: Needs work » Needs review
FileSize
form_links_1.patch975 bytes

Re-rolled for HEAD.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Needs review » Fixed

Committed to CVS HEAD. Thanks for the fast re-roll.

Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)