• last updated 9 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Modernize tcl idioms

Prefer unset to array unset when the whole array should be deleted

See https://wiki.tcl-lang.org/page/Dict+VS+Array+Speed

    • -1
    • +1
    /openacs-4/packages/caldav/tcl/caldav-procs.tcl
Prefer unset to array unset when the whole array should be deleted, rework array idioms into dict idioms to avoid unsetting the array, where appropriate

See https://wiki.tcl-lang.org/page/Dict+VS+Array+Speed

Prefer unset to array unset when the whole array should be deleted

See https://wiki.tcl-lang.org/page/Dict+VS+Array+Speed

Rework array idioms in dict idioms, closer to the intended logics

Cleanup include found to not be in use in upstream codebase already in 2019

Rework array idioms into dict idioms to avoid the need to unset

Prefer unset to array unset when the whole array should be deleted

See https://wiki.tcl-lang.org/page/Dict+VS+Array+Speed

Streamline idiom

Prefer unset to array unset when the purpose is to delete the whole array

See https://wiki.tcl-lang.org/page/Dict+VS+Array+Speed

Prefer unset to array unset when we want to delete the entire array

See https://wiki.tcl-lang.org/page/Dict+VS+Array+Speed

Prefer unset to array unset when the semantic is to delete the whole array

See https://wiki.tcl-lang.org/page/Dict+VS+Array+Speed

State Highcharts package dependency in xotcl-request-monitor

Other leftovers

    • -30
    • +0
    /openacs-4/packages/chat/www/transcript-edit-2.tcl
Other files deleted

Delete more zombie files

    • -24
    • +0
    /openacs-4/packages/chat/www/moderator-grant-2.tcl
Delete zombie file from merge with MAIN

Trap illegal encoding exception thrown by tcl9 when a file contains invalid UTF-8 and skip the package with a warning

Unmaintained packages may contain illegal characters and we do not want the UI to crash in this situation

Limit files__tcl_file_common_errors test to installed packages

Unmaintained packages may ship files with invalid encodings, which tcl9 won't tolerate anymore.

Improve test

- skip packages that are not installed before we check the info file

- ensure all installed packages have an info file

This fixes the test for tcl9, where some unmaintained packages may fail the file parsing because of invalid encoding.

Tcl9 regression test: deactivated case broken in tcl9 with unclear semantics

whitespace change

tcl9 changes in regression test: "2000-00-00 00:00:00" is not a valid date anymore

tcl9 changes: "string bytelength" does not exist in tcl9

tcl9 changes in regression test: fix read of latin1 file with wrong encoding

tcl9 changes in regression test: address changes in tcl9 tilde explansion

tcl9 change: "... -encoding binary ..." not valid in tcl9

regenerated resources

removed file deleted in branch oacs-5-10

    • -3810
    • +0
    /openacs-4/packages/acs-tcl/tcl/00-database-procs.tcl
child-resources: render bulk-actions iff rows are selectable

    • -7
    • +9
    /openacs-4/packages/xowiki/tcl/folder-procs.tcl