• last updated 22 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Various changes on the proctoring artifacts users list:

- display some statistics on the collected artifacts for each user: count, number of reviewed, number of flagged, plus a colored progress bar indicating the review status, orange when in progress, red when completed with flags and green when everything is ok

- switched to a template::list to avoid too much handcrafting

- removed the red delete button from the top of the page, redundant, since we have the bulk actions on the list

- refined the behavior of the bulk-action delete button: it will be disabled when nothing is selected

- now we always trigger a reload when new artifacts are received for this object, but we cap it to 60 seconds

Store the string "false", not the boolean false

Improve spelling

Fine-tuning the new db function interface

#

# Check, wether we have to regenerate the database function interface.

#

# - During initial setup, there are no db-functions, so nothing has to

# be done.

#

# - During regular startup of the server, the generation of the stub

# interface happens in the *init procs (hopefully this is always

# sufficient, but seems so)

#

# - During reloads of acs-db-*-procs, the base classes are interface

# objects are recreated and cleaned up from all prior definitons,

# which means that in this situations, we have to regeneate the

# interface.

#

# - One might call manually the regeneration, when database functions

# have been altered and no restart is desired.

#

Use a full join so that in case of (normally minimal) unalignment between camera and desktop, we still display every artifact

avoid dead operations

Keep making sure permission exists

Reform of site-nodes-procs

- made XOTcl2 based implementation of site-nodes usable during

initial bootstrap

- made new site-node cache implementation working for Oracle

- now, the same code works during bootstrap, for PostgreSQL and Oracle

and independent of xotcl-core being installed or not.

- removed old and broken nsv-based site-node cache

- reduced need of .xql files (removed query for delete.delete_site_node)

- moved deprecated calls to deprecated-procs.tcl

This change make the code much clearer and eases maintenance and debugging.

    • -327
    • +499
    /openacs-4/packages/acs-tcl/tcl/deprecated-procs.tcl
    • -1587
    • +777
    /openacs-4/packages/acs-tcl/tcl/site-nodes-procs.tcl
Update of SQL function calling interface to make it usable during initial bootstrap

- add sanity checks and produce error messages,

when function args are incorrect.

- do not throw errors in case it is called before the

table acs_function_args was created (during boot)

- added preliminary "list" call to acs::dc

- added first *-init.tcl to be called after loading

the *-proc.tcl files.

- removed all references to the ::xo::* namespace

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

    • -0
    • +0
    /openacs-4/packages/acs-tcl/tcl/00-acs-tcl-init.tcl
Make parameter::get_from_package_key usable during inital boot

This change makes parameter::get_from_package_key usable during

initial boot, i.e. before the table apm_parameters is created.

In these cases, the function returns just the default value.

Support memory units as default of the cache sizes

update questionable test slightly

Fixes for Oracle 19c: made set_live_revision compatible with postgres version

Fix expression

Allow to filters also by timestamp, start or end

file upgrade-5.10.1d5-5.10.1d6.sql was initially added on branch oacs-5-10.

improve wording

fix broken function_args definition

file upgrade-5.10.1d2-5.10.1d3.sql was initially added on branch oacs-5-10.

Provide an own translation for "All" that considers the specific item (e.g. for gendering)

Changes after review with downstream stakeholders:

- switch "Confirm OK" and "Flag" buttons

- put the two buttons after the comment section

Bonus change: mark unflagged artifacts with comments with a green border

Normalize the way the flag is represented in the JSON

Use a trash can icon, instead of all that yellow

Fix expression again

The flag coming from the ajax endpoint is an actual boolean

Make the position for the fullscreen preview sticky, and ensure the fullscreen element is positioned as the first body child, so it is put on top and can get as big as possible

Added support for www-toggle-publish-status

This is needed to avoid a bad interaction with [ad_return_url] as it

is used in www-toggle-publish-status in xowiki, since the workflow

definition unsets the actual return_url, which causes ad_return_url to

use the URL leading to this call (m=toggle-publish-status), causing a

redirection loop.

Version number bumped to 5.10.1d28

    • -1
    • +18
    /openacs-4/packages/xowf/lib/inclass-exam.wf
Added the option to parameterize www-delete and www-toggle-publish-status with return_url

In cases, these www* methods are called programmatically, these can be

now parameterized with an "-return_url" parameter to achieve

e.g. workflow-specific behavior.

Many thanks to Thomas Renner for the analysis on

www-toggle-publish-status.

Version number bumped to 5.10.1d18

    • -2
    • +2
    /openacs-4/packages/xowiki/xowiki.info
    • -6
    • +13
    /openacs-4/packages/xowiki/tcl/package-procs.tcl
set arguments to type integer to avoid strange variable names