Skip to content

Commit

Permalink
Make time zone for slugs configurable
Browse files Browse the repository at this point in the history
Slugs of dates are generated during import.
This might lead to confusion if the time zone differs from frontend.
Therefore the time zone is now configurable to allow adjustments for the
actual website.

A new interface `TimeZoneProviderInterface` is provided which can be re
configured to a different implementation.

The default implementation will use TYPO3s `SYS.phpTimeZone` setting,
with fall back to `date_default_timezone_get()` call.
That way it should be useful for most systems out of the box.

Resolves: #11345
  • Loading branch information
d-s-codappix committed Sep 9, 2024
1 parent 2d6c13e commit 6b17b82
Show file tree
Hide file tree
Showing 6 changed files with 160 additions and 3 deletions.
31 changes: 31 additions & 0 deletions Classes/Domain/TimeZone/TimeZoneProviderInterface.php
Original file line number Diff line number Diff line change
@@ -0,0 +1,31 @@
<?php

declare(strict_types=1);

/*
* Copyright (C) 2024 Daniel Siepmann <daniel.siepmann@codappix.com>
*
* This program is free software; you can redistribute it and/or
* modify it under the terms of the GNU General Public License
* as published by the Free Software Foundation; either version 2
* of the License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with this program; if not, write to the Free Software
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
* 02110-1301, USA.
*/

namespace WerkraumMedia\Events\Domain\TimeZone;

use DateTimeZone;

interface TimeZoneProviderInterface
{
public function getTimeZone(): DateTimeZone;
}
37 changes: 37 additions & 0 deletions Classes/Domain/TimeZone/Typo3ConfiguredTimeZoneProvider.php
Original file line number Diff line number Diff line change
@@ -0,0 +1,37 @@
<?php

declare(strict_types=1);

/*
* Copyright (C) 2024 Daniel Siepmann <daniel.siepmann@codappix.com>
*
* This program is free software; you can redistribute it and/or
* modify it under the terms of the GNU General Public License
* as published by the Free Software Foundation; either version 2
* of the License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with this program; if not, write to the Free Software
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
* 02110-1301, USA.
*/

namespace WerkraumMedia\Events\Domain\TimeZone;

use DateTimeZone;

final class Typo3ConfiguredTimeZoneProvider implements TimeZoneProviderInterface
{
public function getTimeZone(): DateTimeZone
{
$timeZone = $GLOBALS['TYPO3_CONF_VARS']['SYS']['phpTimeZone'] ?? null;
$timeZone = $timeZone ?: date_default_timezone_get();

return new DateTimeZone($timeZone);
}
}
8 changes: 6 additions & 2 deletions Classes/Service/DestinationDataImportService/Slugger/Date.php
Original file line number Diff line number Diff line change
Expand Up @@ -24,18 +24,22 @@
namespace WerkraumMedia\Events\Service\DestinationDataImportService\Slugger;

use DateTimeImmutable;
use DateTimeZone;
use TYPO3\CMS\Core\Database\ConnectionPool;
use WerkraumMedia\Events\Domain\TimeZone\TimeZoneProviderInterface;

final class Date implements SluggerType
{
public function __construct(
private readonly ConnectionPool $connectionPool
private readonly ConnectionPool $connectionPool,
private readonly TimeZoneProviderInterface $timeZoneProvider,
) {
}

public function prepareRecordForSlugGeneration(array $record): array
{
$start = new DateTimeImmutable('@' . $record['start']);
$start = new DateTimeImmutable('@' . $record['start'], new DateTimeZone('UTC'));
$start = $start->setTimezone($this->timeZoneProvider->getTimeZone());

$record['event-title'] = $this->getEventTitle((int)$record['event']);
$record['start'] = $start->format('Y-m-d');
Expand Down
39 changes: 39 additions & 0 deletions Documentation/Changelog/4.2.1.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1,39 @@
4.3.0
=====

Breaking
--------

Nothing

Features
--------

Nothing

Fixes
-----

* Make time zone for slugs configurable.

Slugs of dates are generated during import.
This might lead to confusion if the time zone differs from frontend.
Therefore the time zone is now configurable to allow adjustments for the actual
website.

A new interface `TimeZoneProviderInterface` is provided which can be re configured to a
different implementation.

The default implementation will use TYPO3s `SYS.phpTimeZone` setting, with fall
back to `date_default_timezone_get()` call.
That way it should be useful for most systems out of the box.

Tasks
-----

Nothing

Deprecation
-----------

Nothing
Original file line number Diff line number Diff line change
@@ -0,0 +1,46 @@
<?php

declare(strict_types=1);

namespace WerkraumMedia\Events\Tests\Functional\Import\DestinationDataTest;

use DateTimeImmutable;
use DateTimeZone;
use GuzzleHttp\Psr7\Response;
use PHPUnit\Framework\Attributes\Test;
use PHPUnit\Framework\Attributes\TestDox;
use TYPO3\CMS\Core\Utility\ArrayUtility;

#[TestDox('DestinationData import')]
class ImportsWithSystemConfiguredTimeZoneTest extends AbstractTestCase
{
public function setUp(): void
{
ArrayUtility::mergeRecursiveWithOverrule($this->configurationToUseInTestInstance, [
'SYS' => [
'phpTimeZone' => 'Europe/Berlin',
],
]);

parent::setUp();

$this->setUpConfiguration([
'restUrl = https://example.com/some-path/',
]);
$this->importPHPDataSet(__DIR__ . '/Fixtures/Database/DefaultImportConfiguration.php');
$this->setDateAspect(new DateTimeImmutable('2022-07-13', new DateTimeZone('UTC')));
}

#[Test]
public function withTYPO3TimeZone(): void
{
$this->setUpResponses([new Response(200, [], file_get_contents(__DIR__ . '/Fixtures/ResponseWithSingleDate.json') ?: '')]);

$this->executeCommand();

$dates = $this->getAllRecords('tx_events_domain_model_date');
self::assertSame('kurzfuehrung-historische-altstadt-2022-07-13t15-00-00', $dates[0]['slug']);

$this->assertEmptyLog();
}
}
2 changes: 1 addition & 1 deletion ext_emconf.php
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
'author' => 'Dirk Koritnik, Daniel Siepmann',
'author_email' => 'koritnik@werkraum-media.de, coding@daniel-siepmann.de',
'state' => 'stable',
'version' => '4.2.0',
'version' => '4.2.1',
'constraints' => [
'depends' => [
'typo3' => '',
Expand Down

0 comments on commit 6b17b82

Please sign in to comment.