Gerätefilter usw. mit Regexpattern

[IT-VBFK]

Describe the solution you'd like

Anstatt nur mit string matches zu filtern, wäre es hin und wieder ganz hilfreich, mittels Regex-Pattern zu filtern.

Hintergrund: Ich habe Aktoren und Thermostate die bis auf das Prefix gleich heißen. zB. Ak1_Raum1 und Th1_Raum1
Da die beiden mehr oder weniger zusammengehören, möchte ich nun zB hiermit filtern: (Th|Ak)1.*, mit genau diesen zwei Ergebnissen.

Describe alternatives you've considered

Manuell filtern

Is your feature request related to a problem?

Manuell filtern fühlt sich halt ein bisschen umständlich an :)

Additional information

No response

[jp112sdl]

1.) Einstellungen->Geräte
Dort geht es jetzt schon, dass man mittels Pattern (|-getrennt) mehrere Begriffe angeben und suchen kann.

Allerdings nicht "Regex-like", sondern es wird nur geschaut, ob die Suchzeichenfolge irgendwo im Text vorkommt.

Lässt sich jedoch relativ einfach in der webui.js anpassen (aktuellen Code habe ich // auskommentiert):

  /**                                                                                                                                                           
   * Pr?ft, ob der Filter auf einen Text zutrifft                                                                                                               
   **/                                                                                                                                                          
  this.match = function(text)                                                                                                                                   
  {                                                                                                                                                             
    if (false === isSet()) { return true; }                                                                                                                     
                                                                                                                                                                
    var patternList = m_value.toLowerCase().split("|");                                                                                                         
    //var patternList = m_value.split("|");                                                                                                                     
    text            = text.toLowerCase();                                                                                                                       
                                                                                                                                                                
    //for (var i = 0, len = patternList.length; i < len; i++)                                                                                                   
    //{                                                                                                                                                         
    //  if (0 <= text.indexOf(patternList[i])) { return true; }                                                                                                 
    //}                                                                                                                                                         
    var r = new RegExp(m_value.toLowerCase());                                                                                                                  
    if (r.test(text) === true) {return true; }                                                                                                                  
                                                                                                                                                                
    return false;                                                                                                                                               
  };   

2.) Status und Bedienung, sowie Einstellungen->Systemvariablen

Hier funktioniert 1. nicht.

Aber wenn ich auch hier in der webui.js aus

    if (this.filName !== "")                 
    {                                                                                 
      if (typeof obj['name'] != 'undefined')                                          
      {                                                                               
        if (obj['name'] === "")                      { return false; }                    
        var transName = translateString(obj['name']);              
        //if (transName.toLowerCase().indexOf(this.filName) == -1) { return false; }
        var r = new RegExp(this.filName);                                           
        if (r.test(transName.toLowerCase()) === false) {return false; } 
      }                                             
    } 

die aktuell genutzte Such-Methode auskommentiere und gegen eine Regex-Funktion austausche, dann sieht es erstmal gut aus.

Und es funktioniert dann auch noch weiterhin wie gehabt: Teilstrings werden als Ergebnis ausgeworfen.

Wobei man auf die Groß-/Kleinschreibung weiterhin verzichten müsste, da sowohl Filtertext als auch Wert in lowerCase gewandelt werden.

Das würde ich auch so beibehalten wollen.

[IT-VBFK]

Cool :)

Danke fürs checken

Wenn mir noch jemand erklärt, wie man das hier via PR einreicht, kann ich mir das mal anschauen ;)

Wenn diese Änderung überhaupt gewünscht ist..

[jp112sdl]

Wenn mir noch jemand erklärt, wie man das hier via PR einreicht, kann ich mir das mal anschauen ;)

• Fork ziehen
• in buildroot-external/patches/occu anschauen, wie andere Patche aussehen
• neues Patchverzeichnis erstellen und betroffene/benötigte Patchdateien darin ablegen
• create_patches.sh ausführen
• Änderungen in einem neuen eigenen Branch committen
• PR erstellen

Wenn diese Änderung überhaupt gewünscht ist..

Selbst wenn nicht, kannst du ja für dich und dein System einen Patch bauen.
Ich werd es auf jeden Fall in meine persönliche Customize-Sammlung mit aufnehmen.

[jens-maus]

Bin mir nicht mal sicher ob es so eine gute Idee ist jetzt Otto-Normal-Verbraucher so etwas wie eine RegExp erklären zu müssen. Auch wird das sicherlich wieder irgendwelche Nebeneffekt haben wenn jemand jetzt nur ein * oder so anderes Sonderzeichen dort eingibt....

[jp112sdl]

@jens-maus Denk in Ruhe drüber nach.
Ggf. könnte man auch eine extra [ ] Regex-Suche-Checkbox in das Filter-Popup einbauen.

@IT-VBFK : Du kannst es ja mal testen. Ich bin vorerst zufrieden damit.
Falls das hier in RM nicht reinkommt, kannst du den Patch ja auch in einem Homematic-Forenbeitrag bereitstellen und dann kann sich das jeder, der meint damit umgehen zu können, selbst installieren.

1006-WebUI-RegexFilter.txt

[IT-VBFK]

@jens-maus Denk in Ruhe drüber nach. Ggf. könnte man auch eine extra [ ] Regex-Suche-Checkbox in das Filter-Popup einbauen.

@IT-VBFK : Du kannst es ja mal testen. Ich bin vorerst zufrieden damit. Falls das hier in RM nicht reinkommt, kannst du den Patch ja auch in einem Homematic-Forenbeitrag bereitstellen und dann kann sich das jeder, der meint damit umgehen zu können, selbst installieren.

1006-WebUI-RegexFilter.txt

Kann man diesen Patch einfach so mit Boardmitteln anwenden?

[jp112sdl]

Ja natürlich.
mount -o remount,rw / ; cd / ; patch -p 1 -i /pfad/zum/patchfile

[IT-VBFK]

Herzlichen Dank für die Hilfe :)

Das mit dem PR schau ich mir am WE mal in Ruhe an

[IT-VBFK]

neues Patchverzeichnis erstellen und betroffene/benötigte Patchdateien darin ablegen

Noch eine Frage: da braucht man eine webui.js.orig woher bekomm ich die? aus dem OCCU-Package?

[jp112sdl]

Schau mal in die anderen Patchverzeichnisse. Dann nimmst die neueste die du da finden kannst

[IT-VBFK]

Ok,

sorry wenn ich nochmal nachhake..

Was ich gemacht habe:

• die die webui.js.orig aus dem patch 168 genommen
• die aktuelle webui.js aus meinem System (mit den Änderungen) auch in dieses Verzeichnis gesteckt
• create_patches.sh aufgerufen

Nun habe ich aber komischerweise extrem viele Änderungen im patchfile, die gar nichts damit zu tun haben. Kann das sein?
Des weiteren hab ich ein paar weiterer Patches die auch als "modified" gekennzeichnet sind.. Aber gut.. die muss man ja nicht committen

[jens-maus]

Optimalerweise nimmt man immer die webui.js direkt aus dem OCCU als basis bzw. orig patchdatei und fügt dort seine änderungen ein. Wenn man allerdings an der selben stelle wie ein anderer WebUI Patch agiert bzw. etwas ändert, dann muss man natürlich die webui.js dieses früheren Patches nutzen. Deshalb auch die aufsteigenden Patch IDs die auch nicht recyclt werden dürfen.