Issue #3261452 by Spokje, andypost, naveenvalecha, quietone, smustgrave:...

Issue #3261452 by Spokje, andypost, naveenvalecha, quietone, smustgrave: [11.x] Remove tracker module from core

core/.cspell.json

@@ -27,7 +27,6 @@
       "modules/statistics/tests/fixtures/drupal6.php",
       "modules/statistics/tests/fixtures/drupal7.php",
       "modules/system/tests/logo.svgz",
-      "modules/tracker/tests/fixtures/drupal7.php",
       "node_modules/*",
       "profiles/demo_umami/modules/demo_umami_content/default_content/languages/es/**/*",
       "tests/fixtures/files/*",

core/MAINTAINERS.txt

@@ -404,9 +404,6 @@ Toolbar
 Tour
 - Nick Schuch 'nick_schuch' https://www.drupal.org/u/nick_schuch
 
-Tracker
-- ?
-
 Transliteration
 - Andrei Mateescu 'amateescu' https://www.drupal.org/u/amateescu
 

core/modules/migrate/tests/src/Kernel/Plugin/MigrationPluginListTest.php

@@ -57,8 +57,6 @@ class MigrationPluginListTest extends KernelTestBase {
     'system',
     'taxonomy',
     'text',
-    // @todo Remove tracker in https://www.drupal.org/project/drupal/issues/3261452
-    'tracker',
     'update',
     'user',
   ];

core/modules/migrate_drupal/migrations/state/migrate_drupal.migrate_drupal.yml

@@ -36,8 +36,6 @@ finished:
     ping: core
     poll: core
     throttle: core
-    # @todo Remove tracker in https://www.drupal.org/project/drupal/issues/3261452
-    tracker: core
     translation: core
     trigger: core
     variable: core

core/modules/migrate_drupal/tests/src/Kernel/StateFileExistsTest.php

@@ -76,8 +76,6 @@ class StateFileExistsTest extends MigrateDrupalTestBase {
     'taxonomy',
     'telephone',
     'text',
-    // @todo Remove tracker in https://www.drupal.org/project/drupal/issues/3261452
-    'tracker',
     'update',
     'user',
   ];

core/modules/migrate_drupal/tests/src/Kernel/d6/MigrateDrupal6AuditIdsTest.php

@@ -44,8 +44,6 @@ protected function setUp(): void {
     $this->installSchema('forum', ['forum_index']);
     $this->installSchema('node', ['node_access']);
     $this->installSchema('search', ['search_dataset']);
-    // @todo Remove tracker in https://www.drupal.org/project/drupal/issues/3261452
-    $this->installSchema('tracker', ['tracker_node', 'tracker_user']);
 
     // Enable content moderation for nodes of type page.
     $this->installEntitySchema('content_moderation_state');

core/modules/migrate_drupal/tests/src/Kernel/d7/MigrateDrupal7AuditIdsTest.php

@@ -42,8 +42,6 @@ protected function setUp(): void {
     $this->installSchema('forum', ['forum_index']);
     $this->installSchema('node', ['node_access']);
     $this->installSchema('search', ['search_dataset']);
-    // @todo Remove tracker in https://www.drupal.org/project/drupal/issues/3261452
-    $this->installSchema('tracker', ['tracker_node', 'tracker_user']);
 
     // Enable content moderation for nodes of type page.
     $this->installEntitySchema('content_moderation_state');

core/modules/migrate_drupal_ui/tests/src/Functional/d6/MultilingualReviewPageTest.php

@@ -121,8 +121,6 @@ protected function getAvailablePaths() {
       'Taxonomy',
       'Text',
       'Throttle',
-      // @todo Remove Tracker in https://www.drupal.org/project/drupal/issues/3261452
-      'Tracker',
       'Trigger',
       'Update status',
       'Upload',
@@ -151,6 +149,7 @@ protected function getMissingPaths() {
       'Devel node access',
       'Forum',
       'Statistics',
+      'Tracker',
       // Option Widgets is set not_finished in migrate_state_not_finished_test.
       'Option Widgets',
       'Views',

core/modules/migrate_drupal_ui/tests/src/Functional/d6/NoMultilingualReviewPageTest.php

@@ -107,8 +107,6 @@ protected function getAvailablePaths() {
       'Taxonomy',
       'Text',
       'Throttle',
-      // @todo Remove tracker in https://www.drupal.org/project/drupal/issues/3261452
-      'Tracker',
       'Trigger',
       'Update status',
       'Upload',
@@ -157,6 +155,7 @@ protected function getMissingPaths() {
       'String translation',
       'Synchronize translations',
       'Taxonomy translation',
+      'Tracker',
       'Views',
       'Views translation',
     ];

core/modules/migrate_drupal_ui/tests/src/Functional/d7/MultilingualReviewPageTest.php

@@ -166,7 +166,6 @@ protected function getMissingPaths() {
       'References',
       'References UUID',
       'Statistics',
-      // @todo Remove tracker in https://www.drupal.org/project/drupal/issues/3261452
       'Tracker',
       'Translation redirect',
       'Translation sets',

core/modules/migrate_drupal_ui/tests/src/Functional/d7/NoMultilingualReviewPageTest.php

@@ -179,7 +179,6 @@ protected function getMissingPaths() {
       // These modules are in the missing path list because they are installed
       // on the source site but they are not installed on the destination site.
       'Syslog',
-      // @todo Remove tracker in https://www.drupal.org/project/drupal/issues/3261452
       'Tracker',
       'Update manager',
     ];

core/modules/migrate_drupal_ui/tests/src/Functional/d7/Upgrade7Test.php

@@ -215,7 +215,6 @@ protected function getMissingPaths() {
       // These modules are in the missing path list because they are installed
       // on the source site but they are not installed on the destination site.
       'Syslog',
-      // @todo Remove tracker in https://www.drupal.org/project/drupal/issues/3261452
       'Tracker',
       'Update manager',
     ];

core/modules/tracker/config/install/tracker.settings.yml

-cron_index_limit: 1000

core/modules/tracker/config/schema/tracker.schema.yml

-# Schema for the configuration files of the tracker module.
-
-tracker.settings:
-  type: config_object
-  label: 'Tracker settings'
-  mapping:
-    cron_index_limit:
-      type: integer
-      label: 'Cron index limit'

core/modules/tracker/config/schema/tracker.views.schema.yml

-# Schema for the views plugins of the Tracker module.
-
-views.argument.tracker_user_uid:
-  type: views.argument.argument_comment_user_uid
-  label: 'Nodes user posted/commented on'
-
-views.filter.tracker_user_uid:
-  type: views.filter.user_name
-  label: 'UID of a nodes user posted/commented on'

core/modules/tracker/help_topics/tracker.tracking_changed_content.html.twig

----
-label: 'Viewing lists of recently-updated content'
-related:
-  - core.tracking_content
-  - history.tracking_user_content
----
-{% set recent_link_text %}
-{% trans %}Recent content{% endtrans %}
-{% endset %}
-{% set recent_link = render_var(help_route_link(recent_link_text, 'tracker.page')) %}
-<h2>{% trans %}What displays of recently-updated content are available?{% endtrans %}</h2>
-<p>{% trans %}Assuming that you have the core Activity Tracker module installed, these pages that show recently-updated content are available:{% endtrans %}</p>
-<ul>
-  <li>{% trans %}{{ recent_link }}: Shows the content that has been most recently added, updated, or commented on.{% endtrans %}</li>
-  <li>{% trans %}The <em>My recent content</em> tab on the <em>Recent content</em> page (for logged-in users) limits the list to content created or commented on by the user viewing the page.{% endtrans %}</li>
-  <li>{% trans %}The <em>Activity</em> tab on a user profile shows the same list for the user whose profile is being viewed.{% endtrans %}</li>
-</ul>

core/modules/tracker/js/tracker-history.js

-/**
- * Attaches behaviors for the Tracker module's History module integration.
- *
- * May only be loaded for authenticated users, with the History module enabled.
- */
-(function ($, Drupal, window) {
-  function processNodeNewIndicators(placeholders) {
-    const newNodeString = Drupal.t('new');
-    const updatedNodeString = Drupal.t('updated');
-
-    placeholders.forEach((placeholder) => {
-      const timestamp = parseInt(
-        placeholder.getAttribute('data-history-node-timestamp'),
-        10,
-      );
-      const nodeID = placeholder.getAttribute('data-history-node-id');
-      const lastViewTimestamp = Drupal.history.getLastRead(nodeID);
-
-      if (timestamp > lastViewTimestamp) {
-        const message =
-          lastViewTimestamp === 0 ? newNodeString : updatedNodeString;
-        $(placeholder).append(`<span class="marker">${message}</span>`);
-      }
-    });
-  }
-
-  function processNewRepliesIndicators(placeholders) {
-    // Figure out which placeholders need the "x new" replies links.
-    const placeholdersToUpdate = {};
-    placeholders.forEach((placeholder) => {
-      const timestamp = parseInt(
-        placeholder.getAttribute('data-history-node-last-comment-timestamp'),
-        10,
-      );
-      const nodeID = placeholder.previousSibling.previousSibling.getAttribute(
-        'data-history-node-id',
-      );
-      const lastViewTimestamp = Drupal.history.getLastRead(nodeID);
-
-      // Queue this placeholder's "X new" replies link to be downloaded from the
-      // server.
-      if (timestamp > lastViewTimestamp) {
-        placeholdersToUpdate[nodeID] = placeholder;
-      }
-    });
-
-    // Perform an AJAX request to retrieve node view timestamps.
-    const nodeIDs = Object.keys(placeholdersToUpdate);
-    if (nodeIDs.length === 0) {
-      return;
-    }
-    $.ajax({
-      url: Drupal.url('comments/render_new_comments_node_links'),
-      type: 'POST',
-      data: { 'node_ids[]': nodeIDs },
-      dataType: 'json',
-      success(results) {
-        Object.keys(results || {}).forEach((nodeID) => {
-          if (placeholdersToUpdate.hasOwnProperty(nodeID)) {
-            const url = results[nodeID].first_new_comment_link;
-            const text = Drupal.formatPlural(
-              results[nodeID].new_comment_count,
-              '1 new',
-              '@count new',
-            );
-            $(placeholdersToUpdate[nodeID]).append(
-              `<br /><a href="${url}">${text}</a>`,
-            );
-          }
-        });
-      },
-    });
-  }
-
-  /**
-   * Render "new" and "updated" node indicators, as well as "X new" replies links.
-   */
-  Drupal.behaviors.trackerHistory = {
-    attach(context) {
-      // Find all "new" comment indicator placeholders newer than 30 days ago that
-      // have not already been read after their last comment timestamp.
-      const nodeIDs = [];
-      const nodeNewPlaceholders = once(
-        'history',
-        '[data-history-node-timestamp]',
-        context,
-      ).filter((placeholder) => {
-        const nodeTimestamp = parseInt(
-          placeholder.getAttribute('data-history-node-timestamp'),
-          10,
-        );
-        const nodeID = placeholder.getAttribute('data-history-node-id');
-        if (Drupal.history.needsServerCheck(nodeID, nodeTimestamp)) {
-          nodeIDs.push(nodeID);
-          return true;
-        }
-
-        return false;
-      });
-
-      // Find all "new" comment indicator placeholders newer than 30 days ago that
-      // have not already been read after their last comment timestamp.
-      const newRepliesPlaceholders = once(
-        'history',
-        '[data-history-node-last-comment-timestamp]',
-        context,
-      ).filter((placeholder) => {
-        const lastCommentTimestamp = parseInt(
-          placeholder.getAttribute('data-history-node-last-comment-timestamp'),
-          10,
-        );
-        const nodeTimestamp = parseInt(
-          placeholder.previousSibling.previousSibling.getAttribute(
-            'data-history-node-timestamp',
-          ),
-          10,
-        );
-        // Discard placeholders that have zero comments.
-        if (lastCommentTimestamp === nodeTimestamp) {
-          return false;
-        }
-        const nodeID = placeholder.previousSibling.previousSibling.getAttribute(
-          'data-history-node-id',
-        );
-        if (Drupal.history.needsServerCheck(nodeID, lastCommentTimestamp)) {
-          if (nodeIDs.indexOf(nodeID) === -1) {
-            nodeIDs.push(nodeID);
-          }
-          return true;
-        }
-
-        return false;
-      });
-
-      if (
-        nodeNewPlaceholders.length === 0 &&
-        newRepliesPlaceholders.length === 0
-      ) {
-        return;
-      }
-
-      // Fetch the node read timestamps from the server.
-      Drupal.history.fetchTimestamps(nodeIDs, () => {
-        processNodeNewIndicators(nodeNewPlaceholders);
-        processNewRepliesIndicators(newRepliesPlaceholders);
-      });
-    },
-  };
-})(jQuery, Drupal, window);

core/modules/tracker/migrations/d7_tracker_node.yml

-id: d7_tracker_node
-label: Tracker node
-migration_tags:
-  - Drupal 7
-  - Content
-source:
-  plugin: d7_tracker_node
-process:
-  nid: nid
-  published: published
-  changed: changed
-destination:
-  plugin: entity:node
-migration_dependencies:
-  required:
-    - d7_user

core/modules/tracker/migrations/d7_tracker_settings.yml

-id: d7_tracker_settings
-label: Tracker settings
-migration_tags:
-  - Drupal 7
-  - Configuration
-source:
-  plugin: variable
-  variables:
-    - tracker_batch_size
-  source_module: tracker
-process:
-  cron_index_limit: tracker_batch_size
-destination:
-  plugin: config
-  config_name: tracker.settings

core/modules/tracker/migrations/d7_tracker_user.yml

-id: d7_tracker_user
-label: Tracker user
-migration_tags:
-  - Drupal 7
-  - Content
-source:
-  plugin: d7_tracker_user
-process:
-  nid: nid
-  uid: uid
-  published: published
-  changed: changed
-destination:
-  plugin: entity:user
-migration_dependencies:
-  required:
-    - d7_user