Nachrichten-Artikel wird bereits vor dem Speichern veröffentlicht

[ghost]

Issue by @d3vm4t
June 26th, 2017, 19:16 GMT

Folgendes Problem (Contao 3.5.2x):
Wird in einem Nachrichtenarchiv ein neuer Artikel erstellt und unter "Veröffentlichung" die Checkbox "Beitrag veröffentlichen" gesetzt, dann wird der Artikel sofort veröffentlicht, ohne dass zuvor auf Speichern geklickt wird. Man sieht also im Frontend den Artikel bereits veröffentlicht nach dem die Checkbox angeklickt wurde (keine Preview-Ansicht, sondern auch auf einem beliebigen anderen Gerät sichtbar).
Das ist natürlich nicht gut, falls man den Artikel erst ab einem späteren Zeitpunkt veröffentlichen will. Aber damit das Feld "Anzeigen ab" erscheint, muss ja erstmal die Checkbox gesetzt werden.

Können andere das so nachvollziehen oder ist das sogar das gewünschte Verhalten?
Besten Dank im Voraus.

[ghost]

Comment by @fritzmg
June 26th, 2017, 19:33 GMT

Kann ich nachvollziehen. Früher hatte man das Problem noch nicht, da das Anzeigen ab/bis keine Subpalette war.

[ghost]

Comment by @leofeyer
June 27th, 2017, 16:51 GMT

@contao/developers /cc

[ghost]

Comment by @aschempp
June 27th, 2017, 17:31 GMT

Am einfachsten wäre es wohl Start & Stop wieder aus einer Subpalette raus zu nehmen…

[ghost]

Comment by @Toflar
June 27th, 2017, 17:57 GMT

Nö, die findByPublished müsste einfach auch tstamp>0 machen im Model?

[m-vo]

I know the behavior that changing subpalettes forces the dataset to be stored has been like this for a long time. I really don't like it, because it leads to many unintuitive cases including this one but also premature storing of invalid data,...

What exactly was the reason for this design? Simplicity in the implementation? And why don't we change it? :-)

[agoat]

Um das Problem des 'Zwischenspeicherns' bei den SubPaletten generell zu lösen, müsste tatsächlich alles erstmal zwischen gespeichert werden und erst bei Betätigung von 'Speichern' in die Datenbank geschrieben werden.

Für das 'Veröffentlichen'-Problem ist aber erstmal die von @Toflar vorgeschlagene Lösung nahe liegend.

[leofeyer]

Leider löst @Toflar's Vorschlag das Problem nur, wenn der Datensatz noch nicht gespeichert wurde; sonst ist tstamp nämlich schon gesetzt. Und in der Regel ist der Workflow

• News anlegen
• Speichern & bearbeiten
• Inhaltselemente anlegen
• FE-Preview
• Veröffentlichen

Und bereits nach "Speichern & bearbeiten" ist tstamp nicht mehr 0. 😢

[Toflar]

Das ist imho kein Problem. Wenn man die News ohne Inhalt veröffentlichen will, dann tut man das eben. Das Problem ist, dass der News-Beitrag angezeigt wird, sobald man die Veröffentlichen-Checkbox anhakt, welche den Datensatz speichert. Imho speichert sie ihn aber eben mit tstamp=0 bis effektiv ein Speicher-Vorgang ausgelöst wird.

[leofeyer]

Niemand will die News ohne Inhalt veröffentlichen. Das ist ja gerade das Problem. 😄

[Toflar]

Ich glaube nicht, dass das das Problem ist. Das war bisher schon immer so und kann auch durchaus gewollt sein.

[fritzmg]

Das Problem ist, dass du die Option "Anzeigen ab" nur dann benutzen kannst, wenn du die Option "Veröffentlichen" aktivierst. Sobald man das aber macht, ist der Artikel im Frontend bereits veröffentlicht - obwohl man ihn aber eigentlich mit "Anzeigen ab" später veröffentlichen lassen will. Die Überprüfung auf tstamp ist daher nicht ausreichend.

[leofeyer]

So habe ich das Ticket auch verstanden.

[d3vm4t]

So habe ich das Ticket auch verstanden.

So war es auch gemeint. @fritzmg hat das nochmal gut zusammengefasst.

[leofeyer]

Also, nachdem @Toflar's Vorschlag nur einen Teil des Problems löst und @m-vo's Vorschlag weder auf die Schnelle noch ohne BC-Breaks umgesetzt werden kann, nehmen wir wohl @aschempp's Lösung. 😄

[leofeyer]

Hier ist das Ticket zur Änderung: contao/core#7148

[aschempp]

@Toflar löst das Problem leider überhaupt nicht. Man kann ja auch einer "fertigen" News ein Startdatum in der Zukunft geben wollen, dann wären diese beim Klick auf "Veröffentlichen" auch für einen Moment sichtbar.

[d3vm4t]

Wie sind denn die Subpaletten implementiert? Lässt sich die Checkbox nicht ausgliedern, sodass der Wert der Checkbox erst beim Klicken des Speichern-Buttons in die Datenbank geschrieben wird? Die Optionen "Anzeigen ab" und "Anzeigen bis" können ja weiterhin über ein JS Event getoggelt werden.

[leofeyer]

Geändert in 03fbdf6.