"Zeitdifferenz formatieren" liefert "falsches" Format

[Paddex]

Hallo an alle,

ich habe letzte Woche den (für mich) neuen Block "Zeitdifferenz formatieren" entdeckt, musste aber feststellen, dass das Format nicht richtig angezeigt wird.

06:09 -> 6 Minuten 9 Sekunden wird mit falsch angezeit:

Formatierung ist:

Er entfernt die Nullen! Was bei der Minute noch ok wäre, ist aber bei den Sekunden absolut falsch.

Leider ist ein umformatieren mit bem Konvertierungsblock Datum/Zeit auch nicht möglich.

Wenn die Zeitdifferenz negativ wird, dann wir z.B. 0-1:55 angezeigt. Dann ist eine Auswertung ob negativ nicht mehr möglich.

Schreibe hier, weil ich nicht genau weiß, wo der Bug zu melden ist. Hinweise nehme ich gerne.

Danke an die Programmieren, ioBroker ist echt super. Danke für eure Arbeit.

Gruß

Maik

[paul53]

@paddex sagte: wo der Bug zu melden ist.

Als Issue auf Github.

Ich konnte es nachvollziehen (Version 8.3.0).

Negative Werte werden offenbar auch falsch berechnet.

[paul53]

@paddex
Ich habe die Funktion aus dem Javascript-Adapter mal nachvollzogen: Es werden führende Nullen erzeugt. Lediglich die Berechnung von negativen Differenzen ist falsch.

Spoiler

       function formatDiffTime(diff, format) {
           if (!format) {
               format = 'hh:mm:ss';
           }

           let text = format;

           log(`formatTimeDiff(format=${format}, diff=${diff})`);

           const second = 1000;
           const minute = 60 * second;
           const hour = 60 * minute;
           const day = 24 * hour;

           if (/DD|TT|ДД|D|T|Д/.test(text)) {
               const days = Math.floor(diff / day);

               text = text.replace(/DD|TT|ДД/, days < 10 ? `0${days}` : days);
               text = text.replace(/D|T|Д/, days);

               log(`formatTimeDiff(format=${format}, text=${text}, days=${days})`);

               diff -= days * day;
           }

           if (/hh|SS|чч|h|S|ч/.test(text)) {
               const hours = Math.floor(diff / hour);

               text = text.replace(/hh|SS|чч/, hours < 10 ? `0${hours}` : hours);
               text = text.replace(/h|S|ч/, hours);

               log(`formatTimeDiff(format=${format}, text=${text}, hours=${hours})`);

               diff -= hours * hour;
           }

           if (/mm|мм|m|м/.test(text)) {
               const minutes = Math.floor(diff / minute);

               text = text.replace(/mm|мм/, minutes < 10 ? `0${minutes}` : minutes);
               text = text.replace(/m|м/, minutes);

               log(`formatTimeDiff(format=${format}, text=${text}, minutes=${minutes})`);

               diff -= minutes * minute;
           }

           if (/ss|сс|мм|s|с/.test(text)) {
               const seconds = Math.floor(diff / second);

               text = text.replace(/ss|сс/, seconds < 10 ? `0${seconds}` : seconds);
               text = text.replace(/s|с/, seconds);

               log(`formatTimeDiff(format=${format}, text=${text}, seconds=${seconds})`);

               diff -= seconds * second;
           }

           log(`formatTimeDiff(format=${format}, text=${text})`);

           return text;
       }

log(formatDiffTime(369000, 'mm:ss'));

Blockly übergibt das falsche Format an Javascript:

console.info(formatTimeDiff(diff, 'm:s'));
console.info(formatTimeDiff((0 - diff), 'm:s'));

EDIT: PR für Korrektur bei negativen Werten ist erstellt.

[paul53]

@paddex sagte: Formatierung ist:

Das wird von Blockly falsch übergeben, aber anwenderformatiert funktioniert es mit positiven Differenzen:

EDIT: Neue Erkenntnis: "mm:ss" und "m:s" sind vertauscht:

[haus-automatisierung]

@paul53 sagte in "Zeitdifferenz formatieren" liefert "falsches" Format:

Blockly übergibt das falsche Format an Javascript:

Oh man wie blöde, war alles richtig, nur die Übersetzungen waren vertauscht. Danke!

[Paddex]

@paul53 said in "Zeitdifferenz formatieren" liefert "falsches" Format:

anwenderformatiert funktionier

Hallo,

"anwenderformatiert" war leider auch nicht die Lösung wegen der negativ Darstellung.

Was mir gerade aufgefallen ist:

positiv 6 min 9 sek -> negativ 7 min 51 sek ??
Wo ist mein Denkfehler?

Ansonsten ist der Block richtig gut und spart das selber zusammstellen der Zeit.

Gruß Maik

[paul53]

@paddex sagte: nicht die Lösung wegen der negativ Darstellung.

Das ist in der nächsten Version korrigiert.

[haus-automatisierung]

@paul53 sagte in "Zeitdifferenz formatieren" liefert "falsches" Format:

Das ist in der nächsten Version korrigiert.

Genau, ich hab noch 1-2 Bugfixes heute Abend und dann gebe ich die nächste Version frei.

[paul53]

@haus-automatisierung
Anmerkung: formatDate() kann auch Zeitdifferenzen formatieren (Werte < 1.1.2000), liefert aber bei negativen Werten auch ein falsches Ergebnis: -369000 --> "53:51".

[paul53]

@paddex sagte: negativ 7 min 51 sek ??

JS 8.3.1: