• last updated 2 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Test some system information api

The test shows that ad_acs_admin_node has so far returned a boolean, countrary to what the documentation says

    • -0
    • +52
    ./acs-tcl/tcl/test/acs-kernel-procs.tcl
file acs-kernel-procs.tcl was initially added on branch oacs-5-10.

    • -0
    • +0
    ./acs-tcl/tcl/test/acs-kernel-procs.tcl
When demoting a user, flush the permissions cache as well

    • -1
    • +7
    ./acs-tcl/tcl/community-core-procs.tcl
Test demoting/promoting of a user + a buch of other api

This test fails now when checking if a demoted user is still an swa, one needs to flush the permissions as well

    • -1
    • +103
    ./acs-tcl/tcl/test/community-core-test-procs.tcl
Test acs_user::ScreenName

Test acs_object_type_hierarchy

    • -1
    • +76
    ./acs-tcl/tcl/test/object-test-case-procs.tcl
Test acs_object_type::get_table_name

    • -1
    • +21
    ./acs-tcl/tcl/test/object-test-case-procs.tcl
Test acs_object_name api

    • -1
    • +21
    ./acs-tcl/tcl/test/object-test-case-procs.tcl
Test acs_magic_object

    • -1
    • +18
    ./acs-tcl/tcl/test/object-test-case-procs.tcl
Test various URL generating procs

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

    • -0
    • +0
    ./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

    • -2
    • +2
    ./acs-admin/www/users/complex-search.tcl
    • -3
    • +3
    ./acs-tcl/tcl/request-processor-procs.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

    • -2
    • +2
    ./acs-admin/www/users/complex-search.tcl
    • -2
    • +2
    ./acs-developer-support/www/request-info.tcl
    • -2
    • +2
    ./acs-templating/tcl/dimensional-procs.tcl
    • -2
    • +2
    ./general-comments/www/view-comment.tcl
    • -2
    • +2
    ./general-comments/www/admin/index.tcl
Reimplement util_subset_p

    • -39
    • +25
    ./acs-tcl/tcl/utilities-procs.tcl
Deprecate util_report_successful_library_load

Deprecate util_commify_number, replace occurrences and translate automated tests

    • -3
    • +3
    ./acs-subsite/www/admin/group-types/delete.tcl
    • -2
    • +2
    ./acs-subsite/www/admin/rel-segments/one.tcl
    • -20
    • +20
    ./acs-tcl/tcl/test/acs-tcl-test-procs.tcl
Whitespace cleanup

    • -2
    • +2
    ./acs-subsite/www/admin/rel-segments/one.tcl
Deprecate export_entire_form

Deprecate util_AnsiDatetoPrettyDate

Deprecate util_report_library_entry

Formatting changes

    • -14
    • +30
    ./acs-tcl/tcl/utilities-procs.tcl
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

    • -11
    • +8
    ./acs-templating/tcl/list-procs.tcl
Use faster idiom to tell if a string is only space

    • -2
    • +2
    ./acs-automated-testing/www/admin/testcase.tcl
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