For a better integration with other drupal modules, administration settings page should be inside admin/settings.

- **nodejs/nodejs.module**

-    'admin/config/nodejs' => array(
+    'admin/settings/nodejs' => array(

-    'admin/config/nodejs/config' => array(
+    'admin/settings/nodejs/config' => array(

- **nodejs/nodejs_config/nodejs_config.module**

-    'admin/config/nodejs/js' => array(
+    'admin/settings/nodejs/js' => array(

- **nodejs/nodejs_notify/nodejs_notify.module**

-    'admin/config/nodejs/nodejs_notify' => array(
+    'admin/settings/nodejs/nodejs_notify' => array(

-    'admin/config/nodejs/nodejs_notify/settings' => array(
+    'admin/settings/nodejs/nodejs_notify/settings' => array(

-    'admin/config/nodejs/nodejs_notify/broadcast' => array(
+    'admin/settings/nodejs/nodejs_notify/broadcast' => array(

Comments

julien’s picture

julien commented
Status: Active » Fixed

@carnau, thanks, it has been commited here ddc99b3

carnau’s picture

carnau commented
Status: Fixed » Needs work

I forgot to mention that also there is an unnecessary weight property in 'admin/settings/nodejs'. Can you remove it? If someone needs to change this, it can be done using hook_menu_alter(), so a default value is better.

'weight' => -20,
julien’s picture

julien commented
Status: Needs work » Fixed

@carnau, thanks, it's commited. I'm changing the status to fixed.

carnau’s picture

carnau commented

Thank you so much ;)

Status: Fixed » Closed (fixed)

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