• last updated 8 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Deprecate proc apm_pretty_name_for_db_type as is not used anywhere and also relies on questionable datamodel

make spelling of names more consistent

site-nodes cache reform: use ns_urlsspace instead of brute-force caching.

For OpenACS site with recent versions of NaviServer and XOTcl2,

cache sitenode information much more efficiently by the built-in

URL trie, which supports tree match operations, for

tree information. This means that for example for .vuh

handlers it is not necessary to cache the full url for

obtaining the site-node, like it was until now:

3839 id-/storage/view/installers/windows-installer/installer.htm

3839 id-/storage/view/aolserver/install.tgz

3839 id-/storage/view/tutorial/OpenACS_Tutorial.htm

3839 id-/storage/view/openacs-dotlrn-conference-2007-spring/Methodology_ALPE.pdf

3839 id-/storage/view/xowiki-resources/Assessment.jpg

3839 id-/storage/view/tutorial-page-map.png

...

Providing a single entry like

ns_urlspace set -key sitenode /storage/* 3839

is sufficient.

Why is this needed: current cache sizes of larger sites show 330K

(site A) and 390K (site B) cache entries. Matchbase flushes have to

gor thhrough all these entries, causing long lock times. Running

thhis code at OpenACS.org shows an improvement by a factor of more

than 200x.

    • -252
    • +343
    ./tcl/site-nodes-procs.tcl
Use proper thread cached variables instead of legacy proc

Deprecate proc 'apm_db_type_keys' as the info it provides is already in 'ad_known_database_types' without need for queries

TODO: find out if we really need a table of database types

Improve indentation, use non indentation-breaking idioms to create report string

TODO: refactor this bug-tracker dependency outside the core, e.g. using a callback

Revert massive replacement of empty list creation sentences. The use of '[list]' instead of '{}' adds semantics that could be used for performance improvements in the future, such as using a different internal representation. There is already work in this direction, avoiding the generation of the string representation during comparison of empty strings (huge thanks to Stefan Sobernig for the pointer: https://core.tcl.tk/tcl/info/44527c632ed609c2).

  1. … 461 more files in changeset.
Rework variable assigning loops (set -> lassign) + Whitespace changes

  1. … 12 more files in changeset.
improve CSP in accordance with Google's csp-evaluator.withgoogle.com

  1. … 1 more file in changeset.
Prefer 'lappend' to 'set ... concat' for appending elements to lists + Whitespace changes

  1. … 3 more files in changeset.
remove duplicated words

  1. … 9 more files in changeset.
Prefer '{}' to '[list]' when creating empty lists

  1. … 204 more files in changeset.
Prefer '{}' to '[list]' when creating empty lists

  1. … 57 more files in changeset.
improve documentation, remove obsolete documentation

improve documentation

    • -11
    • +10
    ./tcl/utilities-procs-aolserver.tcl
    • -11
    • +10
    ./tcl/utilities-procs-naviserver.tcl
  1. … 2 more files in changeset.
Rework variable assignment (set -> lassign)

    • -173
    • +160
    ./tcl/apm-install-procs.tcl
  1. … 1 more file in changeset.
fix typos

  1. … 7 more files in changeset.
Fix typos

    • -11
    • +11
    ./tcl/test/acs-tcl-test-procs.tcl
  1. … 1 more file in changeset.
Fix typos

  1. … 11 more files in changeset.
fix singular/plural typos

  1. … 6 more files in changeset.
Fix typos

  1. … 28 more files in changeset.
Remove extra line

Prefer 'in' and 'ni' operators over lsearch for testing list containment

  1. … 3 more files in changeset.
Prefer lappend over concat for appending lists (lappend modifies the variable 'in place')

  1. … 3 more files in changeset.
Mark procs as deprecated, which were previously just documented as

deprecated and document recommended versions via @see. These procs

should be moved to *deprecated*.tcl after a while to give people

a chance to react (maybe in 5.11 or 5.12).

site_node::conn_url -> ad_conn

site_nodes_sync -> site_node::init_cache

site_node_id -> site_node::get_node_id -url

site_map_unmount_application -> site_node::unmount

site_node_delete_package_instance -> site_node::delete

Prefer lappend over concat for appending lists (lappend modifies the variable 'in place')

modermize tcl

modernize tcl

  1. … 1 more file in changeset.
Fix typo (db_list_of_list -> db_list_of_lists)

  1. … 2 more files in changeset.
Fix typos

  1. … 33 more files in changeset.