• last updated 15 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Test acs_magic_object

Test various URL generating procs

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

    • -0
    • +0
    /openacs-4/packages/acs-tcl/tcl/test/defs-procs.tcl
Deprecate util_get_current_url, superseded by ad_return_url

Deprecate export_entire_form_as_url_vars and replace occurrences, add a new -formvars flag to export vars to implement the behavior of the proc, that is, export a subset of the variables coming from the current request

    • -1
    • +1
    /openacs-4/packages/chat/www/search-2.tcl
Deprecate export_ns_set_vars and extend export_vars to be able to export also a custom ns_set, which was the real added value provided by this api

    • -1
    • +1
    /openacs-4/packages/chat/www/search-2.tcl
Reimplement util_subset_p

Deprecate util_report_successful_library_load

Deprecate util_commify_number, replace occurrences and translate automated tests

    • -1
    • +1
    /openacs-4/packages/chat/www/search.tcl
Whitespace cleanup

Deprecate export_entire_form

Deprecate util_AnsiDatetoPrettyDate

    • -2
    • +2
    /openacs-4/packages/survey/www/admin/one.tcl
Deprecate util_report_library_entry

Formatting changes

Deprecate util::string_check_urlsafe

Simplify implementation

After some experiments, reinstate the faster version of util_sets_equal_p: this wins (~20% faster) in particular when the sets to be compared are larger

Cleanup duplicated and slower proc definition

Small improvements:

- use "string is space" instead of trimming the string and checking if empty, at least 2x faster on development, wherever we don't need the trimmed value

- modernize leftover foreach trick with lassign

Use faster idiom to tell if a string is only space

Fall back to en_US also when a value for nls_language is there, but empty

Use api to determine whether a field is disabled, as the "disabled" variable will always be set now

    • -1
    • +1
    /openacs-4/packages/xowiki/tcl/repeat-procs.tcl
Claim additional coverage

Count already exisring handlers before introducing new ones

Improve acs::test::form_reply so that when the attributes of a file upload (*.tmpfile or *.content-type) are part of the update argument, it will switch automatically to a multipart request and implement a "real" file upload

This is important on newer NaviServer versions that will reject any *.tmpfile parameter in a request that was not produced internally after a file upload

add the amount of questions from the exam in the title

Test also nonce rendering

Make test more precise

Test a bunch more api

Test template::adp_level, fix test coverage declaration