• last updated 8 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Fix typo

Support for extra extra hints texts (feedback, correction hints)

The updated version of the inclass-exam supports now extra hint texts for lecturers and students. These extra information is displayed after the question, and is used either for feedback to the student (e.g. Einsicht) or as information for the grader (evaluation guidelines, esp. useful when multiple graders are involved). It can be controlled via setup what kind of information the question author can provide. The providable feedback is controlled via the feedback level:

Feedback level

- The feedback level can be

* "full": two input fields for feedback for correct and incorrect answers

* "single": single input fields for feedback

* "none": no input field for feedback.

- The feedback level is specified in the forms used in the

pull down menus for creating "New" test items.

- The feedback level is applicable to all test item types

(SC, MC, Text, ShortText, Ordering, Upload, Composite, Pool)

- The predefined setup is as follows:

"single" is used for all item type, except for "Pool" questions (since the feedback

is taken from the replacement question from the Pool)

- Nothing has change here

Correction notes:

- optionally, correction notes can be specified.

Since in the past, the general feedback was not shown to the students,

it was sometimes abused for correction hints for the lecturer.

Therefore, we have now a separate field

- Correction notes can be added to all item types

(SC, MC, Text, ShortText, Ordering, Upload, Composite, Pool) and are

turned on by default (except for Pool).

Visibility of general feedback and correction notes

- general feedback is shown (when provided)

* full exam-protocol is shown

* filtering single question

* filtering single submission (view with revision selection)

* exam-review for students (Einsicht)

- corrections notes are shown (when provided)

* full exam-protocol is shown

* filtering single question

* filtering single submission (view with revision selection)

- when general feedback is not provided, the exam protocol etc. looks as before

- when correction notes are not provided, the exam protocol etc. looks as before

- Negative feedback is provided,

* when percentage is known, and

* percentage is < 50%

- when negative feedback is displayed, the positive feedback is not displayed and vice versa

For composite questions, the system supports hint texts for the full composite question and additional ones for every single part.

Version number bumped to 5.10.1d32

    • -17
    • +150
    /openacs-4/packages/xowf/tcl/test-item-procs.tcl
Document public api

Revert port of downstream feature: the same check can be performed more flexibly via the condition=regexp and condition=match

    • -5
    • +1
    /openacs-4/packages/xowiki/tcl/xowiki-procs.tcl
Port of downstream feature: condition proc on the FormPage checking that a page name respects some regular expression

    • -1
    • +5
    /openacs-4/packages/xowiki/tcl/xowiki-procs.tcl
Fix typo

    • -2
    • +2
    /openacs-4/packages/xowiki/tcl/xowiki-procs.tcl
Make sure result is defined, small improvements

    • -9
    • +8
    /openacs-4/packages/xowiki/tcl/xowiki-procs.tcl
Port of old local modification: when mime type is text/plain, feed the content straight to the indexer, rather than render the file as HTML, then convert it again to text in the search indexer

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

file iprange-init.tcl was initially added on branch oacs-5-10.

    • -0
    • +0
    /openacs-4/packages/xowf/tcl/iprange-init.tcl
file iprange-procs.tcl was initially added on branch oacs-5-10.

    • -0
    • +0
    /openacs-4/packages/xowf/tcl/iprange-procs.tcl
Added support for restricting access to exams based on IP addresses.

In order to avoid the problem that students might participate on exams

intended to be held e.g. on campus from at home, one can now restrict

the access based on IP ranges. It is possible to restrict the access

via multiple range blocks that can be predefined by the sysadmins. One

could predefine e.g. IP range blocks for "Campus LAN", "On Site",

"VPN", etc., which can be selected by lecturers for individual exams.

Every IP range block can be defined via multiple entries of disallowed

and allowed IP ranges, where the addresses can be specified via IPv4

or IPv6 CIDR blocks (e.g. 127.208.0.0/16), or with wild cards or

individual IP addresses. The usage of CIDR blocks requires an update

to the updated version of NaviServer.

The IP restriction is used in the current implementation for

restricting the regular start of the exam (proctored or

non-proctored). Lecturers can still perform test runs for exams with

restricted access.

Bump version number to 5.10.1d31

    • -1
    • +4
    /openacs-4/packages/xowf/lib/inclass-exam.wf
    • -0
    • +24
    /openacs-4/packages/xowf/tcl/iprange-init.tcl
    • -0
    • +67
    /openacs-4/packages/xowf/tcl/iprange-procs.tcl
Revert additional fallbacks, one should take care that ad_conn is invoked correctly

Provide further fallbacks for ad_conn

Reduce number of queries, improve documentation

Queries could be made even less e.g. by locking and querying the tuples in one statement. However, it would be trickyer to make the code portable.

Simplify nsv idiom

Make sure the value we validate is a list

check that provided signature is a list, otherwise lassign will fail afterwards

Cleanup leftover code

Reimplement the add/edit api to use ad_form

file comment-add-edit.adp was initially added on branch oacs-5-10.

file comment-edit.adp was initially added on branch oacs-5-10.

file comment-add.adp was initially added on branch oacs-5-10.

file comment-add-edit.tcl was initially added on branch oacs-5-10.

Cleanup trivial remarks, provide the user_id explictly to the permission check

Fix typo

Enforce the mimetype the same in follow up script

Enforce only valid mime types

harden page contract

Make publish_body required, as we already enforce it down the line

    • -2
    • +2
    /openacs-4/packages/news/www/item-create.tcl