fix: Use TYPO3\CMS\Core\Type\Enumeration instead of backed enums, as …

Classes/Command/BatchCommand.php

@@ -5,8 +5,8 @@
 namespace Netlogix\Nxgooglelocations\Command;
 
 use Netlogix\Nxgooglelocations\Domain\Model\Batch;
-use Netlogix\Nxgooglelocations\Domain\Model\BatchState;
 use Netlogix\Nxgooglelocations\Domain\Repository\BatchRepository;
+use Netlogix\Nxgooglelocations\Enumerations\BatchState;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;

Classes/Domain/Model/Batch.php

@@ -6,6 +6,7 @@
 
 use DateTime;
 use Exception;
+use Netlogix\Nxgooglelocations\Enumerations\BatchState;
 use Netlogix\Nxgooglelocations\Service\BackendUserImpersonator;
 use Netlogix\Nxgooglelocations\Service\GeoCoder;
 use Netlogix\Nxgooglelocations\Service\Importer;
@@ -42,7 +43,7 @@ class Batch extends AbstractEntity
     #[TYPO3\Transient]
     protected $callback;
 
-    protected BatchState $state = BatchState::NEW;
+    protected string $state = BatchState::NEW;
 
     protected int $amount = 0;
 
@@ -108,7 +109,7 @@ public function cancel(): void
         $this->setState(BatchState::CLOSED);
     }
 
-    protected function setState(BatchState $state): void
+    protected function setState(string $state): void
     {
         $this->state = $state;
         $this->emitStateChange();

Classes/Domain/Model/CodingResult.php

@@ -4,7 +4,7 @@
 
 namespace Netlogix\Nxgooglelocations\Domain\Model;
 
-use Netlogix\Nxgooglelocations\Service\GeoCoder;
+use Netlogix\Nxgooglelocations\Enumerations\CodingResultProbability;
 use Netlogix\Nxgooglelocations\Service\GeoCoderStatus;
 use TYPO3\CMS\Extbase\Property\Exception\InvalidSourceException;
 use TYPO3\CMS\Extbase\Reflection\ObjectAccess;
@@ -13,7 +13,7 @@
  * An accessible data structure for Google geocoding results.
  *
  * @property array rawData
- * @property GeoCoderStatus status
+ * @property string status
  * @property string formattedAddress
  * @property string addressResultFromGeocoding
  * @property float latitude
@@ -48,7 +48,7 @@ public function __get($propertyName)
                 'longitude' => $this->longitude,
             ],
             'probability' => ($this->status === GeoCoderStatus::ZERO_RESULTS)
-                ? CodingResultProbability::ZERO_RESULTS->value
+                ? CodingResultProbability::ZERO_RESULTS
                 : max(
                     $this->minProbability,
                     min(

Classes/Domain/Repository/BatchRepository.php

@@ -4,7 +4,7 @@
 
 namespace Netlogix\Nxgooglelocations\Domain\Repository;
 
-use Netlogix\Nxgooglelocations\Domain\Model\BatchState;
+use Netlogix\Nxgooglelocations\Enumerations\BatchState;
 use TYPO3\CMS\Extbase\Persistence\Generic\Typo3QuerySettings;
 use Netlogix\Nxgooglelocations\Domain\Model\Batch;
 use TYPO3\CMS\Core\Utility\GeneralUtility;
@@ -32,13 +32,13 @@ public function findOpenInFolder(int $storagePageId): QueryResultInterface
 
         $query->setQuerySettings($querySettings);
 
-        $query->matching($query->logicalNot($query->equals('state', BatchState::CLOSED->value)));
+        $query->matching($query->logicalNot($query->equals('state', BatchState::CLOSED)));
 
         return $query->execute();
     }
 
-    public function findOneByState(BatchState $state): ?Batch
+    public function findOneByState(string $state): ?Batch
     {
-        return $this->findOneBy(['state' => $state->value]);
+        return $this->findOneBy(['state' => $state]);
     }
 }

Classes/Enumerations/BatchState.php

@@ -0,0 +1,40 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Netlogix\Nxgooglelocations\Enumerations;
+
+use TYPO3\CMS\Core\Type\Enumeration;
+
+final class BatchState extends Enumeration
+{
+    /**
+     * @var string
+     */
+    final public const __default = self::NEW;
+
+    /**
+     * @var string
+     */
+    final public const NEW = 'new';
+
+    /**
+     * @var string
+     */
+    final public const VALIDATING = 'validating';
+
+    /**
+     * @var string
+     */
+    final public const GEOCODING = 'running';
+
+    /**
+     * @var string
+     */
+    final public const PERSISTING = 'persisting';
+
+    /**
+     * @var string
+     */
+    final public const CLOSED = 'closed';
+}

Classes/Enumerations/CodingResultProbability.php

@@ -0,0 +1,25 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Netlogix\Nxgooglelocations\Enumerations;
+
+use TYPO3\CMS\Core\Type\Enumeration;
+
+final class CodingResultProbability extends Enumeration
+{
+    /**
+     * @var int
+     */
+    final public const ZERO_RESULTS = -1;
+
+    /**
+     * @var int
+     */
+    final public const MANUALLY_IMPORT = -255;
+
+    /**
+     * @var int
+     */
+    final public const MANUALLY_EDITOR = -256;
+}

Classes/Enumerations/GeoCoderStatus.php

@@ -0,0 +1,20 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Netlogix\Nxgooglelocations\Enumerations;
+
+use TYPO3\CMS\Core\Type\Enumeration;
+
+final class GeoCoderStatus extends Enumeration
+{
+    /**
+     * @var string
+     */
+    final public const OK = 'OK';
+
+    /**
+     * @var string
+     */
+    final public const ZERO_RESULTS = 'ZERO_RESULTS';
+}

Classes/Service/GeoCoder.php

@@ -5,9 +5,10 @@
 namespace Netlogix\Nxgooglelocations\Service;
 
 use Exception;
-use Netlogix\Nxgooglelocations\Domain\Model\CodingResultProbability;
 use Netlogix\Nxgooglelocations\Domain\Model\CodingResult;
 use Netlogix\Nxgooglelocations\Domain\Model\FieldMap;
+use Netlogix\Nxgooglelocations\Enumerations\CodingResultProbability;
+use Netlogix\Nxgooglelocations\Enumerations\GeoCoderStatus;
 use TYPO3\CMS\Core\Utility\GeneralUtility;
 use TYPO3\CMS\Extbase\Reflection\ObjectAccess;
 
@@ -52,7 +53,7 @@ public function needsToBeGeoCoded(array $tcaRecord): bool
 
     public function setProbabilityToManually(array $tcaRecord): array
     {
-        $tcaRecord[$this->fieldMap->probability] = CodingResultProbability::MANUALLY_IMPORT->value;
+        $tcaRecord[$this->fieldMap->probability] = CodingResultProbability::MANUALLY_IMPORT;
 
         return $tcaRecord;
     }