• last updated 19 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Test all the cornercases of this api as intended, without using ad_tmpnam

Make sure the cal item notification has at least 1 subscriber

increased value checking for paramter that might be influenced by user input

    • -2
    • +2
    /openacs-4/packages/xowiki/tcl/import-procs.tcl
    • -7
    • +7
    /openacs-4/packages/xowiki/tcl/package-procs.tcl
    • -1
    • +1
    /openacs-4/packages/xowiki/tcl/repeat-procs.tcl
    • -3
    • +3
    /openacs-4/packages/xowiki/tcl/xowiki-procs.tcl
Use a test admin user to run the ics test

Fix typo

Test db_foreach with -column_set with at least 2 rows to check that feering the ns_set works properly

Repair -column_set flag

A more efficient approach would use db api returning ns_sets directly, rather than db_list_of_lists, however this would require a deeper refactoring.

Test db_foreach -column_set flag, documented and used e.g. in the schema-browser

Cover additional api

Make Service Contract implementations private

    • -11
    • +19
    /openacs-4/packages/news/tcl/news-procs.tcl
Increase proc coverage

Increase proc coverage

Increase proc coverage, add more interesting values to the test

Test other trivial wrappers

Increase proc coverage

reduce verbosity

Fake the node_id when test is run outside of a connection

Many thanks to Thomas Renner

Fix actual coverage

Don't load the DAV callbacks when oacs-dav is not installed

Fix typos

Commit remaining files

Make oacs-dav an optional dependency for the file-storage package

Callback implementations and folder registrations will happen only when the package is detected at startup. For existing installations, no action is performed, buit the oacs-dav package can now be deleted without complaints.

improve warning message (for legavy applications)

Make calendar::get_month_multirow_information private: its purpose is very specific to the usage in the month view

Deprecate calendar::assign_permissions, trivial wrapper to the permission api

Deprecate calendar::item::assign_permission, trivial wrapper to the permission api

One can in fact replace the only occurrence by granting the admin permission on the item.

Update tests to demonstrate that the behavior remains the same.

Deprecated api, update documentation

Deprecate long-abandoned api

Increase proc coverage

added ability to order by time values