• last updated 17 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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

Complete content::folder api coverage

Whitespace cleanup

Fix content::extlink::delete: this actually just means to delete the content item

Test public content::extlink api: this test will expose that content::extlink::delete is broken (wrong stored procedure called)

file content-extlink-procs.tcl was initially added on branch oacs-5-10.