search

contao / core

Contao 3 → see contao/contao for Contao 4
GNU Lesser General Public License v3.0
490 stars 214 forks source link

Fehler im TinyMCE bei dem einfügen von Links (Multidomain) #132

Closed ghost closed 12 years ago

ghost commented 12 years ago

Hi Leo,

Fehler tritt auf wenn der Link auf die Installationsdomain gesetzt werden soll. (bei 2 Installationen nachgewiesen) bei TL2.6.1

beim verlinken mittels Link setzen im TinyMCE Editor lässt sich kein Link auf die Installationsdomain setzen.

Beispiel:

Installdomain: www.ainschy.net weiter Domain www.legionderehre.de

im Artikel der legionderehre lässt sich folgender Link nicht einfügen: http://www.ainschy.net

umgekehrt ist es möglich also http://www.legionderehre.de geht.

im HTML Code erscheint www.ainschy.net

dieses lässt sich auch manuell nicht umgehen.

Gruß Ainschy

Download the attachments

--- Originally created by Ainschy on September 25th, 2008, at 06:24pm (ID 132)

ghost commented 12 years ago

Hallo Ainschy, ich versuche mal das Problem weiter einzugrenzen: Funktioniert es einen Link ohne www. zu setzten (also nur http://ainschy.net). Das Problem ist bei mir zumindest so gegeben. Eventuell hilft es beim debuggen.

Grüße

--- Originally created by MacKP on September 25th, 2008, at 07:39pm

leofeyer commented 12 years ago

Das hat etwas mit der TinyMCE-Konfiguration zu tun. Du solltest mal die Anleitung auf deren Projektwebseite lesen und wenn sich das Problem nicht beheben lässt, dort ein Ticket eröffnen.

--- Originally created on September 25th, 2008, at 07:55pm

ghost commented 12 years ago

Hallo leo, kann es sein das es hier ein Problem gibt mit den raltiven URL's vom TinyMCE? Ich hab es bei denen auf der Seite mal getestet und dort bekommt man dann diesen Link: '../' (Hier noch mal eben der aktuelle Thread mit der genauen Beschreibung wann das Problem auftaucht)

Hier gibt es auch ein Ticket und eine 'Lösung' dazu -> http://sourceforge.net/tracker/index.php?func=detail&aid=1065687&group_id=103281&atid=635682

Hmm, this is not a bug. All links within the same site are converted to relative_urls by default. So the examples at http://tinymce.moxiecode.com/ are located in http://tinymce.moxiecode.com/examples/ so this makes it to add ../. You may disable this feature using the relative_urls option.

Ich habe das mal selber geteste. In der tinymce.php Zeile 49ff:

tinyMCE.init({
  mode : "exact",
  height : "300",
  **relative_urls : false,**
  language : "<?php echo $this->language; ?>",
  elements : "<?php echo $this->rteFields; ?>",

Schon konnte ich das ohne Probleme speichern. So wie gewünscht ;-)

Jetzt stellt sich die Frage was damit machen? Ich glaube am schönsten wäre es diese Option zu belassen und in TL den link '../' abzufangen und durch den passenden zu ersetzten. Damit hätte man dan den gewünschten Effekt (glaube ich) und gleichzeitig keine Nachteile durch die nicht gesetzten relativen URL's.

Viele Grüße

--- Originally created by MacKP on August 25th, 2009, at 12:00am

NinaG commented 12 years ago

Wann können wir hier ungefähr mit dem Einbau der Lösung rechnen? Klappt das vielleicht zur 2.7.4? :-)

Ich habe mehrere Kunden die bei ihrer redaktionellen Arbeit über den gleichen Fehler stolpern.

--- Originally created on September 9th, 2009, at 07:17pm

leofeyer commented 12 years ago

Welchen Fehler meinst Du denn?

--- Originally created on September 9th, 2009, at 07:24pm

ghost commented 12 years ago

Hallo Leo, das Problem ist, das TL den Link einer Seite (auf sich selber) mit diesem Link ersetzt -> 'href="undefined"'

Ich habe dir dazu eine genaue Beschreibung des Problems weiter oben gepostet....

Viele Grüße

--- Originally created by MacKP on September 9th, 2009, at 07:29pm

ghost commented 12 years ago

Zur Verdeutlichung noch mal eben: z.B.: -> http://demo.typolight.org (http://www.demo.typolight.org geht) - demo.typolight.org als Domainname -> http://www.meinedomain.de (http://meinedomain.de geht) - www.meinedomain.de als Domainname

Viele Grüße

--- Originally created by MacKP on September 9th, 2009, at 07:33pm

leofeyer commented 12 years ago

Ich kann diesen Fehler nicht konstant reproduzieren. Auf "localhost" ist das Verhalten anders als auf "demo.typolight.org" oder auf "tinymce.moxiecode.com". Keine Ahnung, ob das mit der Subdomain zusammenhängt, jedenfalls scheint es ein generelles TinyMCE-Problem zu sein.

Ich werde das Ticket auf die 2.8 verschieben und nach dem TinyMCE-Update noch mal schauen. Auf keinen Fall sollte man "relative_urls" abschalten, da sonst die Links festgeschrieben sind und man die TL-Installation nicht mehr umziehen kann (z.B. aus einem Unterverzeichnis in ein anderes), ohne alle Pfade in allen Inhaltselementen anpassen zu müssen!

--- Originally created on September 26th, 2009, at 04:47pm

leofeyer commented 12 years ago

Ich habe es jetzt in allen Varianten getestet und es handelt sich wohl um einen TinyMCE-Bug.

Dasselbe gilt auch wenn TYPOlight in einem Unterverzeichnis installiert ist. Ich habe in meinem Plugin jetzt einen Patch installiert, der den Trailing-Slash automatisch hinzufügt und das href-Attribut bei Bedarf neu setzt. Allerdings ist das keine Garantie, dass nicht an anderer Stelle weiterhin Probleme auftreten, da TinyMCE die URLs an verschiedenen Stellen prüft. Insofern sollte der Fehler auf jeden Fall auch in deren Bugtracker gemeldet werden.

--- Originally created on October 15th, 2009, at 06:43pm

ghost commented 12 years ago

Hallo leo, vielen Dank für dein Engagement in dieser Sache. Leider haben wir da jetzt das Problem, das die Leute von Tiny sagen, das es kein Bug wäre (daher hatte ich ja oben den Link zum Tracker von TinyMCE gepostet), du aber davon überzeugt bist... Ich werde das auf jeden Fall mal weiter beobachten und gegebenenfalls hier Meldung erstatten wenn es etwas neues gibt.

--- Originally created by MacKP on October 15th, 2009, at 08:23pm

leofeyer commented 12 years ago

Komisch nur, dass das Fehlverhalten auch im ganz normalen TinyMCE auftritt und nicht auf mein Plugin beschränkt ist. Der Link bezieht sich außerdem auf das Einfügen von "../", was durchaus in Ordnung ist. Das Problem ist vielmehr, dass - wie oben beschrieben - entweder "undefined" oder überhaupt kein href-Attribut eingefügt wird. Und das ist eindeutig nicht richtig.

--- Originally created on October 15th, 2009, at 08:26pm

ghost commented 12 years ago

Klar wäre es nicht in Ordnung wenn der TinyMCE von sich aus "undefined" oder gar nicht einfügen würde.. in der Online Demo vom Tiny ist an den Stellen wo das in TL passiert dieses Verhalten auch nie aufgetaucht, sondern es wurde ja immer nur ein "../" eingefügt. Deswegen bin ich ja davon ausgegangen, das TL da irgendwo beteiligt ist und mit genau diesem Eintrag nichts anfangen kann und daher diese unschönen Ausgaben kommen. Es wurde ja auch im Ticket nicht "../" als Link eingefügt, sondern die URL der Installation.

--- Originally created by MacKP on October 15th, 2009, at 08:49pm

leofeyer commented 12 years ago

Der Fehler tritt in der TinyMCE-Demo ganz genauso auf! Hast Du vielleicht das falsche Beispiel genommen?

http://tinymce.moxiecode.com/examples/example_08.php

Dort wählst Du das zweite Beispiel von unten ("TinyMCE with relative URLs on links and images to a specific page") und versuchst mal, als Linkziel "http://tinymce.moxiecode.com" ohne Trailing-Slash einzugeben. Und was kommt da raus?

--- Originally created on October 15th, 2009, at 09:58pm

ghost commented 12 years ago

Äh, anscheinend hatte ich da dann den falschen TinyMCE. Ich guck mal das ich das bei denen reporte, damit das dann bald mal ein Ende hat.

Viele Dank! Und Entschuldige bitte die arbeit, die ich dir ja eigentlich ersparen wollte...

--- Originally created by MacKP on October 15th, 2009, at 11:04pm

ghost commented 12 years ago

Ich hab das ticket von TinyMCE jetzt bearbeitet und hoffe das die schnell zu einer Lösung kommen. Ich werde dich auf dem laufenden halten, was die dazu schreiben.

Viele Grüße

--- Originally created by MacKP on October 15th, 2009, at 11:17pm

ghost commented 12 years ago

Ich hab mal ein neues Ticket bei denen erstellt -> https://sourceforge.net/tracker/?func=detail&aid=2883035&group_id=103281&atid=635682

Ich hoffe wir bekommen da bald eine Brauchbare Reaktion ;-)

Viele Grüße

--- Originally created by MacKP on October 21st, 2009, at 08:11am

ghost commented 12 years ago

Jahre später ... gerade kämpfe ich mit eben diesem Problem. Contao 2.8.2 Was aus dem TinyMCE-Ticket geworden ist, kann ich leider nicht sehen: ERROR - Artifact: This Artifact Has Been Made Private. Only Group Members Can View Private ArtifactTypes.

--- Originally created by wdburgdorf on September 26th, 2011, at 02:30pm

leofeyer commented 12 years ago

--- Originally completed on October 15th, 2009, at 06:43pm

MacKP commented 12 years ago

Soooo, neues Ticket System aber altes Problem. Irgendwie ist da immer noch der Wurm drinn. Ich hab das jetzt noch mal in der Online Demo nachgestellt:

Es geht nicht:

Wenn man aber noch eine Unterseite angibt: http://demo.contao.org/neseite.html geht es wieder... Ist also mal wieder nur auf die URL beschränkt, die im Startpunkt angegeben ist.

Es wäre schön, wenn das Problem wieder behoben werden könnte (ich weiß.. TinyMCE Problem.. aber da wird dann wieder gesagt, das man keine Relativen URLs nehmen soll... irgendwie müssen wir das doch einfach mal in den Griff bekommen).

Viele Grüße

BigBerny commented 12 years ago

Wäre wirklich super, wenn man dieses Problem lösen könnte. Oder gibt es eine andere Möglichkeit manuell einen Link zu setzen?

leofeyer commented 12 years ago

Wie schon gesagt handelt es sich hierbei um ein TinyMCE-Problem, das auch auf deren Webseite reproduziert werden kann (siehe https://github.com/contao/core/issues/132#issuecomment-2931922).

leofeyer commented 12 years ago

Das musst Du die TinyMCE-Entwickler fragen, nicht uns.

MacKP commented 12 years ago

Hallo Leo, ich weiß, das es ein Problem von denen ist. Die sagen dann aber das es ein Problem von Contao ist (wie oben schon geschrieben). Mit anderen Worten: Das wird bleiben solange wir den TinyMCE benutzen. Und das ist wirklich ein Frustrierender Gedanke. Kannst du da echt überhaupt nichts drehen?

Viele Grüße

leofeyer commented 12 years ago

Wie kann es ein Problem von Contao sein, wenn es in der offiziellen TinyMCE-Demo auch auftritt? Hast Du bzw. habt ihr das dort so gemeldet? Gibt es ein Link zu dem Ticket?

MacKP commented 12 years ago

Hallo Leo, das hatte ich doch vor langer langer Zeit schon. Die Lösung der TinyMCE Leute: keine Relativen URLs, dann hat man das Problem auch nicht mehr. Nachteil dürfte bekannt sein... Einfaches schnelles umziehen ohne alle URLs anpassen zu müssen ist dann nicht drinn :-(

Im Moment wird also so das Problem immer nur auf den anderen geschoben. Die TinyMCE Leute kann ich nicht beknien, deswegen versuch ich das bei dir ;-)

leofeyer commented 12 years ago

Ja, am besten als Gist: https://gist.github.com/

leofeyer commented 12 years ago

Ich habe in 8e0cb430017b5b1c44a3e3ad5929ff6af0828e34 ein Update auf TinyMCE 3.5.5 gemacht. Damit tritt das Problem jetzt nicht mehr bei mir auf. Offenbar doch kein Contao-Fehler :)

MacKP commented 12 years ago

SUPER!!! PS: Hab auch nie behauptet, das es ein Contao-Fehler ist ;-)