• last updated 57 mins ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
- Rework acs-api-browser.callgraph__bad_calls automated test so that acs-service-contract calling contract implementation is considered OK

- Reinstate authentication service contracts as private

- Move test authority implementation to acs-authentication to remove intra-package dependency

acs-api-browser.callgraph__bad_call still fails because proc acs-tcl.install::xml::action::create-user calls private acs-authentication.auth::local::registration::Register... to be continued

    • -1
    • +16
    ./tcl/test/acs-api-browser-procs.tcl
  1. … 3 more files in changeset.
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.
added "content_type" to the list of specially handled calls. this fixes two more cases in Test case callgraph__bad_calls

Fix extra brace

- fixed mis-classification of "max", "min" and "random" in "tclcode_to_html"

- added new test case "callgraph__bad_calls" to check for

a) calls to deprecated procs

b) direct calls to private procs from other packages ("private" means "packakge private"

    • -11
    • +29
    ./tcl/acs-api-documentation-procs.tcl
    • -1
    • +62
    ./tcl/test/acs-api-browser-procs.tcl
improve listing of test coverage

  1. … 17 more files in changeset.
make listing of tested procs more complete

  1. … 2 more files in changeset.
make listing of tested procs more complete

  1. … 6 more files in changeset.
make end of options explicit

  1. … 42 more files in changeset.
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.

    • -18
    • +21
    ./tcl/acs-api-documentation-procs.tcl
    • -9
    • +15
    ./tcl/test/acs-api-browser-procs.tcl
  1. … 56 more files in changeset.
Reduce divergency between oracle and postgres codebase

Set some tests as 'production_safe'

Unset the right nsv in the test or leftover dummy proc names will make acs-tcl.naming__proc_naming test fail

Fix typo in proc doc

use calue checker "token" for "orderby" (usual checker)

Add 'case' to the list of Tcl keywords. The command is obsolete, deprecated, not documented anymore and slated for removal in Tcl 9.0, but still exists and makes the api_doc confused and unhappy.

Whitespace changes + editor hints

Avoid looking for proc names in comments after a semicolon or an opening curly brace, so common words in comments are not included in the api-doc as calls to procs with the same name

Add 'exit' to the list of Tcl keywords, so it is properly managed by the api-doc logic

Add 'unknown' to the list of Tcl keywords, so it is properly managed by the api-doc logic

Whitespace cleanup

  1. … 1 more file in changeset.
Use ad_include_contract

  1. … 1 more file in changeset.
improve spelling

improving comments

Fix test description

Include '-' as a valid character in a proc name in acs-api-browser's 'length_proc'. Fixes the 'acs_api_browser_apidoc_tclcode_to_html' test case.

fix for issue #3427

new funcion: util::inline_svg_from_dot: refactored version from private api-procs function.

use new function in api-doc

CVS: ----------------------------------------------------------------------

    • -56
    • +10
    ./tcl/acs-api-documentation-procs.tcl
  1. … 2 more files in changeset.
fix broken HTML renderung (unbalanced </table>)

Allow dots in the proc name when building the doc HTML to accomodate e.g. for procs such as "system.methodHelp"