• last updated 3 hours ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Whitespace cleanup

improve listing of test coverage

  1. … 17 more files in changeset.
consider subject field restriction from the database also in the ui

make end of options explicit

  1. … 16 more files in changeset.
Fix typo

improve spelling

  1. … 16 more files in changeset.
prefer "ns_conn isconnected" over "ad_conn isconnected", since the latter might not notice changes in the connect state

  1. … 16 more files in changeset.
Kill message key translation only when the original user message is included in the notification

Do not kill localization for moderator notifications, as they contain indeed message keys to be localized

Implement a new type of "Moderator" notification type for forums

This notification will be triggered also by unapproved messages. The UI will show the subscription link only to forum moderators, using a different color (customizable via css, vanilla color is red). Message content will always be hidden for such notifications, so user will need to have access to new message.

  1. … 6 more files in changeset.
Step in the direction of implementing a new kind of "moderator notification": move current notification behavior in a private api, remove dependency on message approval in message::new (also avoids a query)

Fix typo

Comment unused computations

Small refactoring

Move logics to create forum notification type to tcl to:

- reduce code duplication

- make it simpler to extend forums with new notification types

This reform was triggered by a downstream request, where users would like a new kind of notification for forum moderators that triggers also when a forum message has not been approved

  1. … 4 more files in changeset.
Complete a long standing transition (before 5.5): rely on callbacks intead of service contracts to implement FTS datasource logics in the forums package

  1. … 1 more file in changeset.
Prefer acs::test::reply_has_status_code over manually checking status code in the reply

added editor hints and white-space cleanup

  1. … 24 more files in changeset.
Fix typo

Use fully-qualified api names

Indentation changes

Deprecate obsolete api

No need to explicitly delete reading info, as both oracle and postgres define the reference constraint as on delete cascade

Use fully-qualified api names

Use fully-qualified api names, assume message is there (something went very wrong otherwise)

Reduce divergency between oracle and postgres codebase

Reduce divergency between oracle and postgres codebase

Whitespace cleanup

Deprecate obsolete api

Implement operation in a way that oracle is supported