update readingsGroup: sortierbarkeit

[Reinerlein]

Hi Andre,

die Möglichkeit klingt super, auch wenn ich sie jetzt erst entdeckt habe

Könntest du Analog zu "<tfoot>" noch ein "<thead>" ermöglichen, ab dessen Vorkommen überhaupt erst sortiert wird?
Bei meinen Readingsgroups gibt es fast immer einen Header mit trennender Linie darunter (genauso wie einen Footer mit Linie abgetrennt). Damit kann ich das auch direkt auf Oberflächen wie TabletUI einbinden, und habe eine nette Optik (zumindest für mich) .

Ich habe das mal versucht, aber es wird immer die obere Trennlinie mit sortiert

Außerdem wird vor dem Pfeil immer umgebrochen, sodass der Pfeil unter dem Titelwort steht. Kann ich das mit einem Style anpassen?

Danke schon mal...

Grüße
Reiner

[Reinerlein]

Hi Andre,

was mir noch aufgefallen ist:
Wenn ich nach einer Spalte sortiert habe, werden die im Fußbereich stehenden Summen nicht per longpoll aktualisiert.
Nach einem Seiten-Reload wird die Summe wieder korrekt angezeigt.

Da scheint etwas mit den Longpoll-FeldIDs schief zu gehen, wenn die dargestellte Reihenfolge wegen der Sortierung verändert wurde...
Das habe ich aber noch nicht weiter untersucht, sondern nur bemerkt. Vielleicht kannst du es ja nachstellen

Grüße
Reiner

[mi.ke]

Bei der Sortierung nach Spalten (sortColumn) scheint es bei 1-stelligen Zahlenwerten zu Problemen zu kommen.

Ich denke, hab 'ne Lösung in Perl gefunden, aber für einen Patch-Vorschlag reicht mein Perl nicht aus.

IST:

Code: [Auswählen]

@Test=(1..20);
sort(@Test));

SOLL:

Code: [Auswählen]

@Test=(1..20);
sort{$a<=>$b}(@Test));

@justme1968
Bin ich da auf dem richtigen Weg?

[mi.ke]

so, hatte mit einem anderen Readings die Erkenntnis, das es funktioniert.

sobald aber die Einheiten drangehängt werden, sortiert es wieder nicht richtig.

Code: [Auswählen]

valueFormat {moisture => "%.0f %%"}
Ohne valueFormat einwandfrei

[justme1968]

sorry für die späte antwort...

@Reinerlein: <thead> wird automatisch über das headerRows attribut gesteuert.

das problem mit dem nicht aktualisieren kann ich nicht reproduzieren. mein standard test beispiel ganz oben aus dem ersten post zeigt das problem nicht. kannst du mal damit testen bzw. ein beispiel posten bei dem es passiert?

@mi.ke: das sortieren passiert komplett auf javascript seite im browser. d.h. auf perl seite etwas zu ändern hilft nichts. der sorttable javascript code versucht den spalten typ anhand des ersten zeichens zu erkennen. das ist leider nicht 100% sicher und scheint gerade bei denen daten schief zu gehen. im prinzip kann man den daten typ einer spalte auch von aussen fest vorgeben. das geht aber aus der readingsGroup nicht.

man kann aber über ein 'missbrauchtes' valueColumns attribut für dieses reading { "sorttable_customkey='$NUM'"} einen workaround bauen. damit wird einer tabellen zelle direkt der numerische teil des readings als key zugeordnet.

[Reinerlein]

Hi Andre,

hmm, das mit der fehlenden Longpoll-Aktualisierung konnte ich jetzt auch nicht mehr nachstellen... sorry dafür... war wahrscheinlich eine komische Konstellation...

Aber zum Header: wenn ich das Attribut "headerRows" auf 2 setze (Eine Textzeile und ein <hr> als Kopf), dann erhalte ich sofort beim Aufruf der Seite die Fehlermeldung aus dem Screenshot 1.
Wenn ich dort eine 1 eintrage, sortiert er das <hr> wieder mit.

Hier mal das Define des Devices:

Code: [Auswählen]

defmod heizung_Sammler_Stellventile readingsGroup <Name>,<Modus>,<Wunschtemperatur>,<Ventilstellung> <hr> type=HeatingThermostat:mode,desiredTemperature,valveposition <tfoot> <hr> heizung_Sammler_Stellventile:<Max.&nbsp;;Ventilstellung>,<>,<>,$max(:3)@ventilmax
attr heizung_Sammler_Stellventile userattr event-on-change-reading
attr heizung_Sammler_Stellventile alias Heizung Stellventile
attr heizung_Sammler_Stellventile alwaysTrigger 2
attr heizung_Sammler_Stellventile group 1. Grundsätzliches
attr heizung_Sammler_Stellventile nameStyle style="font-weight: bold;; text-align: left;;"
attr heizung_Sammler_Stellventile room Heizung
attr heizung_Sammler_Stellventile sortColumn 4
attr heizung_Sammler_Stellventile valueFormat { 'desiredTemperature.on' => "on", 'desiredTemperature.off' => "off", 'desiredTemperature.comfort' => "comfort", 'desiredTemperature.boost' => "boost", 'desiredTemperature.eco' => "eco", desiredTemperature => "%.1f °C", valveposition => "%i %%", ventilmax => "%i %%" }
attr heizung_Sammler_Stellventile valueStyle { desiredTemperature => 'style="text-align:right"', valveposition => 'style="text-align:right"', ventilmax => 'style="text-align:right"' }

Was mir noch aufgefallen ist: Die Sortierung ist nicht deterministisch, bedeutet, dass bei jedem Aktualisieren eines Devices mittels longpoll die Liste u.U. anders sortiert wird, wenn die Einträge der sortcolumn gleich sind.
Folgende Situation:
- Alle meine Heizungsventile sind geschlossen, ich zeige meine ReadingsGroup an, und sortiere absteigend nach der Ventilstellung. Da alle Werte 0% enthalten, wird nach dem Namen der Devices absteigend sortiert -> Super
- Jetzt drücke ich bei einem Thermostat auf Boost, damit ein Eintrag auf 80% wechselt. Nach der Aktualisierung mittels longpoll steht der Eintrag mit den 80% jetzt oben, der Rest der Liste mit den 0% wird jetzt hinter den 80% aufsteigend nach Name sortiert (Screenshot 2) -> Auch Super
- Wenn ich jetzt den Boostmodus wieder abschalte, wird irgendwie gar nicht mehr sortiert, und die Reihenfolge ist beliebig, und wechselt nach jedem longpoll-Aktualisieren eine Devices auch noch (Screenshot 3). Da ist ab jetzt also ständig Bewegung drin, da jedes Geräte ja alle 5 Minuten aktualisiert wird.

Außerdem wäre für mich eine Umkehrung des Pfeil-Icons passender. Momentan ist der Pfeil nach oben, wenn absteigend sortiert wird. Intuitiv wäre doch die dünne Spitze der kleine Zahlenwert, und die breite Kante der große Zahlenwert bei der Sortierung, oder? Aber das ist vielleicht auch einfach Geschmackssache

Grüße
Reiner

[mi.ke]

Außerdem wäre für mich eine Umkehrung des Pfeil-Icons passender. Momentan ist der Pfeil nach oben, wenn absteigend sortiert wird. Intuitiv wäre doch die dünne Spitze der kleine Zahlenwert, und die breite Kante der große Zahlenwert bei der Sortierung, oder? Aber das ist vielleicht auch einfach Geschmackssache

Definitiv Geschmackssache
Ich finde die Pfeilrichtung so wie z.Z. verwendet genau richtig.

man kann aber über ein 'missbrauchtes' valueColumns attribut für dieses reading { "sorttable_customkey='$NUM'"} einen workaround bauen. damit wird einer tabellen zelle direkt der numerische teil des readings als key zugeordnet.

Damit funktioniert die Sortierung auch mit dem valueFormat und den Einheiten wie erwartet.
Klasse, dank Dir.

[stefan-dd]

Kann man auch eine Sortierung versteckt fest vorgeben?
Ich möchte die Räume nach meiner Vorgabe sortieren. Entweder über ein Reading was nicht mit dargestellt wird, oder kann man die Reihenfolge fest vorgeben?
In der 1. Spalte habe ich die Räume stehen, die jetzt Alphabetisch sortiert werden. Das gefällt mir aber nicht.

[HomeAuto_User]

Hallo,

ich habe mal die Funktion ausgetestet mit "sortColumn Spalte" und erhalte

Code: [Auswählen]

sorttable.js line 203:
TypeError: table.tHead is null
Woran liegt dies? FHEM ist aktuell.

MfG

[MadMax-FHEM]

Hallo,

gleiches Problem (fhem Update gestern).

Allerdings (wie hier: https://forum.fhem.de/index.php/topic,69858.0.html) nur wenn ich in den Raum "Everything" gehe.

Rufe ich einen Raum auf wo die readingsGroup und noch ein zwei andere Dinge drin sind funktioniert es ohne Fehler.

Bin gestern über die Möglichkeit zu sortieren gestolpert: TOLL!! DANKE!!

Ich habe eine readingsGroup wo ich mir die Batteriewechsel anzeige und da wäre es toll, wenn das nach Datum ginge

Zunächst hatte ich meine readingsGroup wie folgt definiert:

Code: [Auswählen]

define rgLastBateryChange readingsGroup NAME=dmLastBatteryChange:.*
Allerdings habe ich dann den "Sortierungspfeil" beim ersten Datum.
Optisch unschön und die Zeile wurde dann nicht mit sortiert...
...doppelt unschön.

Daher habe ich erweitert auf:

Code: [Auswählen]

define rgLastBateryChange readingsGroup <Gerät>,<>,<Datum> NAME=dmLastBatteryChange:.*Also mit Headline.

Zuvor war es optisch unschön aber ohne genannten Fehler...
...jetzt war es optisch so wie gewollt aber eben mit dem genannten Fehler.

Ich habe mir mal die Stelle angeschaut (Zeile 203) und eigentlich wird ja etwas weiter oben geprüft, ob thead === undefined ist.
Hätte erwartet, dass dort dann entsprechend rausgesprungen wird?

Allerdings ist 'thead' bei mir eben nicht 'undefined' sondern eben 'null' (hab im Debugger geschaut)...

Hier ein list der readingsGroup:

Code: [Auswählen]

Internals:
   DEF        <Gerät>,<>,<Datum> NAME=dmLastBatteryChange:.*
   NAME       rgLastBatteryChange
   NR         159
   NTFY_ORDER 50-rgLastBatteryChange
   STATE      Initialized
   TYPE       readingsGroup
   mayBeVisible 1
   CONTENT:
     dmLastBatteryChange 1
   CONTENT2:
   DEVICES:
     ARRAY(0x51345b0)
     ARRAY(0x471abb8)
   fhem:
     lastDefChange 51
     last_update 1509447859.39155
   helper:
     DEF
     mapping    $READING
     positions:
       dmLastBatteryChange.Heizkoerperthermostat_EssZi 1:1
       dmLastBatteryChange.Heizkoerperthermostat_FabiZi 1:2
       dmLastBatteryChange.Heizkoerperthermostat_Kueche 1:3
       dmLastBatteryChange.Heizkoerperthermostat_SchlaZi 1:4
       dmLastBatteryChange.Heizkoerperthermostat_WoZi 1:5
       dmLastBatteryChange.Wandthermostat_Bad 1:6
       dmLastBatteryChange.Wandthermostat_EssZi 1:7
       dmLastBatteryChange.Wandthermostat_FabiZi 1:8
       dmLastBatteryChange.Wandthermostat_Kueche 1:9
       dmLastBatteryChange.Wandthermostat_SchlaZi 1:10
       dmLastBatteryChange.Wandthermostat_WC 1:11
       dmLastBatteryChange.Wandthermostat_WoZi 1:12
     values:
       formated:
         undef
         ARRAY(0x50879e0)
         ARRAY(0x3d43de8)
         ARRAY(0x5149330)
         ARRAY(0x4cf6c98)
         ARRAY(0x51417a8)
         ARRAY(0x52bc500)
         ARRAY(0x50c1818)
         ARRAY(0x5166d98)
         ARRAY(0x4f049c8)
         ARRAY(0x52bd100)
         ARRAY(0x510f958)
         ARRAY(0x51071c0)
       orig:
         undef
         ARRAY(0x5135968)
         ARRAY(0x5181e98)
         ARRAY(0x435e610)
         ARRAY(0x52bb0c8)
         ARRAY(0x4ef98f0)
         ARRAY(0x51360a0)
         ARRAY(0x514b578)
         ARRAY(0x4ff96f0)
         ARRAY(0x52bcfb0)
         ARRAY(0x5334e10)
         ARRAY(0x4c9fa50)
         ARRAY(0x508d970)
       prefixsuffix:
         undef
         ARRAY(0x4d66928)
         ARRAY(0x5135998)
         ARRAY(0x4ef9398)
         ARRAY(0x5081af0)
         ARRAY(0x45fbef0)
         ARRAY(0x510ecb8)
         ARRAY(0x4a8f270)
         ARRAY(0x52bb170)
         ARRAY(0x5164c08)
         ARRAY(0x46d9310)
         ARRAY(0x514ba40)
         ARRAY(0x3ff6558)
Attributes:
   alias      Übersicht Batteriewechsel
   mapping    $READING
   nolinks    1
   room       Overview,Overview Battery
   sortColumn 3
   sortby     09

Wenn ich die Zeile drüber von Prüfung 'thead === undefined' auf 'thead === null' ändere habe ich (natürlich) den Fehler nicht aber welche anderen Auswirkungen das hat weiß ich nat. nicht...

Code: [Auswählen]

//      if( table.tHead === undefined ) {
      if( table.tHead === null ) {
        console.log( 'doSort: table has no heading' );
        return;
      }

EDIT: Browser Firefox 56.0 (64-bit) auf Ubuntu 14.04 LTS 64Bit

EDIT3: Tritt auch bei meinem Android-Phone mit Chrome auf (also auch "nur", wenn ich auf die "Everything" Seite gehe. Die Batterie-Übersichtsseite [dort zeige ich nur 2 readingsGroup an eine davon mit dem sortColum] geht auch hier ohne Fehler)

EDIT2: noch ein list des Dummy

Code: [Auswählen]

Internals:
   NAME       dmLastBatteryChange
   NR         161
   STATE      ???
   TYPE       dummy
   READINGS:
     2017-06-18 21:09:49   Heizkoerperthermostat_EssZi Battery changed:
     2017-07-08 12:36:35   Heizkoerperthermostat_FabiZi Battery changed:
     2017-08-31 22:59:44   Heizkoerperthermostat_Kueche Battery changed:
     2017-01-18 01:23:19   Heizkoerperthermostat_SchlaZi Battery changed:
     2016-11-14 00:55:22   Heizkoerperthermostat_WoZi Battery changed:
     2017-08-14 07:38:04   Wandthermostat_Bad Battery changed:
     2017-05-06 23:30:38   Wandthermostat_EssZi Battery changed:
     2017-05-03 07:28:45   Wandthermostat_FabiZi Battery changed:
     2017-03-19 11:55:00   Wandthermostat_Kueche Battery changed:
     2017-04-22 06:51:20   Wandthermostat_SchlaZi Battery changed:
     2017-03-19 11:56:34   Wandthermostat_WC Battery changed:
     2017-04-14 09:11:26   Wandthermostat_WoZi Battery changed:
Attributes:
   event-on-change-reading .*

Gruß, Joachim

[sd]

Hallo,
tolle Funktionalität, aber wenn ich die Readingsgroup in das Dashboard einbinde, kommt folgender Fehler:

sorttable.js line 203:
TypeError: table.tHead is null

Hat jemand eine Idee?

[BaronThoran]

Moin,

sorttable.js line 203:
TypeError: table.tHead is null

ich habe das gleiche Problem. Wenn ich die Tabelle in einem seperaten Raum habe läuft alles, binde ich die Tabelle in einem Raum mit anderen Sachen ein bekomme ich diese Fehlermeldung.

[mi.ke]

Hallo Andrè

man kann aber über ein 'missbrauchtes' valueColumns attribut für dieses reading { "sorttable_customkey='$NUM'"} einen workaround bauen. damit wird einer tabellen zelle direkt der numerische teil des readings als key zugeordnet.

Damit funktioniert die Sortierung auch mit dem valueFormat und den Einheiten wie erwartet.
Klasse, dank Dir.

sorttable.js line 203:
TypeError: table.tHead is null

Den Fehler hab ich auch, allerdings nicht immer.

Was allerdings immer auftritt, sind Perl-Warnings im LOG.
Hier mal mit attr global stacktrace 1

Code: [Auswählen]

2019.02.04 00:08:19 1: PERL WARNING: Use of uninitialized value $NUM in concatenation (.) or string at (eval 62718) line 1.
2019.02.04 00:08:19 1: stacktrace:
2019.02.04 00:08:19 1:     main::__ANON__                      called by (eval 62718) (1)
2019.02.04 00:08:19 1:     (eval)                              called by ./FHEM/33_readingsGroup.pm (357)
2019.02.04 00:08:19 1:     main::lookup2                       called by ./FHEM/33_readingsGroup.pm (875)
2019.02.04 00:08:19 1:     main::readingsGroup_2html           called by ./FHEM/33_readingsGroup.pm (1209)
2019.02.04 00:08:19 1:     main::readingsGroup_detailFn        called by ./FHEM/01_FHEMWEB.pm (3169)
2019.02.04 00:08:19 1:     main::FW_devState                   called by ./FHEM/01_FHEMWEB.pm (1727)
2019.02.04 00:08:19 1:     main::FW_makeDeviceLine             called by ./FHEM/01_FHEMWEB.pm (1892)
2019.02.04 00:08:19 1:     main::FW_showRoom                   called by ./FHEM/01_FHEMWEB.pm (1083)
2019.02.04 00:08:19 1:     main::FW_answerCall                 called by ./FHEM/01_FHEMWEB.pm (535)
2019.02.04 00:08:19 1:     main::FW_Read                       called by fhem.pl (3610)
2019.02.04 00:08:19 1:     main::CallFn                        called by fhem.pl (727)
Gab es bei readingsGroup eigentlich ein Update in der Richtung.
Ich hab im trunk zwar nichts gefunden, aber ich wollte trotzdem mal fragen.

Falls nicht, könntest Du in einer freien Minute bitte noch mal schauen.
Die Funktion in readingsGroup ist ja eigentlich genial.

thx'n'greetz

mi.ke

[FunkOdyssey]

Ich meine, dass der JavaScript-Fehler nur erscheint, wenn zeitgleich ein Plot angezeigt wird.

[mi.ke]

Ich meine, dass der JavaScript-Fehler nur erscheint, wenn zeitgleich ein Plot angezeigt wird.

Nein (zumindest bei meinen Installationen).

Gerade getestet, ich habe einen Raum ohne Plots, aber mit 3 ReadingGroups.
2 davon sind sortierbar.
JavaScript-Fehler kam trotzdem, inkl. der LOG-Einträge.