antoniop in OpenACS

Provide some basic way to create subgroups via the UI

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

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

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

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.

Move logics to delete a group type into the API

Fix quoting of group type

Remove unneeded variable

Add (trivial) ad_include_contract

Avoid unnecessary computations

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

Reuse return url over subsites, remove dummy sql

Use best practice list idiom

Avoid useless check

Localize file-size limits in the UI

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

Prettify code

Remove obsolete xql files

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

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

Repeat this block of code here as well, as vars in the checks are not kept in the script body

File storage max upload limit reform:

the max upload limit will now default to empty. Rationale: this value is

one that ages quickly with respect to new storage requirements, and is

currently set to something incompatible with the vanilla Naviserver configuration

(500MB fs vs 20MB ns).

The upgrade script will make so that instances setting this parameter

higher than the server upload limit configuration will be

set empty as well (this would never have an effect, as server

will prevent the upload altogheter anyway).

Max upload limit in bytes is now retrieved from a proc implementing

this fallback to server configuration logics. Other than centralizing,

its purpose is to be used in client-side code checks in the future

so we could prevent a doomed upload in advance.

Refer to the parameter in the file-storage instance instead than on a local one to decide the maximum upload size and deprecate the one here

Rationale: makes little sense that we set and maintain separate limits referring to the same file-storage instance

Remove trivial default, already set by the api

Whitespace removal

Remove obsolete queries and bring remaining one inline

Add ad_include_contract and documentation to this page

Use ad_include_contract on this page

Replace custom arg checks with proper ad_include_contract

Fix doc