fixes #3892 Fixing + testing filter_truncate on getPlugin

core/DataTable/Row.php

@@ -492,6 +492,9 @@ class Piwik_DataTable_Row
     private function getColumnValuesMerged($operation, $thisColumnValue, $columnToSumValue)
     {
         switch ($operation) {
+            case 'skip':
+                $newValue = null;
+                break;
             case 'max':
                 $newValue = max($thisColumnValue, $columnToSumValue);
                 break;

plugins/UserSettings/API.php

@@ -215,6 +215,11 @@ class Piwik_UserSettings_API
 
             $visitsSum = $visitsSumTotal - $ieVisits;
 
+
+            // When Truncate filter is applied, it will call AddSummaryRow which tries to sum all rows.
+            // We tell the object to skip the column nb_visits_percentage when aggregating (since it's not correct to sum % values)
+            $table->setColumnAggregationOperation('nb_visits_percentage', 'skip');
+
             // The filter must be applied now so that the new column can
             // be sorted by the generic filters (applied right after this loop exits)
             $table->filter('ColumnCallbackAddColumnPercentage', array('nb_visits_percentage', Piwik_Archive::INDEX_NB_VISITS, $visitsSum, 1));

plugins/UserSettings/functions.php

@@ -16,6 +16,9 @@ require_once PIWIK_INCLUDE_PATH . '/libs/UserAgentParser/UserAgentParser.php';
 
 function Piwik_getPluginsLogo($oldLabel)
 {
+    if($oldLabel == Piwik_Translate('General_Others')) {
+        return false;
+    }
     return 'plugins/UserSettings/images/plugins/' . $oldLabel . '.gif';
 }
 

tests/PHPUnit/Integration/OneVisitor_LongUrlsTruncatedTest.php

@@ -24,7 +24,12 @@ class Test_Piwik_Integration_OneVisitor_LongUrlsTruncated extends IntegrationTes
 
     public function getApiForTesting()
     {
-        $apiToCall = array('Referers.getKeywords', 'Actions.getPageUrls');
+        $apiToCall = array(
+            'Referers.getKeywords',
+            'Actions.getPageUrls',
+
+            // Specifically testing getPlugin filter_truncate works
+            'UserSettings.getPlugin');
 
         return array(
             array($apiToCall, array('idSite'                 => self::$fixture->idSite,