• last updated 15 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
merge from oacs-5-10

    • -12
    • +26
    ./notification-callback-procs.tcl
    • -13
    • +16
    ./notification-display-procs.tcl
    • -1
    • +1
    ./notification-request-procs-postgresql.xql
    • -15
    • +19
    ./notifications-security-procs.tcl
  1. … 8085 more files in changeset.
silence regression test warning

Provide a script name for ad_proc

    • -0
    • +13
    ./test/notifications-test-procs.tcl
simplified exception handling by factoring out redundant code

make use of "ad_unless_script_abort"

    • -18
    • +15
    ./test/notifications-test-procs.tcl
  1. … 1 more file in changeset.
handle script abort in proc handler

    • -8
    • +19
    ./test/notifications-test-procs.tcl
improved spelling

  1. … 14 more files in changeset.
improved spelling

  1. … 7 more files in changeset.
improve spelling

  1. … 5 more files in changeset.
prefer usage of parameter::get_from_package_key over plain parameter::get

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
    ./notification-callback-procs.tcl
  1. … 13 more files in changeset.
Whitespace cleanup

Use quicker idiom to check for existance

Whitespace cleanup

Whitespace changes

Prefer api to retrieve the approval state of a user

Complete coverage for the notification package

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

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

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

    • -0
    • +179
    ./test/notifications-test-procs.tcl
Extend coverage

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

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

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

Expand tests

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

Update documentation to best practices

Fix proc signature

  1. … 6 more files in changeset.
Replace leftover occurrences

  1. … 1 more file in changeset.
Deprecate notification::get_delivery_method_id, same as notification::delivery::get_id and looking more established

  1. … 6 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
    ./test/notifications-test-procs.tcl
  1. … 4 more files in changeset.
Prefer flushing the cache, this addresses the automated test, where multiple insert/deletion are performed in the same request