The category module needs an installation script for pgSQL, and the SQL embedded within its various functions needs to be tested with pgSQL.

CommentFileSizeAuthor
#4 category.pgsql.1.txt3.86 KBAnonymous (not verified)
#3 pgsql.txt3.77 KBAnonymous (not verified)

Comments

Anonymous’s picture

The tables will need some changes to work properly on postgresql. One of the tables has no primary key defined? which PostgreSQL doesn' t like and the indexes need to be declared seperatly. I have had it running under postgreSQL, just the tables need a few tweaks.

Jaza’s picture

You've had it running under PostgreSQL? Great! It would be great if you could post the SQL CREATE TABLE statements that you used, as well as reporting any problems that you may have had with the SELECT / INSERT / UPDATE / DELETE statements within the module.

Anonymous’s picture

StatusFileSize
new3.77 KB

This should get it working on PostgreSQL. The only problems are nodes are not being linked to categories and PostgreSQL does support sequences, as a database feature, but does not use the sequences table like MySQL. The SERIAL data type is set in the category table, so categories get a unique ID. Most of the functionality works under PostgreSQL, but its not linking to anything else yet.

Anonymous’s picture

StatusFileSize
new3.86 KB

New SQL for recent updates, seems to work. The nodes seem to stick to categories now, maybe the container was configured wrong. Only thing is the sequence SQL and its listing the node types allowed in a container twice though it only stores the it once in the database.

Jaza’s picture

Status: Active » Fixed

DB install script committed to HEAD. I have noticed some problems with some queries that are run when using category (post-install), but the install script itself seems to be fine.

Problems with other queries within the category package should be filed as separate issues, as this issue is related to the initial creation of the install script (which is now done).

Anonymous’s picture

Status: Fixed » Closed (fixed)