Download & Extend

Should toolbar depend on menu module? (for menu_custom table)

Project:Drupal core
Version:7.x-dev
Component:toolbar.module
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

After installing Drupal with the minimal installation profile (expert.profile), then enabling the new toolbar module, I get:

PDOException: SQLSTATE[42S02]: Base table or view not found: 1146 Table 'drupalcvs.menu_custom' doesn't exist: INSERT INTO {menu_custom} (menu_name, title, description) VALUES (:db_insert_placeholder_0, :db_insert_placeholder_1, :db_insert_placeholder_2); Array ( [:db_insert_placeholder_0] => admin_shortcuts [:db_insert_placeholder_1] => Administration shortcuts [:db_insert_placeholder_2] => The <em>Admininstration shortcuts</em> menu contains commonly used links for administrative tasks. )  in toolbar_install() (line 23 of modules/toolbar/toolbar.install).

So, should toolbar depend on menu.module or does this expose a bigger issue in the way the menu's are configured? ie should menu_custom be a permanent table?

Comments

#1

Status:active» needs review

Yes, I think toolbar needs a menu dependency.

AttachmentSizeStatusTest resultOperations
toolbar-info.patch559 bytesIdlePassed: 11563 passes, 0 fails, 0 exceptionsView details

#2

Status:needs review» reviewed & tested by the community

Yes, agreed.

#3

Status:reviewed & tested by the community» fixed

Committed to CVS HEAD. Thanks.

#4

Status:fixed» closed (fixed)

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

nobody click here