I'm attempting to move the keyauth menu items under each endpoint (as compared to having one Keys page for all endpoints), but not having any luck. Services uses ctools exportable to create menu items, but per @frankcarey I'm just trying to get the menu items to work first.

Patch with services_keyauth_menu() changes is attached.

CommentFileSizeAuthor
#1 1341088_move_services_keyauth_menu_items.patch6.61 KBwonder95
services_keyauth3_menu.patch2.84 KBwonder95

Comments

wonder95’s picture

wonder95 commented
StatusFileSize
new 1341088_move_services_keyauth_menu_items.patch6.61 KB

New patch which moves everything to admin/settings/services_keyauth.

Roze-1’s picture

Roze-1 commented

Subscribing !

frankcarey’s picture

frankcarey commented
Status: Active » Needs work 
+++ b/services_keyauth.moduleundefined
@@ -18,7 +18,7 @@ function services_keyauth_cron() {
 function services_keyauth_access() {
-  return user_access('administer services') && variable_get('services_use_key', TRUE);
+  return user_access('administer services');// && variable_get('services_use_key', TRUE);

This line that is commented out, is it no longer needed?

+++ b/services_keyauth.moduleundefined
@@ -61,31 +74,24 @@ function services_keyauth_menu() {
+function services_keyauth_admin_page() {
+  $output = t('This is the admin page for entering authorization keys.');
+  ¶
+  return $output;
+}
+

I assume this is more complete in the next patch.

frankcarey’s picture

frankcarey commented
Status: Needs work » Fixed

Applied patch from #1. Please take a look at #3 for the next patch if they aren't already fixed.

Status: Fixed » Closed (fixed)

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