CREATE TABLE forward_log (
nid integer NOT NULL,
type text NOT NULL,
timestamp integer NOT NULL
);
CREATE index idx_forward_log_nid on forward_log (nid);

CommentFileSizeAuthor
#1 forward_1.module30.04 KBseanr

Comments

seanr’s picture

seanr
he/him
commented
StatusFileSize
new forward_1.module30.04 KB

Have you verified that that works? I unfortunately do not have any way to test that. If others could try it and verify it, I'd appreciate it. I've attached a modified copy of the module with that code in the install function.

Shiny’s picture

Shiny commented

It works in Postgresql 8.0 and 7.4

I used php 5.0 and php 4.4
tested on on ubuntu linux, solaris, and free bsd.

however, i'd like to change the definitions to include a reference to the node table.

CREATE TABLE forward_log (
nid integer references node (nid) NOT NULL,
type text NOT NULL,
timestamp integer NOT NULL
);
CREATE index idx_forward_log_nid on forward_log (nid);

(i was suprised that timestamp isn't a reserved word - you might want to change that for future versions as it may prevent compatibility with other database servers people want to use)

seanr’s picture

seanr
he/him
commented
Status: Active » Fixed

Done. Won't change timestamp, however, as that's used extensively throughout drupal (accesslog, aggregator_item, flood, history, node_counter, etc.), so I'd assume that's been tested pretty thoroughly by now.

Anonymous’s picture

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