• last updated 2 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Deprecate (former) service contract implementations

Their body was always empty according to the cvs history and there is no mention of them in the code.

Their counterpart in the forum::notification namespace takes care of both forums and messages, therefore supersedes them.

    • -5
    • +15
    ./tcl/forum-message-reply-procs.tcl
Clearly separate callback hooks from callback implementations, uniform documentation style, remove duplicated callback implementation definitions, uniform indentation, add documentation to public procs (fixes automated tests)

    • -155
    • +111
    ./tcl/forums-callback-procs.tcl
Add documentation to public procs (fixes automated tests)

Uniform documentation style

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

Put cache flushing logics into procs and use them when forum info needs to be updated (fixes automated tests)

Add documentation and a fallback body for this deprecated proc

Provide a minimal documentation to public procs

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

  1. … 8 more files in changeset.
Remove hotfix and exploit new api, that won't fail on the guest user

Use finer grained person:: api to retrieve the name

Use new fine grained api for the screen name

Remove hotfix and use new user api

Remove user hotfix and replace with new api (which won't fail for non-registered users)

Flush templating cache also when approving a message, as this might be a new root message (thread) and therefore thread list should be rebuilt

Re-estate query variable, as programs around expect it

Fix typos

fix diacrytical characters in german messages

Fix regression when checking if user can anonimously post

Fix regressions in forums when user is not logged in

    • -7
    • +23
    ./lib/forums/forums-history-chunk.tcl
Disambiguate table reference in subquery

    • -13
    • +13
    ./tcl/messages-procs-postgresql.xql
Avoid two times the same query just because we might filter by forum

    • -20
    • +2
    ./lib/search/search-chunk-oracle.xql
    • -20
    • +2
    ./lib/search/search-chunk-postgresql.xql
Pass user_name through the message array, as message-view expects it there

This obscure script doesn't do much (basically, displays the content of a message), but nevertheless was passing arrays, defining queries, duplicating code... streamlined to the bones

    • -46
    • +0
    ./lib/message/content-updater-oracle.xql
Remove redundant variables, dead code and bring variables closer to where they are actually used... still room for improvement here

    • -10
    • +4
    ./lib/message/thread-chunk-postgresql.xql
Use more api to streamline queries and reduce divergency between oracle and postgres codebase

    • -28
    • +6
    ./tcl/messages-procs-postgresql.xql
move template cacheing function "cache" into template namespace (resulting in template::cache)

  1. … 2 more files in changeset.
Reduce divergencies between oracle and postgres codebase, reduce the need for joins and stored procedures

    • -41
    • +0
    ./lib/message/threads-chunk-oracle.xql
    • -41
    • +0
    ./lib/message/threads-chunk-postgresql.xql
Centralize templating cache "black magic"

Use the new proc into forum api

Replace deprecated 'forum::security::can_moderate_forum_p' by 'permission::permission_p'