Allow to star and unstar stored segment (#23771)

* Allow to start/unstar stored segment
* Add micro-animation on star/unstar segment
* Add tooltips on star/unstar segment depending on user rights +  who starred a segment
* Reviews tooltips content on all segment actions 
* Use button to improve a11y on all segment actions 
* Bugfix segment search - race condition on fast query change
* Bugfix segment search - clean search special character again
* Improve how the segment list and actions are displayed for anonymous users

Author: tzi

core/Plugin/Controller.php

@@ -726,6 +726,7 @@ protected function setBasicVariablesNoneAdminView($view)
     {
         $view->clientSideConfig = PiwikConfig::getInstance()->getClientSideOptions();
         $view->isSuperUser = Access::getInstance()->hasSuperUserAccess();
+        $view->userCurrentRole = Access::getInstance()->getRoleForSite($this->idSite);
         $view->hasSomeAdminAccess = Piwik::isUserHasSomeAdminAccess();
         $view->hasSomeViewAccess  = Piwik::isUserHasSomeViewAccess();
         $view->isUserIsAnonymous  = Piwik::isUserIsAnonymous();

core/Updates/5.7.0-b1.php

@@ -0,0 +1,42 @@
+<?php
+
+/**
+ * Matomo - free/libre analytics platform
+ *
+ * @link    https://matomo.org
+ * @license https://www.gnu.org/licenses/gpl-3.0.html GPL v3 or later
+ */
+
+namespace Piwik\Updates;
+
+use Piwik\Updater;
+use Piwik\Updater\Migration\Factory as MigrationFactory;
+use Piwik\Updates;
+
+class Updates_5_7_0_b1 extends Updates
+{
+    /**
+     * @var MigrationFactory
+     */
+    private $migration;
+
+    public function __construct(MigrationFactory $factory)
+    {
+        $this->migration = $factory;
+    }
+
+    public function getMigrations(Updater $updater)
+    {
+        return [
+            $this->migration->db->addColumns('segment', [
+                'starred' => 'TINYINT(1) NOT NULL DEFAULT 0',
+                'starred_by' => 'VARCHAR(100) NULL DEFAULT NULL',
+            ]),
+        ];
+    }
+
+    public function doUpdate(Updater $updater)
+    {
+        $updater->executeMigrations(__FILE__, $this->getMigrations($updater));
+    }
+}

core/Version.php

@@ -22,7 +22,7 @@ final class Version
      * The current Matomo version.
      * @var string
      */
-    public const VERSION = '5.7.0-alpha';
+    public const VERSION = '5.7.0-b1';
 
     public const MAJOR_VERSION = 5;
 

lang/en.json

@@ -29,6 +29,18 @@
         "BrokenDownReportDocumentation": "It is broken down into various reports, which are displayed in sparklines at the bottom of the page. You can enlarge the graphs by clicking on the report you'd like to see.",
         "Cancel": "Cancel",
         "CannotUnzipFile": "Cannot unzip file %1$s: %2$s",
+        "CanNotEditGlobalSegment": "This is a global segment. Only super users can edit global segments.",
+        "CanNotStarGlobalSegment": "This is a global segment. Only super users can star global segments.",
+        "CanNotUnstarGlobalSegment": "This is a global segment. Only super users can unstar global segments.",
+        "CanEditGlobalSegment": "This is a global segment. Any changes will apply across all websites.",
+        "CanStarGlobalSegment": "This is a global segment. Adding to Starred will apply across all websites.",
+        "CanUnstarGlobalSegment": "This is a global segment. Removing from Starred will apply across all websites.",
+        "CanNotEditSiteSegment": "You can only edit the segments you created yourself.",
+        "CanNotStarSiteSegment": "You can only add to Starred the segments you created yourself.",
+        "CanNotUnstarSiteSegment": "You can only remove from Starred the segments you created yourself.",
+        "CanEditSiteSegment": "Edit the segment for this website.",
+        "CanStarSiteSegment": "Add to Starred segments for this website.",
+        "CanUnstarSiteSegment": "Remove from Starred segments for this website.",
         "ChangeInX": "Change in %1$s",
         "ChangePassword": "Change password",
         "ChangeTagCloudView": "Please note, that you can view the report in other ways than as a tag cloud. Use the controls at the bottom of the report to do so.",
@@ -464,6 +476,9 @@
         "SmtpServerAddress": "SMTP server address",
         "SmtpUsername": "SMTP username",
         "Source": "Source",
+        "Star": "Star",
+        "StarredBy": "Starred by",
+        "StarredByYou": "Starred by you",
         "StatisticsAreNotRecorded": "Matomo Visitor Tracking is currently disabled! Re-enable tracking by setting record_statistics = 1 in your config/config.ini.php file.",
         "Subtotal": "Subtotal",
         "Summary": "Summary",

phpstan-baseline.neon

@@ -4658,11 +4658,6 @@ parameters:
 			count: 1
 			path: plugins/ScheduledReports/ScheduledReports.php
 
-		-
-			message: "#^Negated boolean expression is always true\\.$#"
-			count: 1
-			path: plugins/SegmentEditor/API.php
-
 		-
 			message: "#^Parameter \\#2 \\$idSites of class Piwik\\\\Segment constructor expects array, int\\|null given\\.$#"
 			count: 1

plugins/CoreHome/vue/dist/CoreHome.umd.js

@@ -216,6 +216,21 @@ export default class ComparisonsStore {
     );
   }
 
+  removeSegmentComparisonByDefinition(segmentDefinition: string): void {
+    if (!this.isComparisonEnabled()) {
+      throw new Error('Comparison disabled.');
+    }
+    let segmentIndex = null;
+    this.getSegmentComparisons().forEach((segment: SegmentComparison, index: number) => {
+      if (segment && segment.params && segment.params.segment === segmentDefinition) {
+        segmentIndex = index;
+      }
+    });
+    if (segmentIndex !== null) {
+      this.removeSegmentComparison(segmentIndex);
+    }
+  }
+
   addSegmentComparison(params: { [name: string]: string }): void {
     if (!this.isComparisonEnabled()) {
       throw new Error('Comparison disabled.');

plugins/CoreHome/vue/dist/CoreHome.umd.min.js

@@ -171,6 +171,8 @@ declare global {
     languageName: string;
     isPagesComparisonApiDisabled: boolean; // can be set to avoid checks on Api.getPagesComparisonsDisabledFor
     userLogin?: string;
+    userCurrentRole: string;
+    isUserAnonymous: boolean;
     userHasSomeAdminAccess: boolean;
     requiresPasswordConfirmation: boolean;
     disableTrackingMatomoAppLinks: boolean;

plugins/CoreHome/vue/src/Comparisons/Comparisons.store.ts

@@ -68,6 +68,8 @@
 
     piwik.hasSuperUserAccess = {{ hasSuperUserAccess|default(0)|e('js')}};
     piwik.userHasSomeAdminAccess = {{ userHasSomeAdminAccess|json_encode|raw }};
+    piwik.userCurrentRole = "{{ userCurrentRole|default('view')|e('js') }}";
+    piwik.isUserIsAnonymous = {{ isUserIsAnonymous|default(0)|e('js') }};
     piwik.userCapabilities = {{ userCapabilities|default([])|json_encode|raw }};
     piwik.config = {};
 {% if clientSideConfig is defined %}