• last updated 4 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
1. Changed the manage page to use acs_object.name rather than do its own

special case checking (Oracle)

2. Added code from ssv2 to allow a user to change their notification

frequency without deleting and adding the notification

3. Changed form-driven pages to use ad_form

sync info files with package data and correct provides identifiers

  1. … 36 more files in changeset.
1. Changed cleanup sweep interval to 15 minutes

2. Fixed broken PostgreSQL and Oracle queries that were supposed to select

notifications for deletion but rather returned all notifications if any

were ripe for deletion, causing seemingly random failure to deliver

notifications. Sloan had hacked this with a kludge to only delete

notifications older than two days but this didn't work on openacs.org,

which supports weekly summaries of notifications.

3. Changed notification::new to not insert notifications that have no

pending requests. This avoids having to waste time deleting them

afterwards.

4. Added notification_request::request_exists which returns true if

there's at least one notification request for a given object and

notification type.

5. Added a couple of indexes to speed queries that ran slow on openacs.org.

    • -1
    • +3
    ./sql/oracle/notifications-core-create.sql
    • -3
    • +19
    ./tcl/notification-display-procs.tcl
    • -1
    • +10
    ./tcl/notification-request-procs.tcl
    • -15
    • +15
    ./tcl/sweep-procs-postgresql.xql
merge from oacs-4-6

    • -3
    • +10
    ./tcl/notification-display-procs.tcl
    • -5
    • +12
    ./tcl/notification-request-procs.tcl
    • -2
    • +14
    ./tcl/notifications-security-procs.tcl
  1. … 90 more files in changeset.
add redirect to manage as index page bug 1951

file index.tcl was initially added on branch oacs-4-6.

I18N: converting the notifications package to using the message catalog

    • -0
    • +25
    ./catalog/notifications.en_US.ISO-8859-1.xml
back out Bens changes per bug 3

added notifications documentation!

    • -3
    • +10
    ./tcl/notification-display-procs.tcl
    • -5
    • +12
    ./tcl/notification-request-procs.tcl
    • -2
    • +14
    ./tcl/notifications-security-procs.tcl
apm_package_url_from_key returns url with leading and trailing slashes

Fixed the EmailDomain, so it doesn't default to openacs.org. Instead, I've changed address_domain so that if there's no domain specified here, we do a regexp on [ad_url] to find out what the domain likely is.

Added new files to .info files

  1. … 38 more files in changeset.
reply_address: Use a reasonable sender email address even if there's no object_id or type_id.

typo in query name, default to disallow mail

    • -2
    • +2
    ./tcl/notifications-security-procs.tcl
    • -1
    • +1
    ./tcl/notifications-security-procs.xql
Fixes from sloan

* Increase field sizes for subject

* adds a manage page

* adds a object_id redirect (object-goto.tcl)

* linking to manage page in outgoing message.

    • -2
    • +2
    ./sql/oracle/notifications-core-create.sql
    • -2
    • +3
    ./tcl/notification-procs-postgresql.xql
    • -3
    • +15
    ./tcl/notifications-security-procs.tcl
    • -0
    • +14
    ./tcl/notifications-security-procs.xql
    • -0
    • +39
    ./www/manage-oracle.xql
    • -0
    • +44
    ./www/manage-postgresql.xql
    • -0
    • +23
    ./www/object-goto.tcl
fixes bug 1731/patch 442 Tilman Singer

Maybe redirect for registration.

cleanup javadoc tags (eg @cvs_id @cvs @cvs-id: instead if @cvs-id)

  1. … 687 more files in changeset.
remove superfluous package local masters and use sitewide master by default

  1. … 700 more files in changeset.
move title and context out of page, propigate context

fixing

1. updated .info file with some missing files for PG

2. fixed the notification.delete func in both DBs

3. bumped pacakge version to 0.2d

    • -0
    • +1
    ./sql/oracle/upgrade/upgrade-0.1d-0.2d.sql
    • -0
    • +1
    ./sql/postgresql/upgrade/upgrade-0.1d-0.2d.sql
removed parens from around select clause in for clause which barfs on pg 7.1. fix from davb

copyright OpenForce -> MIT

    • -2
    • +2
    ./sql/oracle/delivery-method-sc-create.sql
    • -2
    • +2
    ./sql/oracle/notifications-core-create.sql
    • -2
    • +2
    ./sql/oracle/notifications-core-drop.sql
    • -1
    • +1
    ./sql/oracle/notifications-replies-package-create.sql
    • -2
    • +2
    ./sql/postgresql/email-sc-impl-create.sql
  1. … 99 more files in changeset.
Added header to the request-new page. Changed form id from 'request' to 'subscribe' to avoid clash with the form builder magic form id 'request'

I took the liberty to go ahead and implement this pretty_name addition to subscribe_url that I mentioned in the bboard thread.

The insert statement included a sysdate, yet it was in the common .xql file. I moved it out to the -oracle and -postgresql files and changed it to now() for postgresql.

    • -0
    • +11
    ./tcl/notification-procs-oracle.xql
    • -0
    • +10
    ./tcl/notification-procs-postgresql.xql
Missing columns in query. These were in the Oracle query, and at least the delivery_method_id was expected, and caused failure down the line

postgres port stuff

    • -0
    • +6
    ./tcl/notification-request-procs-postgresql.xql
  1. … 6 more files in changeset.
remove ad_system_name call from instant notification subject. left it in

batched notifications.

    • -24
    • +23
    ./tcl/notification-email-procs.tcl