OpenACS / openacs-4 / packages

notifications

Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 44
Last Commit: 11:24
Commits this week: 0
Total Lines of Code (LoC): 8,717
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated 5 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Thursday 07 Jul 2022
4:39 pm

antoniop

Prefer the non cached api when deleting, as the id is cached in the thread and could be stale

Options
    • -2
    • +2
    ./tcl/notification-type-procs.tcl
4:31 pm

antoniop

Some tests for the notifications api, more to come

Options
    • -0
    • +110
    ./tcl/test/notifications-test-procs.tcl
4:31 pm

antoniop

file notifications-test-procs.tcl was initially added on branch oacs-5-10.

Options
    • -0
    • +0
    ./tcl/test/notifications-test-procs.tcl
Thursday 14 Apr 2022
3:30 pm

gustafn

Added parameter for sweep interval for stale notification as stated in a comment already.

Many thanks to Franz Penz for pusing this.

Options
    • -2
    • +3
    ./notifications.info
    • -6
    • +7
    ./tcl/sweep-init.tcl
Thursday 17 Feb 2022
9:28 pm

gustafn

improve input validation

Options
    • -1
    • +3
    ./www/request-new.tcl
Wednesday 16 Feb 2022
2:19 pm

gustafn

Fixes for Oracle 19c: make query portable to Oracle by replacing USING by explicit ON joins

This fixed the following problem, where the problem is not easy to spot on first sight:

[16/Feb/2022:13:26:09][12845.7efd2d3e1700][-sched:0:41:22-] Error: nsoracle.c:1367:OracleSelect: error in `OCIStmtExecute ()': ORA-00904: "NOTIFICATION_REQUESTS"."USER_ID": invalid identifier

: SQL:

: select notification_id,

: notif_subject,

: notif_text,

: notif_html,

: file_ids,

: user_id,

: request_id,

: type_id,

: delivery_method_id,

: response_id,

: notif_date,

: notif_user,

: acs_permission.permission_p(notification_requests.object_id, !>>>!notification_requests.user_id, 'read') as still_valid_p

: from notifications inner join notification_requests using (type_id, object_id)

: inner join acs_objects on (notification_requests.request_id = acs_objects.object_id)

: left outer join notification_user_map using (notification_id, user_id)

: where sent_date is null

: and creation_date <= notif_date

: and (notif_date is null or notif_date < current_timestamp)

: and interval_id = :interval_id

Options
    • -7
    • +10
    ./tcl/sweep-procs.tcl
Tuesday 28 Dec 2021
12:40 pm

antoniop

Deprecate oacs_util::vars_to_ns_set: modern ns_set idioms make this proc obsolete

Options
    • -6
    • +7
    ./tcl/delivery-method-procs.tcl
    • -8
    • +13
    ./tcl/notification-procs.tcl
    • -6
    • +12
    ./tcl/notification-reply-procs.tcl
    • -5
    • +11
    ./tcl/notification-request-procs.tcl
    • -3
    • +8
    ./tcl/notification-type-procs.tcl
  6. … 3 more files in changeset.
Tuesday 05 Oct 2021
2:09 pm

antoniop

Reduce divergency between oracle and postgres codebase

Options
    • -16
    • +0
    ./www/subscribers-oracle.xql
    • -14
    • +0
    ./www/subscribers-postgresql.xql
    • -2
    • +9
    ./www/subscribers.tcl
Thursday 16 Sep 2021
10:27 am

gustafn

fix release dates in .info files

Options
    • -1
    • +1
    ./notifications.info
  2. … 82 more files in changeset.
Wednesday 15 Sep 2021
6:11 pm

gustafn

release work

Options
    • -5
    • +5
    ./notifications.info
  2. … 204 more files in changeset.
Thursday 02 Sep 2021
7:22 pm

gustafn

bump version numbers of 5.9|5.10 packages to 5.10.0b1

adjust dependencies

Options
    • -5
    • +5
    ./notifications.info
  2. … 87 more files in changeset.
Tuesday 17 Aug 2021
11:54 am

gustafn

Notification request cleanup refactoring

One of the most expensive operations in large site is the cleanup for

notification_requests in situations, where the user has lost

permissions on an object, on which the user wanted to receive

notifications. This check was performed previously in

notification::sweep::cleanup_notifications via a permission check over

all notification requests, which can be very costly on large

sites. This change moves this cleanup into the actual notification

sending, where the permissions have to be sent anyhow.

Options
    • -18
    • +54
    ./tcl/sweep-procs.tcl
Wednesday 28 Apr 2021
7:00 pm

antoniop

Fix typo in comment

Options
    • -2
    • +2
    ./tcl/notification-email-procs.tcl
6:59 pm

antoniop

When sending a notification on behalf of a person, if the system is not configured to process replies to notification, do not set the reply-to address to anything different than the sender

Options
    • -4
    • +17
    ./tcl/notification-email-procs.tcl
Monday 19 Apr 2021
3:41 pm

antoniop

Cleanup leftover file

Options
    • -9
    • +0
    ./www/unsubscribe.xql
3:36 pm

antoniop

Remove attack surface

Options
    • -19
    • +0
    ./www/test-cleanup.tcl
    • -1
    • +0
    ./www/test.adp
    • -28
    • +0
    ./www/test.tcl
3:31 pm

antoniop

Check permissions when one accesses the script and also when one performs the actual unsubscribing operation

Options
    • -6
    • +10
    ./www/unsubscribe.tcl
3:19 pm

antoniop

Harden safety: require login, do not let people outside the requester fiddle with the frequency

Options
    • -4
    • +11
    ./www/request-change-frequency.tcl
Saturday 06 Mar 2021
8:36 pm

gustafn

reduce public footprint

Options
    • -2
    • +2
    ./tcl/delivery-method-procs.tcl
    • -4
    • +4
    ./tcl/notification-email-procs.tcl
    • -3
    • +3
    ./tcl/reply-sweep-procs.tcl
    • -3
    • +3
    ./tcl/sweep-procs.tcl
Friday 26 Feb 2021
8:18 pm

gustafn

mark private functions as private

Options
    • -13
    • +13
    ./tcl/apm-callback-procs.tcl
  2. … 9 more files in changeset.
Sunday 17 Jan 2021
10:02 pm

gustafn

make clear from the user-interface that admin is managing the notifications of someone else

Options
    • -1
    • +1
    ./www/manage.adp
    • -3
    • +19
    ./www/manage.tcl
Monday 09 Nov 2020
6:38 pm

antoniop

Remove colon that gets confused as a bind variable

Options
    • -2
    • +2
    ./tcl/sweep-procs.tcl
Friday 06 Nov 2020
4:52 pm

antoniop

Add missing semicolon

Options
    • -1
    • +1
    ./sql/postgresql/upgrade/upgrade-5.10.0d6-5.10.0d7.sql
4:51 pm

antoniop

Solve a potential issue when one would cleanup "dynamic" notification requests: this was done directly on the table and could therefore leave dangling acs_objects around

- cleanup all acs_objects of type 'notification_request' that do not have a corresponding entry in the table (~35000 downstream)

- embed logic of delete_dynamic_requests query inside select_invalid_request_ids query, as there the deletion is done properly

Options
    • -2
    • +2
    ./notifications.info
    • -0
    • +8
    ./sql/postgresql/upgrade/upgrade-5.10.0d6-5.10.0d7.sql
    • -6
    • +14
    ./tcl/sweep-procs.tcl
    • -15
    • +0
    ./tcl/sweep-procs.xql
4:51 pm

antoniop

file upgrade-5.10.0d6-5.10.0d7.sql was initially added on branch oacs-5-10.

Options
    • -0
    • +0
    ./sql/postgresql/upgrade/upgrade-5.10.0d6-5.10.0d7.sql
Tuesday 01 Sep 2020
4:04 pm

antoniop

Rewrite idiom to be portable (limit is psql only)

Options
    • -2
    • +8
    ./tcl/notification-callback-procs.tcl
    • -10
    • +0
    ./tcl/notification-callback-procs.xql
Tuesday 21 Jul 2020
3:58 pm

gustafn

prefer "ns_conn isconnected" over "ad_conn isconnected", since the latter might not notice changes in the connect state

Options
    • -2
    • +2
    ./tcl/notification-procs.tcl
  2. … 18 more files in changeset.
Friday 03 Jul 2020
9:56 am

gustafn

improve spelling: move closer to the linux documentation recommendations

Options
    • -2
    • +2
    ./tcl/notification-procs.tcl
  2. … 9 more files in changeset.
Friday 12 Jun 2020
2:56 pm

gustafn

fix an at lease 11 year old bug... not sure, why this was not discovered earlier

Options
    • -2
    • +2
    ./tcl/notification-email-procs.tcl
Wednesday 03 Jun 2020
3:20 pm

antoniop

Whitespace cleanup

Options
    • -13
    • +13
    ./tcl/apm-callback-procs.tcl