0

In my installation profile, I am trying to add a menu link for the /import path (from feed_import module). menu_link_save() works for all my menu paths with the exception of /import. Anyone have ideas why?

So, this works and creates a menu path to /user:

$link = array('menu_name' => 'menu-authenticated', 'link_path' => 'user', 'link_title' => 'Account');
menu_link_save($link);

But this does not:

$link = array('menu_name' => 'menu-authenticated', 'link_path' => 'import', 'link_title' => 'Import');
menu_link_save($link);

After installation is finished, /import path works (feeds_import is installed and enabled in the installation profile), but the menu link was not created.

Any help would be much appreciated. Thanks!

  • I think this is because the moment you try to create "import" menu item, the menu item doens't exsist yet. I think your code gets executed too early – Robin Nov 1 '14 at 9:07
  • If you put the menu link on a feature the menu link will be created. I know that it's not you are looking for but give a try. – tvl Nov 3 '14 at 15:42
1

As tvl advised, "import" path is not yet defined in when MYPROFILE_install() is called in MYPROFILE.install. So moved the code over to MYPROFILE.profile and used hook_install_tasks() as explained here: https://www.drupal.org/node/1022020

/**
 * Implements hook_install_tasks().
 */
function MYPROFILE_install_tasks($install_state) {
  $tasks['add_import_link'] = array(
    'display_name' => st('Adds import link to menu-administrator'),
    'display' => TRUE,
    'type' => 'normal',
    'run' => INSTALL_TASK_RUN_IF_NOT_COMPLETED,
    'function' => 'callback_add_import_link',
  );

  return $tasks;
}

function callback_add_import_link() {
  // Add the menu link
  $link = array('menu_name' => 'menu-administrator', 'link_path' => 'import', 'link_title' => 'Import', 'weight' => 5);
  menu_link_save($link);
  menu_rebuild();
}
0
  1. Enable the Feeds module in your my_installation.info file.

    dependencies[] = feeds
    dependencies[] = feeds_ui
    
    ; I also use:
    dependencies[] = feeds_xpathparser
    dependencies[] = feeds_tamper
    dependencies[] = feeds_tamper_ui
    
  2. Edit the my_ installation.profile and add there your menu link:

    function my_ installation_install_finished(&$install_state) {
      // Flush all caches to ensure that any full bootstraps during the installer
      // do not leave stale cached data, and that any content types or other items
      // registered by the installation profile are registered correctly.
      drupal_flush_all_caches();
    
      // Add the menu link
      $link = array('menu_name' => 'main-menu', 'link_path' => 'import', 'link_title' => 'Import');
      menu_link_save($link);
    
      // This is not a correct - full install_finished(&$install_state).
      // Copy the Drupal's function and then edit it.
    }
    

    Remember you must login to see the /import link.

  • Thanks for your response tvl... unfortunately this code did not work for me. I'll probably just go with putting it in a feature like you suggested above. – Steven Nov 3 '14 at 23:27
  • @StevenKim there was a menu name mistake (is was my test menu). Did you change it? Is the install_finished() run properly? – tvl Nov 4 '14 at 16:07
  • Yes, I did. I copied the drupal install_finished function from here api.drupal.org/api/drupal/includes!install.core.inc/function/… into start.profile (my installation profile is called "start"). Then renamed the function start_install_finished, and then tried adding the link with the fixed menu name. No luck still. Thanks for trying to help me out though. I'll keep working at it, and if I find the solution, I'll post it here. – Steven Nov 4 '14 at 21:00
  • I got it! Will post solution below. Thanks for helping me out tvl, you got me pointed in the right direction when you said that the menu item doesn't exit yet ... that was true. But I don't think Drupal lets you override install_finished. So instead, I used hook_install_tasks() as explained here: drupal.org/node/1022020 – Steven Nov 4 '14 at 22:18

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service, privacy policy and cookie policy

Not the answer you're looking for? Browse other questions tagged or ask your own question.