• last updated 5 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
improve spelling

factor out loop invariants

thighten parameter checking to valid classes

    • -3
    • +3
    /openacs-4/packages/xowiki/tcl/package-procs.tcl
Provide default value for "return_url" and use "export_vars" for url-construction; whitespace changes

improve spelling

    • -1
    • +1
    /openacs-4/packages/dotlrn/tcl/applets-procs.tcl
    • -1
    • +1
    /openacs-4/packages/dotlrn/www/element-list.tcl
added error message for new validator

    • -2
    • +2
    /openacs-4/packages/xowiki/xowiki.info
added validator for "correct_when" form field types

prefer variable names over method calls

new includelet for documentation purposes: available-formfields

Fix typo in message key

- Fix a tricky Tcl_Obj sharing bug triggered by the usage of C-level

IndexObjs based on volatile tables. Consider the following example:

proc foo {x} {

return [bar -value x ...]

}

The Tcl_Obj "x" is shared as name of argument and as a value of the

non-positional parameter "-value". When bar (e.g. C-implemented)

uses Tcl_GetIndexFromObj*() to lookup "x" in a table of options,

which is volatile, the involved Tcl_Obj will be converted to an

indexObj.

On a call with wrong number of arguments( e.g."foo 1 2 3"), Tcl

will try to give a nice error message, saying that "foo x" can be

called only with one argument. When printing argument "x", it sees

that "x" is an indexObj, and for these kinds of objects, "x" might

be an abbreviated version of a full name. Since the table behind

the indexObj is in the case above volatile, a crash might happen.

The committed fix above is just a temporary measure. NaviServer should

try to avoid such Tcl_Obj conversions in these cases. It is also

questionable, whether the attempt to interpret a argument name as

an abbreviated.

deactivate html hacking (replace textarea by "p") since rendering is

broken due to bootstrap's "form-control" class

    • -2
    • +2
    /openacs-4/packages/xowf/lib/online-exam.wf
strip white space

Make page for managing the parameter "PrivateEmailLevelP" subsite aware.

allow empty correct_when specifications

Whitespace cleanup

Use already existing message-key.

ensure that every formfield is disabled in the disabled_fc list (e.g. mc-interaction)

Export message keys to normalize sorting anx xml quoting (will prevent huge diffs with new message keys)

Fixed typo in message-key. Bump acs-lang version to reload message keys.

reduce verbosity

use new form-field type "correct_when"

provide new form-field "correct_when" for conveniant user interaction

the form-field type is currently just a text field with a tailored

help text, but can be extended in the future via pull-downs etc.

    • -2
    • +2
    /openacs-4/packages/xowiki/xowiki.info
use ad_return_complaint for invalid query parameter parsing (at least in connection threads)

fix workflow such these work as well with shared workflow definitons

    • -26
    • +26
    /openacs-4/packages/xowf/lib/online-exam-answer.wf
    • -38
    • +44
    /openacs-4/packages/xowf/lib/online-exam.wf
New test for json and utf-8 encoding against Postman Echo webservice (https://docs.postman-echo.com)

Remove unused variable

provide missing args for references call

Use template::util::tcl_to_sql_list for proper list element quoting

impreve robustness of ad_doc parsing