[Contao 3] Teaserbild in Event wird nicht korrekt in der DB gesichert. #5002

Closed
markusthomasde opened this Issue Nov 7, 2012 · 12 comments

Projects

None yet

6 participants

@markusthomasde

Wenn ich ein Teaserbild in einem Event hinzufüge, dann bleibt in der Datenbank das Feld addImage leer.

@leofeyer
Member
leofeyer commented Nov 7, 2012

Wie lässt sich das reproduzieren?

@markusthomasde

Ich habe im Eventteaser ein Bild hinzugefügt und dann mal die Teaserausgabe mit showTemplateVars(); ?> überprüft.

@Christoph1987

Ich habe das gleiche Problem. Bei singleSRC wird nur eine Zahl ausgegeben.

@JohannesFerner

@Christoph1987 du musst bei Contao3 den Pfad + Metadaten dann erst über das neue FilesModel rausholen. Die Zahl referenziert auf den Eintrag in der Tabelle zu den Files.

@Christoph1987

Ok, wäre es aber nicht sinnvoll, das Bild standardmäßig im Template auszugeben, so dass das nicht von jedem Benutzer selbst gemacht werden muss? Ich finde es ansonsten etwas verwirrend, dass man die Einstellungen für die Bildgrößen etc. im Modul festlegen kann, aber es nicht ausgegeben wird. Im Newsmodul ist es ja auch standardmäßig enthalten.

@leofeyer
Member

Behoben in e4ae49f.

@leofeyer leofeyer closed this Nov 13, 2012
@Anke
Anke commented Sep 17, 2014

Die Redakteure sind verwirrt und ich bin es auch. In Contao 3.3.5 zeigt event_teaser.html5 immer noch kein Bild an, obwohl man im Teaser eines auswählen und in Teaser und Modul Einstellungen dazu vornehmen kann. Wurde das doch nicht geändert? (siehe auch #7001)

@Aybee
Contributor
Aybee commented Sep 18, 2014

Die Variablen sind vorhanden - print_r($this->arrData). Aber das Template event_teaser berücksichtigt die Ausgabe des Bildes nicht. Sollte mal überarbeitet werden damit sich niemand mehr wundert. Es ist ja einfacher aus einem Template etwas zu entfernen als etwas hinzuzufügen. Deshalb finde ich auch, dass event_teaser auch das Bild ausgeben sollte falls vorhanden.

@leofeyer
Member

Nur weil _teaser im Dateinamen ist, muss das Template nicht automatisch auch das Teaser-Bild ausgeben. Das Template news_short gibt auch nur den Teaser-Text ohne Bild aus, das Template news_latest den Teaser-Text mit Bild.

@Anke
Anke commented Sep 19, 2014

Das Teaser-Template bietet sich an, wenn man die Teaser auflisten will (news_teaser gibt es ja nicht). Und da die Teaser mit einem Bild versehen werden können, erwartet man eigentlich auch die entsprechende Ausgabe - vor allem, weil hier öfter ein "behoben" steht bzw. "Correctly add event images to the templates (see #5002 )".

Wie Aybee sagt, ist es einfacher, etwas aus einem Template zu entfernen, als (als Designer mit rudimentären Programmierkenntnissen) nach dem benötigten Code zu suchen, damit das Bild den Positionierungs- und Größeneinstellungen entsprechend platziert wird.

@Aybee
Contributor
Aybee commented Sep 23, 2014

Sowohl News als auch Events bieten unterhalb des Teasers die Option ein Bild hinzuzufügen.

Wenn man in diesen Templates

  • event_full
  • event_list
  • event_teaser
  • event_upcoming
  • news_full
  • news_latest
  • news_short
  • news_simple

nach $this->addImage oder $this->src sucht, findet man es lediglich in news_latest.

Vielleicht sollte es auch ein Event Template geben mit welchem das Bild ausgegeben wird, so dass man sich den Codeblock von dort kopieren kann. Und damit wenigsten ein Template dabei ist, bei welchem das Bild ausgegeben wird. Dann könnte man die Templates durchprobieren bis man es gefunden hat.

@leofeyer leofeyer added a commit to contao/news-bundle that referenced this issue May 12, 2015
@leofeyer leofeyer Show the teaser image if applicable (see contao/core#5002). 7e5579c
@leofeyer leofeyer added a commit to contao/calendar-bundle that referenced this issue May 12, 2015
@leofeyer leofeyer Show the teaser image if applicable (see contao/core#5002). e311eec
@leofeyer leofeyer added feature and removed defect labels May 12, 2015
@leofeyer leofeyer modified the milestone: 4.0.0, 3.0.1 May 12, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment