[PATCH] - 98_help.pm - Ersetzen von <a href="#..."> durch Link auf commandref

[Markus Bloch]

Hallo betateilchen,

anbei ein Patch für 98_help.pm welches alle Links im Output von <a href="#...."> nach bspw. <a target="_blank" href="/fhem/commandref.html#..." ändert. Dadurch sind alle Verweise auf z.B. readingsFn, do_not_notify, disable, ... wieder intakt und werden in einem neuen Tab/Fenster geöffnet.

Getestet mit Chrome/Firefox/Edge.

Viele Grüße

Markus

[betateilchen]

Hallo Markus,

das halte ich an dieser Stelle nicht für sinnvoll. Die devicespezifische Hilfe wird direkt aus dem Modul gelesen und angezeigt. Die Idee dahinter war eine "schnelle Hilfe" bereitzustellen, an dieser Stelle sucht man meist die Anzeige einer korrekten Syntax für das define oder eine Liste von Attributen. Die Erweiterung von help war nie als vollständiger Ersatz der gesamten commandref konzipiert.

Dein vorgeschlagenes Ersetzen der Links führt zu Problemen, wenn es beispielsweise auf dem System überhaupt keine commandref gibt. Wenn schon, müsste man an dieser Stelle nach extern verlinken und die commandref auf fhem.de verwenden. Aber ehrlich gesagt fände ich das auch unschön.

Wer tiefergehende Informationen zu einem Modul braucht, hat ja immer noch die Möglichkeit, direkt in die commandref (online) zu gehen, dort sollten die Links alle funktionieren.

[betateilchen]

Vielleicht können wir das ja so lösen:

• prüfen, ob im globalen attribute excludefromupdate commandref steht
• wenn ja: ersetze die Links durch externe Links http://fhem.de/commandref...
• wenn nein: ersetze die Links durch lokale Links /fhem/commandref....

Das wäre eine Lösung, mit der ich mich durchaus anfreunden kann.

[Markus Bloch]

Hallo betateilchen,

das wäre toll.

Jetzt habe ich gerade einen ellenlangen Beitrag verfasst, in der ich als Konsequenz das gleiche vorschlagen wollte und dann hast du in der Zwischenzeit das selbe vorgeschlagen :-D

Viele Grüße

Markus

[Markus Bloch]

Mir ist auch aufgefallen, dass man anstatt $FW_ME/docs/commandref.... doch eher docs/commandref... verwenden sollte, falls man FHEM durch Apache Rewrite-Rules und ProxyPass Schweinereien umbiegt.

Kann ich aber gerade nicht testen, da ich auf Arbeit bin.

Viele Grüße

Markus

[betateilchen]

Kann ich aber gerade nicht testen, da ich auf Arbeit bin.

lass Dir Zeit, bei Deinen vorgeschlagenen Patch um, dann schau ich mir das gerne nochmal an.

Viele Grüße
Udo

[Markus Bloch]

Hi Udo,

anbei der überarbeitete Patch, wie du ihn vorgeschlagen hast. Ich habe noch die <a> Ersetzungen für telnet geändert, da ich beim testen meines Patches unter anderem über <a>-Tags gestolpert bin, die über 2 Zeilen definiert waren:

Code Auswählen Erweitern

<a
href="#...>

Daher habe ich deine Ersetzungsregeln mit \s+ ergänzt um solche Links ebenfalls zu erwischen.

Viele Grüße

Markus

[betateilchen]

Danke. Die Änderungen habe ich eben eingecheckt.

[Markus Bloch]

Vielen Dank.

Ich habe noch einen CHANGED Eintrag spendiert.

Viele Grüße

Markus

[Markus Bloch]

@betateilchen fyi: http://forum.fhem.de/index.php/topic,44300.0.html

[betateilchen]

Schau ich mir gerne an, wenn ich aus dem Urlaub zurück bin (bin grade am Packen).

Wenn Du sicher bist, dass es funktioniert, kannst Du die 98_help.pm im Rahmen von Amtshilfe gerne selbst einchecken.

[Markus Bloch]

Das lass ich mir natürlich nicht 2 mal sagen.

Checke ich nachher ein.

Vielen Dank und einen schönen Urlaub wünsch ich Dir.

Gruß

Markus