• last updated 10 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Replace deprecated proc

Specify an explicit format that clock scan can recognize, otherwise results will be unexpected (port from downstream)

fix typos and provide adp doc page

  1. … 1 more file in changeset.
Add editor hints + Whitespace cleanup

Mention explicitly the tested procs in the test cases.

Sanitize date stings for parsability with clock scan and allow HTTPS requests using 'util::http::post' (many thanks to Markus Moser).

Revert massive replacement of empty list creation sentences. The use of '[list]' instead of '{}' adds semantics that could be used for performance improvements in the future, such as using a different internal representation. There is already work in this direction, avoiding the generation of the string representation during comparison of empty strings (huge thanks to Stefan Sobernig for the pointer: https://core.tcl.tk/tcl/info/44527c632ed609c2).

  1. … 473 more files in changeset.
Prefer '{}' to '[list]' when creating empty lists

  1. … 202 more files in changeset.
Rework variable assignment (set -> lassign) + Whitespace changes

  1. … 3 more files in changeset.
fix typos

  1. … 13 more files in changeset.
Use ns_getcontent to access content of the POST requests, when using NaviServer

Handle empty XML strings explicitly

free doc only, when it exists

- modernize tcl

- prefer ns_quotehtml over ad_quotehtml

  1. … 4 more files in changeset.
- replace deprecated command "ns_unlink" with native tcl command "file delete"

  1. … 7 more files in changeset.
- normalize string comparisons

- allow for byte compiled expressions

  1. … 1 more file in changeset.
fix by christoph haberberger: empty parameters in xml-rpc lead to crash

change the manyTypesTest to use 0010704T11:50:30Z (note Z) since w/o it you would get a result that depended on the server local timezone

Make xmlrpc::httppost error out instead of returning a useless -1 in case of error. Vinod-approved change.

all TCL's clock clicks always with the -milliseconds option. The TCL manual

explicitly states that without this switch the return result is platform

dependant. The OpenACS code works at the millisecond granulartiy and silently

assumes that 1 millisecond equals 1000 clicks which is not necessarily true.

I used this shell script to automatize the replacing:

#!/bin/bash

for file in `grep -lsr "\[clock clicks\]" /web/openacs-head/packages/ `; do

echo "repairing $file"

sed "s/\[clock clicks\]/\[clock clicks -milliseconds\]/g" $file > fout

mv fout $file

done

And then grepped for

grep -r "expr.*clock clicks" /web/openacs-head/packages/

to manually remove the divisions by 1000.

  1. … 10 more files in changeset.
Spit out errorInfo in errorlog when a call fails

Initial version of XML-RPC package

    • -0
    • +114
    ./system-procs.tcl
    • -0
    • +17
    ./validator-init.tcl
    • -0
    • +307
    ./validator-procs.tcl
    • -0
    • +708
    ./xml-rpc-procs.tcl
    • -0
    • +181
    ./test/xml-rpc-test-procs.tcl
  1. … 7 more files in changeset.