I was excited about installing simple_access so I can get my site to do what I want it to do and voila! It's very broken. Sure enough there wasn't PostgreSQL support. After adding the appropriate code into simple_access.install I installed the module and voila! Still broken. This time because the SQL statements weren't adhering to the SQL standard of strings being enclosed in single-quotes. Norty norty!

Attached is the patch (4.7) to get it to work on PostgreSQL and to fix the norty use of double quotes. yeee haaaa! simple_access now works for me! Whoever is maintaining this module now owes me 30 minutes of favours! ;)

--
Sammy Spets
Synerger Pty Ltd
http://www.synerger.com

CommentFileSizeAuthor
#2 simple_access.pgsql.HEAD.patch_0.txt5.03 KBsammys
#1 simple_access.pgsql.HEAD.patch.txt4.64 KBsammys
simple_access.pgsql.4.7.patch.txt4.67 KBsammys

Comments

sammys’s picture

sammys commented
StatusFileSize
new simple_access.pgsql.HEAD.patch.txt4.64 KB

Here is the patch for HEAD.

sammys’s picture

sammys commented
StatusFileSize
new simple_access.pgsql.HEAD.patch_0.txt5.03 KB

OK. I've run into more troubles. The sequence for simple_access_groups.gid field isn't programmed in the standard way so group additions aren't working. There don't appear to be sequence additions in the MySQL code so I can only assume this is uber borked.

Attached is a patch for the full fix including the sequence problem. You guys can add the sequence to the MySQL part of .install. Oh, and since 4.7 and HEAD versions are still the same i'm only going to submit the one patch.

gordon’s picture

gordon commented

thanks @sammys I have commited this to both 4.7 and CVS

gordon’s picture

gordon commented
Status: Needs review » Fixed
Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)

  • Commit 69cefd2 on 5.x-1.x, 5.x-2.x, 6.x-2.x, master, 8.x-3.x by gordon: 
    #71455 - fix up pgsql support for pgsql.