Clarified clicks label (#168)

* Clarified clicks label * Added comment clarifying that content is loaded via AJAX (content below is ignored) * Changed label 'clicks' to more accurate 'unique clicks' ('clicks' is used as a label with inconsistent meaning)
phpList · Apr 19, 2017 · 2b5e4b4 · 2b5e4b4
1 parent ba70408
commit 2b5e4b4
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/public_html/lists/admin/actions/statsoverview.php b/public_html/lists/admin/actions/statsoverview.php
@@ -125,7 +125,7 @@
 
     $totalclicked = Sql_Fetch_Row_Query(sprintf('select count(distinct userid) from %s where messageid = %d',
         $GLOBALS['tables']['linktrack_uml_click'], $row['messageid']));
-    $ls->addColumn($element, $GLOBALS['I18N']->get('clicks'), $totalclicked[0],
+    $ls->addColumn($element, $GLOBALS['I18N']->get('Unique Clicks'), $totalclicked[0],
         $totalclicked[0] ? PageURL2('mclicks&id='.$row['messageid']) : '');
 
     $ls->addRow($element, '',

diff --git a/public_html/lists/admin/messages.php b/public_html/lists/admin/messages.php
@@ -410,7 +410,7 @@
       <tr><td>' .s('Unique Views').'</td><td>'.$uniqueviews[0].'</td></tr>';
             if ($clicks[0]) {
                 $resultStats .= '
-           <tr><td>' .s('Clicks').'</td><td>'.$clicks[0].'</td></tr>';
+           <tr><td>' .s('Total Clicks').'</td><td>'.$clicks[0].'</td></tr>';
             }
             $resultStats .= '
          <tr><td>' .s('Bounced').'</td><td>'.$msg['bouncecount'].'</td></tr>';

diff --git a/public_html/lists/admin/statsoverview.php b/public_html/lists/admin/statsoverview.php
@@ -69,6 +69,7 @@
         unset($_SESSION['LoadDelay']);
     }
 
+    // Load page content via AJAX
     echo '<div id="contentdiv"></div>';
     echo asyncLoadContent('./?page=pageaction&action=statsoverview&ajaxed=true&id='.$id.'&start='.$start.addCsrfGetToken());