UI - Einzeilig - Bedienung von Raum legt kein Reading an

[HomeAutoUser]

Hallo,
@bismosa ich habe etwas beim Testen mitbekommen und einen Fix offen gelegt.
Wenn du das UI auf einzeilig setzt und aus der Roomansicht (nicht Detailansicht wenn du das Device anklickst) das Device bedienst, wird DDselect gesetzt aber das Reading vom Kanal nicht.

[bismosa]

Hallo,

ich kann das nicht richtig nachstellen...ode habe das noch nicht richtig verstanden.

1.) Wenn ich die UI auf einzeilig setze:
Kein Reading DDSelected vorhanden. Dies wird erst nach dem ersten ändern in der DropDown-Liste gesetzt -> Würde ich als BUG verstehen.

2.) Wenn ich nun in der Room-Ansicht (oder auch in der Detail-Ansicht...bei mir ist das verhalten gleich) den Kanal auswähle, wird nur das Reading DDSelected gesetzt. Erst wenn ein Befehl ausgeführt wird, wird das Reading Channel verändert.
So sollte es auch eigentlich sein...Das Reading Channel gibt ja nur den letzten Sende bzw. Empfangskanal wieder.

Gruß
Bismosa

[HomeAutoUser]

Hallo,
ich drückte mich vielleicht ein wenig Informationsbehaftet aus :-D

• stell das UI bitte auf einzeilig mal
• schau vorher welche Readings vorhanden sind bzw. wie Ihre Zeitstempel sind
• wechselst du in die Raumansicht und drückst einen Kanal zur Steuerung
• wechselst in die Detailansicht und überprüfst die Readings nach dem Zeitstempel

Nun sollte der soeben gedrückte Kanal nicht aktualisiert sein oder das Reading fehlt wenn der Kanal noch nicht als Reading existent war.

Gruß

[bismosa]

Hallo,

ich habe mal folgendes ergänzt:

if ($DDSelected eq ''){
				readingsSingleUpdate($hash, "DDSelected", 1, 1);
				$DDSelected = 1;
			}

Hier ein größerer Auszug. Müsste in der Zeile 1217 sein...

if ($DDSelected eq ''){
				readingsSingleUpdate($hash, "DDSelected", 1, 1);
				$DDSelected = 1;
			}
			foreach my $rownr (1..$Channels) {
				if ($DDSelected eq "$rownr"){
					$html.= "<option selected value=".($rownr).">".($ChName[$rownr-1])."</option>";
				} else {
					$html.= "<option value=".($rownr).">".($ChName[$rownr-1])."</option>";
				}
			}

Dann wird, wenn kein Reading vorhanden ist, dieses erst angelegt und ggf. der erste Eintrag ausgewählt.

Ist es das?

Gruß
Bismosa

[bismosa]

Hallo!

Jetzt habe ich den Fehler wohl gefunden. Eine Lösung ist mir aber noch nicht bekannt. Ich habe das mal nachgestellt und hier eine Frage gestellt:
https://forum.fhem.de/index.php/topic,102944.0.html

Dies liegt definitiv an der Änderung: #10

Gruß
Bismosa

[HomeAutoUser]

Hallo @bismosa,
ich habe hier https://github.com/HomeAutoUser/Jaro/tree/fix_UI mal den aktuellen Branch gesetzt um hier die fixes für das UI anzugehen.

Die Aktualisierung von dem Kanal sollte kein Problem sein, das habe ich wo anders umgesetzt.
Also würde ich diesen Faden für die bisherigen UI Sachen nutzen um die Struktur nicht durcheinader zu wirbeln. Der Vorteil, wenn wir dann den Fix beheben, wird der Faden als angeschlossen markiert.

Auf gehts, Pö a Pö :-)

[HomeAutoUser]

Hier nach der Stelle

my $changecmd = "cmd.$name=setreading $name DDSelected ";			## this line set DDSelected to value after change in web
$html.= "<select name=\"val.$name\" onchange=\"FW_cmd('$FW_ME$FW_subdir?XHR=1&$changecmd ' + this.options[this.selectedIndex].value)\">";

muss ein Reload der Browserseite passieren dann geht es. Bisher muss ich nur noch die Stelle finden wo ich dazwischen komme. An dieser Stelle geht es nicht direkt.

Das kannst du reproduzieren indem du den Kanal mal wechselst, dann F5 drückst im Browser und dann das Kommando absetzt. So kommt immer das Richtige Kommano mit dem richtigen Kanal.

Wir sind dran :-)

[bismosa]

Hallo!

ja. Genau das ist es. Ein reload (aber bitte nicht der ganzen Seite, das habe ich schon probiert und lastet mein FHEM für mehrere Sekunden aus, wenn viele Browserfenster geöffnet sind) muss passieren, sobald das reading "DDSelected" geändert wird.
Ein Reload der ganzen Seite lässt sich erzwingen mit
FW_directNotify($filter, $fhemweb_instance, $javascript, "");
bzw.
FW_directNotify("FILTER=room=XXX", "#FHEMWEB:WEB", "location.reload('true')", "")
Aber das ist wirklich nicht schön bei vielen offenen Browserfenstern...
Soll wohl besser machbar sein mit einer separaten JavaScript Datei...und sich dort um die Aktualisierung kümmern.

Oder man baut das Kommando um. Man setzt das Reading, aber bei der Auswertung der Buttons wird auf das Reading geachtet...

Gruß
Bismosa

[HomeAutoUser]

Um ein Reload kommen wir nicht drum herum. Ich teste mal und kenne das Problem der mehreren Fenster weil ich es aktuell wo anders. Es gibt die Möglichkeit nur das aktive tab zu aktualisieren.

[bismosa]

Oder halt per JavaScript und den betreffenden Bereich aktualisieren...so wie es auch bisher (vor der Modifizierung) per LongPoll erfolgte und auch in allen anderen Devices funktioniert....
Aber wie auch schon im FHEM-Forum geschrieben...das ist neuland für mich...und ich hatte bisher noch keine Zeit und Gelegenheit mich da einzuarbeiten :(

Oder alternativ -> Den einen Sonderfall ignorieren und weiterhin im summaryFn die Anzeige machen. So hatte es ja auch sehr gut funktioniert. Es gibt dann halt die Darstellungsprobleme bei einzelnen Usern...

[HomeAutoUser]

@bismosa , gibt es ne Variante, das wir hinter dem Kommando

$html.= "<select name=\"val.$name\" onchange=\"FW_cmd('$FW_ME$FW_subdir?XHR=1&$changecmd ' + this.options[this.selectedIndex].value)\">";

mal das eingefügt bekommten?

fhem("trigger WEB.* JS:location.reload(true)");

Ich finde nur nicht den Faden um den Syntax zu beseitigen :-D

[bismosa]

Hallo!

Nein, das habe ich auch nicht geschafft mit in der Zeile einzufügen. Ich habe mal zum verdeutlichen/Testen einige Demos vorbereitet...
1.) So haben wir es bisher. Stark vereinfacht mit einem DropDown und einem Button:
99_Test_01-Wie jetzt.pm.txt
Der Effekt ist logischerweise der gleiche...aber eine einfache Grundlage zum testen :)

2.) Mit einem Reload:
99_Test_02-Reload.pm.txt
Hier habe ich den Setreading-Befehl im Dropdown auf einen "hidden" set-Befehl verändert. Hier wird dann auch das Reload angestossen.
Das funktioniert nur in der Raumansicht. Solange man in der Detailansicht ist, funktioniert dies nicht.
Ich bin auch dagegen. Wenn man etwas mehr im Raum (z.B. noch ein paar SVG-Statistiken) und dann ggf. noch mehrere Tabs offen hat ist (zumindest bei meinem Raspberry) FHEM für einige Zeit blockiert und mit dem neu Laden der kompletten Seiten beschäftigt!

3.) Mit einer Änderung in den Buttons:
99_Test_03-ohneAktualisierung.pm.txt

Hier braucht dann nicht mehr der Kanal mit dem Button verknüpft werden. Hier wird das Reading beim Ausführen des Befehls übernommen.

4.) Oder so wie andere es machen...mit einer externen JavaScript-Datei und einem Austausch nur des betreffenden HTMLs. Also eine eigene ID setzten und dann mittels
document.getElementById("element1").innerHTML = Neues HTML
...
Zitat von rudolfkoenig aus dem Forum:

Das erzeugte HTML muss die Aktualisierung in diesem Fall selbst uebernehmen.
Das kann (soweit ich es sehe) nur durch Verwendung von fhemweb_widgets erfolgen, wo updateLine (Beispiel dafuer ist fhemweb_readingsGroup.js oder fhemweb_readigsHistory.js, updateLine ist aber unerwuenscht/deprecated) oder updateDevs (Beispiel dafuer ist svg.js) aufgerufen wird.

Aber das habe ich auch alles noch nicht verstanden...

Gruß
Bismosa

[bismosa]

Hallo!

Ich habe nochmal ein wenig "rumgespielt". Ich habe mal versucht den entsprechenden JavaScript Code ins HTML mit einzubauen und die Funktion denn mit neuem HTML-Code aufzurufen. Leider ist es mir nicht gelungen. Ich konnte bisher nur Text ersetzen.
99_Test_04-JavaScript.pm.txt

Gruß
Bismosa