• last updated 7 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Extend regression test

    • -5
    • +64
    ./acs-tcl/tcl/test/http-client-procs.tcl
if truncate_len is provided we have to call util_close_html_tags for truncating the string

Fix self-inflicted bug: one should indeed be able to specify the same form var multiple time, test the behavior for the future

    • -17
    • +21
    ./acs-tcl/tcl/http-client-procs.tcl
    • -0
    • +59
    ./acs-tcl/tcl/test/http-client-procs.tcl
    • -1
    • +1
    ./file-storage/tcl/test/webtest-procs.tcl
Reimplement upload automated test using a real multipart request, as newer naviserver will reject handcrafted .tmpfile parameters

    • -9
    • +19
    ./file-storage/tcl/test/webtest-procs.tcl
Extend util::http::post_payload and util::http::post to receive form varaibles also in dict form, update documentation

    • -22
    • +58
    ./acs-tcl/tcl/http-client-procs.tcl
Cleanup leftover arguments

Test ad_return_exception_template

    • -0
    • +76
    ./acs-templating/tcl/test/acs-integration-procs.tcl
file acs-integration-procs.tcl was initially added on branch oacs-5-10.

    • -0
    • +0
    ./acs-templating/tcl/test/acs-integration-procs.tcl
Deprecate acs-subsite/www/shared/db-error and move a copy to acs-subsite/lib/shared/db-error, replace occurrences, adjust dependencies

    • -0
    • +20
    ./acs-subsite/lib/shared/db-error.adp
    • -0
    • +1
    ./dotlrn-homework/dotlrn-homework.info
    • -2
    • +2
    ./dotlrn-homework/www/folder-create.tcl
file db-error.adp was initially added on branch oacs-5-10.

    • -0
    • +0
    ./acs-subsite/lib/shared/db-error.adp
Make this template more robust to the casual invocation from the outside

This should actually be moved to the lib directory, as it is intended as an include

Whitespace changes

    • -14
    • +15
    ./acs-subsite/www/shared/db-error.adp
Test ad_dimensional_sql

    • -0
    • +45
    ./acs-templating/tcl/test/dimensional-procs.tcl
Test ad_dimensional

    • -0
    • +69
    ./acs-templating/tcl/test/dimensional-procs.tcl
file dimensional-procs.tcl was initially added on branch oacs-5-10.

    • -0
    • +0
    ./acs-templating/tcl/test/dimensional-procs.tcl
Make spelling consistent

    • -1
    • +1
    ./acs-subsite/catalog/acs-subsite.de_DE.ISO-8859-1.xml
Quote unallowed HTML according to acs-tcl.documentation__check_proc_doc

Rename proc according to convention enforced in acs-tcl: naming__proc_naming

Reinstate the notnull validation constraint where missing

    • -2
    • +2
    ./forums/lib/message/threads-chunk.tcl
    • -2
    • +2
    ./forums/lib/user/post-history-chunk.tcl
    • -5
    • +5
    ./forums/www/admin/subscribe-others-2.tcl
    • -2
    • +2
    ./forums/www/admin/subscribe-others.tcl
  1. … 10 more files in changeset.
re-add notnull value constraint on message_id

This constraint was removed about 14 days ago. Without this constraint,

errors like the following will show up....

[30/Aug/2022:14:07:29.820625][3102633.7f152c28d700][-conn:openacs.org:default:0:196214-] Error: rp_handler no-script-abort: errorMsg can't read "message(forum_id)": no such variable while serving GE

T /forums/message-view?message_id HTTP/1.1

: ad_url </forums/message-view> maps to file </var/www/openacs.org/packages/forums/www/message-view.adp>

: TCL LOOKUP VARNAME message

: can't read "message(forum_id)": no such variable

: while executing

: "forum::get -forum_id $message(forum_id) -array forum"

Probably, in many more cases, the incorrect removal of the notnull constrain performed by [1]

requires the same fix.

[1] https://github.com/openacs/forums/commit/c403e3134ce4e73ba94a80540a8b0f3cb1651ed0#diff-d35c736495ede0174fb39791ab660c5cd1a4ca467db2c4741cfc935167d7e5a8

handle clases, where "nt_load_end" is not provided

Replace message keys inside message keys. Many thanks to Monika Andergassen.

    • -1
    • +1
    ./acs-subsite/catalog/acs-subsite.de_DE.ISO-8859-1.xml
    • -1
    • +1
    ./acs-subsite/catalog/acs-subsite.en_US.ISO-8859-1.xml
    • -1
    • +1
    ./acs-subsite/catalog/acs-subsite.it_IT.ISO-8859-1.xml
Harden page contract

    • -9
    • +2
    ./acs-subsite/www/user/portrait/upload.tcl
Make script more robust to agents that won't specify the tmpfile via using the api to retrieve the file information

    • -6
    • +14
    ./acs-subsite/www/user/portrait/upload.tcl
Fix typo

    • -2
    • +4
    ./acs-templating/tcl/element-procs.tcl
template::element validation reform: do not run custom user-defined validation on fields that are not formally correct

Rationale: user-defined validation often assumes some basic consistency of the submitted value. When this is not formally correct, users might have to reimplement the whole formal validation in their custom fields in order to prevent exceptions. By preventing their custom blocks from being executed, users can rely on their custom fields to be run only against sane values.

    • -20
    • +38
    ./acs-templating/tcl/element-procs.tcl
Restore formatting in procs doc

    • -79
    • +99
    ./acs-templating/tcl/element-procs.tcl
Normalize indentation according to editor hints

    • -366
    • +366
    ./acs-templating/tcl/element-procs.tcl
comment tags, which are NOT supported by HTML5, allow "abbr" in enhanced text

Fix query syntax