• last updated 13 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Fix typos

  1. … 8 more files in changeset.
Whitespace changes

    • -39
    • +39
    ./tcl/file-storage-callback-procs.tcl
    • -101
    • +98
    ./tcl/test/file-storage-procs.tcl
Fix typos and wording.

create folder only if it not already exists

Removed obsolete @arch-tag

    • -3
    • +2
    ./sql/oracle/upgrade/upgrade-5.1.0a13-5.1.0a14.sql
  1. … 23 more files in changeset.
Removed obsolete @arch-tag

  1. … 58 more files in changeset.
- removed extra colons

- streamline layout of test cases

    • -12
    • +62
    ./tcl/test/file-storage-procs.tcl
Whitespace changes

  1. … 2 more files in changeset.
Fix parameter name in proc doc and add brief descriptions

  1. … 3 more files in changeset.
Remove commented leftover debugging statement, add comment about why we need to name the list by folder_id

Put template::list name back as parametric, because this code might be included multiple times

Redefine this page as ad_include_contract, therefore removing all hardcoded variable checks and exploit builtin checks, remove duplicated definitions and queries, streamline template::list definitions in a way its is readable and simpler to maintain

Pass also orderby to the includelet

Trim whitespaces

Remove unnecessary parentheses and idioms, state query variables explicitly

don't pass deprecated argumentent "-user_id" to fs::get_folder_contents_count

  1. … 1 more file in changeset.
Move callback hooks definitions together with their siblings, add documentation to public procs (fixes automated tests)

    • -94
    • +135
    ./tcl/file-storage-callback-procs.tcl
    • -39
    • +1
    ./tcl/file-storage-install-procs.tcl
Add documentation for public procs (fixes automated tests)

Add documentation to public callback hook (fixes automated tests)

Set package_id explicitly as the main subsite package, as ad_conn package_id could not be properly initialized when a system was freshly restarted

Prefer 'person::name' to 'acs_user::get_element ... -name'

Fiy typo

Deprecated leftover parameter from year 2002

Exploit api instead of using local sql

Localize possible I18n message in folder name

While working on reducing divergencies between oracle and postgres codebase and testing changes I have found the following issues:

- fs::add_created_version.new_text_revision query was defined in postgres only

- version-delete and folder-delete were broken in case notification was enabled,

because this was issued after object was deleted and therefore all references

needed to build notification message were missing (and query failed looking up the object)

    • -25
    • +0
    ./tcl/file-storage-procs-oracle.xql
    • -43
    • +0
    ./tcl/file-storage-procs-postgresql.xql
Whitespace changes

Reduce divergency between oracle and postgres codebase and remove redundant statements

Dont even offer to delete the root folder as this is prevented anyway

whitespace cleanup

    • -453
    • +453
    ./tcl/file-storage-procs.tcl