Sequence needs brackets

MMachnik - January 20, 2009 - 20:12
Project:Path Access
Version:5.x-1.2
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:active
Description

In a multisite environment where we have the sequences table shared, I discovered an entry in that table for path_access which did not follow the convention of using the db/table name prefixes. This presents a problem because each site has its own path_access table but they share the sequence. The prefixes would resolve that issue, so the code needs patching to handle this correctly. I also appended "_pid" to the table name to give the sequence a more descriptive name.

I attempted to address the Postgres handling of the sequence too but I do not have PG, so hopefully someone can test this on the PG side. It worked for me under MySQL.

I have included patches for the install and modules files separately here. I hope that is acceptable. Thanks!

AttachmentSize
path_access.install.seqid_.patch1.68 KB
path_access.module.seqid_.patch630 bytes

#1

MMachnik - January 20, 2009 - 20:18

Note -- this patch should only be applied when doing a new install of the module. An update will be needed to adjust the name of the sequence so it will work with existing installations.

 
 

Drupal is a registered trademark of Dries Buytaert.