Merge pull request #33 from davkraid/dev-11

TYPO3 11 updates
Leuchtfeuer · Apr 12, 2022 · 29a3cc5 · 29a3cc5
2 parents b7e8f7f + 7df3f27
commit 29a3cc5
Show file tree

Hide file tree

Showing 18 changed files with 113 additions and 89 deletions.
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -12,7 +12,7 @@ jobs:
     runs-on: ubuntu-18.04
     strategy:
       matrix:
-        php-version: [7.2, 7.3, 7.4]
+        php-version: [7.2, 7.3, 7.4, 8.0]
     steps:
       - name: Checkout
         uses: actions/checkout@v2
@@ -32,20 +32,32 @@ jobs:
     strategy:
       matrix:
         experimental: [false]
-        php-version: [7.2, 7.3, 7.4]
+        php-version: [7.4]
         typo3-version: [^10.4]
         include:
           - php-version: 7.4
             typo3-version: 10.4.x-dev
             experimental: true
 
           - php-version: 7.4
-            typo3-version: ^11.0
+            typo3-version: ^11.5
             experimental: false
 
           - php-version: 7.4
             typo3-version: dev-master
             experimental: true
+
+          - php-version: 8.0
+            typo3-version: 11.5.x-dev
+            experimental: true
+
+          - php-version: 8.0
+            typo3-version: ^11.5
+            experimental: false
+
+          - php-version: 8.0
+            typo3-version: dev-master
+            experimental: true
     env:
       PHP_VERSION: ${{ matrix.php-version }}
       TYPO3_VERSION: ${{ matrix.typo3-version }}
@@ -96,9 +108,10 @@ jobs:
           composer req typo3/testing-framework typo3/cms-core:"${{ matrix.typo3-version }}" typo3/cms-backend:"${{ matrix.typo3-version }}" typo3/cms-extbase:"${{ matrix.typo3-version }}" typo3/cms-extensionmanager:"${{ matrix.typo3-version }}" typo3/cms-fluid:"${{ matrix.typo3-version }}" typo3/cms-frontend:"${{ matrix.typo3-version }}"
           export TYPO3_PATH_WEB=$PWD/.Build/web
 
-      - name: Run PHP Unit
+      - name: Run PHPUnit
         run: |
-          .Build/bin/phpunit --colors --configuration Tests/Build/FunctionalTests.xml Tests/Functional --bootstrap .Build/vendor/typo3/testing-framework/Resources/Core/Build/FunctionalTestsBootstrap.php --coverage-clover=coverage.xml
+          .Build/bin/phpunit --configuration Tests/Build/FunctionalTests.xml --coverage-clover=coverage.xml
+          .Build/bin/phpunit --configuration Tests/Build/UnitTests.xml
 
           if [[ "${{ matrix.php-version }}" == "7.4" ]] && [[ "${{ matrix.typo3-version }}" == "^10.4"  ]]; then
             bash <(curl -s https://codecov.io/bash)

diff --git a/.gitignore b/.gitignore
@@ -1 +1,3 @@
 .idea
+.Build
+composer.lock
diff --git a/Classes/Command/UpdateIpDatabaseCommand.php b/Classes/Command/UpdateIpDatabaseCommand.php
@@ -103,7 +103,10 @@ private function validateArguments(): bool
     private function truncateTable(): void
     {
         $connection = GeneralUtility::makeInstance(ConnectionPool::class)->getConnectionForTable($this->table);
-        $connection->executeUpdate($connection->getDatabasePlatform()->getTruncateTableSQL($this->table, true));
+        $databasePlatform = $connection->getDatabasePlatform();
+        if ($databasePlatform) {
+            $connection->executeStatement($databasePlatform->getTruncateTableSQL($this->table, true));
+        }
     }
 
     /**

diff --git a/Classes/Domain/Repository/RegionRepository.php b/Classes/Domain/Repository/RegionRepository.php
@@ -33,7 +33,7 @@ public function getCountriesForPage(int $id): array
             ->join('rmm', 'static_countries', 'c', 'c.uid = rmm.uid_foreign')
             ->where($qb->expr()->eq('pmm.uid_local', $qb->createNamedParameter($id, \PDO::PARAM_INT)))
             ->execute()
-            ->fetchAll();
+            ->fetchAllAssociative();
 
         foreach ($results as $result) {
             $iso2Codes[$result['cn_iso_2']] = true;
@@ -52,7 +52,7 @@ public function shouldApplyWhenNoIpMatches(int $id): bool
             ->where($qb->expr()->eq('uid_local', $qb->createNamedParameter($id, \PDO::PARAM_INT)))
             ->andWhere($qb->expr()->eq('uid_foreign', $qb->createNamedParameter(self::APPLY_WHEN_NO_IP_MATCHES, \PDO::PARAM_INT)))
             ->execute()
-            ->fetchAll();
+            ->fetchAllAssociative();
 
         return !empty($results);
     }

diff --git a/Classes/FactProvider/AbstractFactProvider.php b/Classes/FactProvider/AbstractFactProvider.php
@@ -27,10 +27,12 @@ abstract class AbstractFactProvider
 
     /**
      * @param string $basename The basename for the factsArray. This name comes from configuration.
+     * @param array $configuration TypoScript configuration array for this fact provider
      */
-    public function __construct(string $basename = '')
+    public function __construct(string $basename = '', array $configuration = [])
     {
         $this->basename = $basename;
+        $this->configuration = $configuration;
     }
 
     public function isMultiple(): bool
@@ -65,18 +67,6 @@ protected function getFactPropertyName(string $property): string
         return mb_strtolower($property);
     }
 
-    /**
-     * @return array|mixed
-     */
-    public function getSubject()
-    {
-        if (count($this->facts) > 1) {
-            return $this->facts;
-        }
-
-        return array_shift($this->facts);
-    }
-
     /**
      * Priority is only set if there are multiple facts (e.g. for browser accept languages)
      */

diff --git a/Classes/FactProvider/BrowserAcceptedLanguage.php b/Classes/FactProvider/BrowserAcceptedLanguage.php
@@ -48,9 +48,9 @@ public function process(): self
     public function isGuilty($prosecution): bool
     {
         LocateUtility::mainstreamValue($prosecution);
-        $this->priority = (int)($this->getSubject()[$prosecution] ?? 0);
+        $this->priority = (int)($this->facts[$prosecution] ?? 0);
 
-        return isset($this->getSubject()[$prosecution]);
+        return isset($this->facts[$prosecution]);
     }
 
     protected function getAcceptedLanguages(): array

diff --git a/Classes/FactProvider/IP2Country.php b/Classes/FactProvider/IP2Country.php
@@ -33,7 +33,9 @@ public function getBasename(): string
      */
     public function process(): self
     {
-        $iso2 = GeneralUtility::makeInstance(LocateUtility::class)->getCountryIso2FromIP();
+        $simulateIp = $this->configuration['settings']['simulateIp'] ? : null;
+
+        $iso2 = GeneralUtility::makeInstance(LocateUtility::class)->getCountryIso2FromIP($simulateIp) ?? '';
         LocateUtility::mainstreamValue($iso2);
         $this->facts[$this->getBasename()] = $iso2;
 
@@ -48,6 +50,6 @@ public function isGuilty($prosecution): bool
         $prosecution = (string)$prosecution;
         LocateUtility::mainstreamValue($prosecution);
 
-        return $this->getSubject() === $prosecution;
+        return $this->facts[$this->getBasename()] === $prosecution;
     }
 }
diff --git a/Classes/Hook/OverrideIconOverlayHook.php b/Classes/Hook/OverrideIconOverlayHook.php
@@ -52,6 +52,6 @@ private function countRegions(string $table, array $row): int
             ->from($table)
             ->where($qb->expr()->eq('uid', $row['uid']))
             ->execute()
-            ->fetchColumn(0);
+            ->fetchOne();
     }
 }
diff --git a/Classes/Judge/Condition.php b/Classes/Judge/Condition.php
@@ -24,7 +24,7 @@ public function adjudicate(AbstractFactProvider $factProvider, int $priority = A
     {
         $prosecution = $this->configuration['prosecution'] ?? $this->configuration['prosecution.'] ?? null;
 
-        if ($prosecution !== null && $factProvider->isGuilty($prosecution) && isset($this->configuration['verdict'])) {
+        if ($prosecution !== null && isset($this->configuration['verdict']) && $factProvider->isGuilty($prosecution)) {
             $this->decision = (new Decision())->withVerdictName($this->configuration['verdict']);
             $this->decision->setPriority($priority);
 

diff --git a/Classes/Middleware/LanguageRedirectMiddleware.php b/Classes/Middleware/LanguageRedirectMiddleware.php
@@ -43,6 +43,7 @@ public function process(ServerRequestInterface $request, RequestHandlerInterface
                     'overrideSessionValue' => (bool)($locateSetup['overrideSessionValue'] ?? 0),
                     'sessionHandling' => (bool)($locateSetup['sessionHandling'] ?? 0),
                     'excludeBots' => (bool)($locateSetup['excludeBots'] ?? 1),
+                    'simulateIp' => $locateSetup['simulateIp'] ? : null,
                 ],
             ];
 

diff --git a/Classes/Processor/Court.php b/Classes/Processor/Court.php
@@ -95,7 +95,7 @@ protected function processFacts(): void
             }
 
             /* @var $factProvider AbstractFactProvider */
-            $factProvider = GeneralUtility::makeInstance($className);
+            $factProvider = GeneralUtility::makeInstance($className, $key, $this->configuration);
 
             if (!$factProvider instanceof AbstractFactProvider) {
                 throw new IllegalFactProviderException(
@@ -141,8 +141,7 @@ protected function callJudges(): ?Decision
             $configuration = $this->configuration['judges'][$key . '.'] ?? [];
 
             if (empty($configuration)) {
-                $this->logger->warning('');
-                // TODO: Do something?
+                $this->logger->warning('No judges are configured.');
             }
 
             $this->logger->info(sprintf('Judge with key "%s" will be called.', $key));

diff --git a/Classes/Utility/LocateUtility.php b/Classes/Utility/LocateUtility.php
@@ -27,7 +27,7 @@ class LocateUtility
     public function getCountryIso2FromIP(?string $ip = null)
     {
         $ip = $this->getNumericIp($ip);
-        $tableName = self::getTableNameForIp($ip);
+        $tableName = $this->getTableNameForIp($ip);
         $queryBuilder = GeneralUtility::makeInstance(ConnectionPool::class)->getQueryBuilderForTable($tableName);
 
         return $queryBuilder
@@ -36,12 +36,12 @@ public function getCountryIso2FromIP(?string $ip = null)
             ->where($queryBuilder->expr()->lte('ip_from', $queryBuilder->createNamedParameter($ip)))
             ->andWhere($queryBuilder->expr()->gte('ip_to', $queryBuilder->createNamedParameter($ip)))
             ->execute()
-            ->fetchColumn(0);
+            ->fetchOne();
     }
 
     public function getNumericIp(?string $ip = null): string
     {
-        $ip = $ip ?? GeneralUtility::getIndpEnv('REMOTE_ADDR');
+        $ip = $ip ?? (string)GeneralUtility::getIndpEnv('REMOTE_ADDR');
 
         return strpos($ip, '.') !== false ? (string)ip2long($ip) : $this->convertIpv6($ip);
     }
@@ -63,7 +63,8 @@ private function convertIpv6(string $ip): string
                 break;
 
             case function_exists('bcadd'):
-                for ($i = 0; $i < strlen($bin); $i++) {
+                $max = strlen($bin);
+                for ($i = 0; $i < $max; $i++) {
                     $decimalIp = bcmul($decimalIp, '2');
                     $decimalIp = bcadd($decimalIp, $bin[$i]);
                 }
@@ -80,7 +81,7 @@ private function convertIpv6(string $ip): string
         return $decimalIp;
     }
 
-    public static function mainstreamValue(string &$value)
+    public static function mainstreamValue(string &$value): void
     {
         $value = mb_strtolower(str_replace('-', '_', $value));
     }

diff --git a/Classes/Verdict/Redirect.php b/Classes/Verdict/Redirect.php
@@ -28,7 +28,7 @@ class Redirect extends AbstractVerdict
     const SESSION_KEY = 'language';
     const OVERRIDE_PARAMETER = 'setLang';
 
-    private $cookieMode = false;
+    private $sessionMode = false;
 
     private $redirectLanguageUid = 0;
 
@@ -43,9 +43,9 @@ public function execute(): ?ResponseInterface
         $this->redirectLanguageUid = (int)$this->configuration['sys_language'];
         $this->requestedLanguageUid = GeneralUtility::makeInstance(Context::class)->getAspect('language')->getId();
 
-        // Initialize Cookie mode if necessary and prepare everything for possible redirects
-        $this->initializeCookieMode();
-        $this->handleCookieStuff();
+        // Initialize Session mode if necessary and prepare everything for possible redirects
+        $this->initializeSessionMode();
+        $this->handleSessionStuff();
 
         // Skip if no redirect is necessary
         if (!$this->shouldRedirect()) {
@@ -66,60 +66,60 @@ public function execute(): ?ResponseInterface
     }
 
     /**
-     * Set CookeMode Param to true if sessionHandling is enables
+     * Set sessionMode Param to true if sessionHandling is enables
      */
-    private function initializeCookieMode(): void
+    private function initializeSessionMode(): void
     {
         if ((bool)($this->configuration['sessionHandling'] ?? false) === true) {
-            $this->logger->info('Cookie Handling is set.');
-            $this->cookieMode = true;
+            $this->logger->info('Session Handling is set.');
+            $this->sessionMode = true;
         }
     }
 
-    private function handleCookieStuff()
+    private function handleSessionStuff()
     {
         $currentLanguageUid = $this->requestedLanguageUid;
 
-        if ($this->isCookieSet()) {
-            // Cookie is not in current language
-            $this->logger->info('Cookie is set.');
+        if ($this->isSessionValueSet()) {
+            // Session is not in current language
+            $this->logger->info('Session value is set.');
 
-            if ($this->isCookieInCurrentLanguage() === false && $this->shouldOverrideSessionValue() === true) {
-                // Override cookie
-                $this->logger->info('Cookie is not in current language, so we override it.');
+            if ($this->isSessionInCurrentLanguage() === false && $this->shouldOverrideSessionValue() === true) {
+                // Override session
+                $this->logger->info('Session is not in current language, so we override it.');
                 $this->redirectLanguageUid = $currentLanguageUid;
-                $this->setCookie($currentLanguageUid);
-            } elseif ($this->isCookieInCurrentLanguage()) {
-                // Cookie is in current language
-                $this->logger->info('Cookie is in current language.');
+                $this->setSessionValue($currentLanguageUid);
+            } elseif ($this->isSessionInCurrentLanguage()) {
+                // Session is in current language
+                $this->logger->info('Session is in current language.');
                 $this->redirectLanguageUid = $currentLanguageUid;
             } else {
-                // Override config array by cookie value
-                $this->logger->info('Cookie is not in current language and overriding is not allowed.');
-                $this->redirectLanguageUid = $this->getCookieValue();
-                $this->configuration['sys_language'] = $this->getCookieValue();
+                // Override config array by session value
+                $this->logger->info('Session is not in current language and overriding is not allowed.');
+                $this->redirectLanguageUid = $this->getSessionValue();
+                $this->configuration['sys_language'] = $this->getSessionValue();
             }
-        } elseif ($this->cookieMode === true) {
-            $this->logger->info('Cookie is not set, but we are in cookie mode.');
+        } elseif ($this->sessionMode === true) {
+            $this->logger->info('Session is not set, but we are in session mode.');
 
             if ($currentLanguageUid !== $this->redirectLanguageUid) {
-                // Set cookie value to target language
-                $this->setCookie($this->redirectLanguageUid);
+                // Set session value to target language
+                $this->setSessionValue($this->redirectLanguageUid);
             } else {
-                // Set cookie value to current language
-                $this->setCookie($currentLanguageUid);
+                // Set session value to current language
+                $this->setSessionValue($currentLanguageUid);
             }
         }
     }
 
-    private function isCookieSet(): bool
+    private function isSessionValueSet(): bool
     {
-        return $this->getCookieValue() !== null;
+        return $this->getSessionValue() !== null;
     }
 
-    private function isCookieInCurrentLanguage(): bool
+    private function isSessionInCurrentLanguage(): bool
     {
-        return $this->requestedLanguageUid === $this->getCookieValue();
+        return $this->requestedLanguageUid === $this->getSessionValue();
     }
 
     private function shouldOverrideSessionValue(): bool
@@ -131,7 +131,7 @@ private function shouldOverrideSessionValue(): bool
         return false;
     }
 
-    private function setCookie(?int $value)
+    private function setSessionValue(?int $value)
     {
         if ($value === null) {
             $value = (int)$this->configuration['sys_language'];
@@ -140,15 +140,15 @@ private function setCookie(?int $value)
         $this->session->set(self::SESSION_KEY, $value);
     }
 
-    private function getCookieValue(): ?int
+    private function getSessionValue(): ?int
     {
         return $this->session->get(self::SESSION_KEY);
     }
 
     private function shouldRedirect(): bool
     {
-        // Always redirect when we are not in cookie mode
-        if ($this->cookieMode === false) {
+        // Always redirect when we are not in session mode
+        if ($this->sessionMode === false) {
             return true;
         }
 
@@ -162,8 +162,8 @@ private function shouldRedirect(): bool
             }
         }
 
-        // Do not redirect, when cookie is set and cookie value matches given language id
-        if ($this->isCookieInCurrentLanguage()) {
+        // Do not redirect, when session is set and session value matches given language id
+        if ($this->isSessionInCurrentLanguage()) {
             return false;
         }