Issue #1103938: Menu constants doc not formatted right and @name misues changes improper @name directives to @defgroup in D6, and essentially back-ports D7 comments to do so.

A couple small changes to these comments were suggested -- see jhodgdon's a) and b) in comment #6 in that issue -- which should therefore be changed in D7/D8 as well.

In menu.inc:

/**
 * @defgroup menu_item_types Menu item types
 * @{
 * Definitions for various menu types.

"various menu types" should be "various menu item types"

And in pager.inc:

/**
 * @defgroup pagerpieces Pager pieces
 * @{
 * Use these pieces to construct your own custom pagers in your theme.

the first line should not be a command, but a description.

Patch coming.

CommentFileSizeAuthor
#1 1110064-defgroup_comments-1.patch903 bytesjeffschuler
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jeffschuler’s picture

Assigned: jeffschuler » Unassigned
Status: Active » Needs review
FileSize
903 bytes
- * Definitions for various menu types.
+ * Definitions for various menu item types.
- * Use these pieces to construct your own custom pagers in your theme.
+ * Theme functions for customizing pager elements.
jhodgdon’s picture

Version: 7.x-dev » 8.x-dev
Status: Needs review » Reviewed & tested by the community

Thanks Jeff. Please commit to d8 and d7.

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 7.x and 8.x. Thanks.

Status: Fixed » Closed (fixed)

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