• last updated 9 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Don't pass fine tcl variables through the query, avoid unnecessary quoting (bulk-mail api would pass variable to the stored procedure using ':' notation)

    • -9
    • +4
    /openacs-4/packages/dotlrn/www/spam.tcl
    • -7
    • +2
    /openacs-4/packages/dotlrn/www/spam.xql
Reduce divergencies between oracle and postgres codebase

Use more API when retrieving user info

Replace leftover Naviserver-only cache idioms with retro compatible ones

    • -5
    • +5
    /openacs-4/packages/chat/tcl/chat-procs.tcl
Put this last query also in the code as the others, so no xql is required

Use more API, reduce divergency between postgres and oracle code

Use more API

Use more API

Prefer API over handcrafted queries

Create chat room cache in a way to provide compatibility with AOLserver

make timeout and cache size configurable.

    • -1
    • +20
    /openacs-4/packages/chat/tcl/chat-init.tcl
    • -7
    • +1
    /openacs-4/packages/chat/tcl/chat-procs.tcl
make require_test_folder more configurable

use address from driver_info

add address to driver_info

    • -2
    • +2
    /openacs-4/packages/acs-tcl/acs-tcl.info
Remove unneded dict creation

    • -2
    • +2
    /openacs-4/packages/dotlrn/www/spam.tcl
modernize tcl

modernize tcl

Reduce code duplication, exploit API when retrieving user info

    • -73
    • +0
    /openacs-4/packages/dotlrn/www/spam-oracle.xql
    • -2
    • +6
    /openacs-4/packages/dotlrn/www/spam.tcl
    • -8
    • +63
    /openacs-4/packages/dotlrn/www/spam.xql
Remove redundant queries: user name is already retrieved above from cached user api

Reuse person API instead of querying by hand

Modernize idiom

Restore previous behavior: when a person is not found by email, just return empty string instead of an error

Reduce code duplication when retrieving person info

TODO centralize caching

provide compatibility with AOLserver

make timeout cnd cache size configurable

follow usual naming conventions

    • -0
    • +23
    /openacs-4/packages/acs-tcl/tcl/community-core-init.tcl
Pass to a ns_cache implementation to avoid race conditions

Implement acs_user:: cache through nsv, instead of the util_memoize cache. Note that now this information won't expire anymore unless explicitly flushed.

Remove unnecessary backslashes, remove leftover usages of get_from_username_not_cached cache, minimize catch wrapping

Reuse acs_user::get_by_username to retrieve user_id in order to reduce query duplication and cache entries for acs_user::get, simplify idiom

Repair broken link: refer to page 'object' instead of to page 'list'

Fix missing variable substitution

Dont require role at the ends of relationships as this is not required in the data model and standard relationships (e.g. membership_rel, admin_rel) lack these roles as well