Rewrite implementation of hook_menu() to allow sub-pages of profile/edit

+++ b/wysiwyg.admin.incundefined
@@ -517,7 +517,7 @@ function wysiwyg_profile_overview($form, &$form_state) {
-        '#markup' => l(t('Edit'), "admin/config/content/wysiwyg/profile/$id/edit"),
+        '#markup' => l(t('Edit'), "admin/config/content/wysiwyg/profile/$id"),

I'd like to keep this linked to /edit to follow the pattern set by other modules. We don't currently have a way to just "view" a profile without editing it, but maybe that will be useful in the future and I'd like to reserve the non-edit path for that.

+++ b/wysiwyg.moduleundefined
@@ -54,19 +54,21 @@ function wysiwyg_menu() {
-  $items['admin/config/content/wysiwyg/profile/%wysiwyg_profile/edit'] = array(
-    'title' => 'Edit',
+  $items['admin/config/content/wysiwyg/profile/%wysiwyg_profile'] = array(
+    'title' => 'Edit profile',

We need a title callback here. It would make it a lot easier to implement Admin menu's hook_admin_menu_map() later if we simply return the name of the format a profile is attached to here, since that's the primary way to identify a profile.

+++ b/wysiwyg.moduleundefined
@@ -74,10 +76,8 @@ function wysiwyg_menu() {
-    'type' => MENU_LOCAL_TASK,
     'weight' => 10,
+   'type' => MENU_LOCAL_TASK,

No need to move this.

Took the liberty of modifying your patch. What do you think?

Committed to 7.x-2.x and 6.x-2.x. with appropriate path changes. Expanded the comment to the title callback to clarify the parameter and return value.

This change will be in the -dev snapshots within 12 hours and it will be part of the next official releases.

Added bonus for D7: Implementation of hook_admin_menu_map().

Thanks for suggesting, patching and testing this!