TYPO3 13

.github/workflows/ci.yaml

@@ -15,7 +15,7 @@ jobs:
       - name: Install PHP
         uses: shivammathur/setup-php@v2
         with:
-          php-version: 7.4
+          php-version: 8.3
           coverage: none
           tools: composer:v2
         env:
@@ -29,10 +29,9 @@ jobs:
     strategy:
       matrix:
         php-version:
-          - 7.4
-          - 8.0
           - 8.1
           - 8.2
+          - 8.3
     steps:
       - name: Checkout
         uses: actions/checkout@v3
@@ -56,7 +55,7 @@ jobs:
       - name: Install PHP
         uses: shivammathur/setup-php@v2
         with:
-          php-version: "7.4"
+          php-version: "8.3"
           coverage: none
           tools: composer:v2
         env:
@@ -104,24 +103,21 @@ jobs:
     strategy:
       matrix:
         include:
-          - db-version: '8'
-            php-version: '7.4'
-            typo3-version: '^11.5'
-          - db-version: '8'
-            php-version: '8.0'
-            typo3-version: '^11.5'
           - db-version: '8'
             php-version: '8.1'
-            typo3-version: '^11.5'
+            typo3-version: '^12.4'
           - db-version: '8'
             php-version: '8.2'
-            typo3-version: '^11.5'
+            typo3-version: '^12.4'
           - db-version: '8'
-            php-version: '8.1'
+            php-version: '8.3'
             typo3-version: '^12.4'
           - db-version: '8'
             php-version: '8.2'
-            typo3-version: '^12.4'
+            typo3-version: '13.0'
+          - db-version: '8'
+            php-version: '8.3'
+            typo3-version: '13.0'
     steps:
       - uses: actions/checkout@v3
 
@@ -164,18 +160,16 @@ jobs:
     strategy:
       matrix:
         include:
-          - php-version: '7.4'
-            typo3-version: '^11.5'
-          - php-version: '8.0'
-            typo3-version: '^11.5'
-          - php-version: '8.1'
-            typo3-version: '^11.5'
-          - php-version: '8.2'
-            typo3-version: '^11.5'
           - php-version: '8.1'
             typo3-version: '^12.4'
           - php-version: '8.2'
             typo3-version: '^12.4'
+          - php-version: '8.3'
+            typo3-version: '^12.4'
+          - php-version: '8.2'
+            typo3-version: '13.0'
+          - php-version: '8.3'
+            typo3-version: '13.0'
     steps:
       - uses: actions/checkout@v3
 

Classes/Command/UpdateDataCommand.php

@@ -31,15 +31,9 @@
 
 class UpdateDataCommand extends Command
 {
-    /**
-     * @var Pageview
-     */
-    private $repository;
-
-    public function __construct(Pageview $repository)
-    {
-        $this->repository = $repository;
-
+    public function __construct(
+        private readonly Pageview $repository
+    ) {
         parent::__construct();
     }
 

Classes/Dashboard/Provider/NewestPageviews.php

@@ -23,42 +23,22 @@
 
 namespace DanielSiepmann\Tracking\Dashboard\Provider;
 
+use Exception;
 use TYPO3\CMS\Core\Database\Connection;
 use TYPO3\CMS\Core\Database\Query\QueryBuilder;
 use TYPO3\CMS\Dashboard\Widgets\ListDataProviderInterface;
 
 class NewestPageviews implements ListDataProviderInterface
 {
     /**
-     * @var QueryBuilder
+     * @param int[] $languageLimitation
      */
-    private $queryBuilder;
-
-    /**
-     * @var int
-     */
-    private $maxResults;
-
-    /**
-     * @var array
-     */
-    private $pagesToExclude;
-
-    /**
-     * @var array<int>
-     */
-    private $languageLimitation;
-
     public function __construct(
-        QueryBuilder $queryBuilder,
-        int $maxResults = 6,
-        array $pagesToExclude = [],
-        array $languageLimitation = []
+        private readonly QueryBuilder $queryBuilder,
+        private readonly int $maxResults = 6,
+        private readonly array $pagesToExclude = [],
+        private readonly array $languageLimitation = []
     ) {
-        $this->queryBuilder = $queryBuilder;
-        $this->maxResults = $maxResults;
-        $this->pagesToExclude = $pagesToExclude;
-        $this->languageLimitation = $languageLimitation;
     }
 
     public function getItems(): array
@@ -98,10 +78,13 @@ public function getItems(): array
             $this->queryBuilder->where(...$constraints);
         }
 
-        $items = $this->queryBuilder->execute()->fetchAll();
+        $items = $this->queryBuilder->executeQuery()->fetchAllAssociative();
         foreach ($items as $item) {
-            if (is_array($item) === false) {
-                continue;
+            if (is_string($item['url']) === false) {
+                throw new Exception('url of item was not string: ' . var_export($item['url'], true), 1707327319);
+            }
+            if (is_string($item['user_agent']) === false) {
+                throw new Exception('user_agent of item was not string: ' . var_export($item['user_agent'], true), 1707327344);
             }
 
             $preparedItems[] = sprintf(

Classes/Dashboard/Provider/PageviewsPerDay.php

@@ -24,6 +24,7 @@
 namespace DanielSiepmann\Tracking\Dashboard\Provider;
 
 use DanielSiepmann\Tracking\Extension;
+use Doctrine\DBAL\Platforms\SqlitePlatform;
 use TYPO3\CMS\Core\Database\Connection;
 use TYPO3\CMS\Core\Database\Query\QueryBuilder;
 use TYPO3\CMS\Core\Localization\LanguageService;
@@ -32,49 +33,20 @@
 
 class PageviewsPerDay implements ChartDataProviderInterface
 {
-    /**
-     * @var LanguageService
-     */
-    private $languageService;
-
-    /**
-     * @var QueryBuilder
-     */
-    private $queryBuilder;
-
-    /**
-     * @var int
-     */
-    private $days;
+    private LanguageService $languageService;
 
     /**
-     * @var array<int>
+     * @param int[] $pagesToExclude
+     * @param int[] $languageLimitation
      */
-    private $pagesToExclude;
-
-    /**
-     * @var string
-     */
-    private $dateFormat;
-
-    /**
-     * @var array<int>
-     */
-    private $languageLimitation;
-
     public function __construct(
-        QueryBuilder $queryBuilder,
-        int $days = 31,
-        array $pagesToExclude = [],
-        array $languageLimitation = [],
-        string $dateFormat = 'Y-m-d'
+        private readonly QueryBuilder $queryBuilder,
+        private readonly int $days = 31,
+        private readonly array $pagesToExclude = [],
+        private readonly array $languageLimitation = [],
+        private readonly string $dateFormat = 'Y-m-d'
     ) {
         $this->languageService = $GLOBALS['LANG'];
-        $this->queryBuilder = $queryBuilder;
-        $this->days = $days;
-        $this->pagesToExclude = $pagesToExclude;
-        $this->languageLimitation = $languageLimitation;
-        $this->dateFormat = $dateFormat;
     }
 
     public function getChartData(): array
@@ -102,16 +74,16 @@ private function calculateData(): array
         $data = [];
 
         for ($daysBefore = $this->days; $daysBefore >= 0; $daysBefore--) {
-            $label = date($this->dateFormat, (int)strtotime('-' . $daysBefore . ' day'));
+            $label = date($this->dateFormat, (int) strtotime('-' . $daysBefore . ' day'));
             $labels[$label] = $label;
             $data[$label] = 0;
         }
 
-        $start = (int)strtotime('-' . $this->days . ' day 0:00:00');
-        $end = (int)strtotime('tomorrow midnight');
+        $start = (int) strtotime('-' . $this->days . ' day 0:00:00');
+        $end = (int) strtotime('tomorrow midnight');
 
         foreach ($this->getPageviewsInPeriod($start, $end) as $day) {
-            $data[$day['label']] = (int)$day['count'];
+            $data[$day['label']] = (int) $day['count'];
         }
 
         return [
@@ -154,13 +126,15 @@ private function getPageviewsInPeriod(int $start, int $end): array
             ->groupBy('label')
             ->orderBy('label', 'ASC')
         ;
-
-        if ($this->queryBuilder->getConnection()->getDatabasePlatform()->getName() === 'sqlite') {
+        if (
+            (class_exists(SqlitePlatform::class) && $this->queryBuilder->getConnection()->getDatabasePlatform() instanceof SqlitePlatform)
+            || (method_exists($this->queryBuilder->getConnection()->getDatabasePlatform(), 'getName') && $this->queryBuilder->getConnection()->getDatabasePlatform()->getName() === 'sqlite')
+        ) {
             $this->queryBuilder->addSelectLiteral('date(crdate, "unixepoch") as "label"');
         } else {
             $this->queryBuilder->addSelectLiteral('FROM_UNIXTIME(crdate, "%Y-%m-%d") as "label"');
         }
 
-        return $this->queryBuilder->execute()->fetchAll();
+        return $this->queryBuilder->executeQuery()->fetchAllAssociative();
     }
 }

Classes/Dashboard/Provider/PageviewsPerOperatingSystem.php

@@ -23,6 +23,7 @@
 
 namespace DanielSiepmann\Tracking\Dashboard\Provider;
 
+use Exception;
 use TYPO3\CMS\Core\Database\Connection;
 use TYPO3\CMS\Core\Database\Query\QueryBuilder;
 use TYPO3\CMS\Dashboard\WidgetApi;
@@ -31,35 +32,14 @@
 class PageviewsPerOperatingSystem implements ChartDataProviderInterface
 {
     /**
-     * @var QueryBuilder
+     * @param int[] $languageLimitation
      */
-    private $queryBuilder;
-
-    /**
-     * @var int
-     */
-    private $days;
-
-    /**
-     * @var int
-     */
-    private $maxResults;
-
-    /**
-     * @var array<int>
-     */
-    private $languageLimitation;
-
     public function __construct(
-        QueryBuilder $queryBuilder,
-        int $days = 31,
-        int $maxResults = 6,
-        array $languageLimitation = []
+        private readonly QueryBuilder $queryBuilder,
+        private readonly int $days = 31,
+        private readonly int $maxResults = 6,
+        private readonly array $languageLimitation = []
     ) {
-        $this->queryBuilder = $queryBuilder;
-        $this->days = $days;
-        $this->maxResults = $maxResults;
-        $this->languageLimitation = $languageLimitation;
     }
 
     public function getChartData(): array
@@ -112,13 +92,13 @@ private function getPageViewsPerPage(): array
             ->orderBy('total', 'desc')
             ->addOrderBy('operating_system', 'asc')
             ->setMaxResults($this->maxResults)
-            ->execute()
-            ->fetchAll()
+            ->executeQuery()
+            ->fetchAllAssociative()
         ;
 
         foreach ($result as $row) {
-            if (is_array($row) === false) {
-                continue;
+            if (is_string($row['operating_system']) === false) {
+                throw new Exception('operating_system of row was not string: ' . var_export($row['operating_system'], true), 1707326866);
             }
 
             $labels[] = mb_strimwidth($row['operating_system'], 0, 50, '…');