The acl.install script assumes that everyone is running MySQL.

I've attached a tar containing an updated acl.install file and a patch file to same (whichever is more convenient) which will add postgresql support to ACL.

CommentFileSizeAuthor
#5 acl-pgsql-patch-3.tar7 KBNugget
#2 acl-pgsql-patch-2.tar10 KBNugget
acl-pgsql-patch.tar7 KBNugget

Comments

merlinofchaos’s picture

merlinofchaos commented

This is committed. I'm not going to spin another release immediately, so for at least a little bit it will appear in the 5.x-1.x-dev branch.

Nugget’s picture

Nugget commented
StatusFileSize
new acl-pgsql-patch-2.tar10 KB

Oof. Just as well, there's a bug. the acl_id fields need to be type serial in pgsql to get auto-incrementing. Revised patch attached.

Nugget’s picture

Nugget commented

I think this may require more work. Will submit a more comprehensive patch once I've tested. I'm running into problems but I haven't determined if they're in ACL or in forum_access (which I'm also in the process of patching for pgsql support).

merlinofchaos’s picture

merlinofchaos commented

Someone mentioned a forum_access bug about not saving the moderator (which is probaby really an ACL bug) so now I have to figure out what's changed since I released this. I'm gong to try to work on that this weekend.

Nugget’s picture

Nugget commented
StatusFileSize
new acl-pgsql-patch-3.tar7 KB

OK, Got it. It was a simple fix. Just need to create the acl_id_seq sequence manually. Tested with forum_access and it all seems to be working just great. Sorry for the confusion earlier.

merlinofchaos’s picture

merlinofchaos commented
Status: Active » Fixed

Ok, this will go out in acl-5.x-1.1 -- thanks for the submission!

Anonymous’s picture

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