FHEM Forum

FHEM - Entwicklung => FHEM Development => Thema gestartet von: Markus Bloch am 16 Januar 2016, 15:06:26

Titel: [PATCH] - commandref_join.pl - Optionaler Verify eines einzelnen Moduls
Beitrag von: Markus Bloch am 16 Januar 2016, 15:06:26
Hallo Rudi,

anbei ein Patch für commandref_join.pl, welcher es ermöglicht, optional eine übergebene Datei nur auf eine korrekte Commandref zu prüfen.

Diese Prüfung würde ich dann in einem zweiten Schritt in den pre-commit Hook einbauen. So spart man sich das pflegen von beiden Skripten, wenn es Anpassungen in den commandref-Regeln gibt. Im pre-commit würde man dann jedes Geräte-Modul damit prüfen.

Ein normaler Aufruf von commandref_join.pl:

NAS> ./contrib/commandref_join.pl
EN FHEM/95_Alarm.pm: Unbalanced tr (-1, last line ok: 856)
EN FHEM/57_Calendar.pm: Unbalanced table (-3, last line ok: 1259)
EN FHEM/49_IPCAM2.pm: No a-tag with name="IPCAM2"
*** EN FHEM/82_LGTV2012.pm: No document text found
*** EN FHEM/75_LGTV_RS232.pm: No document text found
EN FHEM/00_MAXLAN.pm: Unbalanced tr (1, last line ok: 868)
EN FHEM/00_MAXLAN.pm: Unbalanced td (1, last line ok: 868)
*** EN FHEM/11_OWX_DS2480.pm: No document text found
*** EN FHEM/11_OWX_Executor.pm: No document text found
*** EN FHEM/11_OWX_FRM.pm: No document text found
*** EN FHEM/11_OWX_SER.pm: No document text found
EN FHEM/40_RFXCOM.pm: Unbalanced tr (1, last line ok: 381)
EN FHEM/40_RFXCOM.pm: Unbalanced td (1, last line ok: 381)
EN FHEM/51_RPI_GPIO.pm: Unbalanced table (-4, last line ok: 705)
*** EN FHEM/74_S0COUNTER.pm: No document text found
EN FHEM/45_TRX.pm: Unbalanced tr (1, last line ok: 411)
EN FHEM/45_TRX.pm: Unbalanced td (1, last line ok: 411)
EN FHEM/95_remotecontrol.pm: Unbalanced td (-1, last line ok: 409)
DE FHEM/57_Calendar.pm: Unbalanced table (-3, last line ok: 1465)
DE FHEM/51_RPI_GPIO.pm: Unbalanced table (-4, last line ok: 898)
DE FHEM/95_remotecontrol.pm: Unbalanced td (-1, last line ok: 495)
NAS>


Es werden normal die entsprechenden HTML-Dateien erstellt.

Ein Aufruf mit Dateipfad zum überprüfen:

NAS> ./contrib/commandref_join.pl FHEM/74_S0COUNTER.pm
*** EN FHEM/74_S0COUNTER.pm: No document text found
NAS>


NAS> ./contrib/commandref_join.pl FHEM/73_PRESENCE.pm
NAS>


Es wird dabei keine commandref*.html erstellt. Nur die Fehlerausgaben auf der Konsole werden ausgegeben.

Die -noWarnings Funktionalität von justme1968 funktioniert nachwievor unverändert, auch bei einem Verify.

Da die Änderungen im Diff doch recht eigenartig aussehen, anbei Diff und das komplette Skript.

Gruß
Markus
Titel: Antw:[PATCH] - commandref_join.pl - Optionaler Verify eines einzelnen Moduls
Beitrag von: rudolfkoenig am 16 Januar 2016, 18:00:31
Habs eingecheckt.
Titel: Antw:[PATCH] - commandref_join.pl - Optionaler Verify eines einzelnen Moduls
Beitrag von: betateilchen am 16 Januar 2016, 19:23:19
Gute Idee, damit kann man endlich die commandref seines eigenen Moduls ohne großen Zeitaufwand testen.