Eventlist und Eventreader: Bei Zeitraum Leerzeichen vom Bindestrich entfernen?

[Jens Pielawa]

Hallo an alle,

ich habe ein typographisches Problem: wenn man ein Event erstellt, das einen Zeitraum enthält, also z. B. von 20:00 bsi 22:00 Uhr, wird mit Contao folgendes ausgegeben:

20:00 – 22:00

Das ist typographisch leider ein falscher Bindestrich oder "Bis-Strich". Ich möchte die Leerzeichen entfernen, denn erst dann hat der Bindestrich die richtige Bedeutung "bis". Mit Leerzeichen hat er eine Bedeutung "gegen", etwa bei Spielen, z. B. "Hamburger SV – Hannover 96". Siehe https://de.wikipedia.org/wiki/Halbgeviertstrich
"Laut Duden stehen vor und nach einem Bis-Strich kein Leerzeichen"

In den Templates findet sich die PHP-Funktion

<?php if ($this->time): ?><?= $this->time ?><?php endif; ?></time>

Wo aber werden dann im weiteren Code die zwei Leerzeichen und der Bindestrich hinzugefügt? Wie gesagt sind die zwei Leerzeichen überflüssig, und ich möchte sie loswerden. Und wann bessert Contao im Core nach?

[Spooky]

Und wann bessert Contao im Core nach?

Wenn du das nur hier im Forum postest gar nicht . Du solltest dafür ein Ticket auf GitHub erstellen.

[Spooky]

Hier wird der String für $this->time erstellt: https://github.com/contao/core/blob/....php#L146-L162

Du könntest dir im Template folgenden Code einfügen:

<?php

global $objPage;
$span = \Calendar::calculateSpan($this->begin, $this->end);

if ($this->addTime)
{
    if ($span > 0)
    {
        $this->date = \Date::parse($objPage->datimFormat, $this->begin) . '&ndash;' . \Date::parse($objPage->datimFormat, $this->end);
    }
    elseif ($this->begin == $this->end)
    {
        $this->time = \Date::parse($objPage->timeFormat, $this->begin); // not really necessary
    }
    else
    {
        $this->time = \Date::parse($objPage->timeFormat, $this->begin) . '&ndash;' . \Date::parse($objPage->timeFormat, $this->end);
    }
}

?>

[Jens Pielawa]

OK, so weit, so gut. Deinen Code habe ich zwar eingefügt, aber dann streikte der Eventreader. Fürs Erste habe ich die Stellen mit "Leerzeichen Bindestrich Leerzeichen" mit dem "&ndash" (offenbar nicht updatesicher) ersetzt.
Bleibt nur noch die Eventliste, hier schleppt sich das Problem auch noch durch, und ich finde die Stelle zum Ausbessern im PHP nicht. Hier wird sehr viel mit Datumsangaben sortiert.
Beispiel meiner Baustellen-Seite: http://neu.hudh.de/termine.html
Ich glaube, ich mach mal dieses Ticket auf. Das ist wirklich verbesserungswürdig.

[Spooky]

OK, so weit, so gut. Deinen Code habe ich zwar eingefügt, aber dann streikte der Eventreader.

Ja, der Code von mir war ungetestet

Bleibt nur noch die Eventliste, hier schleppt sich das Problem auch noch durch, und ich finde die Stelle zum Ausbessern im PHP nicht.

Alle Vorkommen:

• /system/modules/calendar/classes/Events.php#L222
• /system/modules/calendar/classes/Events.php#L232
• /system/modules/calendar/classes/Events.php#L240
• /system/modules/calendar/classes/Calendar.php#L412
• /system/modules/calendar/classes/Calendar.php#L416
• /system/modules/calendar/dca/tl_calendar_events.php#L736 (nur Backend)
• /system/modules/calendar/dca/tl_calendar_events.php#L744 (nur Backend)
• /system/modules/calendar/modules/ModuleEventReader.php#L143
• /system/modules/calendar/modules/ModuleEventReader.php#L152
• /system/modules/calendar/modules/ModuleEventReader.php#L160

Ich glaube, ich mach mal dieses Ticket auf. Das ist wirklich verbesserungswürdig.

Am besten wäre es wohl, wenn das über $GLOBALS['TL_LANG'] konfigurierbar wäre. Also zB. dann:

\Date::parse($objPage->timeFormat, $intStartTime) . $GLOBALS['TL_LANG']['MSC']['timeSpanDash'] . \Date::parse($objPage->timeFormat, $intEndTime); 

[Jens Pielawa]

Danke für's Finden, Spooky. Ich habe mein erstes Ticket erstellt:
https://github.com/contao/core/issues/8690
Mal sehen, was rauskommt...

[Exzentrik]

Also falls es sich hier um ein dringendes Problem handelt, das schnellstmöglich gelöst werden muss:

Ich habe bei mir im Template für die Eventanzeige

<?= $this->time ?>

ersetzt durch

<?= $this->startTime ?>-<?= $this->endTime ?>

der gibt dann natürlich die unformatierten timestamps aus, aber die parseDate-Sache habe ich jetzt aus übersichtlichkeits-Gründen weggelassen.

[Jens Pielawa]

Wurde gefixed in
https://github.com/contao/core/commi...17aabd0f3d0e43
Danke sehr!