• last updated 14 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Improve proc documentation

Refactor acs-lang 'message-list', using a single query for counting the different message states instead of one per state, and simplifying tcl logic

Fix xotcl-core cr-items test: modifying IP comes in fact from the cr_item's acs_object (it is also on the revision, but on the items seems to take into account e.g. proxying)

Fix 3 inconsistent log messages in the test

Fix typo

improve spelling

    • -1
    • +1
    /openacs-4/packages/xowf/tcl/xowf-procs.tcl
improve spelling

  1. … 3 more files in changeset.
Rework acs-lang 'package-list' query 'packages_locale_status' query (89ms -> 4ms)

Rework acs-lang 'package-list' query 'packages_locale_status_default' query (48ms -> 2ms)

Prefer portable current_timestamp idiom to NOW()

Exploit existing acs::test::user::create api

Normalize indentation, add editor hints

    • -10
    • +17
    /openacs-4/packages/xolp/tcl/xolp-iri-procs.tcl
    • -0
    • +7
    /openacs-4/packages/xolp/tcl/xolp-procs.tcl
    • -31
    • +38
    /openacs-4/packages/xolp/tcl/xolp-user-procs.tcl
    • -10
    • +17
    /openacs-4/packages/xolp/tcl/xolp-util-procs.tcl
Modernize XoTcl idioms

    • -12
    • +11
    /openacs-4/packages/xolp/tcl/xolp-iri-procs.tcl
Use xotcl-core indentation convention

    • -95
    • +96
    /openacs-4/packages/xolp/tcl/xolp-apm-procs.tcl
Whitespace cleanup, normalize indentation, add editor hints

    • -94
    • +100
    /openacs-4/packages/xolp/tcl/xolp-apm-procs.tcl
Use xotcl db idioms

Rollback tests or they will interfere with the others

Reimplement query to obtain user competencies to fix xolp.user_get_competencies automated test

Personal perplexities:

- why do we specify the competency_set_id on the indicator facts? If one fact can be e.g. an exam, this might be related with multiple competency sets

- should we take into account also the validity of an activity?

    • -14
    • +36
    /openacs-4/packages/xolp/tcl/xolp-user-procs.tcl
Add deleted column to acs-lang 'package-list', and fixed number of messages depending on status, making it coherent with 'message-list'. Also fixed translated/untranslated count on default locale in 'message-list'. Version bump.

file package-list.xql was initially added on branch oacs-5-10.

    • -0
    • +0
    /openacs-4/packages/acs-lang/www/admin/package-list.xql
Port downstream commit cab327aeda04f1f1f6b8f2310ddc88c7c081a84f from Günter Ernst:

Minor adjustments to ensure install- and uninstallability

Port of downstream commit 9cbc3f04aad075e902e64d0c67d53158160a933c from Felix Möedritscher:

feature / issue #10359: added message keys, extended the xolp API - get_competencies, get_activities_of_package, update_title

    • -0
    • +4
    /openacs-4/packages/xolp/catalog/xolp.de_DE.ISO-8859-1.xml
    • -0
    • +4
    /openacs-4/packages/xolp/catalog/xolp.en_US.ISO-8859-1.xml
Fix untranslated count and view on acs-lang 'message-list' + some refactoring

remove .SELF. prefix also in link_info dict

    • -1
    • +2
    /openacs-4/packages/xowiki/tcl/xowiki-procs.tcl
fix typo in anonymous array

    • -3
    • +3
    /openacs-4/packages/xowiki/tcl/xowiki-procs.tcl
Insert missing performance indicators for the user

Fix typo

As xolp_indicator_facts_check1 wants end_timestamp <= storage_timestamp, force end_timestamp some time in the past (api won't allow to specify storage_timestamp)

As the original test expects it, make sure begin timestamp and end timestamp are the same

Fix indicator_create test case:

- handle timezone correctly

- don't rely on database language