• last updated 2 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Port of downstream behavior:

- never display all of the rooms, but only those where we have access (e.g. read permission and not banned) on the specified packages

- when the portlet is rendered outside of a community, e.g. in the personal home, split chat rooms by package (community name)

- considered rendering outside a connection or lack of package as shaded

- display number of currently connected users

- (Postgres only) provide a more efficient query when one might have access to many chats

Also cleanup acs_attribute

Deprecate trivial wrapper dotlrn_chat::add_portlet_helper

Cus short computations when the portlet is shaded

Reduce divergency between Oracle and Postgres codebase

Cleanup the vestigial moderated_p flag

    • -5
    • +5
    /openacs-4/packages/chat/tcl/chat-procs.tcl
    • -2
    • +1
    /openacs-4/packages/chat/www/index.tcl
    • -6
    • +0
    /openacs-4/packages/chat/www/room-edit.tcl
Get rid of the intermediate "room-enter" page and enforce all of the relevant permission in place in the "chat" page, cleanup variables and update automated tests

    • -24
    • +9
    /openacs-4/packages/chat/www/chat.tcl
    • -2
    • +2
    /openacs-4/packages/chat/www/index.tcl
    • -36
    • +0
    /openacs-4/packages/chat/www/room-enter.tcl
Update api

Modernize api

Modernize idiom

Use modern api

Update api to streamline idiom

Update api

No need to access the form directly here

ns_getform will return the empty string only when invoked outside of a connection thread, which is a non-issue at installation time

Deprecate trivial wrappers to the NaviServer API

Modern ns_getform will not return the empty string for empty requests, but only when we are operating outside of a connection. In such circumstances is arguably useful to manipulate the request parameters.

Commit missing change to the users api

Prefer Naviserver API directly

Prefer NaviServer API directly

Rework idiom

Rework idiom

Exploit page contract, rework idioms

Don't run the spiel when we are not connected

Streamline idiom

Streamline idiom

Whitespace cleanup

Modernize api

Inline rp_getform usage

rp_getform will never return the empty string, small cleanup

Remove no_alarms_until column

Rationale: the column or the related api is never used in upstream code to check whether notifications should be sent or not or not. The UI to manage the column's value was not linked anywhere. On the instances accessible by me downstream, no tuple was found where this column had a value.

  1. … 4 more files in changeset.