Autor Thema: [PATCH] - commandref_join.pl - Optionaler Verify eines einzelnen Moduls  (Gelesen 1194 mal)

Offline Markus Bloch

  • Administrator
  • Hero Member
  • *****
  • Beiträge: 3665
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
Developer für Module: YAMAHA_AVR, YAMAHA_BD, FB_CALLMONITOR, FB_CALLLIST, PRESENCE, Pushsafer, LGTV_IP12, version

aktives Mitglied des FHEM e.V. (Technik)

Offline rudolfkoenig

  • Administrator
  • Hero Member
  • *****
  • Beiträge: 24397
Habs eingecheckt.

Offline betateilchen

  • Developer
  • Hero Member
  • ****
  • Beiträge: 17370
  • s/fhem\.cfg/configDB/g
Gute Idee, damit kann man endlich die commandref seines eigenen Moduls ohne großen Zeitaufwand testen.
-----------------------
Unaufgeforderte Anfragen per email werden von mir nicht beantwortet. Dafür ist das Forum da.
-----------------------
Lesen gefährdet die Unwissenheit!

 

decade-submarginal