This module requires the database user to have either CREATE TEMPORARY TABLES or LOCK TABLES right (I did not test for which, but suspect you know) or gives an error that it cannot find custom_breadcrumb table.

Quote from drupal 6 requirements:
As of 6.x, Drupal core no longer requires CREATE TEMPORARY TABLES or LOCK TABLES, and can be installed without them. However some contributed modules may still rely on them.

It would be nice for novices like myself if this were mentioned in the release notes.

Thanks.

CommentFileSizeAuthor
#6 custom_breadcrumbs-DRUPAL-6--1.install.patch712 bytessun
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

terkkila’s picture

I take that back. I am still getting this error:

user warning: Table 'drupal.custom_breadcrumb' doesn't exist query: SELECT * FROM custom_breadcrumb WHERE node_type = 'page' in H:\www\apache2\htdocs\trojan_website\modules\custom_breadcrumbs\custom_breadcrumbs.module on line 84.

It had disappeared for a while.

beauz’s picture

I'm getting this error as well... any idea how to fix? cannot seem to get this module working...

jmdoran’s picture

Priority: Minor » Normal

The problem is in the custom_breadcrumbs.install file. Basically the module isn't installing its database correctly, because it tries to assign a blank string as a default for a text field. As you run into problems later, it's because the custom_breadcrumbs database doesn't exist.

To fix the problem:
(If you've already tried to install the module once start here)
- Disable the module.
- Uninstall the module. (using the uninstall tab)
(if you haven't installed the module start here)
- Comment out line 33 of custom_breadcrumbs.install
This is the line in the custom_breadcrumbs_schema function that adds the value '' to the key 'default' in the $schema variable.
or: 'default' => '',
- Install the module.

This should properly create the database, and make the module usable.
How did this not get noticed? Further inquiry likely needed.

hdcwebteam’s picture

It's either line 25 or 38 in 6.x-1.3, not line 33. Neither one solves the problem when commented out for me.

Proct0t’s picture

commenting out line 33 ('default' => '',) worked for me for version 6.x-1.3...

sun’s picture

Version: 6.x-1.3 » 6.x-1.4
Component: Documentation » Code
Category: support » bug
Status: Active » Reviewed & tested by the community
FileSize
712 bytes

Correct, columns of the type text must not define a default value.

thePanz’s picture

Your patch is working fine.
also I setted #310294: Installation Warning as a duplicated issue.

yang_yi_cn’s picture

This patch works! without it I cannot add any settings

sun’s picture

Title: Module requirements do not match drupal 6 » Wrong table definition in install schema
MGN’s picture

Version: 6.x-1.4 » 6.x-1.x-dev
Status: Reviewed & tested by the community » Fixed

Committed to 6.x-1.x-dev.

Status: Fixed » Closed (fixed)

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