The Drupal 6.2 menu ripping patch missed this string in the schema description:

modules/system/system.install:684:
'description' => t('A serialized array of function names (like user_current_to_arg) to be called to replace a part of the router path with another string.'),

CommentFileSizeAuthor
#2 system-schema-244942.d7.patch897 bytesFreso
#1 system-schema-244942.patch928 bytespwolanin

Comments

pwolanin’s picture

pwolanin commented
Status: Active » Needs review
StatusFileSize
new system-schema-244942.patch928 bytes

I rolled this against 6.x, but should also apply to HEAD.

Freso’s picture

Freso commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new system-schema-244942.d7.patch897 bytes

Hunk #1 succeeded at 750 with fuzz 2 (offset 69 lines).

Re-rolled against HEAD and RTBC from what I can tell. (Applies without offset to current DRUPAL-6 too, and RTBC there as well (from what I can tell).)

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to DRUPAL-6 and CVS HEAD. Thanks. :-)

Freso’s picture

Freso commented
Status: Fixed » Reviewed & tested by the community

@Dries: Your commit #111031 (#175841: UMN Usability: Remove comment controls for users) effectively rolled this back. The original patch (edit: rolled for D7, that is - D6 branch is still fine and dandy) still applies cleanly though, so setting back to RTBC. (And Dries, please commit from an updated tree next time? :p)

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed. Hopefully we got it right this time. Thanks. :)

Anonymous’s picture

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

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