antoniop in OpenACS

About

1,027 commits
  • Committer is not mapped to a user
  • First commit: 20 November 2006
  • Last commit: 04 January 2019
Last Week All Time
Commits: 0 1,027
Files changed: 0 3,106
Change in LOC: -28 16,424

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
 
Friday 04 Jan
4:54 pm

antoniop

Port from downstream a more failsafe cleanup in the importer

Options
    • -2
    • +2
    /openacs-4/packages/xowiki/tcl/import-procs.tcl
4:50 pm

antoniop

Remove redundant cleanup which might fail in case object does not exist (we clean up anyway in the finally branch)

Options
    • -6
    • +7
    /openacs-4/packages/xowiki/www/admin/import.tcl
2:27 pm

antoniop

Refine approach by which index page is detected in bulk-delete method: previous way would treat all first level children of the root_folder as the index page, preventing bulk deletion focused on them (e.g. folders)

Options
    • -9
    • +20
    /openacs-4/packages/xowiki/tcl/xowiki-www-procs.tcl
Thursday 03 Jan
6:18 pm

antoniop

Allow to filter group members by membership state in the API

Options
    • -24
    • +22
    /openacs-4/packages/acs-subsite/tcl/group-procs.tcl
Friday 28 Dec 2018
12:24 pm

antoniop

Greatly reduce complexity of code required to support -export flag for ad_form:

instead of juggle variables around to mimick export_vars, just set -export flag

value as a form property, then use the real export_vars on it later when form

is rendered. This will support all current features (e.g. :multiple, :sign, :array)

more reliably and reduce code duplication.

Options
    • -80
    • +5
    /openacs-4/packages/acs-tcl/tcl/form-processing-procs.tcl
    • -1
    • +10
    /openacs-4/packages/acs-templating/tcl/form-procs.tcl
Tuesday 18 Dec 2018
5:13 pm

antoniop

Put event handler inline so is attached at element creation, fix URLencoding of image deletion

Options
    • -1
    • +8
    /openacs-4/packages/xowiki/www/ckeditor-images/thumb-view.adp
    • -9
    • +6
    /openacs-4/packages/xowiki/www/ckeditor-images/thumb-view.tcl
4:42 pm

antoniop

Bring event handler inline to force it being attached after the upload button is added to the page

Options
    • -2
    • +5
    /openacs-4/packages/xowiki/www/ckeditor-images/upload_image.adp
4:26 pm

antoniop

Add missing parameter in calc_wiki_image_links_to_image_tags

See bug tracker #3372

Options
    • -2
    • +2
    /openacs-4/packages/xowiki/www/resources/ckeip.js
Monday 17 Dec 2018
11:39 am

antoniop

Replace single quotes wuith double quotes in javascript expressions involving acs_ListBulkActionClick

this is really needed only for the URL part, but was done on the name as well for consistency. The reason

is even a properly encoded URL could still contain single quotes, therefore breaking the expression,

while double quotes are expected to be encoded in valid, urlencoded URLs. This relies on trusting the developer

to properly encode URLs, which should be expected. An alternative could involve active escaping of

javascript special characters by e.g. ad_js_escape.

Options
    • -1
    • +1
    /openacs-4/packages/acs-templating/resources/lists/courses.adp
    • -1
    • +1
    /openacs-4/packages/acs-templating/resources/lists/table-2third.adp
    • -1
    • +1
    /openacs-4/packages/acs-templating/resources/lists/table.adp
    • -1
    • +1
    /openacs-4/packages/openacs-bootstrap3-theme/resources/lists/table-2third.adp
    • -1
    • +1
    /openacs-4/packages/openacs-bootstrap3-theme/resources/lists/table.adp
    • -2
    • +2
    /openacs-4/packages/xotcl-core/tcl/30-widget-procs.tcl
    • -2
    • +2
    /openacs-4/packages/xowiki/tcl/bootstrap-procs.tcl
Friday 14 Dec 2018
4:11 pm

antoniop

Introduce new ad_js_escape with the purpose of sanitizing tcl strings used inside javascript code from escape sequences and from quotes in particular and provide some test cases

TODO: consider its usage into templates when e.g. we are putting a URL into javascript functions such as acs_ListBulkActionClick. URLs might in fact contain single and double quotes.

Options
    • -1
    • +22
    /openacs-4/packages/acs-tcl/tcl/text-html-procs.tcl
    • -0
    • +19
    /openacs-4/packages/acs-tcl/tcl/test/text-html-procs.tcl
Wednesday 12 Dec 2018
7:20 pm

antoniop

Provide some basic way to create subgroups via the UI

Options
    • -2
    • +3
    /openacs-4/packages/acs-subsite/www/admin/groups/one.adp
    • -2
    • +7
    /openacs-4/packages/acs-subsite/www/admin/groups/one.tcl
Tuesday 11 Dec 2018
5:48 pm

antoniop

Actually store the group name in the UI by intercepting the right form var

Options
    • -2
    • +3
    /openacs-4/packages/acs-subsite/www/admin/groups/new.tcl
5:23 pm

antoniop

Fix test test by considering that group pretty name is automatically localized

Options
    • -3
    • +10
    /openacs-4/packages/acs-subsite/tcl/test/acs-subsite-procs.tcl
5:21 pm

antoniop

Don't quote some function defaults that are meant to be evaluated literally

Options
    • -7
    • +17
    /openacs-4/packages/acs-subsite/tcl/plpgsql-utility-procs.tcl
3:35 pm

antoniop

New automated test doing the roundrtrip of group-type related API

Note that this is currently failing due to a bug in package_instantiate_object, which triggered the creation of this test.

Options
    • -1
    • +53
    /openacs-4/packages/acs-subsite/tcl/test/acs-subsite-procs.tcl
3:14 pm

antoniop

Move logics to delete a group type into the API

Options
    • -1
    • +34
    /openacs-4/packages/acs-subsite/tcl/group-type-procs-oracle.xql
    • -1
    • +31
    /openacs-4/packages/acs-subsite/tcl/group-type-procs-postgresql.xql
    • -1
    • +71
    /openacs-4/packages/acs-subsite/tcl/group-type-procs.tcl
    • -79
    • +0
    /openacs-4/packages/acs-subsite/www/admin/group-types/delete-2-oracle.xql
    • -85
    • +0
    /openacs-4/packages/acs-subsite/www/admin/group-types/delete-2-postgresql.xql
    • -62
    • +6
    /openacs-4/packages/acs-subsite/www/admin/group-types/delete-2.tcl
    • -14
    • +0
    /openacs-4/packages/acs-subsite/www/admin/group-types/delete-2.xql
2:25 pm

antoniop

Fix quoting of group type

Options
    • -1
    • +1
    /openacs-4/packages/acs-subsite/www/admin/group-types/delete-2-postgresql.xql
12:26 pm

antoniop

Remove unneeded variable

Options
    • -7
    • +1
    /openacs-4/packages/acs-subsite/tcl/subsite-procs.tcl
12:20 pm

antoniop

Add (trivial) ad_include_contract

Options
    • -1
    • +5
    /openacs-4/packages/acs-subsite/lib/subsites.tcl
12:16 pm

antoniop

Avoid unnecessary computations

Options
    • -1
    • +0
    /openacs-4/packages/acs-subsite/lib/subsites-oracle.xql
    • -1
    • +0
    /openacs-4/packages/acs-subsite/lib/subsites-postgresql.xql
    • -9
    • +10
    /openacs-4/packages/acs-subsite/lib/subsites.tcl
Monday 10 Dec 2018
7:26 pm

antoniop

Reduce divergency between oracle and postgres codebase and bring portable sql inline

Options
    • -39
    • +0
    /openacs-4/packages/acs-subsite/tcl/application-group-procs-oracle.xql
    • -41
    • +0
    /openacs-4/packages/acs-subsite/tcl/application-group-procs-postgresql.xql
    • -6
    • +39
    /openacs-4/packages/acs-subsite/tcl/application-group-procs.tcl
    • -33
    • +0
    /openacs-4/packages/acs-subsite/tcl/application-group-procs.xql
5:31 pm

antoniop

Reuse return url over subsites, remove dummy sql

Options
    • -2
    • +3
    /openacs-4/packages/acs-subsite/lib/subsites.tcl
5:27 pm

antoniop

Use best practice list idiom

Options
    • -1
    • +1
    /openacs-4/packages/acs-subsite/lib/subsites.tcl
5:25 pm

antoniop

Avoid useless check

Options
    • -2
    • +1
    /openacs-4/packages/acs-subsite/lib/subsites.tcl
Friday 07 Dec 2018
1:02 pm

antoniop

Localize file-size limits in the UI

Options
    • -4
    • +5
    /openacs-4/packages/file-storage/www/file-add.tcl
12:54 pm

antoniop

Introduce a client-side check for the maximum file size, inform user of current limit and improve file-size related localization

Options
    • -2
    • +20
    /openacs-4/packages/file-storage/www/file-add.tcl
12:34 pm

antoniop

Prettify code

Options
    • -10
    • +20
    /openacs-4/packages/file-storage/www/file-add.tcl
12:12 pm

antoniop

Remove obsolete xql files

Options
    • -31
    • +0
    /openacs-4/packages/file-storage/www/version-delete-oracle.xql
    • -26
    • +0
    /openacs-4/packages/file-storage/www/version-delete-postgresql.xql
    • -25
    • +0
    /openacs-4/packages/file-storage/www/version-delete.xql
1:23 am

antoniop

Simplify code and exploit parameter contract (we know flags are boolean)

Options
    • -17
    • +10
    /openacs-4/packages/acs-tcl/tcl/utilities-procs.tcl
1:02 am

antoniop

Bugfixes et al:

- don't throw a server error if one clicks submit on the empty form without specifying a file

- allow to upload the same zipfile multiple times without throwing an error

- streamline idioms

Options
    • -48
    • +26
    /openacs-4/packages/file-storage/www/file-add.tcl