OpenACS

Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 6
Total Committers: 146
Last Commit: 21 Nov
Commits this week: 6
Total Lines of Code (LoC): 7,560,943
Net change in LoC this week: 122

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated 1 hour ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Thursday 25 Aug 2022
1:29 pm

antoniop

Bring test closer to reality

Options
    • -6
    • +14
    /openacs-4/packages/file-storage/tcl/test/webtest-procs.tcl
1:25 pm

antoniop

Introduce security::safe_tmpfile_p checking whether a file belongs to the configured tmpfolder and respects other constraints

The plan is to use it to improve input validations

Options
    • -2
    • +2
    /openacs-4/packages/acs-tcl/acs-tcl.info
    • -1
    • +54
    /openacs-4/packages/acs-tcl/tcl/security-procs.tcl
    • -0
    • +45
    /openacs-4/packages/acs-tcl/tcl/test/security-procs.tcl
11:53 am

gustafn

Fixed serious bug killing at least short-text questions in inclass exam

The bug was introduced in [1], by testing for the existence of the

disabled attribute, and when it exists, it was omitting values

reading. The problem is that when form-fields are reset, the

"disabled" attribute is set to 0, leading the exists check to

succeed. In essence, This change sets now the default value of the

form-field to "0", such that it is safe to test it everywhere.

Originally, it was not set by default to save resources (memory and

processing power), but this requires a more careful analysis when

changes happen.

[1] https://fisheye.openacs.org/browse/OpenACS/openacs-4/packages/xowiki/tcl/xowiki-www-procs.tcl?r1=1.368.2.125&r2=1.368.2.126

Options
    • -2
    • +2
    /openacs-4/packages/xowiki/tcl/form-field-procs.tcl
    • -12
    • +11
    /openacs-4/packages/xowiki/tcl/xowiki-www-procs.tcl
Wednesday 24 Aug 2022
7:45 pm

gustafn

Tighten test for is_wf_instance

Previously, the test was based on setting of state and FormPage.

In cases, where plain FormPages are used inside an xowf package

this test was leading to wrong results.

Options
    • -1
    • +4
    /openacs-4/packages/xowf/tcl/xowf-procs.tcl
    • -0
    • +1
    /openacs-4/packages/xowf/tcl/test/test-item-procs.tcl
6:50 pm

gustafn

base all timings on NavigationTiming plugin to get closer to previous behavior

Options
    • -5
    • +13
    /openacs-4/packages/boomerang/tcl/boomerang-procs.tcl
    • -0
    • +456
    /openacs-4/packages/boomerang/www/resources/plugins-1.737.0/navtiming.js
6:50 pm

gustafn

file navtiming.js was initially added on branch oacs-5-10.

Options
    • -0
    • +0
    /openacs-4/packages/boomerang/www/resources/plugins-1.737.0/navtiming.js
6:00 pm

gustafn

improved error message

make clear, what the name and what the item_id in the message are

Options
    • -2
    • +2
    /openacs-4/packages/xowiki/tcl/folder-procs.tcl
5:59 pm

gustafn

use new path xowiki/lib/portlets/ instead of .../www/...

Options
    • -3
    • +4
    /openacs-4/packages/xowiki/tcl/xowiki-procs.tcl
5:57 pm

gustafn

comment method

Options
    • -1
    • +4
    /openacs-4/packages/xowiki/tcl/xowiki-uploader-procs.tcl
4:51 pm

gustafn

Deactivated script-dynamic again

activating "script-dynamic" for script-src prevents resources as the following from loading

<script type="text/javascript" src="/resources/xowiki/bootstrap-treeview/1.2.0/bootstrap-treeview.min.js" nonce="F8843D5CD542FB1CDB4C94C5D13C4E5ECC2E1DCA">

Options
    • -2
    • +2
    /openacs-4/packages/acs-tcl/tcl/security-procs.tcl
4:30 pm

gustafn

improve test coverage of apm_* API

Options
    • -0
    • +24
    /openacs-4/packages/acs-tcl/tcl/test/apm-procs.tcl
4:28 pm

gustafn

reduce verbosity

Options
    • -1
    • +1
    /openacs-4/packages/openacs-bootstrap5-theme/resources/widgets/login.tcl
2:13 pm

gustafn

added test for checking documentation in regard of valid HTML markup

Options
    • -2
    • +53
    /openacs-4/packages/acs-tcl/tcl/test/doc-check-procs.tcl
10:58 am

antoniop

Test the behavior of the file-storage when a malicious user would try to store a pre-existing file on the server as its own

The fix for the file-storage is a simple validation to make sure that the tmpfile exists, however, for the generic case of the file widget, we cannot trust the tmpfile value when this was not generated by the server. This will probably cause regression when one wants to show a "preview" of a form, to be continued.

Options
    • -19
    • +4
    /openacs-4/packages/acs-templating/tcl/file-procs.tcl
    • -1
    • +121
    /openacs-4/packages/file-storage/tcl/test/file-storage-procs.tcl
    • -1
    • +5
    /openacs-4/packages/file-storage/www/file-add.tcl
9:05 am

antoniop

Whitespace cleanup

Options
    • -6
    • +6
    /openacs-4/packages/file-storage/tcl/test/webtest-procs.tcl
Tuesday 23 Aug 2022
9:10 pm

gustafn

replace occurrences of <br/> in api doc strings

Options
    • -8
    • +8
    /openacs-4/packages/acs-mail-lite/tcl/email-inbound-procs.tcl
    • -12
    • +16
    /openacs-4/packages/xotcl-core/tcl/cr-procs.tcl
    • -9
    • +12
    /openacs-4/packages/xotcl-core/tcl/generic-procs.tcl
8:44 pm

gustafn

Properly escape "<" and ">" in api-doc documentation.

Since all documentation is rendered via HTML, the characters

"<" and ">" have to be HTML-quoted, otherwise strange things

(omission, unintended renderings) might occur.

E.g. the sentence

Define an interface between a page and an

ADP <include> similar to the page_contract.

was rendered as

Define an interface between a page and an

ADP similar to the page_contract.

which is incorrect.

Options
    • -3
    • +3
    /openacs-4/packages/acs-api-browser/tcl/acs-api-documentation-procs.tcl
    • -2
    • +2
    /openacs-4/packages/acs-bootstrap-installer/tcl/00-proc-procs.tcl
    • -1
    • +1
    /openacs-4/packages/acs-content-repository/tcl/publish-procs.tcl
    • -7
    • +7
    /openacs-4/packages/acs-lang/tcl/lang-util-procs.tcl
    • -2
    • +4
    /openacs-4/packages/acs-mail-lite/tcl/acs-mail-lite-procs.tcl
    • -3
    • +3
    /openacs-4/packages/acs-mail-lite/tcl/email-inbound-procs.tcl
    • -2
    • +2
    /openacs-4/packages/acs-mail-lite/tcl/legacy-inbound-procs.tcl
    • -6
    • +7
    /openacs-4/packages/acs-tcl/tcl/deprecated-procs.tcl
    • -5
    • +9
    /openacs-4/packages/acs-tcl/tcl/security-procs.tcl
    • -17
    • +22
    /openacs-4/packages/acs-tcl/tcl/tcl-documentation-procs.tcl
    • -7
    • +6
    /openacs-4/packages/acs-tcl/tcl/utilities-procs.tcl
    • -10
    • +18
    /openacs-4/packages/notifications/tcl/notification-callback-procs.tcl
    • -5
    • +6
    /openacs-4/packages/tsearch2-driver/tcl/tsearch2-driver-procs.tcl
    • -0
    • +8
    /openacs-4/packages/xowf/tcl/test-item-procs.tcl
5:39 pm

antoniop

Incorporate feedback from downstream penetration tests:

do not skip basic validation in the file widget when a tmpfile is not found in the request.

Options
    • -47
    • +54
    /openacs-4/packages/acs-templating/tcl/file-procs.tcl
5:35 pm

antoniop

forum_id is actually optional here

Options
    • -2
    • +2
    /openacs-4/packages/forums/lib/search/search-form.tcl
4:53 pm

antoniop

Port of downstream hotfix: questions defined with an empty scoring might later fail the score calculations

One might consider preventing empty scores altogether via validation

Options
    • -3
    • +3
    /openacs-4/packages/xowiki/tcl/form-field-procs.tcl
2:58 pm

gustafn

improving rendering in api-doc

- fixed validity of HTML (missing <dd>)

- use

<dt>foo</dt><dd>...</dd>

instead of

<dt><b>foo></b><dd>...</dd>

and other variants. This improves stylability.

Options
    • -29
    • +29
    /openacs-4/packages/acs-api-browser/tcl/acs-api-documentation-procs.tcl
    • -2
    • +2
    /openacs-4/packages/acs-api-browser/tcl/api-doc-init.tcl
2:54 pm

gustafn

Moved styling to site-master.css

Options
    • -7
    • +0
    /openacs-4/packages/acs-api-browser/www/proc-view.adp
    • -0
    • +6
    /openacs-4/packages/acs-subsite/www/resources/site-master.css
2:53 pm

gustafn

properly quote tags when meant literally

Options
    • -3
    • +3
    /openacs-4/packages/acs-bootstrap-installer/installer/www/blank-master.tcl
2:06 pm

antoniop

Extend test depending on the setting of EmailForgottenPasswordP

Options
    • -18
    • +51
    /openacs-4/packages/acs-subsite/tcl/test/acs-subsite-procs.tcl
1:10 pm

gustafn

Changed "db_multirow" to use "db_list_of_lists" instead of "db_list_of_ns_sets"

Thanks to the recent improvements in NaviServer for

"db_list_of_lists", it is more efficient to use this in "db_multirow"

(actually in "db_multirow_helper"). The implementation revealed a bug

in "db_list_of_lists", when both options "-columns_var" and

"-with_headers" were set.

Extended regression test to cover the former bug.

Options
    • -9
    • +19
    /openacs-4/packages/acs-tcl/tcl/01-database-procs.tcl
    • -1
    • +17
    /openacs-4/packages/acs-tcl/tcl/test/db-proc-test-procs.tcl
Monday 22 Aug 2022
7:06 pm

gustafn

new proc "ad_log_deprecated": unified interface for logging deprecated usages

The existing code used a larger variety of different messages to

denote invocations of deprecated procs and other

artifacts. "ad_log_deprecated" provides a unified interface, and

provides a usage hint what to use instead based on the API-doc

definitions in the log-file.

Options
    • -2
    • +2
    /openacs-4/packages/acs-bootstrap-installer/acs-bootstrap-installer.info
    • -2
    • +33
    /openacs-4/packages/acs-bootstrap-installer/tcl/00-proc-procs.tcl
    • -2
    • +3
    /openacs-4/packages/xotcl-core/xotcl-core.info
    • -3
    • +3
    /openacs-4/packages/xotcl-core/tcl/05-db-procs.tcl
    • -3
    • +3
    /openacs-4/packages/xotcl-core/tcl/06-package-procs.tcl
    • -3
    • +3
    /openacs-4/packages/xotcl-core/tcl/06-param-procs.tcl
    • -3
    • +3
    /openacs-4/packages/xowiki/xowiki.info
    • -6
    • +8
    /openacs-4/packages/xowiki/lib/view.tcl
    • -1
    • +1
    /openacs-4/packages/xowiki/tcl/menu-procs.tcl
    • -3
    • +2
    /openacs-4/packages/xowiki/tcl/package-procs.tcl
    • -6
    • +9
    /openacs-4/packages/xowiki/tcl/xowiki-procs.tcl
    • -4
    • +4
    /openacs-4/packages/xowiki/tcl/xowiki-utility-procs.tcl
6:08 pm

gustafn

Added Drag and Drop interface for feedback files

- When grading a student, feedback files can be added via drag and drop

- Caution was spent to avoid potentially slow refresh operations on

log exam protocols. All interactions use AJAX and update the

DOM tree directly

- The feedback files are displayed as thumbnails

- When permissions allow, the thumbnail files can be directly viewed and/or deleted

- Supports Bootstrap3 and Bootstrap 5

- Support for composite questions.

- bumped version number to 5.10.1d43

Options
    • -3
    • +3
    /openacs-4/packages/xowf/xowf.info
    • -0
    • +2
    /openacs-4/packages/xowf/catalog/xowf.de_DE.ISO-8859-1.xml
    • -0
    • +2
    /openacs-4/packages/xowf/catalog/xowf.en_US.ISO-8859-1.xml
    • -55
    • +246
    /openacs-4/packages/xowf/tcl/test-item-procs.tcl
    • -0
    • +48
    /openacs-4/packages/xowf/www/resources/test-item.css
4:51 pm

antoniop

Fix missing variable, test the bug

Options
    • -1
    • +49
    /openacs-4/packages/acs-subsite/tcl/test/acs-subsite-procs.tcl
    • -2
    • +3
    /openacs-4/packages/acs-subsite/www/register/recover-password.tcl
4:44 pm

gustafn

preserve comments hinting to ad_apply and apply whitespace changes

Options
    • -1
    • +2
    /openacs-4/packages/acs-service-contract/tcl/acs-service-contract-procs.tcl
    • -1
    • +2
    /openacs-4/packages/rss-support/tcl/rss-generation-service-procs.tcl
    • -24
    • +25
    /openacs-4/packages/rss-support/www/subscr-ae.tcl
Sunday 21 Aug 2022
6:46 pm

gustafn

rename "iconified file" to "thumbnail file"

Options
    • -4
    • +5
    /openacs-4/packages/xowiki/tcl/bootstrap-procs.tcl
    • -1
    • +57
    /openacs-4/packages/xowiki/tcl/package-procs.tcl
    • -2
    • +2
    /openacs-4/packages/xowiki/tcl/xowiki-uploader-procs.tcl
    • -3
    • +3
    /openacs-4/packages/xowiki/tcl/xowiki-www-procs.tcl