typo in 42_SMARTMON

[RichardCZ]

42_SMARTMON.pm:        $hash->{DEVICE_FIRMARE}=$v

sollte vmtl. FIRMWARE heissen.

[hexenmeister]

Huch, stimmt, werde ich mal korrigieren. Danke!

EDIT: Habe mal schnell geändert und eingecheckt.

[RichardCZ]

Huch, stimmt, werde ich mal korrigieren. Danke!

EDIT: Habe mal schnell geändert und eingecheckt.

Super. Bei der Gelegenheit hätte man Smartmon auch aus der ISO-8859-1 Zwangsjacke befreien können, aber das Nächste Mal vielleicht.

303c303
<   # Natuerlich muss der user auch der Gruppe "sudo" angeh�ren.
---
>   # Natuerlich muss der user auch der Gruppe "sudo" angehören.
404c404
<         $hash->{DEVICE_FIRMWARE}=$v;
---
>         $hash->{DEVICE_FIRMARE}=$v;
582c582
< =item summary_DE liefert einige Statistiken ueber S.M.A.R.T. kompatible Ger�te
---
> =item summary_DE liefert einige Statistiken ueber S.M.A.R.T. kompatible Geräte
688c688

Hmm wobei... vielleicht bekomme ich es auch so automatisiert hin.

[hexenmeister]

Auch eine gute idee. Schaue heute abend mal.

[RichardCZ]

Auch eine gute idee. Schaue heute abend mal.

Hm. Auch das ist zwar richtig:

https://svn.fhem.de/trac/changeset?reponame=&new=21598%40%2F&old=21597%40%2F

aber das File ist immer noch iso-8859-1 (=Latin1) kodiert. Ich habe es noch nicht ausprobiert,
aber ich würde sagen JETZT würde commandref_DE.html#SMARTMON falsch kodiert Umlaute anzeigen.

Die HTML-Entities haben davor geschützt.

Also entweder man macht es ganz richtig (IMHO) -> UTF-8 kodierte Umlaute (= Quelltext ist in UTF-8, vollkommen unabhängig davon ob man use utf8 nutzt oder nicht), oder man hat iso-8859-1 und lässt HTML Entities.

edit:

Kommando zurück, ich bin ein Idiot.

Wenn man schon nicht lesen kann. Mein system spuckt mir hier die Liste der ISO-8859-kodierten Dateien im FHEM repo aus und dort steht nach dem aktuellsten commit

42_SYSMON.pm:                Perl5 module source, ISO-8859 text, with very long lines

Und ich lese natürlich 42_SMARTMON.pm

... 

[hexenmeister]

Bin verwirrt - mein Editor meint es sei jetzt UTF-8.

Im Commandref-Part sind jetzt alle Umlaute als HTMP-Entities codiert. Das ist doch auch für UTF-8-Codierte Dateien erlaubt und korrekt, oder?

[RichardCZ]

Bin verwirrt - mein Editor meint es sei jetzt UTF-8.

Im Commandref-Part sind jetzt alle Umlaute als HTMP-Entities codiert. Das ist doch auch für UTF-8-Codierte Dateien erlaubt und korrekt, oder?

siehe edit.

Gekommen bin ich dann darauf als ich explizit file 42_SMARTMON.pm gemacht habe. Sorry für das Rauschen.

[hexenmeister]

Kein Problem

SYSMON kommt auch von mir und auch wenn ich das Modul nicht mehr für besonders nützlich halte, werde ich bei Gelegenheit umstellen.

[RichardCZ]

Kein Problem

SYSMON kommt auch von mir und auch wenn ich das Modul nicht mehr für besonders nützlich halte, werde ich bei Gelegenheit umstellen.

Ok. Hier sind im übrigen SYSMON/SMARTMON nach einem automatischen Waschgang:

https://gl.petatech.eu/root/HomeBot/-/commit/f8fdf1954dc2d17968b4d9195436c2cd5846c4e1

wenn Du das als Grundlage hernehmen möchtest - nur zu.