Current implementation does not support PostgreSQL. Attached patches address that. Also implemented the 'un-install' method.

CommentFileSizeAuthor
#1 path_access.install.diff848 bytesvsimkus
path_access.module.diff404 bytesvsimkus

Comments

vsimkus’s picture

vsimkus commented
StatusFileSize
new path_access.install.diff848 bytes

Apparently the other diff was jettisoned. Here it is.

budda’s picture

budda
Primary language English
Location Cheshire
commented
Assigned: Unassigned » budda
Status: Needs review » Fixed

I've added the PostgreSQL code to the install file.

I'm a little wary about the auto incrementing sequence thing that you've added. Why is this being used over Drupals sequence system?

You also forgot to enclose all table names in braces - ie {path_access}

Anonymous’s picture

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

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