• last updated 15 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Complete coverage for the package

file lang-widget-procs.tcl was initially added on branch oacs-5-10.

    • -0
    • +0
    ./tcl/test/lang-widget-procs.tcl
Go ahead and reimplement also lc_time_utc_to_local

    • -16
    • +0
    ./tcl/localization-procs-oracle.xql
    • -14
    • +0
    ./tcl/localization-procs-postgresql.xql
Reimplement lc_time_local_to_utc using lc_time_tz_convert

    • -11
    • +0
    ./tcl/localization-procs-oracle.xql
    • -9
    • +0
    ./tcl/localization-procs-postgresql.xql
Improve test for lc_time_local_to_utc, showing that its behavior is inconsitent with respect to invalid input

    • -0
    • +33
    ./tcl/test/acs-lang-localization-procs.tcl
Return the system locale as the user locale also when user preferences are there for the user, but the locale is empty

Replace api outside of the package dependencies

Deprecate Oracle-specific collation api

Although it could be interesting to have OpenACS cross-db support for collation, it is unclear if just using different idioms in xql files would not achieve the same.

Update thead variable them setting the translator mode

Increase coverage

This test will currently fail because the translator mode flag is cached per thread, but the set api won't update this cache.

Increase proc coverage

Increase proc coverage

Re-load also all other enabled locales to avoid test side effects

Increase proc coverage

Wrap test in a transaction

    • -116
    • +118
    ./tcl/test/lang-catalog-procs.tcl
file lang-util-procs.tcl was initially added on branch oacs-5-10.

    • -0
    • +0
    ./tcl/test/lang-util-procs.tcl
Increase proc coverage

Improve coverage

Increase proc coverage

Cleanup private interface lang::system::timezone_support_p: ref-timezones has been made a dependency of acs-lang long ago

    • -83
    • +73
    ./tcl/test/acs-lang-test-procs.tcl
Allow to specify a user for lang::user::language, default to the package default locale when a locale for user+package is not found

Increase proc coverage

This test will fail because of two reasons:

- when using package-level locales and a package default locale is specified, but no user+package locale, the expected behavior should be that the default from the package is used, but this is not the case

- lang::user::language does not allow to specify a user id

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

    • -0
    • +0
    ./tcl/test/locale-procs.tcl
Increase proc coverage

    • -5
    • +104
    ./tcl/test/lang-catalog-procs.tcl
Increase proc coverage

file lang-catalog-procs.tcl was initially added on branch oacs-5-10.

    • -0
    • +0
    ./tcl/test/lang-catalog-procs.tcl
Export the catalog via the api to normalize quoting

    • -5
    • +5
    ./catalog/acs-lang.en_US.ISO-8859-1.xml
Cleanup leftover, fix typos

    • -5
    • +2
    ./tcl/test/acs-lang-localization-procs.tcl
lc_time_tz_convert reform:

- enforce that only dates in the ISO format "%Y-%m-%d %H:%M:%S" are accepted

- apply clock api normalization to seemingly out of range time_values, as per https://www.tcl.tk/man/tcl/TclCmd/clock.html#M25

- return the empty string every time input is invalid or a conversion cannot be determined

- return a value, as ISO date, only when conversion is successful

- prefer the clock api to perform the conversion, but fall back to the ref-timezones datamodel when the timezone is not supported by Tcl

    • -15
    • +0
    ./tcl/localization-procs-oracle.xql
    • -12
    • +0
    ./tcl/localization-procs-postgresql.xql
Introduce a thorough test of the lc_time_tz_convert api checking:

- dates in the distant past

- dates in the distant future

- daylight saving for different times of the year across the two timezones

- historical circumstances (e.g. Pre-timezones US)

- Invalid input

The test shows that the current api does not behave consistently in certain circumstances:

- the same time_value is sometimes returned with invalid input

- the empty string is sometimes returned with valid input

- for dates where the ref-timezones package does not provide information, no result is returned

    • -0
    • +118
    ./tcl/test/acs-lang-localization-procs.tcl