Problem/Motivation

By definition, this is the LATEST group activity and the module stores multiple latest activities (per learning path and per module).

Proposed resolution

Leave just the latest, update query not to store multiple entities anymore.

Remaining tasks

Update the latest activities view not to use aggregation (not included in the solution).

CommentFileSizeAuthor
#4 opigno_lp-3291892-4.patch2.18 KBgraber
#2 opigno_lp-3291892-2.patch2.03 KBgraber

Comments

Graber created an issue. See original summary.

graber’s picture

graber commented
Status: Active » Needs review
StatusFileSize
new opigno_lp-3291892-2.patch2.03 KB

The patch is made on top of our list of patches (that are also posted here in different issues), so it may not apply, but easy enough to re-roll.

catch’s picture

catch
he/him
Primary language English
commented 
+++ b/opigno_learning_path.install
@@ -555,3 +555,42 @@ function opigno_learning_path_update_9010() {
 }
+
+/**
+ * Remove multiple entries from opigno_latest_group_activity.
+ */
+function opigno_learning_path_update_9011(&$sandbox) {
+  if (!isset($sandbox['list'])) {

This could be a post-update since it's 100% a content change rather than schema.

graber’s picture

graber commented
StatusFileSize
new opigno_lp-3291892-4.patch2.18 KB