From 5447b95c0a0d8d8759fffda350ecd87708cf3c9f Mon Sep 17 00:00:00 2001
From: Daniel Siepmann <coding@daniel-siepmann.de>
Date: Thu, 29 Sep 2022 15:17:06 +0200
Subject: [PATCH] Show last import time beside each import configuration in
 backend module (#77)

---
 .../Model/Backend/ImportConfiguration.php     | 22 +++++++++++++++++++
 .../TCA/tx_thuecat_import_configuration.php   |  9 ++++++++
 Documentation/Changelog/1.3.0.rst             | 10 +++++----
 Resources/Private/Language/locallang.xlf      |  6 +++++
 .../Templates/Backend/Overview/Index.html     | 11 ++++++++++
 ext_tables.sql                                |  1 +
 6 files changed, 55 insertions(+), 4 deletions(-)

diff --git a/Classes/Domain/Model/Backend/ImportConfiguration.php b/Classes/Domain/Model/Backend/ImportConfiguration.php
index 3fcc810..270cedc 100644
--- a/Classes/Domain/Model/Backend/ImportConfiguration.php
+++ b/Classes/Domain/Model/Backend/ImportConfiguration.php
@@ -26,6 +26,7 @@ namespace WerkraumMedia\ThueCat\Domain\Model\Backend;
 use TYPO3\CMS\Core\Utility\ArrayUtility;
 use TYPO3\CMS\Core\Utility\GeneralUtility;
 use TYPO3\CMS\Extbase\DomainObject\AbstractEntity;
+use TYPO3\CMS\Extbase\Persistence\ObjectStorage;
 use WerkraumMedia\ThueCat\Domain\Import\Entity\Properties\ForeignReference;
 use WerkraumMedia\ThueCat\Domain\Import\ImportConfiguration as ImportConfigurationInterface;
 use WerkraumMedia\ThueCat\Domain\Import\ResolveForeignReference;
@@ -52,6 +53,11 @@ class ImportConfiguration extends AbstractEntity implements ImportConfigurationI
      */
     protected $tstamp = null;
 
+    /**
+     * @var ObjectStorage<ImportLog>
+     */
+    protected $logs;
+
     /**
      * @var string[]|null
      */
@@ -62,6 +68,11 @@ class ImportConfiguration extends AbstractEntity implements ImportConfigurationI
      */
     protected $allowedTypes = [];
 
+    public function __construct()
+    {
+        $this->logs = new ObjectStorage();
+    }
+
     public function getTitle(): string
     {
         return $this->title;
@@ -82,6 +93,17 @@ class ImportConfiguration extends AbstractEntity implements ImportConfigurationI
         return $this->tstamp;
     }
 
+    public function getLastImported(): ?\DateTimeImmutable
+    {
+        $positionOfLastLog = count($this->logs) - 1;
+        $lastImport = $this->logs->offsetGet((string) $positionOfLastLog);
+        if (!$lastImport instanceof ImportLog) {
+            return null;
+        }
+
+        return $lastImport->getCreated();
+    }
+
     public function getStoragePid(): int
     {
         if ($this->configuration === '') {
diff --git a/Configuration/TCA/tx_thuecat_import_configuration.php b/Configuration/TCA/tx_thuecat_import_configuration.php
index 05186b5..3867b78 100644
--- a/Configuration/TCA/tx_thuecat_import_configuration.php
+++ b/Configuration/TCA/tx_thuecat_import_configuration.php
@@ -68,6 +68,15 @@ return (static function (string $extensionKey, string $tableName) {
                     'readOnly' => true,
                 ],
             ],
+            // Configured for usage within Extbase, not TCA itself
+            'logs' => [
+                'config' => [
+                    'type' => 'inline',
+                    'foreign_table' => 'tx_thuecat_import_log',
+                    'foreign_field' => 'configuration',
+                    'readOnly' => true,
+                ],
+            ],
         ],
         'types' => [
             '0' => [
diff --git a/Documentation/Changelog/1.3.0.rst b/Documentation/Changelog/1.3.0.rst
index 5b3b5f8..c1b7831 100644
--- a/Documentation/Changelog/1.3.0.rst
+++ b/Documentation/Changelog/1.3.0.rst
@@ -9,7 +9,9 @@ Nothing
 Features
 --------
 
-Add support for TYPO3 v11 and PHP 8.0 + 8.1.
+* Add support for TYPO3 v11 and PHP 8.0 + 8.1.
+
+* Last import date is now shown within backend module beside each import configuration.
 
 Fixes
 -----
@@ -34,9 +36,9 @@ Fixes
 Tasks
 -----
 
-Removed API Key from site configuration.
-The key was already moved to extension configuration as documented.
-Still we extended the site configuration, which is now cleaned up, see: https://github.com/werkraum-media/thuecat/issues/55
+* Removed API Key from site configuration.
+  The key was already moved to extension configuration as documented.
+  Still we extended the site configuration, which is now cleaned up, see: https://github.com/werkraum-media/thuecat/issues/55
 
 Deprecation
 -----------
diff --git a/Resources/Private/Language/locallang.xlf b/Resources/Private/Language/locallang.xlf
index 2b98390..9775c60 100644
--- a/Resources/Private/Language/locallang.xlf
+++ b/Resources/Private/Language/locallang.xlf
@@ -46,6 +46,12 @@
             <trans-unit id="module.importConfigurations.lastChanged" xml:space="preserve">
                 <source>Last changed</source>
             </trans-unit>
+            <trans-unit id="module.importConfigurations.lastImported" xml:space="preserve">
+                <source>Last imported</source>
+            </trans-unit>
+            <trans-unit id="module.importConfigurations.lastImported.never" xml:space="preserve">
+                <source>Never</source>
+            </trans-unit>
             <trans-unit id="module.importConfigurations.missing.text" xml:space="preserve">
                 <source><![CDATA[No import configuration is available yet. Please <a href="%1$s">create the first</a> to get started.]]></source>
             </trans-unit>
diff --git a/Resources/Private/Templates/Backend/Overview/Index.html b/Resources/Private/Templates/Backend/Overview/Index.html
index 2dd223e..e88e28e 100644
--- a/Resources/Private/Templates/Backend/Overview/Index.html
+++ b/Resources/Private/Templates/Backend/Overview/Index.html
@@ -55,6 +55,7 @@
                 <tr>
                     <th>{f:translate(id: 'module.importConfigurations.title')}</th>
                     <th>{f:translate(id: 'module.importConfigurations.lastChanged')}</th>
+                    <th>{f:translate(id: 'module.importConfigurations.lastImported')}</th>
                     <th>{f:translate(id: 'module.actions')}</th>
                 </tr>
             </thead>
@@ -63,6 +64,16 @@
                     <tr>
                         <td>{importConfiguration.title}</td>
                         <td>{importConfiguration.lastChanged -> f:format.date(format: 'd.m.Y H:i')}</td>
+                        <td>
+                            <f:if condition="{importConfiguration.lastImported}">
+                                <f:then>
+                                    {importConfiguration.lastImported -> f:format.date(format: 'd.m.Y H:i')}
+                                </f:then>
+                                <f:else>
+                                    {f:translate(id: 'module.importConfigurations.lastImported.never')}
+                                </f:else>
+                            </f:if>
+                        </td>
                         <td>
                             <f:link.editRecord
                                 uid="{importConfiguration.uid}"
diff --git a/ext_tables.sql b/ext_tables.sql
index d77c7c5..daea5ab 100644
--- a/ext_tables.sql
+++ b/ext_tables.sql
@@ -6,6 +6,7 @@ CREATE TABLE tx_thuecat_import_configuration (
     title varchar(255) DEFAULT '' NOT NULL,
     type varchar(255) DEFAULT '' NOT NULL,
     configuration text,
+    logs int(11) unsigned DEFAULT '0' NOT NULL,
 );
 
 CREATE TABLE tx_thuecat_import_log (