Elementtyp Youtube fehlerhaft?

[DetlefT]

Hallo,

ich versuche z.Zt. vergeblich, ein Youtube-Video im Projekt einzubinden (z.Zt. noch nur auf XAMPP). Die Meldung lautet immer: "Dieses Video ist nicht verfügbar".

Bisher habe ich herausgefunden, dass Contao 3.5.26 den unter "Youtube-ID" eingegebenen Wert beim Speichern in lowercase konvertiert. Also:
HZNM2Ki42T8 ist der korrekte v= Wert, Video würde in Youtube gefunden!
hznm2ki42t8 ist der von Contao konvertierte Wert, Video wird in Youtube nicht gefunden!

Ist dieses Verhalten schon mal jemandem aufgefallen? Weiß jemand, wo ich die strtolower- (oder mb_strtolower-) Funktion entfernen kann - ich finde es einfach nicht.

Danke für Eure Aufmerksamkeit.

[Spooky]

» https://github.com/contao/core/issues/8704

[Andreas]

Ich hatte gerade die gleiche Lösung gefunden und bei mir eingebaut.

  /** Patch the youtube ID - don't change to lowercase */
  unset($GLOBALS['TL_DCA']['tl_content']['fields']['youtube']['eval']['rgxp']); 

Aber ich frage mich, warum die Videos trotzdem nicht laufen. Ich benutze youtube_iframe (@DetlevT: Solltest du auch.).

Die Console zeigt einen Fehler auf https://www.youtube.com/sw.js - Zeile 5:

TypeError: "iterator" is not an object

function r(){l();var a=k.h.iterator;a||(a=k.h.iterator=k.h("iterator"));"function"!=typeof Array.prototype[a]&&h(Array.prototype,a,{configurable:!0,writable:!0,value:function(){return t(this)}}); 

Klickt man im Youtube-Iframe auf den Button "Auf youtube.com ansehen", dann wird bei Youtube das Video mit der klein geschriebenen ID versucht aufzurufen.

Kann das jemand bestätigen?

[DetlefT]

Aber ich frage mich, warum die Videos trotzdem nicht laufen.

Mit Einsatz des Workarounds wird schon mal nicht mehr konveriert - Danke dafür! - aber Andreas Aussage kann ich bestätigen: läuft nicht.

[Spooky]

Bei mir funktioniert es. Im Source wird dann

<source type="video/x-youtube" src="https://www.youtube.com/watch?v=HZNM2Ki42T8" title="">

ausgegeben, bzw. mit youtube_iframe Extension

<iframe type="text/html" src="https://www.youtube.com/embed/HZNM2Ki42T8" allowfullscreen></iframe>

[DetlefT]

Richtig im HTML ausgegeben wird es, es werden auch Bilder angezeigt, wenn man mit Slider vorwärts geht, aber es bewegt sich halt nix, sind Standbilder.

[Spooky]

Slider? Was genau meinst du? Poste mal einen Link zur Seite.

[DetlefT]

Wie oben geschrieben: XAMPP.

Der Slider im YouTube Video, wo Du im Film vor oder zurück gehen kannst.

[Spooky]

Wie hast du das YouTube Video nun eingebunden? Ganz normal mit MediaElement.js oder mit der youtube_iframe Erweiterung?

[DetlefT]

Elementtype: Youtube, kein youtube_iframe (das teste ich, wenn es wieder läuft).

Händisch über <video>-Tags bzw. über Elementtyp: Video/Audio läuft alles wie geschmiert

[Spooky]

Das heißt du benutzt MediaElement.js. Wie wird von MediaElement.js das YouTube Video eingebunden? Als iframe?

Jedenfalls denke ich nicht, dass dein aktuelles Problem etwas mit dem ursprünglichen Problem zu tun hat. Lösche mal deinen Browser Cache und starte den gesamten PC neu

[DetlefT]

<video width="500" height="300" controls>
              <source type="video/x-youtube" src="https://www.youtube.com/watch?v=HZNM2Ki42T8" title="">
          </video>

MediaElement.js? Keine Ahnung. Einen <iframe>-Tag kann ich nirgendwo finden. Hier das ist noch im Quelltext:

<script src="assets/jquery/mediaelement/2.21.2/js/mediaelement-and-player.min.js"></script>

[Spooky]

Das iframe wird von MediaElement.js erzeugt. Siehst du also nur, wenn du den DOM im Browser mit den Developer Tools inspizierst.

[DetlefT]

Ja, MediaElement.js ist drin.

Habe jetzt voutube_iframe installiert, damit funktioniert es.

Danke für Eure Hilfe!

[BugBuster]

Äh, also was muss ich machen um den Fehler mit der Kleinschreibung zu bekommen? Ich kann das bisher nicht nachvollziehen.

[Andreas]

Jetzt läuft es in meinem Firefox wieder, nachdem ich die Seite mit Shift neu geladen habe. Etwas seltsam, weil mein FF steht auf "Jedesmal auf neue Seitenversion prüfen". Bezog sich anscheinend nicht auf Seiten in einem Iframe. Im Chrome lief es sofort.

[DetlefT]

Jetzt ergibt sich noch eine Frage nach responsive. Ich bekomme die youtube_iframe-Lösung einfach nicht responsive. Wenn ich keine Größenangaben mache, klappt es, sobald ich irgendwo eine Max-Breite vorgebe, ist Ende mit responsive - das Video bzw. iframe soll halt nicht über die gesamte Seitenbreite gehen.

<div class="ce_youtube video-container last block">
  <!-- indexer::stop -->
      <div class="video-wrapper">
      <iframe src="https://www.youtube.com/embed/thuJ2uPE11I?hl=de" allowfullscreen></iframe>
    </div>
    <!-- indexer::continue --></div>

.video-wrapper {
  position: relative;
  padding-bottom: 56.25%;
  padding-top: 30px;
  height: 0;
  overflow: hidden;
  margin-top: 30px;
}
.video-wrapper iframe, .video-wrapper object, .video-wrapper embed {
  position: absolute;
  top: 0;
  left: 0;
  width: 100%;
  height: 100%;
}

Quelle von http://www.blog.medianetic.de/contao...e-integrieren/

/* Einer von vielen Versuchen */
.video-container {
	width :500px;
	height: auto;
}

Wo immer ich die Max-Breite vorgebe, im BE, im .video-wrapper oder im .videocontainer, Ende Gelände.

Könnt Ihr mir auf's Pferd helfen?

[BugBuster]

Also in Contao 3.5 bekomme ich das nicht hin, aber in Contao 4.3.7 ja, da wird es gewandelt. Wie Leo schon schrieb, an Punycode liegt es nicht, die 2er Version kam ja erst mit Contao 4.3.8. (heute)

Komischerweise, wenn ich mir ein Testscript schreibe was die Idna Klasse benutzt, dann wird das bei mir nicht gewandelt. Ich kann die Ursache noch nicht sehen.

[Spooky]

Äh, also was muss ich machen um den Fehler mit der Kleinschreibung zu bekommen? Ich kann das bisher nicht nachvollziehen.

Prinzipiell so wie im Ticket beschriebne. Einfach in den neuesten Contao Versionen ein YouTube Element anlegen und Großbuchstaben in das youtube Feld schreiben und speichern.

[BugBuster]

In Contao 4.3.7 habe ich es ja auch jetzt.
Habe aber irgendwie noch nicht die Stelle gefunden wo das auf lowercase gesetzt wird.
Alle if ($this->rgxp == 'url') Stellen verwenden encodeUrl, damit sollte das schon mal nicht bei Speichern gewandelt werden.
Daher suchte ich nach den decode Aufrufen und habe die mal in decodeUrl umgebaut inkl. Idna:decodeUrl ergänzt.

Wird aber immer noch gewandelt. Irgendwo übersehe ich da noch ne Stelle bei der DCA Geschichte.

[Spooky]

Die Stelle wo es in Contao 3 passiert habe ich im Ticket ja schon gepostet. Ich habe es nicht extra getestet, aber ich vermute, dass es in Contao 4 genau so sein wird.

[BugBuster]

Gerade in Contao 3 konnte ich das bei mir nicht nachvollziehen.
Nun zieht Leo das Punycode Update auch noch zurück, ich hoffe nicht auch in Contao 4, dann habe ich ein großes Problem mit meinen Erweiterungen.

[Spooky]

Erzeuge im Root einer Contao 3.5.26 Installation mal eine Datei namens test.php mit folgenden Inhalt:

<?php

require __DIR__ . '/system/initialize.php';
var_dump(\Idna::encodeUrl('LOREM'));

Und rufe diese dann im Frontend auf. Bei mir ist die Ausgabe

string(5) "lorem"

[BugBuster]

Ach, da wird nur die Youtube ID als URL übergeben? Das muss ja schief laufen.

Jetzt verstehe ich auch die Markierung die Leo als Fehlerursache angegeben hat, da macht er ja dann http://LOREM draus und das dann geht zu Punycode.
Muss ich heute Abend noch mal durchgehen.

[Spooky]

Ach, da wird nur die Youtube ID als URL übergeben? Das muss ja schief laufen.

Ja natürlich, du gibst ja (in Contao 3 zumindest) nur die YouTube ID im Inhaltselement an . Hast du die Reproduction Steps des Tickets nicht versucht?

[BugBuster]

Doch, das ist ja das merkwürdige, das bei mir in Contao 3 alles super lief. Verstehe ich im Nachhinein auch nicht ganz.

Wie im Ticket geschrieben finde ich es nur merkwürdig, warum das ganze als URL betrachtet und daher auch geprüft wird, wenn es intern gar nicht als solche gespeichert wird.

[Znrl]

Ja der eigentliche Bug scheint ja dann die zu sein, dass an der Stelle überhaupt punycode verwendet wird.
Punycode sollte ja nur auf die TLD angewendet werden.
Alles andere z.B. per urlencode().

Müsste man mal die anderen Verwendungen von punycode durchgehen und eher das fixen anstatt das auf punycode v 2 zu schieben.

edit: Ich mein natürlich nicht (nur) TLD sondern Domainnamen.

[Jayster]

Blöde Frage: Warum wird beim Feld "youtube" eigentlich "rgxp" => "url" verwendet? Es ist doch keine URL.

[Spooky]

Das nicht - aber die ID wird als Parameter an eine URL dran gehängt und bei 'rgxp' => 'url' wird unter anderem auch das Encoding sicher gestellt.

[folkfreund]

Das heißt, es wird ein Seiteneffekt ausgenutzt. Eigentlich wird nur das Encoding gebraucht... Jede Änderung an der Implementierung von "rgxp" => "url" kann also unerwünschte Folgen haben. Hab ich das richtig verstanden?

[Andreas]

Ich wäre nie auf die Idee gekommen eine Youtube ID auf "url => expects a valid URL" zu testen. Dann schon eher auf alnum, folderalias oder alias. https://docs.contao.org/books/api/dc...tml#evaluation

Wenn ich das richtig sehe muss auf [a-zA-Z0-9]* getestet werden.

[Jayster]

Ich glaube, es gibt zwei Möglichkeiten, wie es mit neuen Eval-Optionen schöner wäre.
1. Eine neu Rgxp für Youtube IDs: "rgxp" => "youtubeid".
2. Nicht validieren sondern nur encoden mittels "urlencode" => true.
Beides gibt es noch nicht, müsste also angelegt werden.

[BugBuster]

Das heißt, es wird ein Seiteneffekt ausgenutzt. Eigentlich wird nur das Encoding gebraucht... Jede Änderung an der Implementierung von "rgxp" => "url" kann also unerwünschte Folgen haben. Hab ich das richtig verstanden?

Ja :-) Aus dem (3.5.x) Ticket:

Bin gespannt wie du in der Idna Klasse erkennen willst, das es sich nicht um eine unvollständige URL handelt sondern um eine Youtube-ID.

Genau das ist das Problem. Deswegen müssen wir überall wo wir 'rgxp'=>'url' oder die Idna-Klasse verwenden prüfen, was genau wir übergeben.

Für Contao 4.4 werde ich versuchen, die Version 2 einzubauen.

Ich hoffe er schafft das, denn ohne Punycode 2 gibt es keine Botdetection Erweiterung (eine Abhängigkeit verlangt Punycode 2) und somit keine Banner, Visitors, dlstats Erweiterung für Contao 4
Dabei ist Botdetection und Visitors als Bundle schon so gut wie fertig.

Ich wünschte ich könnte Leo da unterstützen, aber ich habe schon ewig gebraucht halbwegs zu verstehen wie die DCA rgxp Angabe zur Prüfung bei der Idna Klasse landet.

[Andreas]

1. Eine neu Rgxp für Youtube IDs: "rgxp" => "youtubeid".

Das ist zu speziell und kann dadurch schlecht wiederverwendet werden. Besser wäre "rgxp" => "realalnum" o.ä..

$strings = array('Az12', 'Az1ü', 'Az1.2');
foreach ($strings as $testcase) {
  dump($testcase, preg_match('/^[[:alnum:]]*$/', $testcase));
} 

[folkfreund]

Wie wäre es mit "rgxp" => "regex([a-zA-Z0-9]*)" ?
In den Klammern könnte eine selbst definierte Expression stehen. Damit wären Sonderfälle sehr leicht abbildbar.

[BugBuster]

Also falls ihr da Ideen habt, Leo hat das Ticket für Contao 4.4 grad reopened.
https://github.com/contao/core-bundle/issues/748

Weiß nicht ob das zu Leo's Ideen passt, aber ein wenig Input kann nicht schaden, also rein damit als Kommentar.

[Jayster]

Ich war so frei, die bisherigen Vorschläge ins Ticket zu schreiben.

[Andreas]

Irgendwie kann ich nicht glauben, dass die Validierung einer Youtube-ID in Zusammenhang mit Punycode stehen soll und darüber entscheiden soll, welche Punycodeversion benutzt wird. Eine Youtube ID ist doch eindeutig /^[[:alnum:]]*$/. Oder?

Deshalb gehört das IMHO gar nicht zu diesem Ticket. Aber trotzdem danke fürs Hinzufügen.

Gute Idee, eine freie Regex eingeben zu können, kann man vielleicht mal gebrauchen, andererseits ist eine eigene "rgxp" schnell aufgesetzt. Benötige ich die nur einmal, dann wäre so ein Freitextfeld praktisch.

[Znrl]

Der Fehler liegt auch eigentlich woanders.
Nur tritt der Fehler eben erst bei Punycode v2 auf.

Im Grunde hätte man im Modul statt nach der Youtube ID nach der kompletten URL fragen müssen, dann würde das so funktionieren.

Hier wird aber immer HTTP:// vorangestellt, wenn das nicht angegeben ist dann wird daraus:
HTTP://YoutubeID und deshalb auch encoded, Punycode muss davon ausgehen, dass das eine Domain ist.

Das heißt der "schnelle" Fix war eben einfach wieder auf Punycode v1 zu wechseln, das eigentliche Problem ist damit aber natürlich nicht gelöst.

Punycode v1 nutzt kein mb_strtolower()
Punycode v2 schon, deshalb wird es hier nicht mehr "verziehen", dass hier fälschlicherweise Punycode zum Einsatz kommt.
Könnten Youtube IDs Sonderzeichen enthalten hätte es wohl schon bei v1 gekracht.

Deshalb muss das mit Youtube gefixed werden, damit Punycode v2 wieder genutzt werden kann.

[BugBuster]

Kommt mit der 4.4.0, Leo hat ua. die "'rgxp'=>'url'" Regel entfernt und noch weitere Dinge angepasst, das ursprüngliche Ziel war ja Punycode 2.x zu unterstützen.
Laut Changelog soll es sogar ein 4.4.0-beta1 geben, bin mal gespannt wann, da dass ursprüngliche Datum bereits verstrichen ist.