• last updated 4 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Store statistics in a separate page instead of the workflow itself

- advantages:

* smaller workflow instance

- no danger that page is too large for caching, even for large exams.

- smaller objects mean also better performance

* modification date in exam is not modified when statistics

are updated

* easier extensible

- New function WorkflowPage->childpage

easy to use interface for creating child pages

- render_submission=exam_protocol: changed name of

parameter "-form_objs" to "-filter_form_ids", since

form_objs are used typically to keep objects, not ids

- Still to do: policy for inclass-exam-statistics.wf

- Bumped version number to 5.10.1d25

    • -8
    • +12
    /openacs-4/packages/xowf/lib/inclass-exam.wf
    • -40
    • +113
    /openacs-4/packages/xowf/tcl/test-item-procs.tcl
    • -8
    • +55
    /openacs-4/packages/xowf/tcl/xowf-procs.tcl
file inclass-exam-statistics.wf.page was initially added on branch oacs-5-10.

Updated inferface for Page->create_form_page_instance

Since "create_form_page_instance" creates always a form

page, we can also pass the state at creation to it

(like other essential parameters). The default value

is "initial" (the default state in workflows).

Bumped version number to 5.10.1d15

    • -2
    • +2
    /openacs-4/packages/xowiki/xowiki.info
    • -5
    • +3
    /openacs-4/packages/xowiki/tcl/xowiki-procs.tcl
updated list of tested functions

Fix comment

Simplify test logics, bring comment closer to relevant code

make calculation of [ns_set array [ns_getform]] more robust

in case, invalid UTF-8 is passed in, [ns_getform] might be empty.

probably this could also be addressed in NaviServer

    • -5
    • +8
    /openacs-4/packages/xowf/www/index.vuh
improve styling of horizonal form fields (many thanks to Monika Andergassen)

Improve robustness of xowf.create_test_items automated test: make sure that the exercise appearing is actually supposed to render 2 images

Fixes xowf.create_test_items automated test

Compute amplitude and wavelength explicitly in pixels and make it stable with respect to the captcha size, introduce a random offset for the wave instead

Fixes acs-templating.tesseract_cannot_crack_catpcha automated test

Fix typo

Fix tag

Minor modernization

- use atomic stack operations to reduce locks and avoid race conditions

- improve spelling

- break overlong lines

- better align comments with actual code

refactor code

    • -20
    • +44
    /openacs-4/packages/xowiki/tcl/package-procs.tcl
    • -35
    • +8
    /openacs-4/packages/xowiki/tcl/xowiki-procs.tcl
Make creation of ::xo::cc more robust

In case the argument processing of

ConnectionContext create ::xo::cc -p1 ... -p2 ....

::xo::cc destroy_on_cleanup

runs into an exception leading to an ad_script_abort, the half-in

initialized object without cleanup definition will survive. At later

times, this object might be reused, containing potentially

cached results from earlier runs, which can cause troubles in error

cases.

This change takes care that errors during argument processing will

not leave half-initialized objects behind.

improve wording in comments

use "ns_strcoll" for ordered composite when available

improve wording

safetybelt for acunetix testing: refetch parameter page, when it is not initialized

hopefully a temporary measure

    • -1
    • +9
    /openacs-4/packages/xowiki/tcl/xowiki-procs.tcl
Do not consider unchecked radio buttons as form values, or when an unselected radio exists, the last value will might be picked

Fixes xowiki.create_form_with_form_instance, breaks xowf.create_test_items

Test reproducing an upstream use case: a compound field made of both regular, and compound field, with the compound field also repeated

This test exposes a bug in the acs::test::xpath::get_form_values proc

update message catalog

Provide a message key for help text, localize in German

Drop the pointsize parameter, use size instead, yielding more predictable results across Imagemagik versions and more controllable in general, make the wavelength randomized in the distortion

Port downstream change: adding label for question selection box

Many thanks to Felix Mödritscher

Normalize ordering in the catalog files to facilitate downstream merges

include timing information in DEBUG lines only, when nsf was configured properly with it

avoid unneccesary warnings (and unneccessary work)

In case the exam-protocol is shown filtered (e.g. only selected questions)

there is distinction between the available answer attributes and the

answer attributes, which is shown. Earlier versions tried to lookup

all available answer attributes, now we restrict the lookup to only

the shown answer attributes, and therefore avaiding running into

warnings of the form "No form field with name ... found"

Deprecate template::util::tcl_to_sql_list, completely replaced by ns_dbquotelist, a native NaviServer command

  1. … 3 more files in changeset.
Improve test portability