• last updated 6 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Reuse computation

    • -2
    • +2
    ./acs-subsite/www/user/portrait/index.tcl
Avoid catch where a more appropriate idiom exists

    • -8
    • +10
    ./acs-subsite/www/user/portrait/index.tcl
Improve documentation

    • -1
    • +6
    ./acs-subsite/www/user/portrait/index.tcl
Cleanup case

    • -4
    • +0
    ./acs-subsite/www/user/portrait/index.adp
Whitespace cleanup

    • -3
    • +3
    ./acs-subsite/www/user/portrait/index.tcl
Harden page contract

    • -10
    • +2
    ./acs-subsite/www/user/portrait/index.tcl
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