reduce usage of ad_arg_parser by using c-level implementation; fixes as well broken cases of bind-var handling

added test cases for db_0or1row and db_1row

whitespace changes

add test cases for db_list*

reduce verbosity

whitespace changes

remove debug code

extended regression test: increase coverage of page contract filters

fix util::external_url_p when security::locations returns empty element

ease navigation: provide links to coverage pages

added new test cases for testing page contract filters

simplify and modernize Tcl code

improve navigation structure (context bar) and actions labeling

the reousrce actions is not necessary, since rerun performs an automatic resources

adjust to api-browser changes for trailing non-positional arguments

Added support for trailing non-positional arguments

The change makes the old hack with the own trailing-argument-parser

"ad_arg_parser" obsolete by using directly the c-level argument parser

which is anyhow capable of handling trailing arguments. A few changes

were necessary for api doc to handle there trailing arguments as well.

The proc "db_string" is already converted to use native arguments.

    • -3
    • +3
added test case for db_string with various bind options

whitespace changes

move db-test case to db-regression test files

remove unneeded method

complete listing of covered procs in test cases

use fine-grade percentage to color mapping

added a comment about the usefulness of ad_arg_parser

Fix for issue #3442 to support range types in .xql files

PostgreSQL supports range types since 9.5. When using range types,

square braces have to be used in SQL statements. Since OpenACS uses

always Tcl substitution in .xql files, and OpenACS does NOT allow

backslash substitution in these files, square brackets could not be

escaped and therefore not be used in .xql files so far. This change

allows now a developer to deactivate the substitution by passing

e.g. "-subst none" to the db_* command using the .xql file. Valid

values for "-subst" are "all", "none", "vars", and "commands", default

is "all" which is exactly the behavior of before. Therefore, this

change is fully backward compatible.

    • -1
    • +1
Remove leftovers

mark private functions as private

whitespace changes

improve test coverage

improve consistency of interface: use instance variable when no folder_id is provided

Whitespace changes

Add new test 'acs_subsite_attributes' covering the 'attribute::add', 'attribute::exists_p' and 'attribute::delete' procs