• last updated 5 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Remove apparent leftover + trailing whitespace

Add new test case 'files__page_contracts', which checks for the existance of both 'ad_page_contract' and 'ad_include_contract' in the .tcl files inside 'www' and 'lib' package directories.

There are cases, where includelets are not stored in 'lib' but 'www', or have 'ad_page_contract' instead of 'ad_include_contract'. Checking if the location of includelets is correct is not so clear, so we avoid doing this in this particular test.

rework test case documentation__check_typos

- make test case more robust (don't abort tests when there is a proc_doc without a "main" section

- improve performance and sanity:

* move invariantes out of the loop

* use built-in hashes

* don't check the same term multiple times

... times went from >15 sec to <1sec

remove useless semicolon

Add new test case 'files__trailing_whitespace', which looks for trailing whitespace (spaces or tabs at the end of lines).

Currently, only .tcl files are checked.

no reason for casting values to integer

incr would return a decimal number, but test wants an integer to be returned. Restore expr and make clear that we want an integer out of it

Streamline util_sets_equal_p (passes automated test)

Add more information about typos loaded and checks passed

Fix typo + Whitespace changes

Test refactoring

Improve test documentation.

Add more common spelling errors to the 'documentation__check_typos' test case, collected from previous typo fixes in openacs.

    • -1
    • +87
    ./tcl/test/doc-check-procs-common-typos.txt
Reduce test verbosity

Add new test case 'documentation__check_typos', which checks for spelling errors in the proc documentation.

The test uses a file containing common spelling errors (doc-check-procs-common-typos.txt), based in the one

provided by Debian's lintian, which can be found here: https://github.com/Debian/lintian/tree/master/data/spelling

The test can take some time, depending on the amount of packages and how powerful the machine is.

    • -0
    • +4234
    ./tcl/test/doc-check-procs-common-typos.txt
prefer "aa_equals" over "aa_true"+"string equal"

    • -11
    • +11
    ./tcl/test/test-membership-rel-procs.tcl
  1. … 2 more files in changeset.
replace "expr" by "incr"

whitespace changes, added editor hints

    • -59
    • +59
    ./tcl/utilities-procs-aolserver.tcl
  1. … 1 more file in changeset.
add partial abckword compatibility for ns_getcontent for AOLserver

    • -5
    • +54
    ./tcl/utilities-procs-aolserver.tcl
Cache the portrait in the right cache (Thanks to Felix Mödritscher)

Translation of file descriptor must be binary as well when flushing in-memory payload to the spool file, or we would corrupt this content

util::http performance improvements:

- cache util::http::available as namespace variable, as in the worst case it would go to the db

- replace array idioms with dict idioms

Trim whitespace

fix typo

Fix typo

Replace array idioms with dict idioms

improve testcases by removing useless expr commands

    • -14
    • +14
    ./tcl/test/application-data-link-procs.tcl
    • -16
    • +16
    ./tcl/test/test-permissions-procs.tcl
Declass constraint names breaking the very strict OpenACS naming convention from errors to warnings

Rationale: it is sometimes impossible to have constraint names that are both < 30 chars and consistent with the naming scheme 'table_name'_'column_name'_'constraint_type' for all those defined on a particular table (see e.g. acs_named_objects).

Breaking this convention might be suboptimal, but is not harmful in itself.

flush per-request cache (fixes regression test properly)

simplify code