• last updated 1 hour ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Skip processing for all formfields that are defined as disabled:

the browser should not send us these data in the first place.

Extend test: show that even when a field is defined as disabled, a request crafted to carry these fields will get through and the values will be stored

A fix will come shortly, similar to what was done for compound fields

undo part of last change

unfortunatly, the 0.9.3 issue can't be fixed so simple as hoped. The "-html" flag is necessary for dealing with autoclosed entries.

for orthogonaly, remove "-html" flag from dom parse to avoid a potential top-level <html> element

fix compatibility with tdom 0.9.3

fix warning (including some false positives) in regression test

  1. … 4 more files in changeset.
we need the upgrade script, since the automatic aler just kicks in, when the constraint does not exist. Antonio was right.

Nevertheless, it would be a nice feature, if the ORM would handle this.

remove useless upgrade scripts (xotcl-core does this automatically on reloads)

Document and enforce expected behavior when Forms are deleted: this is forbidden and returns an error as long as they have instances

Follow up of https://cvs.openacs.org/changelog/OpenACS?cs=oacs-5-10%3Agustafn%3A20220622174910

In the end we need the upgrade logic, as the ORM will kick in only if the column is just being created

  1. … 1 more file in changeset.
Make page_template foreign key deferrable

This solves the problem of deleting implicitly child-items, which

depend on each other when deleting an item. When deleting an item, the

deletion of child-items in the content repository happens in an

arbitrary order, which could raise fk constraints. Since the deletion

happens in a transaction, the deferring checking of the constraints

helps, since then the constraint violating item is also already

deleted.

There is no update script necessary, since reload alters the

constraint.

undo recent change, which is not sufficiently general (and solve the issue by db means, next commit)

Address the cornercase of form pages having their template stored in the same folder: when deleting items containing such a setup, cleanup these formpages first

Addresses a consequence of https://cvs.openacs.org/changelog/OpenACS?cs=oacs-5-10%3Agustafn%3A20220613165033

remove traces of md5::md5, since ns_md5 is emulated also for AOLserver

  1. … 2 more files in changeset.
Drop "ON DELETE CASCADE" on page_template, since this might lead to crippled entries in the database.

When someone delets via operations in the database a form, protentially existing

form pages refering to this item might be left in a crippled state.

The cascade operation deletes the revision (xowiki_page_instance), but leaves

the half-crippled item. The problem does not exist over the web interface,

since it does not allow such items to be deleted while form pages exist.

  1. … 1 more file in changeset.
Add support for menu-entries, where simply a "-link" is provided

Provide a fix for "dom parse -html ..." for adp tags.

"dom parse -html" has two problems with ADP tags like "<adp:icon ...>":

a) If the tag name contains a colon or underscore, the tag is

treated like plain text, i.e. "<" and ">" are converted into

HTML entities.

b) These tags have to be closed "<adp:icon ...>" is invalid.

Several existomg ADP tags have not closing tag.

Therefore, we resolve the ADP tags before parsing the text by

tdom. There should be some future framework support to do this in

general, but until we have this, resolve this problem here locally.

New method "marshall_all_to_file"

This method is similar to "marshall_all", but exports the objects

directly to a file. This can save memory when exporting a large

collection of objects, since the plain "marshall_all" appends to

string, which can get very large, especially due to Tcl's

"double the size when space is needed" policy during "append"

operations.

fix for old-style ad_form based forms with radio or checkbox controls for bootstrap 5

These controls were used with a bootstrap3 classes, which had the consequence

that no input was possible for these fields. Probably, similar changes

are as well necessary in other situations as well.

fix typo

tdom sometimes html-escapes unknown tags

Consider code like

#

# dom parse -simple -html $form doc

# ...

# return [$root asHTML]

Unfortunately, this causes that some tags unknown to tdom

(like <adp:icon>) are converted to escaped tags (&lt; ...).

This can be regarded as a bug. To avoid this problem, we

substitute here the adp_tags in advance. This needs more

investigation in other cases.... The potential harm in this

cases here is very little, but probably, there are other

cases as well where this might harm.

escape also brackets

added context_id to excluded vars in copy_content_vars

added download icon for CSV

  1. … 1 more file in changeset.
make regression test more robust with respect to yui rendering

Styling changes for minimal setups

This change is necessary for minimal setups, e.g. for plain installs

during regression test, when no bootstrap packages are available.

- define yui as last-resort fallback for "toolkit"

- handle CSS class composition for YUI as well

(merge of class-specific and item specific CSS classes,

used for color management for publish status button)

- remove underline for publish-status in YUI mode

  1. … 1 more file in changeset.
Cleanup dummy commands and objects to avoid cross interactions between tests

  1. … 1 more file in changeset.
Cleanup test for deleted api

get rid of "xowiki::adp_parse_tags", since this is handled now already in acs-templating

  1. … 1 more file in changeset.
do not cache some volatile info in xowiki::file objects