ENGCOM-6427: Resolve Refresh Statistics: Updated At = Null should be display as "Never" instead of "undefined". issue25931 #25932

Author: VladimirZaets

app/code/Magento/Reports/Controller/Adminhtml/Report/AbstractReport.php

@@ -18,6 +18,7 @@
 /**
  * Reports api controller
  *
+ * phpcs:disable Magento2.Classes.AbstractApi
  * @api
  * @since 100.0.2
  * @SuppressWarnings(PHPMD.AllPurposeAction)
@@ -140,7 +141,7 @@ protected function _showLastExecutionTime($flagCode, $refreshCode)
         $flag = $this->_objectManager->create(\Magento\Reports\Model\Flag::class)
             ->setReportFlagCode($flagCode)
             ->loadSelf();
-        $updatedAt = 'undefined';
+        $updatedAt = __('Never');
         if ($flag->hasData()) {
             $updatedAt = $this->timezone->formatDate(
                 $flag->getLastUpdate(),

app/code/Magento/Reports/i18n/en_US.csv

@@ -224,3 +224,5 @@ Action,Action
 Report,Report
 Description,Description
 undefined,undefined
+Never,Never
+

app/code/Magento/Reports/view/adminhtml/layout/reports_report_statistics_index.xml

@@ -71,7 +71,7 @@
                                 <argument name="sortable" xsi:type="string">0</argument>
                                 <argument name="id" xsi:type="string">updated_at</argument>
                                 <argument name="index" xsi:type="string">updated_at</argument>
-                                <argument name="default" xsi:type="string" translate="true">undefined</argument>
+                                <argument name="default" xsi:type="string" translate="true">Never</argument>
                                 <argument name="column_css_class" xsi:type="string">col-period</argument>
                                 <argument name="header_css_class" xsi:type="string">col-period</argument>
                             </arguments>

dev/tests/integration/testsuite/Magento/Reports/Controller/Adminhtml/Report/Statistics/IndexTest.php

@@ -0,0 +1,25 @@
+<?php
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+
+declare(strict_types=1);
+
+namespace Magento\Reports\Controller\Adminhtml\Report\Statistics;
+
+/**
+ * @magentoAppArea adminhtml
+ */
+class IndexTest extends \Magento\TestFramework\TestCase\AbstractBackendController
+{
+    /**
+     * Test load page
+     */
+    public function testExecute()
+    {
+        $this->dispatch('backend/reports/report_statistics');
+        $actual = $this->getResponse()->getBody();
+        $this->assertContains('Never', $actual);
+    }
+}