[live] Der Suchtimer unterstützt einige Formularfelder nicht, ...

nobanzai · August 15, 2025 at 5:43 PM

Hi,

ich bekomme beim Anlegen eines neuen Suchtimers die Meldung

Der Suchtimer unterstützt einige Formularfelder nicht, sodass deren Werte nicht gespeichert werden können. Bitte auf die aktuelle Version von EPGsearch upgraden.

Ich verwende aber epgsearch 2.4.4 - und das ist doch die neueste Version aus dem git Repo, oder?

Danke und ciao.
Michael.

SHofmann · August 15, 2025 at 7:06 PM

Dummer kleiner Logikfehler, der mir leider nicht aufgefallen ist:

Diff

diff --git a/pages/edit_searchtimer.ecpp b/pages/edit_searchtimer.ecpp
index 0745621..4661f9e 100644
--- a/pages/edit_searchtimer.ecpp
+++ b/pages/edit_searchtimer.ecpp
@@ -278,7 +278,10 @@ const char *TNT_ARRAY = "";
   SearchTimer* searchtimer;
   if (!searchtimerid.empty()) {
     searchtimer = searchtimers.GetByTimerId(searchtimerid);
-    if (!searchtimer)
+    if (searchtimer) {
+      hasInsufficientFields = searchtimer->HasInsufficientFields();
+      hasUnknownFields = searchtimer->HasUnknownFields();
+    } else
       throw HtmlError( tr("Couldn't find search timer. Maybe you mistyped your request?") );
   } else {
     // get an existing search timer for detemining the compatibility flags
@@ -388,9 +391,6 @@ const char *TNT_ARRAY = "";
     delmode = searchtimer->DelMode();
     delaftercountrecs = searchtimer->DelAfterCountRecs();
     delafterdaysoffirstrec = searchtimer->DelAfterDaysOfFirstRec();
-    // amend the compatibility flags
-    hasInsufficientFields |= searchtimer->HasInsufficientFields();
-    hasUnknownFields |= searchtimer->HasUnknownFields();
     editsearchtimer = searchtimer;
   }
   else

Display More

Hatte wohl gerade eine Änderung in EPGsearch am Laufen, bei der diese Fehlermeldung zu erwarten war.

PS: Der Patch vermeidet lediglich den (in der gemeldeten Plugin-Kombination) ungerechtfertigten Kompatibilitätshinweis. Und mehr ist es auch nicht, denn das Anlegen eines Suchtimers funktioniert ja trotz des Hinweises.

MarkusE · September 1, 2025 at 8:44 AM

Ich habe das im git upgedatet.

SHofmann , dabei ist mir

Code

searchtimer = new SearchTimer();

aufgefallen. An welcher Stelle wird dieser SearchTimer denn wieder gelöscht, z.B. wenn der Anwender auf "Abbrechen" clickt? Ich hätte jetzt irgendwie ein "delete searchtimer" erwartet.

SHofmann · September 1, 2025 at 2:37 PM

Stimmt, das hatte ich übersehen… und auch, dass selbst neue Suchtimer immer zum Bearbeiten angezeigt werden. Diese – wie auch die Suche – haben zudem die Ausschlusslisten des ersten Suchtimers vorausgewählt, was mir nicht aufgefallen ist, weil ich bislang ohne Ausschlusslisten gearbeitet (jetzt aber testhalber welche angelegt) habe.

Hier der entsprechende Patch mit der Bitte um Prüfung (nicht dass ich noch etwas übersehen habe):

Diff

diff --git a/pages/edit_searchtimer.ecpp b/pages/edit_searchtimer.ecpp
index 1ebd9a8..15c65b8 100644
--- a/pages/edit_searchtimer.ecpp
+++ b/pages/edit_searchtimer.ecpp
@@ -88,6 +88,7 @@ ChannelGroups channelGroups;
 Blacklists blacklists;
 RecordingDirs recordingdirs;
 SearchTimer* editsearchtimer;
+SearchTimer newSearchTimer;
 bool hasInsufficientFields = false;
 bool hasUnknownFields = false;
 std::string contentsFilter;
@@ -277,22 +278,21 @@ const char *TNT_ARRAY = "";
   }
   SearchTimer* searchtimer;
   if (!searchtimerid.empty()) {
-    searchtimer = searchtimers.GetByTimerId(searchtimerid);
-    if (searchtimer) {
-      hasInsufficientFields = searchtimer->HasInsufficientFields();
-      hasUnknownFields = searchtimer->HasUnknownFields();
-    } else
+    searchtimer = editsearchtimer = searchtimers.GetByTimerId(searchtimerid);
+    if (!searchtimer)
       throw HtmlError( tr("Couldn't find search timer. Maybe you mistyped your request?") );
+    hasInsufficientFields = searchtimer->HasInsufficientFields();
+    hasUnknownFields = searchtimer->HasUnknownFields();
   } else {
     // get an existing search timer for detemining the compatibility flags
-    searchtimer = &*searchtimers.begin();
-    if (searchtimer) {
+    if (searchtimers.begin() != searchtimers.end()) {
+      searchtimer = &*searchtimers.begin();
       hasInsufficientFields = searchtimer->HasInsufficientFields();
       hasUnknownFields = searchtimer->HasUnknownFields();
     }
-    searchtimer = new SearchTimer();
+    searchtimer = &newSearchTimer;
   }
-  if (searchtimer) {
+  if (searchtimer) { // always true
     search = searchtimer->Search();
     mode = searchtimer->SearchMode();
     tolerance = searchtimer->Tolerance();
@@ -391,7 +391,6 @@ const char *TNT_ARRAY = "";
     delmode = searchtimer->DelMode();
     delaftercountrecs = searchtimer->DelAfterCountRecs();
     delafterdaysoffirstrec = searchtimer->DelAfterDaysOfFirstRec();
-    editsearchtimer = searchtimer;
   }
   else
   {
@@ -875,7 +874,7 @@ const char *TNT_ARRAY = "";
 %               if (blacklists.size() > 0) {
                   <select name="blacklists" multiple="multiple" size="5" onchange="changedblacklistsel(this)">
 %                 int i = 0; for (Blacklists::iterator blacklist = blacklists.begin(); blacklist != blacklists.end(); ++blacklist, i++) {
-                    <option value="<$ blacklist->Id() $>" <{ SELECTIF(editsearchtimer && editsearchtimer->BlacklistSelected(blacklist->Id())) }>><$ blacklist->Search() $></option>
+                    <option value="<$ blacklist->Id() $>" <{ SELECTIF(searchtimer && searchtimer->BlacklistSelected(blacklist->Id())) }>><$ blacklist->Search() $></option>
 %                 }
                   </select>
 %               }
diff --git a/pages/searchepg.ecpp b/pages/searchepg.ecpp
index 9b7d08e..5b07f57 100644
--- a/pages/searchepg.ecpp
+++ b/pages/searchepg.ecpp
@@ -171,12 +171,13 @@ const char *TNT_ARRAY = "";
 </html>
 <%cpp>
   }
-  SearchTimer* searchtimer;
+  SearchTimer* searchtimer = nullptr;
   // get an existing search timer for detemining the compatibility flags
-  searchtimer = &*searchtimers.begin();
-  if (searchtimer) {
+  if (searchtimers.begin() != searchtimers.end()) {
+    searchtimer = &*searchtimers.begin();
     hasInsufficientFields = searchtimer->HasInsufficientFields();
     hasUnknownFields = searchtimer->HasUnknownFields();
+    searchtimer = nullptr;  // to aovid references to blacklists of the first search timer
   }
   for (unsigned int i = 0; i < extEPGInfos.size(); i++)
     extepgvalues.push_back("");

Display More

Der Suchtimer wird damit im Kontext der Seite anlegt und damit später automatisch gelöscht. Die anderen Probleme sind ebenfalls adressiert.

Den else-Zweig in edit_searchtimer.ecpp, Zeile 295 ff. ("always true"), könnte man entfernen und die übrigen Befehle ausrücken. Aber das habe ich mir nach deinem letzten Hinweis verkniffen, damit du den Patch nicht deswegen zurückweist.

SHofmann · September 1, 2025 at 5:52 PM

Eine Kleinigkeit in der Anzeige von Popups ist mir noch aufgefallen:

Diff

diff --git a/live/css/styles.css b/live/css/styles.css
index a47b7f9..509ac8a 100644
--- a/live/css/styles.css
+++ b/live/css/styles.css
@@ -885,6 +885,9 @@ div.cast_crew.tabular {
 }
 
 .ok-info-body {
+  display: flex;
+  column-gap: var(--icon-column-gap);
+  align-items: center;
   margin: var(--popup-body-margin);
 }
 
@@ -913,6 +916,9 @@ div.cast_crew.tabular {
 }
 
 .err-info-body {
+  display: flex;
+  column-gap: var(--icon-column-gap);
+  align-items: center;
   margin: var(--popup-body-margin);
 }

Display More

Damit sind Schrift und Icon perfekt zentriert.

MarkusE · September 1, 2025 at 7:04 PM

Die Patches sind im git.

Etwas ist mir noch aufgefallen: in pages/searchepg.ecpp, Zeile 615

Code

<option value="<$ blacklist->Id() $>" <{ SELECTIF(searchtimer && searchtimer->BlacklistSelected(blacklist->Id())) }>><$ blacklist->Search() $></option>

Hier ist searchtimer immer == nullptr. Sollten wir searchtimer da rauswerfen, oder ist das ein Fehler?

SHofmann · September 1, 2025 at 7:27 PM

Danke.

Das habe ich mit Absicht drin gelassen. Wie ich schon vor einiger Zeit angemerkt habe, sind edit_searchtimer.ecpp und searchepg.ecpp so aufgebaut, dass sich die gemeinsamen Teile per Vergleich leicht synchronisieren lassen. Man ändert Code in der einen Datei und übernimmt die Änderungen dann bequem in die andere.

Diese Stelle ist in einem "gemeinsamen" Teil und sollte deshalb auch identisch sein:

Damit das alles passt, wird searchtimer in Zeile 180 explizit der nullptr zugewiesen. Ist also keine funktionale Notwendigkeit, sondern vielmehr eine Bequemlichkeit für die Codepflege. Von dieser Herangehensweise habe ich auch bei meiner nächsten Änderung, die ich in der Pipeline habe, wieder umfassend Gebrauch gemacht.

Also ja: bitte drin lassen.

SHofmann · September 1, 2025 at 8:01 PM

Noch eine weitere Kleinigkeit: Bei Suchtimern und der EPG-Suche werden leere Auswahl-Boxen für Kategorien angezeigt, die von EPGsearch also Texteingabefelder charakterisiert sind. Das kommt daher, dass EPGsearch in diesen Fällen eine einelementige Liste mit einem leeren String liefert.

Dieser Patch:

Diff

 pages/edit_searchtimer.ecpp | 24 +++++++++++++-----------
 1 file changed, 13 insertions(+), 11 deletions(-)

diff --git a/pages/edit_searchtimer.ecpp b/pages/edit_searchtimer.ecpp
index d7ff4df..5449c05 100644
--- a/pages/edit_searchtimer.ecpp
+++ b/pages/edit_searchtimer.ecpp
@@ -706,23 +706,25 @@ const char *TNT_ARRAY = "";
                 <input type="checkbox" name="useextepginfo" value="1" <{ CHECKIF(useextepginfo) }> onclick="changeduseextepginfo(this)" id="useextepginfo"/>
                 <div id="use_extepg_settings" style="display: none">
                   <table class="dependent" border="0" cellspacing="0" cellpadding="0">
-%                 int i = 0; for (ExtEPGInfos::iterator extinfo = extEPGInfos.begin(); extinfo != extEPGInfos.end(); ++extinfo, i++) {
+                    <!-- Extended EPG categories -->
+%                   int i = 0; for (ExtEPGInfos::iterator extinfo = extEPGInfos.begin(); extinfo != extEPGInfos.end(); ++extinfo, i++) {
                     <tr>
                       <td class="label"><div class="withmargin"><$ extinfo->Name() $>:</div></td>
                       <td><input type="text" name="extepgvalues<$TNT_ARRAY$>" value="<$extepgvalues[i]$>" size="20" id="extepg_<{ reply.out() << extinfo->Id(); }>_value"/></td>
-%                     if (extinfo->Values().size() > 0) {
-                      <td>
-                        <select size="3" multiple="multiple" name="extepg_<{ reply.out() << extinfo->Id(); }>" onchange="changedextepginfo(this)">
-%                       for (unsigned int j = 0; j < extinfo->Values().size(); j++) {
-                          <option value="<{ reply.out() << extinfo->Values()[j]; }>"
-%                         if (extinfo->Selected(j, extepgvalues[i])) {
-                            selected="selected"
+                      <td><# we need an empty cell for retaining the layout #>
+                        <# EPGsearch returns a one-element list with an empty string for text-type categories #>
+%                       if (extinfo->Values().size() > 1 || (extinfo->Values().size() == 1 && !extinfo->Values()[0].empty())) {
+                          <select size="3" multiple="multiple" name="extepg_<{ reply.out() << extinfo->Id(); }>" onchange="changedextepginfo(this)">
+%                         for (unsigned int j = 0; j < extinfo->Values().size(); j++) {
+                            <option value="<{ reply.out() << extinfo->Values()[j]; }>"
+%                           if (extinfo->Selected(j, extepgvalues[i])) {
+                              selected="selected"
+%                           }
+                            > <{ reply.out() << extinfo->Values()[j]; }> </option>
 %                         }
-                          > <{ reply.out() << extinfo->Values()[j]; }> </option>
+                          </select>
 %                       }
-                        </select>
                       </td>
-%                     }
                     </tr>
 %                   }
 
 pages/searchepg.ecpp | 24 +++++++++++++-----------
 1 file changed, 13 insertions(+), 11 deletions(-)

diff --git a/pages/searchepg.ecpp b/pages/searchepg.ecpp
index 8b55639..1da308b 100644
--- a/pages/searchepg.ecpp
+++ b/pages/searchepg.ecpp
@@ -462,23 +462,25 @@ const char *TNT_ARRAY = "";
                 <input type="checkbox" name="useextepginfo" value="1" <{ CHECKIF(useextepginfo) }> onclick="changeduseextepginfo(this)" id="useextepginfo"/>
                 <div id="use_extepg_settings" style="display: none">
                   <table class="dependent" border="0" cellspacing="0" cellpadding="0">
-%                 int i = 0; for (ExtEPGInfos::iterator extinfo = extEPGInfos.begin(); extinfo != extEPGInfos.end(); ++extinfo, i++) {
+                    <!-- Extended EPG categories -->
+%                   int i = 0; for (ExtEPGInfos::iterator extinfo = extEPGInfos.begin(); extinfo != extEPGInfos.end(); ++extinfo, i++) {
                     <tr>
                       <td class="label"><div class="withmargin"><$ extinfo->Name() $>:</div></td>
                       <td><input type="text" name="extepgvalues<$TNT_ARRAY$>" value="<$extepgvalues[i]$>" size="20" id="extepg_<{ reply.out() << extinfo->Id(); }>_value"/></td>
-%                     if (extinfo->Values().size() > 0) {
-                      <td>
-                        <select size="3" multiple="multiple" name="extepg_<{ reply.out() << extinfo->Id(); }>" onchange="changedextepginfo(this)">
-%                       for (unsigned int j = 0; j < extinfo->Values().size(); j++) {
-                          <option value="<{ reply.out() << extinfo->Values()[j]; }>"
-%                         if (extinfo->Selected(j, extepgvalues[i])) {
-                            selected="selected"
+                      <td><# we need an empty cell for retaining the layout #>
+                        <# EPGsearch returns a one-element list with an empty string for text-type categories #>
+%                       if (extinfo->Values().size() > 1 || (extinfo->Values().size() == 1 && !extinfo->Values()[0].empty())) {
+                          <select size="3" multiple="multiple" name="extepg_<{ reply.out() << extinfo->Id(); }>" onchange="changedextepginfo(this)">
+%                         for (unsigned int j = 0; j < extinfo->Values().size(); j++) {
+                            <option value="<{ reply.out() << extinfo->Values()[j]; }>"
+%                           if (extinfo->Selected(j, extepgvalues[i])) {
+                              selected="selected"
+%                           }
+                            > <{ reply.out() << extinfo->Values()[j]; }> </option>
 %                         }
-                          > <{ reply.out() << extinfo->Values()[j]; }> </option>
+                          </select>
 %                       }
-                        </select>
                       </td>
-%                     }
                     </tr>
 %                   }

Display More

… prüft genau darauf und blendet die Auswahl-Boxen aus, wenn sie nicht benötigt werden:

MarkusE · September 2, 2025 at 3:57 PM

Quote from SHofmann

Also ja: bitte drin lassen.

O.K. ich lasse es drin. Auch wenn mir modulare Programmierung lieber wäre.

SHofmann · September 2, 2025 at 4:50 PM

Noch besser wäre es, die Redundanz herauszunehmen und beides auf eine gemeinsame Basis zurückzuführen…

Keine Ahnung, ob sich das mit TntNet vernünftig lösen lässt. Vielleicht, indem man für jedes Element eine Komponente erstellt?

MarkusE · September 2, 2025 at 5:03 PM

Der Patch aus #8 ist im git

SHofmann · September 3, 2025 at 1:21 PM

Danke dir! Gestern Abend kam mir noch der Gedanke, dass es für das Layout wohl besser ist, wenn sich die Text-Kategorien über zwei Spalten erstrecken (anstatt die dritte Spalte leer zu lassen). Ebenso sollte sich die Option für fehlende Kategorie entsprechend über drei Spalten erstrecken. Das wäre dann folgender Patch zur Korrektur:

Diff

 pages/edit_searchtimer.ecpp | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/pages/edit_searchtimer.ecpp b/pages/edit_searchtimer.ecpp
index 5449c05..88be603 100644
--- a/pages/edit_searchtimer.ecpp
+++ b/pages/edit_searchtimer.ecpp
@@ -710,10 +710,13 @@ const char *TNT_ARRAY = "";
 %                   int i = 0; for (ExtEPGInfos::iterator extinfo = extEPGInfos.begin(); extinfo != extEPGInfos.end(); ++extinfo, i++) {
                     <tr>
                       <td class="label"><div class="withmargin"><$ extinfo->Name() $>:</div></td>
-                      <td><input type="text" name="extepgvalues<$TNT_ARRAY$>" value="<$extepgvalues[i]$>" size="20" id="extepg_<{ reply.out() << extinfo->Id(); }>_value"/></td>
-                      <td><# we need an empty cell for retaining the layout #>
                         <# EPGsearch returns a one-element list with an empty string for text-type categories #>
-%                       if (extinfo->Values().size() > 1 || (extinfo->Values().size() == 1 && !extinfo->Values()[0].empty())) {
+%                     bool isTextCategory = extinfo->Values().size() == 0 || (extinfo->Values().size() == 1 && extinfo->Values()[0].empty());
+                      <td <?? isTextCategory ? "colspan=\"2\"" ?>>
+                        <input type="text" name="extepgvalues<$TNT_ARRAY$>" value="<$extepgvalues[i]$>" size="20" id="extepg_<{ reply.out() << extinfo->Id(); }>_value"/>
+                      </td>
+%                     if (!isTextCategory) {
+                        <td>
                           <select size="3" multiple="multiple" name="extepg_<{ reply.out() << extinfo->Id(); }>" onchange="changedextepginfo(this)">
 %                         for (unsigned int j = 0; j < extinfo->Values().size(); j++) {
                             <option value="<{ reply.out() << extinfo->Values()[j]; }>"
@@ -723,14 +726,14 @@ const char *TNT_ARRAY = "";
                             > <{ reply.out() << extinfo->Values()[j]; }> </option>
 %                         }
                           </select>
-%                       }
                       </td>
+%                     }
                     </tr>
 %                   }
 
                     <!-- Ignore missing EPG categories -->
                     <tr>
-                      <td><div class="options">
+                      <td colspan="3"><div class="options">
                         <div class="dotted">
                           <input type="checkbox" name="ignoreMissingEPGCats" value="1" <{ CHECKIF(ignoreMissingEPGCats) }> />
                           <label for="ignoreMissingEPGCats"><$ tr("Ignore missing EPG info") $> </label>
 pages/searchepg.ecpp | 17 ++++++++++-------
 1 file changed, 10 insertions(+), 7 deletions(-)

diff --git a/pages/searchepg.ecpp b/pages/searchepg.ecpp
index 1da308b..57bd1e6 100644
--- a/pages/searchepg.ecpp
+++ b/pages/searchepg.ecpp
@@ -466,10 +466,13 @@ const char *TNT_ARRAY = "";
 %                   int i = 0; for (ExtEPGInfos::iterator extinfo = extEPGInfos.begin(); extinfo != extEPGInfos.end(); ++extinfo, i++) {
                     <tr>
                       <td class="label"><div class="withmargin"><$ extinfo->Name() $>:</div></td>
-                      <td><input type="text" name="extepgvalues<$TNT_ARRAY$>" value="<$extepgvalues[i]$>" size="20" id="extepg_<{ reply.out() << extinfo->Id(); }>_value"/></td>
-                      <td><# we need an empty cell for retaining the layout #>
-                        <# EPGsearch returns a one-element list with an empty string for text-type categories #>
-%                       if (extinfo->Values().size() > 1 || (extinfo->Values().size() == 1 && !extinfo->Values()[0].empty())) {
+                      <# EPGsearch returns a one-element list with an empty string for text-type categories #>
+%                     bool isTextCategory = extinfo->Values().size() == 0 || (extinfo->Values().size() == 1 && extinfo->Values()[0].empty());
+                      <td <?? isTextCategory ? "colspan=\"2\"" ?>>
+                        <input type="text" name="extepgvalues<$TNT_ARRAY$>" value="<$extepgvalues[i]$>" size="20" id="extepg_<{ reply.out() << extinfo->Id(); }>_value"/>
+                      </td>
+%                     if (!isTextCategory) {
+                        <td>
                           <select size="3" multiple="multiple" name="extepg_<{ reply.out() << extinfo->Id(); }>" onchange="changedextepginfo(this)">
 %                         for (unsigned int j = 0; j < extinfo->Values().size(); j++) {
                             <option value="<{ reply.out() << extinfo->Values()[j]; }>"
@@ -479,14 +482,14 @@ const char *TNT_ARRAY = "";
                             > <{ reply.out() << extinfo->Values()[j]; }> </option>
 %                         }
                           </select>
-%                       }
-                      </td>
+                        </td>
+%                     }
                     </tr>
 %                   }
 
                     <!-- Ignore missing EPG categories -->
                     <tr>
-                      <td><div class="options">
+                      <td colspan="3"><div class="options">
                         <div class="dotted">
                           <input type="checkbox" name="ignoreMissingEPGCats" value="1" <{ CHECKIF(ignoreMissingEPGCats) }> />
                           <label for="ignoreMissingEPGCats"><$ tr("Ignore missing EPG info") $> </label>

Display More

Damit sieht das Ganze – vor allem bei schmalen Displays und Fenstern – auch besser aus:

Sorry für die nachgeschobene Änderung und die Extra-Arbeit, die du damit hast.

MarkusE · September 3, 2025 at 4:45 PM

ist im git

SHofmann · September 5, 2025 at 5:25 PM

Je mehr ich mich mit den erweiterten EPG-Informationen befasse, desto unübersichtlicher finde ich das Inspizieren bzw. Zusammenstellen von Kategorien per Mehrfachauswahl in einer nur dreizeiligen Auswahl-Box; auch ist das Layout damit recht unschön. Und das Textfeld mit dem daraus generierten String sollte sich zudem nicht ändern lassen.

Wie wäre es denn mit einer Lösung auf Basis von Check-Boxen, die sich eher an den Inhaltskategorien orientiert:

Hier der zugehörige Patch:

0001 Ext. EPG-info selection with check boxes.diff.txt

Wie sind die Meinungen hierzu?

PS: Die Felder mit den Strings der ausgewählten Kategorien braucht man eigentlich nicht. Man könnte sie deshalb natürlich auch gleich auf "hidden" setzen:

Hier das Diff für diese Ergänzung:

Diff

diff --git a/pages/edit_searchtimer.ecpp b/pages/edit_searchtimer.ecpp
index e0b1e4c..868eb17 100644
--- a/pages/edit_searchtimer.ecpp
+++ b/pages/edit_searchtimer.ecpp
@@ -719,7 +719,7 @@ const char *TNT_ARRAY = "";
 %                     cToSvConcat<16> id("extepg_");
 %                     id.appendInt<1>(extinfo->Id());
                       <td>
-                        <input type="text" name="extepgvalues<$TNT_ARRAY$>" value="<$extepgvalues[i]$>" size="20" id="<$ id $>_value" <?? !isTextCategory ? " readonly=\"1\"" ?> />
+                        <input type="text" name="extepgvalues<$TNT_ARRAY$>" value="<$extepgvalues[i]$>" size="20" id="<$ id $>_value" <?? !isTextCategory ? " hidden=\"1\" readonly=\"1\"" ?> />
 %                       if (!isTextCategory) {
                           <div class="options">
 %                         for (unsigned int j = 0; j < extinfo->Values().size(); j++) {
diff --git a/pages/searchepg.ecpp b/pages/searchepg.ecpp
index a9bdb58..11f038a 100644
--- a/pages/searchepg.ecpp
+++ b/pages/searchepg.ecpp
@@ -485,7 +485,7 @@ const char *TNT_ARRAY = "";
 %                     cToSvConcat<16> id("extepg_");
 %                     id.appendInt<1>(extinfo->Id());
                       <td>
-                        <input type="text" name="extepgvalues<$TNT_ARRAY$>" value="<$extepgvalues[i]$>" size="20" id="<$ id $>_value" <?? !isTextCategory ? " readonly=\"1\"" ?> />
+                        <input type="text" name="extepgvalues<$TNT_ARRAY$>" value="<$extepgvalues[i]$>" size="20" id="<$ id $>_value" <?? !isTextCategory ? " hidden=\"1\" readonly=\"1\"" ?> />
 %                       if (!isTextCategory) {
                           <div class="options">
 %                         for (unsigned int j = 0; j < extinfo->Values().size(); j++) {

Display More

Beziehungsweise alles zusammen:

0001a Ext. EPG-info selection with check boxes.diff.txt

Und wenn man es gleich so macht wie bei den Inhaltskategorien, sammelt man die ausgewählten Kategorien erst nach Abschicken des Formulars auf. Das ist aber eine etwas umfangreichere Änderung – MarkusE, was meinst du?

MarkusE · September 5, 2025 at 5:54 PM

Insgesamt gefällt mir diese Änderung gut. Es könnte aber zu Darstellungsproblem kommen, wenn die Werteliste zu groß wird.

MarkusE · September 5, 2025 at 6:08 PM

Die Patches aus #14 sind im git.

Vor größeren Umbauten solltest Du noch überlegen, wie wir mit sehr großen Wertelisten umgehen.

z.B. könntest Du prüfen, wie viel Platz die Liste mit den Checkboxen braucht. Und wenn das zu groß wird, auf altes Verhalten umschalten oder mehrere Zeilen machen oder ...

SHofmann · September 5, 2025 at 6:11 PM

Daran hatte ich auch schon gedacht. Aber Scrollen im Browser ist einfacher als in einer dreizeiligen Auswahlbox. Vor allem die Nutzer von Tablets wären mit den neuen Ansatz wohl besser dran.

Hier mal beispielhaft die doch recht umfangreiche Liste aus der TvMovie-Beispieldatei von EPGsearch:

Ich mag mir gar nicht vorstellen, wie man mit Textfeld und Auswahl-Box eine Übersicht gewinnen kann, wenn vielleicht ein gutes Dutzend Kategorien ausgewählt sind. Oder wie man überhaupt erfassen kann, welche Kategorien es "weiter unten" in der Liste so gibt.

nobanzai · September 5, 2025 at 6:22 PM

Wenn man das da oben dann noch in Spalten sauber ausgerichtet aufteilen könnte, ...

SHofmann · September 5, 2025 at 6:31 PM

Quote from MarkusE

Die Patches aus #14 sind im git.

Danke dir!

Quote from MarkusE

Vor größeren Umbauten solltest Du noch überlegen, wie wir mit sehr großen Wertelisten umgehen.
z.B. könntest Du prüfen, wie viel Platz die Liste mit den Checkboxen braucht. Und wenn das zu groß wird, auf altes Verhalten umschalten oder mehrere Zeilen machen oder ...

Ich fürchte, dass es dafür keine wirklich gute Lösung gibt. Wenn die Listen so umfangreich sind wie oben, wird die Usability mit Check-Boxen (neu) etwas, mit Auswahl-Boxen (alt) deutlich ungünstiger. Wobei ich das Scrollen mit Check-Boxen für das kleinere Übel halte.

Ob es besser ist, bei umfangreichen Datensätzen auf ein mehrstufiges Auswahlverfahren (etwa Slots 0-9, A, ..., Z entsprechend des ersten Zeichens eines Kategoriewerts) zu wechseln, wage ich auch zu bezweifeln. Denn auch damit kann man nicht auf einen Blick erfassen, was es gibt und was ausgewählt ist. Insofern sollte man sich überlegen, ob man da allzu viel Energie reinstecken sollte.

Wir haben in diesem speziellen Fall halt leider keinen Einfluss auf Umfang und Struktur der Ausgangsdaten. Das beginnt schon damit, dass wir nicht wissen, ob wir es mit einem halben Dutzend oder zwei Dutzend von EPG-Kategorien zu tun haben. Und auch nicht, wie umfangreich der Datensatz einer Kategorie ist. Jede VDR-Installation kann da anders sein.

Das von EPGsearch mitgelieferte Tool createcats zum Erstellen einer solchen Konfiguration aus den vorliegenden EPG-Daten kennt deshalb eine Option, bei einer zu umfangreichen Kategorienliste statt einer Auswahlliste nur eine textbasierte Eingabe vorzusehen:

Quote

−v N, −−maxvalues=N
Die Werte einer Kategorie bleiben unberücksichtigt, wenn sie die festgelegte Anzahl überschreiten.
Ist die Anzahl unterschiedlicher Werte für eine Kategorie größer als N, wird für die entsprechende Kategorie keine Werteliste erstellt. Die Kategorie wird stattdessen durch einen String repräsentiert, der mit beliebigem Text gefüllt werden kann. Andernfalls wird die Kategorie durch ein Menüelement zur Auswahl von Kategoriewerten aus der Liste abgebildet.

Ich finde, da muss jeder Nutzer für sich selbst entscheiden, wo der "Sweet Spot" im Einzelfall liegt.

SHofmann · September 5, 2025 at 7:11 PM

Quote from nobanzai

Wenn man das da oben dann noch in Spalten sauber ausgerichtet aufteilen könnte, ...

Das lässt sich mit CSS vermutlich sogar relativ leicht lösen. Was aber, wenn lange Kategoriewerte wie "Auslandsreportage" auf kurze wie "Drama" treffen? Dann haben wir jede Menge ungenutzten Leerraum zwischen den Spalten. Ob das dann besser ist? Vielleicht eine Option für die siteprefs.css…

PS: Man muss sich wohl zwischen fester Breite mit variabler Spaltenzahl oder fester Spaltenzahl mit variabler Breite entscheiden. Wie man die vom Layout ermittelte Breite des Containers mit so vielen Spalten wie möglich nutzen kann, konnte ich nicht herausfinden.

Participate now!