• last updated 2 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Implement the 'dotlrn_applet::remove_applet_from_dotlrn' proc, called from different remove_applet procs, but never implemented

    • -0
    • +17
    /openacs-4/packages/dotlrn/tcl/applets-procs.tcl
    • -0
    • +19
    /openacs-4/packages/dotlrn/tcl/applets-procs.xql
Replace xowf proctoring implementation with that coming from proctoring-support

TODO: nicer detection of whether proctoring-support package is actually installed

    • -7
    • +9
    /openacs-4/packages/xowf/lib/inclass-exam.wf
    • -239
    • +0
    /openacs-4/packages/xowf/lib/proctored-page.adp
    • -208
    • +0
    /openacs-4/packages/xowf/lib/proctored-page.tcl
    • -67
    • +0
    /openacs-4/packages/xowf/lib/proctoring-upload.tcl
prefer SQL max() function over sort and limit 1

    • -3
    • +4
    /openacs-4/packages/xowiki/tcl/package-procs.tcl
Make apm callbacks private and move them together with the others

Maintain also the has_child cache when mounting a package

Fix typo

Make installation procs private

When a new group is created, flush all the group::get_id caches with the same name so that the new group has a chance to be retrieved

Note that it is wrong to assume a group_name will be unique, although this is normally not far from the truth in our everyday practice

Cover more group api

this also exposes a potential problem with group::get_id when a group is created, deleted, and then a new one is created with the same name (e.g. executing the test multiple times). As the memoize cache is not maintained, the returned group_id will be incorrect.

Increase proc coverage

Reinstate lost queries for content::item::content_is_null

file content-item-procs-oracle.xql was initially added on branch oacs-5-10.

file content-item-procs-postgresql.xql was initially added on branch oacs-5-10.

Move examination statement acceptance as last step of the proctoring wizard, localize the accept button, bind ajax logging of acceptance to the actual clicking of the accept button

Improve test

New test for lang::util::convert_to_i18n pinpointing some expected behaviors of this proc

Upgrade CKEditor to 4.14.1

Improve doc and minor refactoring to reduce the diff with possible upcoming change

    • -10
    • +42
    /openacs-4/packages/xowiki/tcl/xowiki-procs.tcl
Fix typo

cache method "has_children" to avoid frequent (but cheap) DB calls

improve handling of errors, which are triggerd by the error template

acs-lang reform:

when calling lang::util::convert_to_i18n, do not always register a en_US translation, which will be otherwise always overridden. Instead, let lang::message::register make sure that a en_US message exists and create it only when needed as a fallback. Before, lang::message::register would return an error when e.g. a new message keys was registered for de_DE and no en_US was there. Now the en_US will be just created on the fly.

This does not break current automated tests

remove unnecessary parameter package_id

    • -2
    • +1
    /openacs-4/packages/xowiki/tcl/weblog-procs.tcl
fix comment and missing arguemnt

use package based instantiate_forms method

whitespace cleanup

whitespace changes

make "instantiate_forms" a method of packages

this reads better than the previous variant based on the Weblog class.

    • -3
    • +3
    /openacs-4/packages/xowiki/xowiki.info
    • -7
    • +6
    /openacs-4/packages/xowiki/tcl/folder-procs.tcl
    • -5
    • +4
    /openacs-4/packages/xowiki/tcl/import-procs.tcl
    • -5
    • +4
    /openacs-4/packages/xowiki/tcl/package-procs.tcl
    • -26
    • +8
    /openacs-4/packages/xowiki/tcl/weblog-procs.tcl
    • -2
    • +2
    /openacs-4/packages/xowiki/tcl/xowiki-procs.tcl
whitespace changes

prefer name resolver over "my" call