PostgreSQL support on install

yagur - November 5, 2007 - 20:53
Project:Login Security
Version:5.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:ilo
Status:closed
Description

postgresql support patch.

AttachmentSize
patch.patch638 bytes

#1

ilo - November 6, 2007 - 09:17
Assigned to:Anonymous» ilo
Status:needs review» reviewed & tested by the community

Thanks a lot for your work! I will close the uninstall flow of the module and hope you could help testing the psql setup. Commiting to the cvs will happend as soon as I finish the uninstall code .

#2

yagur - November 7, 2007 - 23:18

I have a poor english skill(plz excuse me)... so I might missunderstand your reply. Do you mean Login-Security module need a PostgreSQL uninstall query?
Uninstallation works well with mysql query(Only creating table is slitely different). They use same SQL query(which is "drop") so no additional uninstall query code is going to be needed.
And installing and uninstalling the module works fine with PostgreSQL, and the module worked well with PostgreSQL.
Thanks for your login module(just what i needed).

#3

ilo - November 7, 2007 - 23:29

Hey.. no, thanks to you.. And sorry for my annoying english (from spain.. jeje)..

I said that when I finish the unistall code in the module, I will commit all the changes.. And also, if you could do some test in postgreSQL database like having failed login attempts from different IP addresses and so would be great..

I have copied the uninstall code from other module so I guess it should work well.

#4

ilo - January 15, 2008 - 21:55
Status:reviewed & tested by the community» fixed

Fixed and released as stable. Thank you!

#5

ilo - January 15, 2008 - 21:56
Status:fixed» closed

finished

 
 

Drupal is a registered trademark of Dries Buytaert.