I'm of the opinion that i18n_menu_init() probably shouldn't be running during a profile install. It is very expensive, and greatly slows the installation process.

Is there any reason why this hook implementation must run during a profile install? Or is it perfectly reasonable to do this?

/**
 * Implements hook_init().
 */
function i18n_menu_init() {
  // Don't run prior to installation completion.
  if (variable_get('install_task') != 'done') {
    return;
  }

Comments

joseph.olstad’s picture

Issue summary: View changes
Status: Active » Fixed

Thanks for the patch, but went with another approach and it is fixed by this issue being fixed:
#881322: Multilevel menu fails to create active menu trail with same link_path in different languages

Status: Fixed » Closed (fixed)

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