Project

General

Profile

Actions

Feature proposal #8002

closed

Move system action behavior to core CB Activity

Added by krileon almost 4 years ago. Updated over 1 year ago.

Status:
Rejected
Priority:
Normal
Assignee:
Target version:
-
Start date:
08 May 2020
Due date:
% Done:

100%

Estimated time:

Description

Instead of using CB Auto Actions to exclusively implement notification logging and other functionality move this to core CB Activity. This will be a simple on/off feature for several notifications. Creation of new notification behavior would still be done exclusively from CB Auto Actions, but default cases should be handled by CB Activity.

Actions #1

Updated by krileon almost 4 years ago

The actual handling of the activity logging should just be in their respective plugins. For example CB GroupJive is already handling access and display of its activity and notifications then it should also just handle logging that information as well. CB Activity would still be responsible for core activity and notifications.

Basically once and for all need to determine who is responsible for what. This is a bit hard to determine as we want to avoid duplicate code. Having a simple and strong API in CB Activity that plugins then use would be the best approach. This means it needs to be extremely simple to create and log activity and notifications, which it currently is not. Basically plugins shouldn't need to deal with table objects and instead call an API against their streams directly, which will handle the table objects. Basically a more robust stream API that handles more than just display. Example as follows.

$stream = new Activity(); // Establish a stream

$stream->load( STREAM_ID ); // Load the stream we intend to use (this would be a param in GJ)

$stream->setTarget( 'group', GROUP_ID ); // Set the Target and Target ID

$stream->create( ... ); // This would handle creating a new post on this stream

Currently to create a new post you have to use an ActivityTable object directly and that should be avoided. Notifications are a bit trickier. For example if we need to push notifications to dozens of people we don't want to build dozens of objects in PHP to do this. We need to do a mass insert query. So we'd need to first build a list of valid recipients then we can safely go ahead with a mass insert. This means the notifications stream objects need to also handle creating entries to multiple recipients.

The reason a lot of this wasn't put into plugins directly previously is because of how streams are currently built (from a set of parameters), but them changing in #7454 will remove that obstacle.

Actions #2

Updated by krileon almost 4 years ago

It might be best to continue to let CB Auto Actions log activity, but that's all it should realistically be responsible for. It should not be displaying streams though. That should just be directly integrated.

Actions #3

Updated by krileon over 1 year ago

  • Status changed from Assigned to Rejected
  • Target version deleted (6.0.0)
  • % Done changed from 0 to 100

See #8981

Actions

Also available in: Atom PDF