[vz-users] vzlogger an zweite middleware

[Frank Richter]

Hallo Andreas,

Am 26. Januar 2017 um 17:00 schrieb Andreas Goetz <cpuidle at gmail.com>:

> Servus,
>
> 2017-01-26 16:43 GMT+01:00 Frank Richter <frank.richter83 at gmail.com>:
>
>> Hi Andreas,
>>
>> Am 26. Januar 2017 um 10:47 schrieb Andreas Goetz <cpuidle at gmail.com>:
>>
>>> Hi Frank,
>>>
>>> 2017-01-23 11:29 GMT+01:00 Frank Richter <frank.richter83 at gmail.com>:
>>>
>>>> Moin Andreas,
>>>>
>>>> Am 23. Januar 2017 um 09:29 schrieb Andreas Goetz <cpuidle at gmail.com>:
>>>>
>>>>> Hallo Zusammen,
>>>>> ...
>>>>>
>>>>
>>> Bitte mal wui.js so anpassen, dann sollte das abonnieren wieder tun:
>>>
>>>     $('#entity-public input[type=button]').click(function() {
>>>         // clone entity from data attribute
>>>         var entity = $.extend({}, $('#entity-public-entity
>>> option:selected').data('entity'));
>>>         try {
>>>             entity.cookie = Boolean($('#entity-public-cook
>>> ie').prop('checked'));
>>>             delete entity.active;
>>>             vz.wui.addEntity(entity);
>>>         }
>>>         catch (e) {
>>>             vz.wui.dialogs.exception(e);
>>>         }
>>>         finally {
>>>             $('#entity-add').dialog('close');
>>>         }
>>>     });
>>>
>>
>> Hab den Block ausgetauscht, Verhalten ist aber unverändert (neu
>> abonnierter Kanal ist nicht aktiv).
>>
>
> Echt?! Mist, ich schaus an. Issue als Reminder?
>

Mehrfach getestet, also ich würde sagen echt...
Issue mach' ich auf.


>
>
>>
>>
>>>
>>>>
>>>> Das ist tatsächlich das erwartete Verhalten laut Code: das active Flag
>>>> wird nicht gespeichert und überlebt damit nicht die Browser Session. Wenns
>>>> da einen Änderungswunsch gibt bitte Issue aufmachen.
>>>>
>>>>
>>>> ...
>>>>
>>>
>> Finde ich nicht so schön: Bisher habe ich auf allen Geräten alle Kanäle
>> abonniert, aber nur eine kleine Auswahl auf aktiv gesetzt (wegen
>> Performance und Übersichtlichkeit). Wenn ein persistentes active flag jetzt
>> ersatzlos wegfällt, werden bei jedem Reload des Frontends erstmal alle
>> Kanäle geladen, was erstens dauert und zweitens einen kaum lesbaren Chart
>> erzeugt.
>>
>
> Da fällt nix weg, das gabs noch nie bzw. nur durch Versehen- sollte das
> anders sein bräuchte ich einen alten Commit > 2 Monate bei dem das schonmal
> ging.
>
>
>> Dann muss ich erst alles abwählen, was ich gerade nicht sehen will.
>> Alternative wäre dann nur, seltener genutzte Kanäle nur nah Bedarf zu
>> abonnieren, aber das kommt mir jetzt nicht wie ein Fortschritt vor.
>> Besonders unpraktisch für Leute, die zum abbonieren die UUID brauchen
>> (private Kanäle, z.B. Nutzer von demo).
>>
>
> Ich hab dafür einfach unterschiedliche Gruppen mti den gelichen Kanälen.
> Performance ist dank persistenter Middleware (ppm) eh kein Problem mehr.
>
>
>> Gegen Speicherung des active flags im Cookie hätte ich nichts
>> einzuwenden, das würde auch auf verschiedenen Clients verschiedene
>> default-Ansichten ermöglichen.
>> Ein issue dafür kann ich gerne aufmachen, wenn ich die ganze Sache nicht
>> immer noch missverstanden haben sollte...
>>
>
> Ja bitte, ich finde den Request super sinnvoll. Wenns das früher schonmal
> gab würde ich allerdigns lieber reparieren als neu zu bauen.
>

Als Cookie gab es das wohl noch nie, aber bis letzte Woche hat es eben
zumindest mit dem active flag aus der DB funktioniert, auch wenn das nie so
gedacht war...
Weiß nicht ob du das überlesen hast: Wirklich ignoriert wird das active
flag aus der DB meiner Meinung nach bisher nicht: Kanäle die active=false
haben, werden beim Reload des Frontends auch inaktiv geladen. Nur setzen
geht halt nicht mehr.

Grüße
Frank


>
>
>> Viele Grüße
>>
>> Andreas
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://demo.volkszaehler.org/pipermail/volkszaehler-users/attachments/20170126/6b890134/attachment.html>