Forum OpenACS Development: oracle-compat-create.sql in acs-events

Collapse
1: oracle-compat-create.sql in acs-events
Posted by Jarkko Laine on 11/30/04 12:56 PM
Does anyone know why these plpgsql procs are in acs-events? Aren't they kind of general helper procs that could be in core?

OK, but if they are in the right place, there's a bug in acs-mail-lite, since it happily tries to use to_interval() (a proc created in oracle-compat-create.sql) but doesn't have a dependency for acs-events, thus bombing every night.

Collapse
2: Re: oracle-compat-create.sql in acs-events (response to 1)
Posted by Jade Rubick on 12/01/04 06:20 PM
acs-mail-lite should be updated.
Collapse
3: Re: oracle-compat-create.sql in acs-events (response to 1)
Posted by Dave Bauer on 12/01/04 06:27 PM
Jade, In what way? Acs-mail-lite should not depend on acs-events. At least I can't think of a good reason.

If those plpgsql functions are generally useful they should go in acs-kernel/sql/postgresql/postgresql.sql I think. That is where the other functions to mirror oracle syntax are.

Collapse
4: Re: oracle-compat-create.sql in acs-events (response to 3)
Posted by Jade Rubick on 12/01/04 06:44 PM
I'm saying it should be updated to either not use that function, or that function should be moved somewhere else. There's no reason for it to depend on acs-events.