• last updated 13 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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. … 3 more files in changeset.
reduce the size of the public interface

    • -10
    • +10
    ./40-db-query-dispatcher-procs.tcl
whitespace changes: break overlong lines

    • -150
    • +176
    ./40-db-query-dispatcher-procs.tcl
make clear from the name that db_qd_prepare_queryfile_content is intended to be used in different packages

  1. … 2 more files in changeset.
Make api public, complies with acs-api-browser.graph__bad_calls automated test

  1. … 4 more files in changeset.
Make api public, complies with acs-api-browser.graph__bad_calls automated test

  1. … 3 more files in changeset.
Make proc public (it is used in a test is acs-tcl)

Complies with acs-api-browser.callgraph__bad_calls

Whitespace cleanup

    • -69
    • +69
    ./40-db-query-dispatcher-procs.tcl
include package key in proc info

mark functions called only internally as private

  1. … 15 more files in changeset.
just avoid tilde substitution, do not perform sanitizing

sanitizing can lead to file name clashes in case the names

without tilde exists, and they are OS specific. The intention

of "ad_file" is just to be a tilde-proof "file" replacement

Sanitize all trailing tildes from the filename, mapping them to the current folder

Fixes acs-bootstrap-installer.ad_file automated test

Make 'ad_file' test case deterministic

Automated test for ad_file exposing a blind spot with files starting with the '~' character

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

    • -0
    • +0
    ./test/00-proc-procs.tcl
improve spelling

    • -10
    • +12
    ./40-db-query-dispatcher-procs.tcl
  1. … 2 more files in changeset.
allow value checkers in ad_procs

Prefer 'namespace which' over 'info commands', as it is faster (on local tests, around 2x) and returns a single value. Many thanks to Nathan Coulter.

  1. … 56 more files in changeset.
Treat callback hooks and implementations as private api

Reduce acs-tcl.documentation__check_deprecated_see test warnings

  1. … 2 more files in changeset.
introduced "ad_file" as a means to avoid unexpected tilde substitution in file names

  1. … 47 more files in changeset.
deprecate unused function

Restrict switches to not negative Tcl numbers, and avoid non-printable characters as well

Fix typo

Remove private proc 'number_p', used only in 'ad_proc_valid_switch_p', and simplify logic of the latter

improve spelling: prefer comma after "therefore" and friends

  1. … 5 more files in changeset.
Fixed bug while populating the 'api_proc_doc_scripts' nsv on ad_proc, where some procs where not added due to a race condition between the 'acs-automated-testing' and the 'acs-bootstrap-installer' packages. See comments in commit for more details.

improve spelling

  1. … 1 more file in changeset.
Deprecate trivial empty_string_p