Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

C 4.8.1 {{image::*}} im Newsletter Template erzeugt keine Bilder mehr #680

Closed
Kahmoon opened this issue Aug 28, 2019 · 7 comments
Closed
Labels

Comments

@Kahmoon
Copy link

Kahmoon commented Aug 28, 2019

Affected version(s)
Contao 4.8.1

Description
Ich verwende im Newsletter Template ein Logo via replaceInsertTags

<?= $this->replaceInsertTags( '{{image::fa971c31-7300-11e8-8b83-74d02bcb043d?width=450&height=150&mode=crop}}' ); ?>

Bisher hat das auch super funktioniert. Seit Contao 4.8.1 ist das Bild im Newsletter nicht mehr vorhanden.

How to reproduce / Workaround
Erzeuge ich via Inhaltselement "Text" und dem Inserttag selbst {{image::fa971c31-7300-11e8-8b83-74d02bcb043d?width=450&height=150&mode=crop}} das Bild und rufe es im Frontend einmalig auf, funktioniert auch die Grafik im Newsletter.

Ändere ich die Größeneinstellung im Template ist die Grafik wieder nicht verfügbar. Erst wenn ich die Grafik mit den neuen Einstellungen wieder einmalig im Frontend via Inserttag aufrufe, funktioniert sie auch wieder im Newsletter.

Ich gehe also davon aus das {{image::*}} im Template keine Bildversionen mehr erzeugt. Vermutlicher Zusammenhand mit Deferred Image Creation.

VG Frank

@fritzmg
Copy link
Contributor

fritzmg commented Aug 28, 2019

Ich gehe also davon aus das replaceInsertTags im Template keine Bildversionen mehr erzeugt. Vermutlicher Zusammenhand mit Deferred Image Creation.

replaceInsertTags hat damit nichts zu tun. Generell werden keine Bilder mehr erzeugt - sondern erst, wenn auf das Bild zugegriffen wird. Das ist aber in diesem Fall ein Problem - weil das Bild, wahrscheinlich? - ansonsten von Contao direkt integriert worden wäre - anstatt extern verlinkt.

@Kahmoon
Copy link
Author

Kahmoon commented Aug 28, 2019

Da es ein fixes Element in jedem Newsletter ist (was sicher bei vielen der Fall ist), habe ich das Logo im Template platziert um es nicht immer wieder manuell via Newsletter einzufügen. Das macht es dem Kunden auch leichter und reduziert Fehlerquellen :D

@Kahmoon Kahmoon changed the title C 4.8.1 replaceInsertTags im Template erzeugt keine Bilder mehr C 4.8.1 {{image::*}} im Template erzeugt keine Bilder mehr Aug 28, 2019
@Kahmoon Kahmoon changed the title C 4.8.1 {{image::*}} im Template erzeugt keine Bilder mehr C 4.8.1 {{image::*}} im Newsletter Template erzeugt keine Bilder mehr Aug 28, 2019
@ausi
Copy link
Member

ausi commented Aug 28, 2019

Ich denke die E-Mail-Klasse muss ensprechend angepasst werden damit das Bild erzeugt wird bevor es im E-Mail inkludiert wird:

$arrCid[$src] = $this->objMessage->embed(\Swift_EmbeddedFile::fromPath($this->strImageDir . $src));

@leofeyer
Copy link
Member

Wieso verwendest Du denn <?= $this->replaceInsertTag('{{image}}') ?> in einem PHP-Template? Dort kannst Du doch direkt mit der Image-Factory arbeiten.

@leofeyer leofeyer added the bug label Aug 28, 2019
@leofeyer leofeyer added this to the 4.8 milestone Aug 28, 2019
@Kahmoon
Copy link
Author

Kahmoon commented Aug 28, 2019

Was ist die Image-Factory? Ich hatte das dort schon seit 4.6 via mail_default.html5 so gemacht. Einen anderen Weg kenne ich leider nicht.

Wenn es hierfür einen besseren Weg gibt...gerne!

@fritzmg
Copy link
Contributor

fritzmg commented Aug 28, 2019

Wieso verwendest Du denn <?= $this->replaceInsertTag('{{image}}') ?> in einem PHP-Template? Dort kannst Du doch direkt mit der Image-Factory arbeiten.

Würde das Problem aber auch nicht beheben.

@leofeyer
Copy link
Member

Siehe #693.

@leofeyer leofeyer removed this from the 4.8 milestone Aug 30, 2019
leofeyer pushed a commit that referenced this issue Sep 5, 2019
Description
-----------

Fixes #680

Better name for `$objFile->generateIfDeferredImage()`?
`createDeferredImage()`, `generateDeferredImage()`, `createImageIfDeferred()` ?

Commits
-------

7e750d8 Fix deferred images in HTML emails
59dcfd1 Rename to createDeferredImage()
e4f807d Rename to createIfDeferred()
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 18, 2024
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

4 participants