• last updated 3 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Properly escape "<" and ">" in api-doc documentation.

Since all documentation is rendered via HTML, the characters

"<" and ">" have to be HTML-quoted, otherwise strange things

(omission, unintended renderings) might occur.

E.g. the sentence

Define an interface between a page and an

ADP <include> similar to the page_contract.

was rendered as

Define an interface between a page and an

ADP similar to the page_contract.

which is incorrect.

    • -10
    • +18
    ./tcl/notification-callback-procs.tcl
  1. … 13 more files in changeset.
Whitespace cleanup

    • -11
    • +11
    ./tcl/notification-type-procs.tcl
Use quicker idiom to check for existance

Whitespace cleanup

Whitespace changes

Prefer api to retrieve the approval state of a user

    • -2
    • +2
    ./tcl/notifications-security-procs.tcl
    • -14
    • +0
    ./tcl/notifications-security-procs.xql
Complete coverage for the notification package

    • -0
    • +247
    ./tcl/test/notifications-test-procs.tcl
Brutally delete the notification requests before deleting the delivery method (it's fine because it will be rolled back)

    • -0
    • +1
    ./tcl/test/notifications-test-procs.tcl
Fix issue in the query logics: we actually want to check the owner of the request, there might be multiple requests associated to the same object

    • -6
    • +6
    ./tcl/notifications-security-procs.tcl
    • -1
    • +10
    ./tcl/test/notifications-test-procs.tcl
Extend coverage, this will expose an api bug

    • -0
    • +179
    ./tcl/test/notifications-test-procs.tcl
Whitespace changes

Extend coverage

    • -1
    • +27
    ./tcl/test/notifications-test-procs.tcl
Cover more api

    • -0
    • +42
    ./tcl/test/notifications-test-procs.tcl
Extend automated tests

    • -0
    • +124
    ./tcl/test/notifications-test-procs.tcl
Allow to specify a user that does not come from the connection

Expand tests

    • -2
    • +80
    ./tcl/test/notifications-test-procs.tcl
Improve test a bit

    • -2
    • +9
    ./tcl/test/notifications-test-procs.tcl
Update documentation to best practices

    • -10
    • +8
    ./tcl/notification-display-procs.tcl
Fix proc signature

    • -1
    • +1
    ./tcl/test/notifications-test-procs.tcl
  1. … 5 more files in changeset.
Replace leftover occurrences

Deprecate notification::get_delivery_method_id, same as notification::delivery::get_id and looking more established

    • -2
    • +2
    ./tcl/test/notifications-test-procs.tcl
  1. … 5 more files in changeset.
Deprecate notification::get_interval_id, duplicated by notification::interval::get_id_from_name which appears to be more established

    • -21
    • +38
    ./tcl/test/notifications-test-procs.tcl
  1. … 3 more files in changeset.
Prefer flushing the cache, this addresses the automated test, where multiple insert/deletion are performed in the same request

Extend test

    • -12
    • +36
    ./tcl/test/notifications-test-procs.tcl
Cache the new type_id immediately in the (rare) event that a type was created in a previous request with the same name and then deleted

Expand test

    • -1
    • +10
    ./tcl/test/notifications-test-procs.tcl
Fix typo in xql file

    • -1
    • +1
    ./tcl/notification-request-procs-postgresql.xql
Extend notifications tests

    • -0
    • +154
    ./tcl/test/notifications-test-procs.tcl
Prefer the non cached api when deleting, as the id is cached in the thread and could be stale

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

    • -0
    • +0
    ./tcl/test/notifications-test-procs.tcl