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

Comments

terkkila’s picture

terkkila commented

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

beauz commented

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

jmdoran’s picture

jmdoran commented
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

hdcwebteam commented

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

Proct0t commented

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

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Version: 6.x-1.3 » 6.x-1.4
Component: Documentation » Code
Category: support » bug
Status: Active » Reviewed & tested by the community
StatusFileSize
new custom_breadcrumbs-DRUPAL-6--1.install.patch712 bytes

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

thepanz’s picture

thepanz commented

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

yang_yi_cn’s picture

yang_yi_cn commented

This patch works! without it I cannot add any settings

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Title: Module requirements do not match drupal 6 » Wrong table definition in install schema
MGN’s picture

MGN commented
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.