• last updated 3 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Extend and fix xowiki.create_form_with_form_instance automated test with respect to checkbox behavior

- simple checkboxes (normal and repeated): as we do not specify a value for those, the default should be expected, not the last option

- checkboxes in repeated compound fields: upon triavial re-editing, the values should stay the same

This change exposes a bug in acs::test::xpath::get_form_values

Handle differences in data-model between oracle and postgres, where oracle uses an additional filename column in the cr_revisions table when a revision points to a file

Many thanks to Josue Cardona for reporting the regression

Port of downstream modification:

do not restrict the format the user can supply. Sanitize the filename later and complain only if this is made exclusively of invalid characters.

get_form_data reform:

when validation fails, revert all changes performed on the object while filling up the form fields.

Rationale: when validation fails, we do not persist the data. The same we should not let unvalidated data sneak into the object, as this may be e.g. displayed on the page or be otherwise used by the system.

Added support for automatic disconnect when a dynamic cluster node is shut down

When the dynamic cluster configuration has scale for certain

occasions, it makes sense to provide down scaling support when these

occasions are over, which does not rely on the configured cluster

disconnect timeout (ClusterAutodeleteInterval). The new code will send

automatically a disconnect request when a dynamic cluster node is

terminating gracefully.

Depending on the configuration, a new version of NaviServer will be

necessary to reliably execute disconnect requests. Appropriate changes

are in the NaviServer release/4.99 and main branches.

- Bumped version numbers:

* acs-tcl to 5.10.1b4

* acs-admin to 5.10.1b4

    • -2
    • +2
    /openacs-4/packages/acs-tcl/acs-tcl.info
    • -9
    • +19
    /openacs-4/packages/acs-tcl/tcl/cluster-init.tcl
    • -41
    • +75
    /openacs-4/packages/acs-tcl/tcl/cluster-procs.tcl
Do not retrieve extra_css from query_parameters, as this is vulnerable to injections

Many thanks to Markus Moser

Provide a specific message key for when messages are not available

Many thanks to Sebastian Scheder

Adding auto-deletion of dynamic cluster nodes and small refactoring

- New kernel parameter "ClusterAutodeleteInterval" to specify, when a

dynamic cluster node is not regarded as temporarily unavailable but

as definitely gone. The default value for this parameter is 2m (2

minutes).

- small refactoring to reduce duplicated logic

- Bumped version numbers:

* acs-kernel to 5.10.1b4

* acs-tcl to 5.10.1b3

* acs-admin to 5.10.1b3

    • -3
    • +3
    /openacs-4/packages/acs-tcl/acs-tcl.info
    • -0
    • +10
    /openacs-4/packages/acs-tcl/tcl/cluster-init.tcl
    • -10
    • +65
    /openacs-4/packages/acs-tcl/tcl/cluster-procs.tcl
Small update for cluster support

- Improved visualization of cluster nodes that we lost contact to

- New kernel parameters to reduce hard-coded values and to make purpose more explicit

* new parameter ClusterHeartbeatInterval (default 20s),

was hardcoded before to the new default value

* renamed PreferredLocationRegexp -> ClusterPreferredLocationRegexp

* renamed EnableLoggingP -> ClusterEnableLoggingP

- Bumped version numbers:

* acs-kernel to 5.10.1b3

* acs-tcl to 5.10.1b2

* acs-admin to 5.10.1b2

acs-kernel acs-admin acs-tcl/

    • -3
    • +3
    /openacs-4/packages/acs-tcl/acs-tcl.info
file upgrade-5.10.1b2-5.10.1b3.sql was initially added on branch oacs-5-10.

template::widget::merge_tag_attributes has to be public already, as it is used by template::widget::select_locales

Avoid duplicate entries in acs_sc_msg_type_elements

-- The three columns msg_type_id, element_name, and

-- element_msg_type_id identify the paramter uniquely, and must be

-- therefore not null.

Bump version number to 5.10.1b2

file upgrade-5.10.1b1-5.10.1b2.sql was initially added on branch oacs-5-10.

improved spelling

provide the version directory instead of the version on the admin page

Earlier versions had just the version, but not the version directory, which is

the precise information

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

don't trust in stability of version numbers, but check, what we have actually got.

The structure of the distributed zip file unfortunately a moving target.

fixed the path confusion in packaged bootstrap icons files

Unfortunately, the structure of the distributed .zip file of bootrap icons is

version dependent: Versions greater or equal to 1.10.4 and less

than 1.11.0 require a "font" in the path. Also 1.11.2 requires this, but not 1.11.1.

Fix logic to retrieve the Boostrap Icons CSS path:

the situation is even more slippery than thought, as it seems the new "font" path was discontinued for versions 1.11.0 and 1.11.1, then reintroduced in version 1.11.2. We can only speculate what logic will follow the next version, so for now we assume the new path will stay and just skip the two outlier versions.

Update comment

avoid double quoting

Fix typo

Form widget attributes reform:

- consolidate the logics to merge tag attributes coming from the adp and the tcl definitions of widgets into a private api

- change the previous behavior of having attributes coming from the adp (aka the tag_attributes argument) take over attributes coming from the element

See https://openacs.org/forums/message-view?message_id=5799349

Revert in order to provide a meaningful log message

*** empty log message ***

Cut-off version for the added font folder seems to be 1.11.1, newer versions seem to now provide a minified css

cluster setup: use qualified location as well for the local host

improved spelling

reduced verbosity

    • -2
    • +2
    /openacs-4/packages/xowiki/tcl/package-procs.tcl
Do not enforce that pictures are always different on desktop screens, as they may be still for longer than the average picture interval

Add type=oneof to Serializer exportMethods

Many thanks to Sebastian Scheder and Günter Ernst