• last updated 11 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Fix extra braces

Add 'calendar_basic_api' test case, covering 'calendar::new', 'calendar::delete', 'calendar::rename' and 'calendar::name'

Extend 'create_portal_from_template' test case to cover 'portal::delete', 'portal::page_delete', 'portal::page_count', 'portal::get_page_id', 'portal::get_page_pretty_name', 'portal::set_page_pretty_name' and 'portal::first_page_p'

Whitespace changes

Add covered procs to test case

Add trivial test for 'bulk_mail::package_key'

remove dependency on ad_ns_set_keys

improve listing of test coverage

mark functions called only internally as private

bring test coverage to >50% by adding test cases and marking only locally called functions as private

Add editor hints + whitespace changes

Remove leftover comment

Add 'attachments_basic_api' test case. Covers basic api of the attachments package: 'attachments::attach', 'attachments::unattach', 'attachments::get_all_attachments', 'attachments::get_attachments' and 'attachments::toggle_approved'

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

    • -0
    • +0
    /openacs-4/packages/attachments/tcl/test/attachments-procs.tcl
mark unused function "_mr" as deprecated

mark unused functions as deprecated

improve comment

make use of built-in reverse proxy mode of newer versions of NaviServer

overwork get_community_id once more. provide a shortcut is hopless situations, reduce potential warnings and improve comments

make clear that get_community_id_from_url needs connection in case no URL is provided; provide shortcut, when determined package_id is empty

    • -1
    • +1
    /openacs-4/packages/survey/tcl/survey-procs.tcl
    • -2
    • +2
    /openacs-4/packages/survey/www/admin/one.tcl
don't load deprecated code unless explicitly required

deprecate unused and untested code

fix deprecated function

remove call to deprecated proc

removed leftover from debugging

Fix correct-when test of empty input when constraint is "<" or "<=". Many thanks for Florian Mosböck for finding this.

provide more details in log message

    • -2
    • +2
    /openacs-4/packages/xowiki/tcl/import-procs.tcl
mark in log-file begin and end of test cases, support multiple msg arguments in aa_log (similar to ns_log)

reduce verbosity

use tcl "max" idiom