antoniop in OpenACS

About

2,044 commits
  • Committer is not mapped to a user
  • First commit: 20 November 2006
  • Last commit: 03 December 2020
Last Week All Time
Commits: 0 2,044
Files changed: 0 3,666
Change in LOC: 0 19,494

Line History

Line Count Graph
Line Count Graph

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
 
Thursday 03 Dec
2:42 pm

antoniop

Whitespace cleanup

Options
    • -3
    • +3
    /openacs-4/packages/categories/lib/map.tcl
2:42 pm

antoniop

Use page contract

Options
    • -6
    • +8
    /openacs-4/packages/categories/lib/map.tcl
2:36 pm

antoniop

Add missing var

Options
    • -0
    • +1
    /openacs-4/packages/acs-subsite/www/admin/rel-segments/elements-display.tcl
2:30 pm

antoniop

Add include contract

Options
    • -1
    • +7
    /openacs-4/packages/acs-subsite/www/admin/rel-segments/elements-display.tcl
Thursday 26 Nov
6:03 pm

antoniop

New method to cleanup all data associated with a community in the throttle object:

if a community is deleted when a user is accessing it, some leftover information will be kept in the object. At the next request this will be interpreted as the community having been left and the system will try to track this event into request_monitor_community_activities. Depending on who comes first, we might try to insert a tuple in the table for an object that does not exist anymore.

Hence, we use this method to cleanup all community info before the actual deletion in dotlrn (happening only downstream).

Options
    • -2
    • +18
    /openacs-4/packages/xotcl-request-monitor/tcl/throttle_mod-procs.tcl
Tuesday 24 Nov
7:38 pm

antoniop

Don't be so strict upstream, as here we do not restrict community_key to ltree

Options
    • -2
    • +2
    /openacs-4/packages/dotlrn/tcl/community-procs.tcl
7:22 pm

antoniop

Constrain further the characters we allow in the community_key: makes sense upstream as this will become part of a URL, but is even more important downstream, as we implement the community_key as a ltree and therefore we are limited in the type of characters we can allow (See https://www.postgresql.org/docs/current/ltree.html)

Options
    • -3
    • +3
    /openacs-4/packages/dotlrn/tcl/community-procs.tcl
4:40 pm

antoniop

Fix typo

Options
    • -2
    • +2
    /openacs-4/packages/dotlrn/tcl/community-procs.tcl
4:36 pm

antoniop

Add missing arguments

Options
    • -2
    • +4
    /openacs-4/packages/dotlrn/tcl/community-procs.tcl
4:26 pm

antoniop

Rework dotlrn_community::check_community_key_valid_p so that we check also for the community key to not have spaces

Logics have been moved to private api that might or might not become the preferrable api at some point, as it also returns the specific error message together with the boolean

Options
    • -9
    • +43
    /openacs-4/packages/dotlrn/tcl/community-procs.tcl
    • -9
    • +0
    /openacs-4/packages/dotlrn/tcl/community-procs.xql
Monday 23 Nov
3:55 pm

antoniop

Whitespace cleanup

Options
    • -5
    • +5
    /openacs-4/packages/acs-subsite/www/register/user-join.tcl
3:55 pm

antoniop

Remove irrelevant comment

Options
    • -3
    • +1
    /openacs-4/packages/acs-subsite/www/register/user-join.tcl
3:55 pm

antoniop

Port of downstream customization: don't lose the return URL when one tries to join a community before login

Options
    • -2
    • +12
    /openacs-4/packages/acs-subsite/www/register/user-join.tcl
3:11 pm

antoniop

Important, although very simple reform in the page-contract filters: do not kill the case in the filter specs, or filters such as "w:oneof(One|Two)" or "locale:oneof(en_US|de_DE)" would not have a chance to match

Note that this makes the contracts stricter with respect to case, which might break obscure use cases...

Options
    • -2
    • +4
    /openacs-4/packages/acs-tcl/tcl/tcl-documentation-procs.tcl
Thursday 19 Nov
7:26 pm

antoniop

Localize already at the pretty value, otherwise things such as the wikicmds will not be evaluated

Options
    • -1
    • +11
    /openacs-4/packages/xowiki/tcl/form-field-procs.tcl
7:16 pm

antoniop

Address a problem encountered already with proctoring uploads: in cases that count both as timeouts and errors, the rescheduling of the request would happen twice. If the condition persists, the number of requests can grow out of control

Options
    • -6
    • +16
    /openacs-4/packages/proctoring-support/www/resources/proctored-page.js
Monday 09 Nov
6:38 pm

antoniop

Remove colon that gets confused as a bind variable

Options
    • -2
    • +2
    /openacs-4/packages/notifications/tcl/sweep-procs.tcl
Friday 06 Nov
4:52 pm

antoniop

Add missing semicolon

Options
    • -1
    • +1
    /openacs-4/packages/notifications/sql/postgresql/upgrade/upgrade-5.10.0d6-5.10.0d7.sql
4:51 pm

antoniop

Solve a potential issue when one would cleanup "dynamic" notification requests: this was done directly on the table and could therefore leave dangling acs_objects around

- cleanup all acs_objects of type 'notification_request' that do not have a corresponding entry in the table (~35000 downstream)

- embed logic of delete_dynamic_requests query inside select_invalid_request_ids query, as there the deletion is done properly

Options
    • -2
    • +2
    /openacs-4/packages/notifications/notifications.info
    • -0
    • +8
    /openacs-4/packages/notifications/sql/postgresql/upgrade/upgrade-5.10.0d6-5.10.0d7.sql
    • -6
    • +14
    /openacs-4/packages/notifications/tcl/sweep-procs.tcl
    • -15
    • +0
    /openacs-4/packages/notifications/tcl/sweep-procs.xql
4:51 pm

antoniop

file upgrade-5.10.0d6-5.10.0d7.sql was initially added on branch oacs-5-10.

Options
    • -0
    • +0
    /openacs-4/packages/notifications/sql/postgresql/upgrade/upgrade-5.10.0d6-5.10.0d7.sql
Thursday 05 Nov
2:53 pm

antoniop

Reinstate retrieval of the package_id from the connection context in child-resources, the problem was that, when we required a connection context in create_form_page_instance, we did with no information whatsoever, which was then retrieved by e.g. ns_conn, sometimes wrongly (e.g. when one is running under the test package)

Options
    • -0
    • +1
    /openacs-4/packages/xowiki/tcl/folder-procs.tcl
    • -2
    • +2
    /openacs-4/packages/xowiki/tcl/xowiki-procs.tcl
Wednesday 04 Nov
7:21 pm

antoniop

child-resources has already a package_id, getting it from the connection context makes the includelet susceptible to e.g. package initializations from the outside (see e.g. xowiki link_tests automated test)

Options
    • -1
    • +0
    /openacs-4/packages/xowiki/tcl/folder-procs.tcl
Tuesday 27 Oct
4:08 pm

antoniop

Fix overquoting in english message key

Options
    • -2
    • +2
    /openacs-4/packages/proctoring-support/proctoring-support.info
    • -1
    • +1
    /openacs-4/packages/proctoring-support/catalog/proctoring-support.en_US.ISO-8859-1.xml
Thursday 22 Oct
4:22 pm

antoniop

Allow in the include to selectively disable combinations of audio, camera and desktop recording (at least one stream needs to be enabled for proctoring to happen)

This allows to better tailor proctoring behavior to the user needs and privacy requirements

Options
    • -18
    • +32
    /openacs-4/packages/proctoring-support/lib/proctored-page.adp
    • -1
    • +11
    /openacs-4/packages/proctoring-support/lib/proctored-page.tcl
    • -128
    • +152
    /openacs-4/packages/proctoring-support/www/resources/proctored-page.js
    • -4
    • +6
    /openacs-4/packages/proctoring-support/www/resources/proctoring.js
Monday 19 Oct
5:19 pm

antoniop

Keep tolerating parameter_id specified as unnamed argument in legacy code and warn about deprecation

Options
    • -2
    • +20
    /openacs-4/packages/acs-tcl/tcl/apm-procs.tcl
Thursday 08 Oct
4:04 pm

antoniop

Handle the case where the iframe URL points to an external website: just skip the iframe-loop check

Options
    • -7
    • +18
    /openacs-4/packages/proctoring-support/www/resources/proctored-page.js
1:15 pm

antoniop

Whitespace cleanup

Options
    • -5
    • +5
    /openacs-4/packages/search/www/search.tcl
10:13 am

antoniop

Reduce hardcoded execs: the util might be improved to avoid them at some point

Options
    • -4
    • +2
    /openacs-4/packages/xowiki/tcl/import-procs.tcl
10:09 am

antoniop

Humanize indentation

Options
    • -5
    • +10
    /openacs-4/packages/acs-tcl/tcl/utilities-procs.tcl
Wednesday 07 Oct
4:33 pm

antoniop

Always make sure to clear any previously scheduled upload, before scheduling another one

This handles the case when a connection times out (readyState 4 and status = 0). In this case both the rescheduling in the readystatechange handler and in the timeout handler would be executed.

Options
    • -12
    • +26
    /openacs-4/packages/proctoring-support/www/resources/proctored-page.js