smartVISU visu.js uzsu mit Bedingung?

Hans Franz · 21 Februar 2015, 13:43:04

Hallo,
Um mich etwas einzuarbeiten habe ich versucht in der visu.js der uzsu eine Condition mitzugeben.

--- visu.js	2015-02-21 12:00:12.948726807 +0100
+++ visu.js.patched	2015-02-21 12:04:04.000000000 +0100
@@ -255,10 +255,16 @@
         var template = "";
         // tabelle der zeileneinträge abschliessen
         template += "<\/table>";
+        template += "<table style = 'border: 0'> <tr> <td> <form> <fieldset data-mini='true'> ";
+        // hier Bedingung für gesamte uzsu
+        template += //"<div data-role='controlgroup' data-type='horizontal' data-inline='true' >"+
+        "<tr><td><\/td><td> Bedingung<\/td><tr>"+
+        "<tr><td><\/td><td><input type='text' data-clear-btn='false' style = 'width:300px' id='uzsuCondition'"+
+        "<\/td><\/tr>";
         // hier der activierungsbutton für die gesamte uzsu
-        template += "<table style = 'border: 0'> <tr> <td> <form> <fieldset data-mini='true'> " +
-        "<input type='checkbox' id='uzsuGeneralActive'> <label for='uzsuGeneralActive'>UZSU Activate<\/label>" +
-        "<\/fieldset><\/form> <\/td>";
+        template += "<tr><td> <div data-role='controlgroup' data-type='horizontal' data-inline='true' >"+
+        "<input type='checkbox' id='uzsuGeneralActive'><label for='uzsuGeneralActive'>UZSU Activate<\/label><\/td>"+
+        "<\/fieldset><\/form><\/td>";
         // jetzt kommen noch die buttons in der basisleiste mit rein
         template += "<td> <div data-role='controlgroup' data-type='horizontal' data-inline='true' data-mini='true'>"; 
                 template += "<div data-role = 'button' id = 'uzsuAddTableRow'> Add Entry <\/div>";
@@ -294,7 +300,8 @@
         var weekDays =['MO','TU','WE','TH','FR','SA','SU'];
         // jetzt wird die tabelle befüllt
         // allgemeiner Status, bitte nich mit attr, sondern mit prop, siehe https://github.com/jquery/jquery-mobile/issues/5587
-        $('#uzsuGeneralActive').prop('checked',response.active).checkboxradio("refresh");       
+        $('#uzsuGeneralActive').prop('checked',response.active).checkboxradio("refresh"); 
+        $('#uzsuCondition').val(response.condition.replace(/__/g,' '));
         // auswahl format
         switch(customFormat[0]){
                 case '0':
@@ -417,6 +424,7 @@
         // bitte darauf achten, dass das zurückspielen exakt dem der anzeige enspricht.
         // gesamthafte aktivierung
         response.active = $('#uzsuGeneralActive').is(':checked');
+        response.condition = $('#uzsuCondition').val();
         // dispatcher für format
         switch(customFormat[0]){
                 case '0':
@@ -470,7 +478,9 @@
         }
         // über json interface / treiber herausschreiben
         if(saveSmarthome){
-                io.write(item, {active : response.active, list : response.list });
+            var fhemcondition = response.condition;
+                fhemcondition = fhemcondition.replace(/ /g,'__');
+                io.write(item, {active : response.active, condition : fhemcondition, list : response.list});
         }
 }
 // steuerung des Popups
@@ -537,7 +547,7 @@
         $(this).data('uzsu', response[0]);
     } 
     else {
-        $(this).data('uzsu', { active: true, list: [] });
+        $(this).data('uzsu', { active: true, list: [], condition: '' });
         }
 });
 // als zweites der handler für die callbacks des click events

Die kommt auch in der UZSU_execute richtig an, wird dort aber nach dem ersten Leerzeichen, ohne die es in einer Bedingung ja nicht geht, abgeschnitten.
Meine Lösung mit dem Replacement von " " durch "__" scheint mir nicht nur suboptimal sondern sieht allzusehr nach Bastellösung aus, da in der UZSU_execute noch zurückgewandelt werden muss.
Wie macht man so etwas elegant? Den ganzen json-String in Anführungszeichen zu setzen ist mir leider nicht gelungen.

Gruß
Hans

bgewehr · 29 März 2015, 12:20:18

Hallo, Hans,

an den Bedingungen denke ich auch schon einige Wochen herum.

Ich hätte aber gern pro Zeile die Möglichkeit, Bedingungen zu setzen. Dann aber muss man wieder pro Zeile einzelne WDT erzeugen, weil der ja nur eine Bedingung für alle UZSU-Zeilen kann. Ich hätte gern eine Möglichkeit, Feiertage und Ferientage mit einzubeziehen oder Türgriffsensoren usw.

Lass das bitte noch etwas reifen, wir müssen erst klären, ob das weiterhin mit dem smarthome.py-UZSU identisch erfolgen kann, oder ob wir forken müssen für fhem. Bisher haben wir versucht, das zu vermeiden...

Schau mal hier:
https://github.com/mworion/uzsu_widget/issues/13

Hans Franz · 29 März 2015, 13:34:53

Hallo Bernd,

Kein Thema

.
Mir ging es eigentlich auch nur darum, mich mal an js heranzuwagen. Bedingungen kann man ja auch in z.B. myUtils definieren.
Das git von mworion hatte ich vor einigen Tagen gesehen (und beschämt zu Boden geschaut: an Kompatibilität hatte ich nicht gedacht).
Bin gespannt, wie's weitergeht. Super Job, den du hier machst.

Gruß
Hans

bgewehr · 29 März 2015, 13:39:20

Danke, das freut mich!

bgewehr · 01 Dezember 2015, 22:26:25

So, MWorion hat nun die V 3.99 responsive im GitHub.

Siehe ISSUE 13: https://github.com/mworion/uzsu_widget/issues/13

Wir müssen nun feststellen, welche Conditions wir genau brauchen.

Auf jeden Fall müsste die Funktion UZSU_Execute dann so angepasst werden, dass sie bei gegebenen Bedingungen pro Zeile der UZSU einen eigenen WDT erzeugt, weil der ja nur EINE Condition haben kann und nicht pro Profil eine...

Bitte sagt mir Eure Meinung, welche Ausprägungen die Conditions haben müssen, dann leite ich das an MWORION weiter.

Wollt Ihr einen einfachen Perl-String für die Condition oder lieber ein Device Comp Value - Tripel?

Danke und Gruß
Bernd

dev0 · 05 Dezember 2015, 08:38:57

Ich würde einen Perl String vorziehen, da beim Triple eh nur der Komparator über die GUI auszuwählen wäre, Device und Value muss man sowieso eintippen.

bgewehr · 21 Februar 2016, 19:08:36

Ich habe MWorion jetzt vorgeschlagen, es wie abgebildet zu machen.

Im FHEM Code kann dann unterschieden werden: wenn nur das erste Texfeld gefüllt ist, dann als Perl evaluieren, sonst Device comp value.

dev0 · 21 Februar 2016, 19:46:21

Gute Idee.

bgewehr · 13 März 2016, 21:05:17

So, es ist soweit!

Mit dem aktuellen responsive_dev branch ist es MWORION gelungen, einen separaten Designtype 2 für fhem speziell mit einigen Features auszurüsten:

1. Die aktiven Variablen $we und !$we können nun an Stelle der Mo Di Mi Tagesreihe benutzt werden, um Schaltvorgänge auch von holidays und Feiertagen abhängig zu wählen. Sie werden im Designtype 2 immer zusätzlich angeboten.

2. In den expert-Einstellungen jeder Zeitreihe kann nun über eine Kombination aus fhemDevice, Vergleichsoperator und Wert oder aber über einen Perl String im Feld fhemDevice eine Bedingung formuliert werden, die dem WDT zugeordnet wird. Daraus folgt auch, dass nun wieder für jede Zeile (mit conditions) ein eigener WDT erstellt werden muss.

Nun ist die uzsu_execute in der 99_fronthem_utils.pm entsprechend anzupassen. Ich mach mich mal dran...

Um vom neuen Designtype Gebrauch zu machen, müsst Ihr in smartVISU den Designtype 2 mittels 2 im vierten Parameter ansteuern: