Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

[3.5.35] Newsletter-Verteiler bearbeiten: Änderungsdatum fehlerhaft dargestellt #8883

Closed
Mynyx opened this issue May 28, 2018 · 5 comments
Closed
Assignees
Labels
Milestone

Comments

@Mynyx
Copy link
Contributor

Mynyx commented May 28, 2018

Bei einem Newsletter-Verteiler ist als tstamp 1423317420 gespeichert. In der Detailansicht des Verteilers (Blaues Icon) wird als Änderungsdatum korrekt 07.02.2015 14:57 angezeigt.

In der Ansicht Verteiler bearbeiten (Stift Icon) wird stattdessen jedoch als Änderungsdatum fehlerhaft 02.05.2015 15:47 angegeben.

@leofeyer
Copy link
Member

Funktioniert in der Online-Demo einwandfrei.

@Mynyx
Copy link
Contributor Author

Mynyx commented Jun 28, 2018

Ich habe mir das nun noch einmal genauer angesehen. In der Ansicht Verteiler bearbeiten (Stift Icon) wird das Änderungsdatum offenbar über folgende SQL-Abfrage ermittelt: SELECT MAX(tstamp) AS tstamp FROM tl_newsletter WHERE pid='1'. Dies ergibt als Ergebnis das letzte Änderungsdatum der Newsletter in dem Newsletter-Verteiler. Dadurch unterschiedet sich das angezeigte Datum dann von dem Änderungsdatum des Newsletter-Verteilers selbst.
Da das Änderungsdatum in der Ansicht Verteiler bearbeiten (Stift Icon) jedoch zwischen den anderen Eigenschaften des Newsletter-Verteilers angezeigt wird, ist es nicht nachvollziehbar, warum dieses nicht auf den Newsletter-Verteiler sondern auf die Newsletter in diesem bezogen ist.

@leofeyer leofeyer reopened this Sep 2, 2018
@leofeyer leofeyer added defect and removed invalid labels Sep 2, 2018
@leofeyer leofeyer added this to the 4.4.24 milestone Sep 2, 2018
@leofeyer leofeyer removed this from the 4.4.24 milestone Sep 3, 2018
@leofeyer
Copy link
Member

leofeyer commented Sep 3, 2018

Es kommt halt darauf an, wie man "letzte Änderung" definiert.

Nehmen wir als Beispiel einen Artikel in Contao: Wann war die letzte Änderung eines Artikels? Wenn etwas an den Metadaten geändert wurde oder wenn etwas am Inhalt geändert wurde?

Ich gebe Dir aber Recht, dass der Zeitstempel im Parent-View die letzte Änderung der Metadaten anzeigen sollte und nicht des Inhalts.

@contao/developers Seht ihr das genauso?

@Mynyx
Copy link
Contributor Author

Mynyx commented Sep 5, 2018

Um dies eindeutiger zu gestalten, könnte die Bezeichnung zu "Änderungsdatum Einstellungen" geändert werden.

@leofeyer leofeyer added this to the 4.4.27 milestone Sep 27, 2018
@leofeyer leofeyer self-assigned this Oct 30, 2018
@leofeyer
Copy link
Member

Behoben in contao/contao@48a1e8e.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants