Bitte contrib/commandref_join.pl Fehler beachten

[rudolfkoenig]

Bitte nach Doku-Aenderung immer contrib/commandref_join.pl ausfuehren, und evtl. Fehler vor dem Einchecken beheben. Aktuelle Ausgabe:

EN FHEM/00_HMLAN.pm: Unbalanced li (1, last line ok: 1020)
DE CUL_MAX: nonempty line after =begin html ignored
DE FHEM/20_GUEST.pm: Unbalanced li (1, last line ok: 1075)
DE FHEM/20_ROOMMATE.pm: Unbalanced li (1, last line ok: 1068)
DE dewpoint: nonempty line after =begin html ignored
DE FHEM/98_dewpoint.pm: Unbalanced li (-3, last line ok: 629)

[Loredo]

Hallo Rudi,

DE FHEM/20_GUEST.pm: Unbalanced li (1, last line ok: 1075)
DE FHEM/20_ROOMMATE.pm: Unbalanced li (1, last line ok: 1068)

Wenn ich den HTML Code daraus durch HTML-Tidy laufen lasse und hinterher wieder hinein kopiere, bleibt der Fehler trotzdem.
Ich kann keinen fehlenden <li> oder </li> Tag finden 

Kann es sein, dass das commandref_join.pl hier falsch prüft?
Ich habe jetzt mal bei all meinen Modulen die Doku mit HTML-Tidy überarbeitet abgelegt. Dabei fiel mir auch auf, dass commandref_join.pl es wohl auch nicht mag, wenn man korrektes XHTML mit name+id im englischen Bereich verwendet:

Code Auswählen Erweitern


EN FHEM/70_ENIGMA2.pm: No <a name="ENIGMA2"> link
EN FHEM/98_GEOFANCY.pm: No <a name="GEOFANCY"> link
EN FHEM/20_GUEST.pm: No <a name="GUEST"> link
EN FHEM/70_ONKYO_AVR.pm: No <a name="ONKYO_AVR"> link
EN FHEM/10_RESIDENTS.pm: No <a name="RESIDENTS"> link
EN FHEM/20_ROOMMATE.pm: No <a name="ROOMMATE"> link


Für den deutschen Bereich wird seltsamer Weise keine Warnung ausgegeben.

Gruß
Julian

[betateilchen]

vergiss die externe Prüfung 

Der Fehler in der 20_GUEST ist in Zeile 1224:

Code Auswählen Erweitern

    <li><b>durTimerSleep</b> - Timer, der die Schlafdauer in Minuten anzeigt/li>

Da fehlt beim schliessenden </li> die öffnende spitze Klammer.

In der ROOMMATE ist der exakt gleiche Fehler (dank copy & paste) in Zeile 1223

[Loredo]

Mist, dabei hatte ich mich so angestrengt! *grrr*

[betateilchen]

anstrengen reicht nicht (er hat sich stets bemüht...), man muss es auch richtig machen 

[rudolfkoenig]

Fuer Zeilen wie

Code Auswählen Erweitern

<a name="ENIGMA2 id="ENIGMA2">

wird nicht mehr die Warnung

EN FHEM/70_ENIGMA2.pm: No <a name="ENIGMA2"> link

ausgegeben.

[Loredo]

Danke, Rudi und betateilchen.
Auch von meiner Seite gefixt.

[betateilchen]

Auch von meiner Seite gefixt.

nicht wirklich, aktuell liefert commandref_join noch:

EN FHEM/20_GUEST.pm: Unbalanced li (-1, last line ok: 1134)
DE FHEM/20_GUEST.pm: Unbalanced li (-1, last line ok: 1383)

Tipp: Lass die Style-Angaben im <li>-Tag in den Zeilen 1135 und 1384 weg, dann funktionierts fehlerfrei

[betateilchen]

Das kann doch nicht so schwer sein... mal schauen.



in 00_HMLAN.pm fehlt in Zeile 1024 das schließende </li>

Code Auswählen Erweitern

<li><a name="HMLANhmKey3">hmKey3</a></li><br>



in 14_CUL_MAX fehlt die Leerzeile zwischen =begin und <a name>

Code Auswählen Erweitern


=begin html_DE

<a name="CUL_MAX"></a>




Ich hab die Module hier korrigiert, bei Bedarf kann ich die gerne einchecken, aber nicht unaufgefordert.

[Loredo]

Danke, Betateilchen. Hab ich auch korrigiert (in der Hoffnung, dass HTML Tidy es bei Änderungen nicht wieder vermurkst  ).

[Loredo]

Hier noch die Korrektur für GEOFANCY

Code Auswählen Erweitern


=pod

=begin html

  <a name="GEOFANCY" id="GEOFANCY"></a>
  <h3>GEOFANCY</h3>
  <ul>
    <li>Provides webhook receiver for geofencing via the following iOS apps:</li><br/><br/>
    <li>
      <a href="https://itunes.apple.com/de/app/geofency-time-tracking-automatic/id615538630?l=en&amp;mt=8">Geofency</a>
    </li>
    <li>
      <a href="https://itunes.apple.com/de/app/geofancy/id725198453?l=en&amp;mt=8">Geofancy</a>
    </li>
    <li>
      Note: GEOFANCY is an extension to <a href="FHEMWEB">FHEMWEB</a>. You need to install FHEMWEB to use GEOFANCY.
    </li>

  <a name="GEOFANCYdefine" id="GEOFANCYdefine"></a> <b>Define</b>
    <code>define &lt;name&gt; &lt;infix&gt;</code><br>
    <br>
    Defines the webhook server. <code>&lt;infix&gt;</code> is the portion behind the FHEMWEB base URL (usually <code>http://hostname:8083/fhem</code>)<br>
    <br>
    Example:
    <code>define geofancy GEOFANCY geo</code><br>
    <br>
    The webhook will be reachable at http://hostname:8083/fhem/geo in that case.<br>
    <br>

  <a name="GEOFANCYset" id="GEOFANCYset"></a> <b>Set</b>
    <br>
    <ul>
      <li>
        <b>clear</b> &nbsp;&nbsp;readings&nbsp;&nbsp; can be used to cleanup auto-created readings from deprecated devices.
      </li>
    </ul><br>
    <br>

  <a name="GEOFANCYattr" id="GEOFANCYattr"></a> <b>Attributes</b>
    <br>
    <ul>
      <li>devAlias: can be used to rename device names in the format DEVICEUUID:Aliasname.
          Separate using blank to rename multiple devices.</li>
    </ul><br>
    <br>
    <b>Usage information</b><br>
    <br>
    Likely your FHEM installation is not reachable directly from the internet (good idea!).<br>
    It is recommended to have a reverse proxy like nginx or Apache in front of FHEM
    where you can make sure access is only possible to specific subdirectories like /fhem/geo.<br>
    You might also want to think about protecting the access by using HTTP Basic Authentication
    and encryption via SSL.<br>
    Also the definition of a dedicated FHEMWEB instance for that purpose might help
    to restrict FHEM's functionality (note that the 'hidden' attributes of FHEMWEB
    currently do NOT protect from just guessing/knowing the correct URL!)<br>
    <br>
    To make that reverse proxy available from the internet,
    just forward the appropriate port via your internet router.<br>
    <br>
    The actual solution on how you can securely make your Geofancy webhook
    available to the internet is not part of this documentation and depends on your own skills.
    <br>
    <br>
    <b>Integration with Home Automation</b><br>
    <br>
    You might want to have a look to the module family of <a href="#ROOMMATE">ROOMMATE</a>,
    <a href="#GUEST">GUEST</a> and <a href="#RESIDENTS">RESIDENTS</a> for an easy processing of GEOFANCY events.
  </ul>

=end html

=cut


Die ganzen DIV tags sind waren sehr kontraproduktiv.

Danke, hatte ich schon gefixt. Die Div-Tags lasse ich drin, sie scheinen mir konformer zu sein und ich möchte eigentlich gerne die Möglichkeit offen lassen, den Code einfach durch HTML-Tidy zu jagen. Da jedes Mal die DIVs wieder rauszufieseln, geht dann am Ziel vorbei...

[betateilchen]

Die Div-Tags lasse ich drin, sie scheinen mir konformer zu sein

Das sehe ich aber völlig anders

[Loredo]

Das sehe ich aber völlig anders

Darfst du ja 

[betateilchen]

EDIT: es ist geschafft. Heute abend keine Fehlermeldungen mehr beim commandref_join. Mal sehen, wie lange.

[betateilchen]

Hallo Martin,

es besteht mal wieder Handlungsbedarf 

Code Auswählen Erweitern


EN FHEM/10_CUL_HM.pm: Unbalanced ul (1, last line ok: 6353)
EN FHEM/10_CUL_HM.pm: Unbalanced b (-1, last line ok: 7028)
EN FHEM/98_HMinfo.pm: Unbalanced b (-2, last line ok: 1740)


In der 98_HMInfo liegt der Fehler in der Zeile 1953

<li></B>templateName</B>

das muss ein öffnendes B tag sein.

[C_Herrmann]

Hallo,

wenn man die Modul-Dateien mit Notepad++ öffnet und als Sprache html eingestellt, kann man mit aktiviertem Syntax-Highlighting den entsprechenden Bereich ansehen und auf die verdächtigen Tags klicken. Wenn das entsprechende Tag fehlt, wird es nicht hervorgehoben.

10_CUL_HM Zeile 6354-6:

Code Auswählen Erweitern

<h3>CUL_HM</h3>
<ul>
  Support for eQ-3 HomeMatic devices via the <a href="#CUL">CUL</a> or the <a

Das <ul> ist unbalanced.

10_CUL_HM.pm Zeile 7027-9:

Code Auswählen Erweitern

          activate backlight<br></li>
      <li><B>display &lt;text&gt; comma unit tone backlight &lt;symbol(s)&gt;
         </B><br>

Hier ist das </B> in der nächsten Zeile. Das sollte aber kein Fehler sein.

Gruß,
Christian

[betateilchen]

Hallo Christian,

Dein Tipp ist gut gemeint, aber er hilft bei komplexen Texten wie hier nicht weiter.
Auch das erste ul ist korrekt, das wird in Zeile 7579 geschlossen.

Mit dem simplen html-Highlighting kommt man da nicht wirklich weit.

Viele Grüße
Udo

[betateilchen]

Das Problem mit dem <li>in der 10_CUL_HM ist in Zeile 6766, das <ul> dort ist falsch und muss raus.

[betateilchen]

Das Problem mit dem <b>in der 10_CUL_HM ist in Zeile 6769

The command is similar to <a href="#CUL_HMpeerChan">peerChan</a></B>. While peerChan

Das schliessende B-tag muss raus.

[betateilchen]

wer hat sich eigentlich den Wahnsinn angetan, die 10_CUL_HM ins Deutsche zu übersetzen? Die Sache mit den Umlauten in den Texten finde ich nicht so prickelnd.

Davon abgesehen sind jetzt noch mehr Fehler beim commandref_join (nicht nur in der 10_CUL_HM) ...

[betateilchen]

schon cool, dass jetzt sogar schon Module ganz ohne commandref-Teil (nichtmal ein dummy!) eingecheckt werden...

Code Auswählen Erweitern


EN FHEM/00_OWX_ASYNC.pm: No <a name="OWX_ASYNC"> link
*** 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
DE HMLAN: nonempty line after =begin html ignored
DE FHEM/10_MAX.pm: Unbalanced li (1, last line ok: 1060)


[rudolfkoenig]

Eigentlich widerspricht das der Definition von fhem/FHEM, und diese Module muessten nach contrib verschoben werden.
Wie kann man Norbert erreichen/motivieren, um ihn nicht mit dem Verschieben zu ueberraschen?

[betateilchen]

Wie kann man Norbert erreichen

Du bist hier ein "Global Moderator" - soll man Dir jetzt erklären, wie man über das Forum emails oder PN verschicken kann? *lach*

[ntruchsess]

Eigentlich widerspricht das der Definition von fhem/FHEM, und diese Module muessten nach contrib verschoben werden.
Wie kann man Norbert erreichen/motivieren, um ihn nicht mit dem Verschieben zu ueberraschen?

Das sind keine eigenständigen FHEM-module, sondern nur Hilfsklassen. War mir nicht bewusst, dass Die auch ohne perl-doc in der Commandref auftauchen. Liegt das daran, dass die Namen mit einer 2-stellingen Zahl + '_' beginnen? Dann benenne ich sie einfach um und/oder verschiebe sie nach lib.

Gruß,

Norbert

[betateilchen]

Liegt das daran, dass die Namen mit einer 2-stellingen Zahl + '_' beginnen?

Ja.

War mir nicht bewusst, dass Die auch ohne perl-doc in der Commandref auftauchen.

Du hättest nur ein einziges Mal vor dem Einchecken die commandref_join aufrufen brauchen...

[ntruchsess]

Du hättest nur ein einziges Mal vor dem Einchecken die commandref_join aufrufen brauchen...

So, Hilfs-module umbenannt und OWX_ASYNC-doku angepasst. Diesmal auch mit 'commandref_join.pl' getestet

Gruß,

Norbert

[betateilchen]

super

[rudolfkoenig]

Die aktuelle Liste ist wieder angewachsen:

EN FHEM/70_JSONMETER.pm: Unbalanced ul (-1, last line ok: 1017)
EN FHEM/23_LUXTRONIK2.pm: Unbalanced ul (-1, last line ok: 1814)
EN FHEM/00_TCM.pm: Unbalanced li (1, last line ok: 1051)
EN FHEM/32_mailcheck.pm: Unbalanced li (1, last line ok: 476)
EN FHEM/38_netatmo.pm: Unbalanced li (-1, last line ok: 1095)
EN FHEM/98_statistics.pm: Unbalanced ul (-1, last line ok: 1066)
DE FHEM/70_JSONMETER.pm: Unbalanced ul (-1, last line ok: 1140)
DE FHEM/23_LUXTRONIK2.pm: Unbalanced ul (-1, last line ok: 1948)
DE FHEM/10_MAX.pm: Unbalanced li (1, last line ok: 1060)
DE FHEM/98_statistics.pm: Unbalanced ul (-1, last line ok: 1177)

Angeblich unterstuetzt sourceforge jetzt SVN-Hooks, ich muss das mal pruefen. Alternativ koennten wir fehlerhafte Module aus der update-Liste entfernen, leider haetten damit die Modul-Autoren keinen direkten Feedback.

[betateilchen]

Und die Liste wächst weiter...

Code Auswählen Erweitern


mba:fhem udo$ perl contrib/commandref_join.pl
EN FHEM/70_JSONMETER.pm: Unbalanced ul (-1, last line ok: 1024)
EN FHEM/23_LUXTRONIK2.pm: Unbalanced ul (-1, last line ok: 1833)
EN FHEM/59_PROPLANTA.pm: Unbalanced ul (-1, last line ok: 554)
EN FHEM/59_PROPLANTA.pm: Unbalanced code (4, last line ok: 502)
EN FHEM/36_WMBUS.pm: Unbalanced li (1, last line ok: 451)
EN FHEM/98_statistics.pm: Unbalanced ul (-1, last line ok: 1129)
DE FHEM/70_JSONMETER.pm: Unbalanced ul (-1, last line ok: 1147)
DE FHEM/23_LUXTRONIK2.pm: Unbalanced ul (-1, last line ok: 1967)
DE FHEM/59_PROPLANTA.pm: Unbalanced ul (-1, last line ok: 608)
DE FHEM/36_WMBUS.pm: Unbalanced li (1, last line ok: 557)
DE FHEM/98_statistics.pm: Unbalanced ul (-1, last line ok: 1253)


[rudolfkoenig]

Ich habe pre-commit auf perl umgebaut (ist als contrib/pre-commit eingecheckt), und die Tag-Pruefung eingebaut. Fuer die meisten "kaputten" Module habe ich die Fehler gefixt (sorry, und das war ein einmaliger Vorgang), nur PROPLANTA aendert sich fuer mein Geschmack zu haeufig. Aber das bedeutet auch, dass der Autor das Problem demnaechst beheben wird

Achtung: <ul> mit Style ist nicht erlaubt: Styles gehoeren nach CSS und nicht ins HTML.

[betateilchen]

Ich habe pre-commit auf perl umgebaut (ist als contrib/pre-commit eingecheckt), und die Tag-Pruefung eingebaut.

*gefällt mir*

Danke

[betateilchen]

Aktueller Stand...

Code Auswählen Erweitern


EN FHEM/70_ENIGMA2.pm: No <a name="ENIGMA2"> link
EN FHEM/98_GEOFANCY.pm: No <a name="GEOFANCY"> link
EN FHEM/20_GUEST.pm: No <a name="GUEST"> link
EN FHEM/98_HourCounter.pm: No <a name="HourCounter"> link
EN FHEM/70_ONKYO_AVR.pm: No <a name="ONKYO_AVR"> link
*** 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/10_RESIDENTS.pm: No <a name="RESIDENTS"> link
EN FHEM/20_ROOMMATE.pm: No <a name="ROOMMATE"> link


es kann doch nicht so schwer sein... 

[Loredo]

Code Auswählen Erweitern


EN FHEM/70_ENIGMA2.pm: No <a name="ENIGMA2"> link
EN FHEM/98_GEOFANCY.pm: No <a name="GEOFANCY"> link
EN FHEM/20_GUEST.pm: No <a name="GUEST"> link
EN FHEM/70_ONKYO_AVR.pm: No <a name="ONKYO_AVR"> link
EN FHEM/10_RESIDENTS.pm: No <a name="RESIDENTS"> link
EN FHEM/20_ROOMMATE.pm: No <a name="ROOMMATE"> link


es kann doch nicht so schwer sein... 

Würdest du mir in deiner unendlichen Weisheit bitte helfen und sagen, weshalb


1. diese Meldungen plötzlich auftauchen (denn sie sind neu)
2. Das commandref_join.pl Script es nicht schafft zu erkennen, dass die besagten Links in allen Modulen vorhanden sind und nur neben dem "name" Attribut ebenfalls das "id" Attribut gesetzt haben.


Hochachtungsvoll...

[betateilchen]

Würdest du mir in deiner unendlichen Weisheit bitte helfen und sagen, weshalb
1. diese Meldungen plötzlich auftauchen (denn sie sind neu)

Guter Hinweis, danke.

Mit der aktuellen Version der commandref_join treten die Meldungen wegen <a name...> nicht auf.

Nur die Fehlermeldungen bei den 11_OWX... sind noch aktuell.

[rudolfkoenig]

Bei mir meldet contrib/commandref_join.pl aktuell keine Fehler.

Wuerde mich auch ueberraschen, wenn neue Meldugen auftauchen, da beim Einchecken auf sowas geprueft wird.

[ntruchsess]

es kann doch nicht so schwer sein... 

Code Auswählen Erweitern


*** EN FHEM/11_OWX_DS2480.pm: No document text found


richtig, so schwer kann es eigentlich nicht sein einen aktuellen code-stand zu verwenden
Das habe ich schon am 14. April des vergangenen Jahres behoben.

- Norbert

[Sailor]

Ein herzerfrischendes "Moin" vom Deich vorweg!

Ich habe ein wirklich allzu seltsames Problem hier.

Das Ausfuehren des Befehls

Code Auswählen Erweitern

/opt/fhem $ sudo perl contrib/commandref_join.pl

fuehrt bei mir zu der Fehlermeldung

Code Auswählen Erweitern

*** EN FHEM/73_km200.pm: ignoring text due to DOS encoding

Und das obwohl der Text bereits in "UTF-8 ohne BOM" codiert war/ist. 

Der Witz ist, dass ich das Modul mit einem funktionierenden commandref - Eintrag bereits eingecheckt hatte. 

Heute habe ich den Code geaendert und schwupps kommt die Fehlermeldung bei der Ueberpruefung.

Das geaenderte Modul laeuft unter fhem ohne Probleme...
Ich habe es mal zum Testen angehaengt. Waere schoen, wenn mich Jemand auf den Fehler hinweisen koennte. 

Gruss und DANKE vorweg
   Sailor

[betateilchen]

Deine Datei ist Windows-kodiert:

(http://up.picr.de/20674518ht.png)

Ich habe die Dir die Datei Unix-kodiert hier angehängt.

[Sailor]

Deine Datei ist Windows-kodiert:
Ich habe die Dir die Datei Unix-kodiert hier angehängt.

Hallo betateilchen

Jetzt zeigt mir NotePad++ allerdings an, dass die von Dir konvertierte Datei in ANSI kodiert ist.

Ich versuch mal ob commandref_join damit zufrieden ist.

Aber ich würde schon gerne wissen, wie ich mir in Zukunft selbst zu helfen wüsste... Wird wohl nicht am Manual von Notepad++ vorbeigehen. 

Danke schon mal für die Konvertierung.

Gruss
   Sailor

[betateilchen]

Das kannst Du in notepad++ in den Optionen generell festlegen.

Für eine einzelne Datei geht das in notepad++ unter Bearbeiten -> Zeilenende (wenn ich mich recht erinnere)

[Sailor]

Hallo betateilchen

Unter "Edit" -> "EOL Conversion"
habe ich eine Liste

Windows Format
UNIX/OSX Format
Old MAC Format

Doch leider ist UNIX/OSX Format ausgegraut und somit nicht auswählbar.... AAARRRGH! 
Das wäre es gewesen!

Ich forsche weiter und berichte!

Gruss
    Sailor

Nachtrag: Wenn UNIX schon ausgewählt, lässt es sich nicht nach UNIX konvertieren. Also alles gut!

Danke!

[betateilchen]

Nachtrag: Wenn UNIX schon ausgewählt, lässt es sich nicht nach UNIX konvertieren.

fein, dass Du selbst draufgekommen bist *g*