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.