• last updated 9 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
prefer "string first" over "string match" (the first one is slightly faster)

Streamline idiom

Whitespace cleanup

Do the transaction only if necessary

Break overlong lines

Add multiselect to the widgets we check

Fix validated input format

Restrict new widget validation of options to select and radio, not every widget must restrict the values to the options property

Apply a date-aware validation

Trap the case where invalid dates are supplied to the proc

Cleanup redundant validation

Implement a hook for widget-specific validation and use it to enforce that widgets that specify a list of options must return a value that belongs to those options

Add input validation

Don't pass around variables that are not meant to be changed by the UI as hidden fields

Fix typo

no need for curly braces

extend test cases

improve documentation and streamline code

do not allow to edit unresolve links during the exam, but allow this in the preview mode

Inform widget: do not render any input element alongside the form value

Check if o is an object only once, use nsf idiom to check for a class, unwrap idioms

Port edit-new policy rule from original policy1 object, so that unresolved links can be rendered correctly

Fixes xowf create_composite_test_item automated test

Use package_id instead of the package object

    • -2
    • +2
    /openacs-4/packages/xowiki/tcl/package-procs.tcl
fix typo

    • -2
    • +2
    /openacs-4/packages/xowiki/tcl/package-procs.tcl
Disallow "create-from-prototype" to non-admins in exam-answering workflows.

Added policy priviledge "create-from-prototype"

In certain situations, creating pages from prototype pages might not

be allowed for certain users. The off-the-shelf policies allow

"create-from-prototype" to all users having "create" permissions on

the package_id (very similar to edit-new).

Improve program comments.

    • -30
    • +48
    /openacs-4/packages/xowiki/tcl/package-procs.tcl
Don't break the page cyclus protocol if not necessary.

There is no need to call ad_script_abort in a situation,

where the local installation is broken.

Modernize parameter parsing (query and includelet paramters)

- handle invalid UTF-8 exception that might be triggered by

"ns_parsequery" in newer versions of NaviServer

- the new code is about 4 times faster than the old one

- replaced array by dict

- don't create helper procs on the fly but use "nsf::parseargs" if possible

- added icanuse rules for

* "nsf::parseargs -asdict"

* "ns_parsequery -charset"

make function private

New test to detect smells of hacking in the code, raising a warning when certain smelly keywords are found