[live] Migration zu MooTools v1.6.0

SHofmann · June 26, 2026 at 3:22 PM

MarkusE, ich habe gerade im Git gesehen, dass du jetzt wohl doch in Richtung der MooTools v1.6 gehen möchtest. Ich habe mir die Änderungen noch nicht im Detail ansehen können. Doch möchte ich vorsorglich darauf hinweisen, dass ich für die Popups und Aktionen der SVG-Tasten auf der Fernbedienung:

The content cannot be displayed because you do not have authorisation to view this content.

… wie du dich vielleicht erinnerst, die MooTools etwas habe patchen müssen, damit sie auch die SVG-Namespaces korrekt verarbeiten. Ob die neuen MooTools diese automatisch unterstützen, kann ich derzeit nicht einschätzen. Bitte teste bei der Umstellung deshalb auch, ob mit den neuen MooTools die Popups und Aktionen bei der Fernbedienung noch immer funktionieren…

Für Tablets benötigen wir, wenn wir den derzeitigen Tweak nicht beibehalten wollen, für Touch & Drag der Info-Fenster möglicherweise noch die Mobile-Komponente.

MarkusE · June 26, 2026 at 9:33 PM

Wo auf

kann ich die Mobile-Komponente auswählen?

SHofmann · June 26, 2026 at 11:28 PM

Ich habe das nur bei der Recherche aufgeschnappt...

SHofmann · June 27, 2026 at 12:45 PM

Ich habe es nochmals nachgeschaut: Es gibt ein unabhängiges, auf den MooTools aufbauendes Framework namens Moobile. Aber wenn die bisherigen Tweaks auch mit 1.6 funktionieren, haben wir diesbezüglich vielleicht gar keinen Handlungsbedarf…

MarkusE · June 27, 2026 at 12:52 PM

Ich habe im git jetzt auf 1.6 upgedatet. Bitte teste das mal.

SHofmann · June 27, 2026 at 1:11 PM

Danke, das mache ich. Ich habe gerade einen ersten Blick auf die Änderungen geworfen, und ein paar Kleinigkeiten sind mir schon aufgefallen, etwa.

Code

if (this.options.top != undefined) {
   winOpts += ",top=" + this.options.top;
}

Das deckt den Fall this.options.top === null aber nicht ebenfalls ab. Insofern sollten wir in solchen Fällen – sofern die MooTools das nicht anders empfehlen – besser:

Code

if (this.options.top) {
   winOpts += ",top=" + this.options.top;
}

… schreiben. Ich sehe mir die Änderungen einmal gesamthaft an und stelle einen Patch mit Vorschlägen wie diesen zusammen. Dann kannst du entscheiden, wie du das handhaben und was davon du übernehmen möchtest. Das wird aber noch ein paar Tage dauern.

MarkusE · June 27, 2026 at 5:32 PM

Quote from SHofmann
Danke, das mache ich. Ich habe gerade einen ersten Blick auf die Änderungen geworfen, und ein paar Kleinigkeiten sind mir schon aufgefallen, etwa.
Code
if (this.options.top != undefined) {
   winOpts += ",top=" + this.options.top;
}
Das deckt den Fall this.options.top === null aber nicht ebenfalls ab.

Tatsächlich ist das die 1:1 Übersetzung des Codes von mootools.v1.11.js . Da steht:

Code

function $defined(obj){
  return (obj != undefined);
};

Javascript nimmt das mit dem == / != übrigens nicht so genau: null != undefined ist da auch false.

Wir können solche Sachen natürlich auch verändern. Da ist halt dann immer das Risiko, dass irgendetwas danach nicht mehr funktioniert.

> Bitte teste das mal.

Ich dachte da mehr an funktionale Tests, also ob aus Endanwendersicht irgendetwas nicht funktioniert.

Und wie sich das auf dem Tablet macht.

heifisch · June 28, 2026 at 9:50 AM

Hab eine Fehlfunktion gefunden.
Wenn man paar Aufzeichnungen ausgewählt hat und auf Ausgewählte Aufzeichnungen löschen drückt, passiert nichts.
Ich kann aber nicht sicher sagen, ob dass vor dem Mootools-Update nicht auch so war.

live-Version: 3.5.6_git_master_0552ead_20260627124930+0200

SHofmann · June 28, 2026 at 2:04 PM

Quote from MarkusE

Ich dachte da mehr an funktionale Tests, also ob aus Endanwendersicht irgendetwas nicht funktioniert.
Und wie sich das auf dem Tablet macht.

Schon klar, aber ich wollte mir erst einmal einen Überblick über die Codeänderungen verschaffen.

Quote from heifisch

Wenn man paar Aufzeichnungen ausgewählt hat und auf Ausgewählte Aufzeichnungen löschen drückt, passiert nichts.

Das kommt daher:

The content cannot be displayed because you do not have authorisation to view this content.

… und muss wohl damit gefixt werden:

Diff

diff --git a/live/js/live/createHtml.js b/live/js/live/createHtml.js
index e35f6911..47d56a44 100644
--- a/live/js/live/createHtml.js
+++ b/live/js/live/createHtml.js
@@ -229,7 +229,7 @@ async function actionOnMarkedRecordings(form, act) {
       event_ = new DOMEvent(event);
       merged_options = Object.merge(liveEnhanced.options.infoWinOptions, { onDomExtend: liveEnhanced.domExtend.bind(liveEnhanced) });
     }
-    var infowin = new InfoWin.Ajax(epgid, "epginfo.html?epgid="+encodeURIComponent(epgid), merged_options);
+    var infowin = new InfoWin_Ajax(epgid, "epginfo.html?epgid="+encodeURIComponent(epgid), merged_options);
     infowin.options.offsets.y = -400;
     infowin.show(event_);
     event_.stop();
diff --git a/live/js/live/vdr_status.js b/live/js/live/vdr_status.js
index fc1c12ab..c3c2347d 100644
--- a/live/js/live/vdr_status.js
+++ b/live/js/live/vdr_status.js
@@ -241,6 +241,6 @@ class LiveVdrInfo {
           this.timer = null;
         }
       }
-      this.cancel();
+      this.request(false);
     }
 }

Display More

Damit geht zumindest bei mir der Bestätigungsdialog wieder auf:

The content cannot be displayed because you do not have authorisation to view this content.

SHofmann · June 28, 2026 at 6:40 PM

MarkusE, die Tastenbelegung der Fernbedienung öffnet sich nur noch als Webseite, nicht mehr als Info-Fenster. Damit das wieder funktioniert, benötigt man folgende Anpassung:

Diff

diff --git a/live/js/live/pageenhance.js b/live/js/live/pageenhance.js
index af2cc786..d6e4d148 100644
--- a/live/js/live/pageenhance.js
+++ b/live/js/live/pageenhance.js
@@ -11,7 +11,7 @@
 var PageEnhance = new Class({
     Implements: [Options],
     options: {
-      epgLinkSelector: 'a[href^="epginfo.html?epgid"], *[xlink:href="epginfo.html?epgid=rcKeys"]',
+      epgLinkSelector: MooTools.version == '1.11' ? 'a[href^="epginfo.html?epgid"], *[xlink:href="epginfo.html?epgid=rcKeys"]' : 'a[href^="epginfo.html?epgid"], a[*|href="epginfo.html?epgid=rcKeys"]',
       actionLinkSelector: 'a[href^="vdr_request/"]',
       hintTipSelector: '*[title], *[xlink:title]',
       hintClassName: 'hint',

Display More

Ich gehe allerdings davon aus, dass du nach Abschluss der initialen Tests die Unterstützung für die MooTools v1.11 aus dem Code entfernst, sodass man der Ternary nicht mehr benötigt.

Was meine Tests betrifft, sollte mit diesen beiden Anpassungen (#9 und diese hier) alle Funktionen unverändert zur Verfügung stehen. Keep your fingers crossed!

SHofmann · June 28, 2026 at 6:43 PM

Noch ein kleiner Typo:

Diff

diff --git a/live/js/live/infowin.js b/live/js/live/infowin.js
index 067f9e39..88871cb0 100644
--- a/live/js/live/infowin.js
+++ b/live/js/live/infowin.js
@@ -508,7 +508,7 @@ var InfoWin_Ajax = new Class({
 
 
 /*
-Class: Infowin__otifier
+Class: Infowin_Notifier
 
 Creates a notification popup that disappears automatically.
 Useful for a confirmation message after a AJAX action request.

Display More

PS: Dass du die Migration jetzt doch so schnell und problemlos hinbekommen hast, ist echt klasse!

SHofmann · June 28, 2026 at 10:39 PM

Das Verschieben von Info-Fenstern auf Tablets muss noch angepasst werden.

SHofmann · June 29, 2026 at 5:47 PM

Quote from SHofmann

Das Verschieben von Info-Fenstern auf Tablets muss noch angepasst werden.

Den angepassten Patch habe ich im ursprünglichen Thread hinterlegt.

MarkusE · June 29, 2026 at 9:16 PM

Im git ist ein Update, mit #10 und #11

MarkusE · June 29, 2026 at 9:20 PM

Ich habe auch Code entfernt, der für MooTools v1.11 notwendig war. Ich werde nicht mehr zu MooTools v1.11 zurückgehen.

Nach den Tests und Korrekturen gehe ich davon aus, dass der jetzige Code mit MooTools v1.6 fehlerfrei läuft. Und falls doch noch ein Fehler gefunden wird, dann korrigieren wir den eben.

SHofmann · June 30, 2026 at 3:49 PM

Quote from MarkusE

Im git ist ein Update, mit #10 und #11

… und noch ein bisschen mehr, wie beispielsweise einige Änderungen in recordings.ecpp. Schön wäre es schon, wenn du dich an deine eigenen Regeln halten und für jede inhaltlich eigenständige Änderung – wie schon von dem einen oder anderen Package-Maintainer vorgeschlagen – einen eigenen Commit machen würdest…

MarkusE · June 30, 2026 at 4:51 PM

Immerhin teste ich vor jedem Commit. Das ist ein gewisser Aufwand. Daher kein eigener Commit für #10 und #11.

In der Commit message steht ja dann drin, was in dem Commit ist.

SHofmann · July 1, 2026 at 5:27 PM

Quote from SHofmann

und noch ein bisschen mehr, wie beispielsweise einige Änderungen in recordings.ecpp.

Bei dieser Änderung ist mir aufgefallen, dass die CSS-Regeln für die Formatierung der Aufzeichnungen noch nicht perfekt passen, sobald man ohne den TvScraper arbeitet, also einen "echten" Level 0 hat. Hier der entsprechende Patch:

Diff

diff --git a/live/css/styles.css b/live/css/styles.css
index c9e4fa5d..110811e7 100644
--- a/live/css/styles.css
+++ b/live/css/styles.css
@@ -1723,29 +1723,29 @@ div.recordings {
 /* last displayed folder collapsed and hits the form's bottom border at: */
 /* level 0 */
 form div.recordings
-> div.recordingslist > div.folderslist:last-of-type
+> div.recordingslist:not(:has(> table.recordings)) > div.folderslist:last-of-type
 > div.recordings_folder:has(+ div.recordingslist:last-of-type:is([style*="display: none"])),
 /* level 1 */
 form div.recordings
-> div.recordingslist > div.folderslist:last-of-type:not(:has(> div.recordingslist > table.recordings))
+> div.recordingslist:not(:has(> table.recordings)) > div.folderslist:last-of-type:not(:has(> div.recordingslist > table.recordings))
 > div.recordingslist > div.folderslist:last-of-type
 > div.recordings_folder:has(+ div.recordingslist:last-of-type:is([style*="display: none"])),
 /* level 2 */
 form div.recordings
-> div.recordingslist > div.folderslist:last-of-type:not(:has(> div.recordingslist > table.recordings))
+> div.recordingslist:not(:has(> table.recordings)) > div.folderslist:last-of-type:not(:has(> div.recordingslist > table.recordings))
 > div.recordingslist > div.folderslist:last-of-type:not(:has(> div.recordingslist > table.recordings))
 > div.recordingslist > div.folderslist:last-of-type
 > div.recordings_folder:has(+ div.recordingslist:last-of-type:is([style*="display: none"])),
 /* level 3 */
 form div.recordings
-> div.recordingslist > div.folderslist:last-of-type:not(:has(> div.recordingslist > table.recordings))
+> div.recordingslist:not(:has(> table.recordings)) > div.folderslist:last-of-type:not(:has(> div.recordingslist > table.recordings))
 > div.recordingslist > div.folderslist:last-of-type:not(:has(> div.recordingslist > table.recordings))
 > div.recordingslist > div.folderslist:last-of-type:not(:has(> div.recordingslist > table.recordings))
 > div.recordingslist > div.folderslist:last-of-type
 > div.recordings_folder:has(+ div.recordingslist:last-of-type:is([style*="display: none"])),
 /* level 4 */
 form div.recordings
-> div.recordingslist > div.folderslist:last-of-type:not(:has(> div.recordingslist > table.recordings))
+> div.recordingslist:not(:has(> table.recordings)) > div.folderslist:last-of-type:not(:has(> div.recordingslist > table.recordings))
 > div.recordingslist > div.folderslist:last-of-type:not(:has(> div.recordingslist > table.recordings))
 > div.recordingslist > div.folderslist:last-of-type:not(:has(> div.recordingslist > table.recordings))
 > div.recordingslist > div.folderslist:last-of-type:not(:has(> div.recordingslist > table.recordings))
@@ -1753,7 +1753,7 @@ form div.recordings
 > div.recordings_folder:has(+ div.recordingslist:last-of-type:is([style*="display: none"])),
 /* level 5 */
 form div.recordings
-> div.recordingslist > div.folderslist:last-of-type:not(:has(> div.recordingslist > table.recordings))
+> div.recordingslist:not(:has(> table.recordings)) > div.folderslist:last-of-type:not(:has(> div.recordingslist > table.recordings))
 > div.recordingslist > div.folderslist:last-of-type:not(:has(> div.recordingslist > table.recordings))
 > div.recordingslist > div.folderslist:last-of-type:not(:has(> div.recordingslist > table.recordings))
 > div.recordingslist > div.folderslist:last-of-type:not(:has(> div.recordingslist > table.recordings))

Display More

Hätte mir auch damals schon auffallen können…

PS: MarkusE, das hätte vielleicht besser in den Haupt-Thread gehört, aber bitte trotzdem übernehmen…

MarkusE · July 2, 2026 at 3:15 PM

#18 ist im git

Participate now!