Limit access and editing menu by user

Question

I have created a new menu and I'm trying to limit the access to a particular user to edit instead of role with the following code but it's not working, I still can't see the menu and edit the menus items by logging in with the relevant user login (user id 3 in this case). Wondering what's the proper way of doing.

/**
 * Implements hook_menu().
 */

function menu_access_by_uid_menu(){
  $items = array();
  $items['admin/structure/menu/manage/new-menu'] = array(
    'title' => 'New Menu',  
    'access callback' => 'menu_access_by_uid_validate',       
  ); 

  return $items;
}


function menu_access_by_uid_validate() {

 global $user;

 if (user_is_logged_in() && ($user->uid == 1 || $user->uid == 3)){
   return TRUE;
 }

 return FALSE;
}

Answer 1

There is a nice module to handle visibility of menu items. You can use the module or see code in side the module.

May be It will help you. I am not sure :P

https://www.drupal.org/project/menu_item_visibility

Answer 2

If all you want to do is limit access to the admin pages for a specific menu then something like this should work:

function mymodule_menu_alter(&$items) {
  $items['admin/structure/menu/manage/menu-new-menu']['access callback'] = 'mymodule_access_by_uid';
  $items['admin/structure/menu/manage/menu-new-menu/%']['access callback'] = 'mymodule_access_by_uid';
}

function mymodule_access_by_uid() {
  global $user;

  if ($user->uid == 1 || $user->uid == 3) {
    return TRUE;
  }

  return FALSE;
}

You need to use hook_menu_alter() to add your own access callback for an existing page because hook_menu() is for defining your own menu items and page callbacks.

Also, I don't think you need user_is_logged_in() since that's already implied if the uid is something other than 0.