Download & Extend
Issues

The "Parameters" section shows HTML tags are rendered as plain text

Posted by kiamlaluno on August 30, 2011 at 11:07pm
Project:Drupal core
Version:7.x-dev
Component:documentation
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)
Issue tags:needs backport to D7

Issue Summary

API page: http://api.drupal.org/api/drupal/modules--node--node.module/function/nod...

The "Parameters" section is showing HTML tags as if they were plain text.

The issue is also present in the pages for Drupal 6, and Drupal 7.

AttachmentSizeStatusTest resultOperations
node_title_list | Drupal API.png31.33 KBIgnored: Check issue status.NoneNone
Login or register to post comments

Comments

#1

Posted by Devin Carlson on August 31, 2011 at 2:47am
Status:active» needs review

After checking out the Doxygen formatting conventions and referencing other parts of the module, I think node_comment_statistics should be wrapped in curly brackets and comment_count should be wrapped in quotation marks (the existing code tags should be removed).

AttachmentSizeStatusTest resultOperations
fix-node-title-list-documentation-formatting-1264894.patch683 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 32,895 pass(es).View details

#2

Posted by jhodgdon on September 2, 2011 at 3:07pm
Status:needs review» needs work

Thanks for the patch!

The curly brackets are definitely a good idea. I think the field name doesn't need to be in quotes though?

Also, could you:
a) Change "DB" to "database" in the first line of that @param
b) Re-wrap the paragraph lines so they are as close to 80 characters as possible without going over

Thanks!

#3

Posted by James_Stallings on September 27, 2011 at 1:53am
Status:needs work» needs review
AttachmentSizeStatusTest resultOperations
fix-node-title-list-doc_patch1_1264894_2.patch857 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 36,066 pass(es).View details

#4

Posted by jhodgdon on September 27, 2011 at 3:47pm
Status:needs review» reviewed & tested by the community

Thanks! Looks good to me, for 7.x/8.x.

#5

Posted by catch on September 28, 2011 at 3:20am
Version:8.x-dev» 7.x-dev

Committed to 8.x, moving to 7.x for webchick.

#6

Posted by jhodgdon on September 28, 2011 at 2:54pm

#3: fix-node-title-list-doc_patch1_1264894_2.patch queued for re-testing.

#7

Posted by webchick on September 30, 2011 at 7:55pm
Status:reviewed & tested by the community» fixed

Committed and pushed to 7.x. Thanks!

#8

Posted by System Message on October 14, 2011 at 8:01pm
Status:fixed» closed (fixed)

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

nobody click here