• last updated 20 hours ago
Constraints: committers
Constraints: files
Constraints: dates
file proctoring-configure.adp was initially added on branch oacs-5-10.

Port of downstream HTML5 date and time form widgets by Günter Ernst

These widgets provide a better and more consistent user experience and client-side validation across multiple platforms. They should in time become the preferred alternative for dates and time for template forms.

Comment noop result of a 18 years old typo, add remarks

Arrays might or not have been a better choice for the date object, however, such decision happened very long ago and it has worked good enough... cleanup concerning remarks

Whitespace change

Review concerning remark: the algotirhm is indeed correct, assuming the year is expressed in the Gregorian calendar

Replace deprecated idiom

Uniform documentation formatting and indentation, add missing documentation

For Postgres, take advantage of exclusive database api to check for permissions more efficiently, improve doc

Notification request cleanup refactoring

One of the most expensive operations in large site is the cleanup for

notification_requests in situations, where the user has lost

permissions on an object, on which the user wanted to receive

notifications. This check was performed previously in

notification::sweep::cleanup_notifications via a permission check over

all notification requests, which can be very costly on large

sites. This change moves this cleanup into the actual notification

sending, where the permissions have to be sent anyhow.

Fix typo

Fix expression to the original intention: check if ns_conn url ends by ad_conn extra_url

more cleanup for setting of potentially unencoded return_url

    • -2
    • +1
    • -2
    • +2
minor cleanup for returnurls

    • -1
    • +1
don't hard-code "Learner" as role

streamline returnredirect commands

make sure, the provided URL is always properly encoded

    • -6
    • +8
Fix typo in datamodel creation, make it atomic

ad_return_url: experimental change to produce per default a proper encoded result

Background: before this change, ad_return_url encoded always the query

variables but not the URL path. This mixed result is complex to handle

to achieve ex-post a proper encoding. The function has a flag

"-urlencode", which is nowhere used in OpenACS. Other functions like

e.g. export_vars, or pretty_link return per default as well the

encoded URL. "ad_return_url" has a new flag "-path_encode=false" to

obtain the previous behavior (similar to the other mentioned


break overlong lines and improve comments

bugfix: don't call get_non_empty_file_formfields in render_answers when in export mode

improve spelling

provide a compatibility function for ns_parsehostport for earlier versions of NaviServer

make spelling more consistent

prefer variable resolver over method calls for unset

    • -4
    • +4
minor refactoring to correct old-style widgetSpecs; prefer variable resolver over method calls for unset

support editor=none in arbitrary form-field specs

added composite question per default to the "New" pull-down menu, added message keys for German

    • -0
    • +1
make sure that the query variable "show" is only passed non-empty, otherwise the page-contract of admin/message-list.tcl will fail

Reduce usage of ns_mktemp in OpenACS

ns_mktemp uses the deprecated old POSIX call mktemp(), which should

not be used anymore for security reasons (race between the name

creation and opening the file). This change removes several usages of

"ns_mktemp" from OpenACS and replaces it with calls to the

safe Tcl call "file tempfile ..." (introduced by Tcl 8.6).

prefer usage of "xo::write_tmp_file" over "xo::write_file" for writing tmp files

    • -3
    • +3
    • -4
    • +5