Install broken on installs with prefixed tables

ewlloyd - January 31, 2008 - 07:02
Project:Taxonomy Lineage
Version:5.x-1.4
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

When you install the module on a site that uses table prefixes, it fails. Warnings are issued indicating that the table "prefix_term_lineage" can't be found.

I've reviewed the code, and found the issue. The .install script doesn't properly escape the table and index names (e.g., "{term_lineage}"). I've fixed it on my installation, and it now works.

I can provide a patch on request.

#1

ingacosta - October 8, 2008 - 12:15

please, send me a patch. You can send it to [mail deleted]
I have the same problem. I tried to fix it but I couldn't resolve the problem at all. I have problems when others modules need this module.

#2

RTFVerterra - June 3, 2008 - 12:56

I also had the same issue. Please send me your patch, I am not a php coder, and I know very little about database. Please send the patch to (email - I removed my email address), thanks.

#3

guardian - April 24, 2008 - 19:33

here is a patch and also all you need to know about patches is here.

AttachmentSize
lineage.install.patch 1.05 KB

#4

guardian - April 24, 2008 - 19:33
Status:active» reviewed & tested by the community

#5

ingacosta - April 28, 2008 - 19:13

I patched it for myself, but thank you very much for your help.

#6

guardian - June 2, 2008 - 19:18

please wake up :)

AttachmentSize
lineage.install.patch 1.28 KB

#7

guardian - June 2, 2008 - 19:28

sorry for the broken patch

AttachmentSize
lineage.install.patch 1.35 KB

#8

xjm - August 13, 2008 - 15:31

Tested patch; it resolved the same issue for me. Hope to see the fix in a release.

#9

AD-DA - August 25, 2008 - 12:05

Tested the patch.
It works.
Is this module still maintained? Could this patch be committed?

#10

Justin W Freeman - September 1, 2008 - 03:54
Status:reviewed & tested by the community» fixed

This patch is in the development version of the module.

After I've been through the issue queue, I'll create a new release. Then the port to 6.x will begin.

#11

Anonymous (not verified) - September 15, 2008 - 04:12
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.