Updated: Comment #0

Problem/Motivation

While reviewing #2027183: hook_menu() title callback is ignored on routes
I thought this was a mistake.
Turns out it's in 4 other places.
Since it's fixed in one place in that issue, this issue will fix it in the other places so it's not copied around more.

Proposed resolution

Remaining tasks

tbd

User interface changes

No.

API changes

No.

Files: 
CommentFileSizeAuthor
#15 2027351-menu-example-docs-15.patch3.36 KBmicnap
PASSED: [[SimpleTest]]: [MySQL] 40,078 pass(es).
[ View ]
#12 2027351-menu-example-docs-12.patch3.44 KBlongwave
PASSED: [[SimpleTest]]: [MySQL] 57,959 pass(es).
[ View ]
#10 2027351-menu-example-docs-10.patch3.44 KBlongwave
PASSED: [[SimpleTest]]: [MySQL] 56,600 pass(es).
[ View ]
#9 2027351-menu-example-docs-9.patch3.43 KBlongwave
FAILED: [[SimpleTest]]: [MySQL] 58,092 pass(es), 1 fail(s), and 0 exception(s).
[ View ]
#3 2027351-2-array-example.patch1.67 KBhelenkim
PASSED: [[SimpleTest]]: [MySQL] 56,781 pass(es).
[ View ]

Comments

Title:tidy up grammer in doc copied around: (ex: array('node', '5'))tidy up grammar in doc copied around: (ex: array('node', '5'))

$ ag "An array of path arguments \(ex: array\('node', '5'\)\)"
core/includes/menu.inc
511: *   An array of path arguments (ex: array('node', '5'))
581: *   An array of path arguments (ex: array('node', '5'))
730: *   An array of path arguments (ex: array('node', '5'))
805: *   An array of path arguments (ex: array('node', '5'))

StatusFileSize
new1.67 KB
PASSED: [[SimpleTest]]: [MySQL] 56,781 pass(es).
[ View ]

I found the four lists and tidy up the grammar related on the #2027183-6: hook_menu() title callback is ignored on routes.
- * An array of path arguments (ex: array('node', '5'))
+ * An array of path arguments, for example, array('node', '5').

Component:menu system» documentation
Status:Active» Reviewed & tested by the community

Looks good.

Thanks! Good cleanup. :)

@jhodgdon did you mean to commit it? I didn't see it in the git log.

Haven't done so yet. Will soon. When I do, I'll put an unambiguous message in the issue and change the status. :)

Status:Reviewed & tested by the community» Needs work
Issue tags:+needs backport to D7

Actually... Can we fix the punctuation? The comma before "for example" should actually be a semi-colon. If we're going to clean this up, might as well not introduce a new error. :)

Status:Needs work» Needs review
StatusFileSize
new3.43 KB
FAILED: [[SimpleTest]]: [MySQL] 58,092 pass(es), 1 fail(s), and 0 exception(s).
[ View ]

I figured I should fix up all instances of "for example" in menu.inc at the same time.

StatusFileSize
new3.44 KB
PASSED: [[SimpleTest]]: [MySQL] 56,600 pass(es).
[ View ]

Improved consistency a tiny bit more.

Status:Needs review» Needs work

Thanks, this all looks good!

Except for one spot -- in the Translate function:

- *   An array of helper function (ex: array(2 => 'menu_tail_to_arg'))
+ *   An array of helper functions; for example, array(2 => 'menu_tail_to_arg'))

This is missing . at the end, and has an extra ) at the end.

Status:Needs work» Needs review
StatusFileSize
new3.44 KB
PASSED: [[SimpleTest]]: [MySQL] 57,959 pass(es).
[ View ]

Fixed as per #11.

Status:Needs review» Reviewed & tested by the community

Excellent! This is ready to commit. Thanks!

Version:8.x-dev» 7.x-dev
Status:Reviewed & tested by the community» Patch (to be ported)

Thanks again! Committed to 8.x. I think we should backport this to 7.x.

Status:Patch (to be ported)» Needs review
StatusFileSize
new3.36 KB
PASSED: [[SimpleTest]]: [MySQL] 40,078 pass(es).
[ View ]

A patch for changes in D7.

Status:Needs review» Reviewed & tested by the community

Looks good, thanks!

Status:Reviewed & tested by the community» Fixed

Thanks again! Committed to 7.x.

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