Hi,

Here's a PostgreSQL table schema for your module. I hope you will include it in the next version of the software.

Thanks for your time,

CommentFileSizeAuthor
#3 bookmarks_4.pgsql186 bytesdeekayen
#1 bookmarks_3.pgsql15 bytesdeekayen
bookmarks_2.pgsql114 bytesDaniel Lyons

Comments

deekayen’s picture

deekayen commented
StatusFileSize
new bookmarks_3.pgsql15 bytes

Updated to work with http://drupal.org/node/27340

Probably need to check on how I did default NULL values for uname and pword since I don't have pgsql.

Daniel Lyons’s picture

Daniel Lyons commented

The attachment doesn't seem quite right.

I went ahead and peeked at the other patch you referenced, and it looks fine to me on a cursory glance. If you just add the same two lines to bookmarks.pgsql, it should be fine; PostgreSQL can handle DEFAULT NULL.

Do let me know if there's anything else I can help you with PostgreSQL-wise.

deekayen’s picture

deekayen commented
StatusFileSize
new bookmarks_4.pgsql186 bytes

You're right, the file I uploaded is wrong. Perhaps this will make a bit more sense.

deekayen’s picture

deekayen commented
Status: Needs review » Fixed

I committed the original schema to CVS and I'll move the http://drupal.org/node/27340 related changes to the login reminder patch.

Anonymous’s picture

(not verified) commented
Anonymous’s picture

(not verified) commented
Anonymous’s picture

(not verified) commented
Anonymous’s picture

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