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

Verwenden eines Insert-Tags beim Einfügen von Bildern im TinyMCE #5965

Closed
planepix opened this issue Jul 10, 2013 · 30 comments
Closed

Verwenden eines Insert-Tags beim Einfügen von Bildern im TinyMCE #5965

planepix opened this issue Jul 10, 2013 · 30 comments
Labels
Milestone

Comments

@planepix
Copy link

Fügt man über den TinyMCE Bilder ein, wird der Pfad als relativer Pfad gespeichert.
Führt man dann die Synronisation aus, werden ggf. geänderte Bilder, die im TinyMCE eingefügt wurden, nicht aktualisiert.

Prima wäre, wenn es ähnlich wie beim Einfügen von Seitenlinks auch hier mit einem Insert-Tag arbeiten kann - das ein Einfügen eines Bildes in den TinyMCE daraus einen Insert-Tag generiert. So dass dann anstatt des Pfades z. B. einfach {{file_path::12}} drinnen steht.

@leofeyer
Copy link
Member

Du meinst {{file::23}}?

@joeherold
Copy link

Hallo Leo,
Hallo Thomas,

nein meint er nicht. Der {{file::ID}} Insert Tag ist ja auf den templates Ordner referenziert. Es geht aber darum, dass im TinyMCE, wenn ein Bild/Eine Datei verlinkt wird, mittels TinyMCE ja der wirkliche Pfad gesetzt wird.
Die Frage ist, warum gibt es hier nicht auch einen insert tag wie bei links zu Seiten zum Beispiel ( {{link_url::ID}} ).
Warum wird bei Bildern der Pfad gesetzt, wenn es doch eh das Model für Dateien gibt?

Wenn eine Seite verschoben wird, der alias geändert wird (was weiß ich...) dann wird das ja automatisch durch den insert tag mitgezogen, nicht aber bei verlinkten Bildern, Dateien.

Ich weiß, man sollte dafür nicht den TinyMCE nutzen, aber Kunden machen es dennoch....

@ghost
Copy link

ghost commented Jul 10, 2013

+1

@bekanntmacher
Copy link
Contributor

Duplikat zu: #5856

@hmb
Copy link

hmb commented Jul 23, 2013

Irgendwie ist dieses Thema gerade überall gefragt... Habe eine Erweiterung dafür geschrieben:
https://contao.org/de/extension-list/view/db-file-inserttag.de.html

@Samson1964
Copy link

+1

@bekanntmacher
Copy link
Contributor

@hmb deine Lösung ist nicht zielführend
Es geht darum, dass die per TinyMCE eingefügten Bilder nicht als Pfade gespeichert werden sondern als Inserttag (wie beim Einfügen von Links). > Das TinyMCE-Plungin müsste angepasst werden

@Samson1964
Copy link

Die Erweiterung von hmb ist natürlich nur eine Ersatzlösung. Besser fände ich natürlich auch die Anpassung von TinyMCE.

@hmb
Copy link

hmb commented Jul 29, 2013

@bekanntmacher
Im Prinzip ja, eine zielführende Lösung erfordert zwei Schritte. Ein Insert-Tag, das aus der ID den echten Dateipfad generiert. Und im zweiten Schritt muss der TinyMCE dieses Insert-Tag auch benutzen.

Meine Extension realisiert nur Schritt 1. So gesehen hast Du recht, automatisch geht es nicht. Andererseits, kannst Du im Link oder Bild-Dialog vom TinyMCE das Insert Tag von Hand angeben. Dann hast Du die Funktionalität, dass umbenannte Dateien weiter gefunden werden. Ab da spielt es keine Rolle mehr, ob das Tag über den Dateiauswahldialog oder von Hand ins TinyMCE HTML gekommen ist.

Die Extension ist ein Provisorium, wie Samson1964 gesagt hat. Ich finde ein derartiger Insert-Tag gehört in den Core. Ist der im Core, kann man auch den TinyMCE gleich anpassen, dass ihn benutzt. Deswegen habe ich den Tag umbenennbar gestaltet. Sobald wir wissen, wie der endgültige Core-Tag heißen soll, kann man ihn in meiner Extension genauso nennen. Beim nächsten Update wird dann meine Extension überflüssig und Contato funktioniert trotzdem noch. Sozusagen "invers updatesicher"

Ich habe dieses Ticket als Request dafür verstanden. Bei der Gelegenheit könnte man auch die Datei-URLs im CSS-Editor auf das DB-gestützte Dateisystem umstellen. Ich mach da vielleicht ein Extra Ticket.

@jantheofel
Copy link

Das Thema scheint ja doch einigen Leuten (wie ich finde zurecht) wichtig zu sein. Leo: Passiert hier noch was?

@leofeyer
Copy link
Member

Das Inserttag {{file::UUID}} gibt es bereits seit Contao 3.2.0. Oder was meinst Du?

@joeherold
Copy link

Siehe hier im Screenshot: im TinyMCE wird die UUID nicht verwendet
bildschirmfoto 2014-02-16 um 09 59 18

@joeherold
Copy link

Bei den Links aber schon...., hier wird die page_id verwendet
So hätten wir es auch gerne bei den Bildern...
bildschirmfoto 2014-02-16 um 10 01 46

@joeherold
Copy link

Ich denke sowohl Thomas als auch ich haben das bereits vor 7 Monaten in Post 1 und 3 deutlich beschrieben...

@jantheofel
Copy link

So ist es. Es bringt leider nichts, wenn es das Inserttag gibt, aber der TinyMCE es nicht verwendet.

@c4y
Copy link

c4y commented Feb 16, 2014

+1 für die Lösung mit dem Inserttag.
Ich bringe meinen Kunden aktuell immer noch bei, bloß keine Bilder im Tiny zu verwenden. Und jedesmal die ID zu recherchieren, um den neuen Inserttag in 3.2.0 manuell zu nutzen ist ja auch keine Lösung.

@leofeyer
Copy link
Member

@joeherold Ja, und seitdem wurde auch eine Extension zu dem Thema programmiert, oder?

@joeherold
Copy link

Mir nicht bekannt

@leofeyer
Copy link
Member

Sechster Kommentar von oben in genau diesem Thread: #5965 (comment)

@jantheofel
Copy link

Eine Erweiterung löst das nicht wirklich. Es kann doch nicht sein, dass ich eine Erweiterung benötige um zwei Core-Funktionen (TinyMCE und Inserttags) wirklich benutzen zu können. So was muss für mich auf alle Fälle und ausschließlich im Core realisiert werden.

Weiterhin stellt die oben genannte Extension nur ein weiteres Inserttag zur Verfügung, nimmt jedoch nicht die notwendigen Anpassungen am TinyMCE vor.

@joeherold
Copy link

@leofeyer Diese Erweiterung ändert nix am TinyMCE

@hmb
Copy link

hmb commented Feb 17, 2014

Ja, die Erweiterung war sozusagen nur eine Krücke die ich mir mal gebaut habe, um einen Insert-Tag zu haben, mit dem ich DB-basierte Dateien im TinyMCE einfügen kann. Allerdings mit dem "von Hand recherchieren" der ID. Mehr hat mir damals die Zeit nicht erlaubt. Hat mir als Entwickler auch gereicht.
Nachdem der Tag jetzt offiziell im Core ist, ist die Erweiterung obsolet. Wie ist denn die Stimmung, den neuen Insert-Tag auch im TinyMCE zu verwenden?
Vielleicht hat ja Jemand Zeit und Resourcen das zu tun und einen Pull request zu stellen. Theoretisch könnte ich das machen, aber momentan bin ich ziemlich im Projekt, vor Mitte März geht nix. Und dann macht's auch nur Sinn, wenn es auch für den Core gewünscht ist ;-)

@davidmaack
Copy link

Bin ich blind, oder ist der Post mit der Erweiterung gelöscht? Ich kann ihn jedenfall nicht sehen....
Ansonsten muss ich den anderen Postern zustimmen - aus Sicht des Redakteurs ist es komisch, dass der TinyMCE Verlinkungen zu Seiten als Insert-Tags auflöst und bei Bildern nicht.
Zumal sich Bildnamen öfter ändern als Seitennamen :)

@hmb
Copy link

hmb commented Feb 19, 2014

@davidmaack Der Post ist noch da, der 6. von oben.

@davidmaack
Copy link

@hmb Hmm nun ist er da. Gestern fehlte der definitiv. Ich verbuche das mal als Github-Anzeiget-Bug :D

@hmb
Copy link

hmb commented Feb 21, 2014

@davidmaack A glitch in the Matrix. Der Github Robot hatte meinen Account versehentlich als Fake eingestuft. Der Support hat ihn wieder freigeschaltet.

@SGehle
Copy link

SGehle commented Feb 24, 2014

Wie ist der aktuelle Stand?

@leofeyer leofeyer added this to the 3.3.0 milestone Mar 17, 2014
@leofeyer
Copy link
Member

Wie am Montag, den 28. April, besprochen, wollen wir versuchen, die Pfade beim Speichern automatisch zu konvertieren. Dadurch könnte der bestehende Picker so bleiben wie er ist.

@bekanntmacher
Copy link
Contributor

+1

@leofeyer
Copy link
Member

Implementiert in f2f38b5.

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

10 participants