antoniop in OpenACS

About

2,269 commits
  • Committer is not mapped to a user
  • First commit: 20 November 2006
  • Last commit: 05 May 2021
Last Week All Time
Commits: 0 2,269
Files changed: 0 3,730
Change in LOC: 0 19,523

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
 
Wednesday 05 May
5:28 pm

antoniop

In acs-tcl.documentation__check_proc_doc automated test, if the test fails on procs that belong to namespaces outside the OpenACS codebase, just issue a warning rather than failing

Options
    • -2
    • +16
    /openacs-4/packages/acs-tcl/tcl/test/doc-check-procs.tcl
4:02 pm

antoniop

Modernize javascript idiom

Options
    • -3
    • +1
    /openacs-4/packages/acs-templating/tcl/head-procs.tcl
3:46 pm

antoniop

Allow to specify javascrit event handlers at the tcl level using CSS query selectors

Options
    • -4
    • +35
    /openacs-4/packages/acs-templating/tcl/head-procs.tcl
Monday 03 May
7:38 am

antoniop

Comment command that won't be defined in upstream codebase

Options
    • -2
    • +3
    /openacs-4/packages/acs-automated-testing/tcl/aa-test-procs.tcl
Friday 30 Apr
7:42 pm

antoniop

Reduce duplication:

index and update_index were in fact implementing a upsert logic. Modern sql allows to write this in a single query and merge the two apis into one. Replace reference in the service contracts as well.

Options
    • -2
    • +2
    /openacs-4/packages/tsearch2-driver/tsearch2-driver.info
    • -0
    • +7
    /openacs-4/packages/tsearch2-driver/sql/postgresql/upgrade/upgrade-5.10.0d5-5.10.0d6.sql
    • -2
    • +2
    /openacs-4/packages/tsearch2-driver/tcl/tsearch2-driver-install-procs.tcl
    • -18
    • +17
    /openacs-4/packages/tsearch2-driver/tcl/tsearch2-driver-procs-postgresql.xql
    • -22
    • +10
    /openacs-4/packages/tsearch2-driver/tcl/tsearch2-driver-procs.tcl
    • -3
    • +2
    /openacs-4/packages/tsearch2-driver/tcl/test/tsearch2-driver-procs.tcl
7:42 pm

antoniop

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

Options
    • -0
    • +0
    /openacs-4/packages/tsearch2-driver/sql/postgresql/upgrade/upgrade-5.10.0d5-5.10.0d6.sql
11:05 am

antoniop

Fix typo

Options
    • -2
    • +2
    /openacs-4/packages/forums/tcl/forums-callback-procs.tcl
Thursday 29 Apr
4:03 pm

antoniop

Handle correctly the indexing of moderated messages in fulltext search:

so far the fulltext search would also include unapproved messages and threads. With this reform we address this by:

- making sure unapproved threads are deleted from the index

- making sure new threads are added to the index only when the are approved

- making sure we unindex or reindex a thread whenever the approval state changes

- not fetching unapproved messages in the datasource callback, hence excluding them from the index

Note that Oracle has not been targeted by the reform. However, the situation improves for Oracle as well, because we won't render unmoderated messages in the datasource callback anymore.

Options
    • -2
    • +2
    /openacs-4/packages/forums/forums.info
    • -6
    • +59
    /openacs-4/packages/forums/sql/postgresql/forums-sc-create.sql
    • -0
    • +124
    /openacs-4/packages/forums/sql/postgresql/upgrade/upgrade-1.4.0d3-1.4.0d4.sql
    • -1
    • +4
    /openacs-4/packages/forums/tcl/forums-callback-procs.tcl
4:03 pm

antoniop

file upgrade-1.4.0d3-1.4.0d4.sql was initially added on branch oacs-5-10.

Options
    • -0
    • +0
    /openacs-4/packages/forums/sql/postgresql/upgrade/upgrade-1.4.0d3-1.4.0d4.sql
9:14 am

antoniop

Streamline idiom: we are only interested in the user ids

Options
    • -9
    • +11
    /openacs-4/packages/acs-authentication/tcl/local-procs.tcl
    • -14
    • +0
    /openacs-4/packages/acs-authentication/tcl/local-procs.xql
Wednesday 28 Apr
7:00 pm

antoniop

Fix typo in comment

Options
    • -2
    • +2
    /openacs-4/packages/notifications/tcl/notification-email-procs.tcl
6:59 pm

antoniop

When sending a notification on behalf of a person, if the system is not configured to process replies to notification, do not set the reply-to address to anything different than the sender

Options
    • -4
    • +17
    /openacs-4/packages/notifications/tcl/notification-email-procs.tcl
6:20 pm

antoniop

Prefer api to hardcoded urls

Options
    • -2
    • +7
    /openacs-4/packages/news/lib/item.tcl
6:00 pm

antoniop

Address long standing todo: view news_items_live_or_submitted is defined for oracle exactly as the "workaround query" was doing, that is, from cr_revisions.content column

Options
    • -13
    • +0
    /openacs-4/packages/news/lib/item-oracle.xql
    • -15
    • +1
    /openacs-4/packages/news/lib/item.tcl
5:46 pm

antoniop

Whitespace changes

Options
    • -25
    • +1
    /openacs-4/packages/news/lib/item.tcl
3:57 pm

antoniop

Don't display an empty list element when comment_url is empty

Options
    • -1
    • +3
    /openacs-4/packages/news/lib/item.adp
3:55 pm

antoniop

Whitespace cleanup

Options
    • -6
    • +6
    /openacs-4/packages/news/lib/item.tcl
Tuesday 27 Apr
4:28 pm

antoniop

Now that the proctoring display UI has been ported, notify by default new proctoring events via websockets

Options
    • -2
    • +0
    /openacs-4/packages/proctoring-support/lib/proctoring-upload.tcl
    • -1
    • +2
    /openacs-4/packages/proctoring-support/www/upload.adp
    • -0
    • +1
    /openacs-4/packages/proctoring-support/www/upload.tcl
12:30 pm

antoniop

Do not handle 400 responses in any special way: cases found in practice were always the result of a client-side timeout

Options
    • -16
    • +3
    /openacs-4/packages/proctoring-support/www/resources/proctored-page.js
Monday 26 Apr
6:18 pm

antoniop

Hide timing information from the user

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

antoniop

Streamline proctoring upload:

- do not hardcode a timeout, requests will either complete or eventually fail

- use a single event handler at the end of the request to decide if and how to reschedule the upload. This prevents the possibility that the same error event triggers multiple uploads via different handlers

- improve documentation

Options
    • -28
    • +55
    /openacs-4/packages/proctoring-support/www/resources/proctored-page.js
Friday 23 Apr
4:40 pm

antoniop

Improve further: flush the object only when the state changes

Options
    • -3
    • +6
    /openacs-4/packages/xowf/tcl/xowf-procs.tcl
3:57 pm

antoniop

Improve fix:

in FormPage www-edit, just after the save_data operation we render the FormPage to refresh the references. We should flush the form object cache here, as otherwise any formfield spec will come from the form in the previous state

Options
    • -5
    • +7
    /openacs-4/packages/xowf/tcl/xowf-procs.tcl
11:39 am

antoniop

Reinstate oacs-dav as a dependency... there is api usage that is not so easy to replace at the moment

Options
    • -3
    • +4
    /openacs-4/packages/file-storage/file-storage.info
Wednesday 21 Apr
1:03 pm

antoniop

Fix test category, add tested api

Options
    • -2
    • +3
    /openacs-4/packages/file-storage/tcl/test/file-storage-procs.tcl
1:01 pm

antoniop

Make sure that the form object is flushed whenever state might have changed

If code executed after the state change accesses the form object again, this could be that from a previous state and e.g. hold the wrong form definition. This happens in practice downstream, where the submitting of activities also involves accessing the form definition to e.g. compute the grade based on the questions. Downstream we also cache the formfield specs, so if a spec is dependent on the state, might be wrong in also for future requests.

Options
    • -0
    • +5
    /openacs-4/packages/xowf/tcl/xowf-procs.tcl
Tuesday 20 Apr
1:42 pm

antoniop

Fix typo, write a basic test for the involved api

Options
    • -2
    • +2
    /openacs-4/packages/acs-content-repository/tcl/content-revision-procs.tcl
    • -1
    • +43
    /openacs-4/packages/file-storage/tcl/test/file-storage-procs.tcl
1:08 pm

antoniop

Fix typo

Options
    • -2
    • +2
    /openacs-4/packages/file-storage/tcl/file-storage-procs.tcl
11:37 am

antoniop

Sanitize bogus URLs such as "//" or e.g. dotlrn URL lookup would fail

Not clear if this should happen already at the ns_conn level or somewhere in the site_node api

Options
    • -3
    • +7
    /openacs-4/packages/xotcl-request-monitor/tcl/throttle_mod-procs.tcl
11:20 am

antoniop

Accept only integers as value for the rss query parameter

Options
    • -2
    • +3
    /openacs-4/packages/xowiki/tcl/package-procs.tcl