• last updated 13 mins ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Fix acs-lang.test_message_register automated test:

when checking if a message should be created or updated, do not trust the nsv, which won't be there for messages that have been soft-deleted and go to the db instead. When a message is soft deleted, cleanup the nsv cache.

Expose a bug in the message key registration:

when a message key is soft-deleted and the server is restarted, upon re-registration the nsv that signals its presence won't be found and an insertion will be attempted. However, as the db entry is there, a primary key constraint will be triggered.

Stick to packages that are enabled

  1. … 1 more file in changeset.
Use the intended table to get packages that are known to the system

  1. … 1 more file in changeset.
Add comment

Make api to retrieve the dependencies of a package public, as we use it in a test in another package, swith to the api that will retrieve the cached value instead of re-computing the result

  1. … 1 more file in changeset.
New test to check that message keys are used consistently with package dependencies

this can only be enforced as a warning because it is possible to have constructs such as "if installed, then use message key" that would be tricky to detect

Reuse computation of lang::conn::locale when invoked again in the same request

This reduces stress on the util_memoize cache for those installation that invoke this api often.

Reduce divergency between oracle and postgres codebase

  1. … 6 more files in changeset.
list called api procs

Mark 'lang::message::cache' as public, as it is used in 'acs-admin'

Mark 'lang::catalog::package_catalog_dir' as public, as it is used in 'acs-admin'

Mark 'lang::util::get_hash_indices' as public, as it is used in 'acs-admin'

complete listing of covered procs in test cases

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

  1. … 4 more files in changeset.
remove protection "private", since function has to be called from other packages

  1. … 6 more files in changeset.
bring test coverage to >50% by adding test cases and marking only locally called functions as private

    • -0
    • +29
    ./test/acs-lang-localization-procs.tcl
mark unused function "_mr" as deprecated

improve listing of test coverage

  1. … 17 more files in changeset.
use per_thread_cache for flusing rather than raw variables

make listing of tested procs more complete

    • -2
    • +5
    ./test/acs-lang-localization-procs.tcl
  1. … 5 more files in changeset.
make comment more precise

New test to enforce that catalog files are valid XML

Define arguments so that documentation__check_parameters automated test does find -varname flag correctly

Pass missing object_id flag

Improve sql portability

    • -22
    • +22
    ./test/acs-lang-localization-procs.tcl
  1. … 6 more files in changeset.
Improve test

    • -8
    • +13
    ./test/acs-lang-localization-procs.tcl
New test for lang::util::convert_to_i18n pinpointing some expected behaviors of this proc

    • -0
    • +67
    ./test/acs-lang-localization-procs.tcl
Fix typo

acs-lang reform:

when calling lang::util::convert_to_i18n, do not always register a en_US translation, which will be otherwise always overridden. Instead, let lang::message::register make sure that a en_US message exists and create it only when needed as a fallback. Before, lang::message::register would return an error when e.g. a new message keys was registered for de_DE and no en_US was there. Now the en_US will be just created on the fly.

This does not break current automated tests