[PATCH] - fhem.pl Aufnahme von Hilfsmodulen (HttpUtils, ...) in version output

[betateilchen]

Rudi hat einfach die Chance genutzt, die commandref_join.pl zu einem Zeitpunkt zu ändern, an dem Markus sie nicht in Bearbeitung hatte 

[rudolfkoenig]

@Markus: patch eingespielt.
@Markus/@betateilchen/@justme1968/@benni: habe die erwaehnten Module aus commandref_frame.html entfernt.

Falls jemand in der Uebersicht weitere Kategorien wuenscht, bitte melden.

[Markus Bloch]

Vielen Dank, Rudi!

Weil es so viel Spaß macht, hier nochmal ein Patch für kleinere Optimierungen an commandref_join.pl:

- Es werden nur noch Module in die Übersicht aufgenommen, die auch tatsächlich eine Commandref besitzen (matcht auf /^=pod/)
- configDB.pm wird als "command"-Modul eingruppiert (Ich hoffe betateilchen haut mich jetzt nicht, das ich configDB relevante Sachen anfasse  )

Gruß
Markus

edit: Patch zurückgezogen nach einigen Überlegungen

[betateilchen]

- configDB.pm wird als "command"-Modul eingruppiert

NEIN! Lass das bitte, das ergibt überhaupt keinen Sinn.

• Die Datei ./configDB.pm ist als helper einzuordnen, so wie das bisher auch war. Die Klassifizierung wurde gestern von mir in der Datei vorgenommen.
• Die Datei ./FHEM/98_configdb.pm ist eine command Datei, diese wurde von mir auch schon entsprechend klassifiziert.
  

[Markus Bloch]

NEIN! Lass das bitte, das ergibt überhaupt keinen Sinn.

Ja, hast recht. Habe ich erst später gesehen. Sry!

[Markus Bloch]

Die Datei ./configDB.pm ist als helper einzuordnen, so wie das bisher auch war. Die Klassifizierung wurde gestern von mir in der Datei vorgenommen.

Die aktuelle Implementation von commandref_join.pl ignoriert allerdings bei configDB.pm deine Klassifizierung, da sie nicht explizit danach durchsucht wird. Momentan werden nur /\d\d_.*\.pm$/ im Unterordner "FHEM" danach explizit durchsucht.

configDB.pm wird nachträglich in die Liste der Module aufgenommen. Eine Prüfung welches Gruppierung zu nutzen ist, erfolgt nicht. Bei mir ist configDB momentan garnicht gelistet, da es keinerlei Eintrag in $modIdx{configDB} aktuell gibt:

Code Auswählen Erweitern


  $mods{configDB} = "configDB.pm" if(-f "configDB.pm");


Daher mein Vorschlag (jetzt als "helper"):

Code Auswählen Erweitern

  if(-f "configDB.pm") {
    $mods{configDB} = "configDB.pm";
    $modIdx{configDB} = "helper";
  } 

Viele Grüße

Markus

[betateilchen]

Genau das hatte ich auch gerade getestet, nachdem ich das Frühstück beendet hatte 

Code Auswählen Erweitern


Index: commandref_join.pl
===================================================================
--- commandref_join.pl (revision 10549)
+++ commandref_join.pl (working copy)
@@ -41,7 +41,12 @@
       close(MOD);
     }
   }
-  $mods{configDB} = "configDB.pm" if(-f "configDB.pm");
+ 
+  if(-f "configDB.pm") {
+    $mods{configDB}   = "configDB.pm";
+    $modIdx{configDB} = "helper";
+  }
+   
} else { # check for syntax only
   my $modname = $verify;
   $modname =~ s/^.*[\/\\](?:\d\d_)?(.+).pm$/$1/;


[Markus Bloch]

Genau das hatte ich auch gerade getestet, nachdem ich das Frühstück beendet hatte 

Ich hab das Frühstück ausgelassen, daher war ich schneller

[betateilchen]

Das ausgiebige Frühstück am Sonntag ist für mich ein heiliger Kult 

[rudolfkoenig]

@betateilchen: Habe dein Patch eingecheckt.

[betateilchen]

Danke.

Meine beiden Module InfoPanel und weco habe ich eben von device nach helper umklassifiziert, die können bei Gelegenheit aus helper@commandref_frame entfernt werden.

[Loredo]

Ich habe "meine" commands/helper entsprechend gekennzeichnet, und sie aus commandref_frame.html entfernt.
Nebeneffekt: die Sortierung wird jetzt vom Computer durchgefuehrt, und Klein-/Grossschreibung wird dabei ignoriert.

Folgende Module können ebenfalls aus commandref_frame entfernt werden:


10_RESIDENTS
20_GUEST
20_ROOMMATE
70_Pushover
75_MSG
75_msgConfig
98_GEOFANCY

[rudolfkoenig]

@betateilchen: habs entfernt.
@Loredo: ich tu mich damit schwer, da keiner dieser Module je in commandref_frame erwaehnt war.

[Loredo]

@Loredo: ich tu mich damit schwer, da keiner dieser Module je in commandref_frame erwaehnt war.

Komisch, hatte sie dort als Helper Module bzw. Command eingefügt.
Hast du geschaut, ob du die Module ohne den XX_ Präfix dort findest? Hatte den der Eindeutigkeit halber mit angegeben.

Code Auswählen Erweitern


RESIDENTS
GUEST
ROOMMATE
MSG
msgConfig
GEOFANCY



Im Falle von RESIDENTS gibts da eh einen Zuordnungsfehler, es wird dort auf #RSS statt #RESIDENTS verlinkt. Aber die Zeile zu entfernen löst das ja dann...

[Loredo]

Ich hätte da noch eine "Sonderlocke" (wobei ich glaube das auch schon bei anderen Modulen gesehen zu haben):


Die Schreibweise der Moduldatei 75_MSG.pm unterscheidet sich von der, wie sie in der Commandref verwendet werden soll (msg).
Das war bei der manuellen Verlinkung in commandref_frame kein Problem, dort konnte man einfach den Link auf "#MSG" setzen und den angezeigten Text auf "msg". Der Hintergrund bei diesem Kommando ist historisch bedingt, da es vorher ein gleichnamiges Modul gab, dessen Namensraum nun für das Kommando übernommen wurde. Daher wurde der Dateiname entsprechend beibehalten.


Ließe sich der Überschriften-Name ggf. auch noch als Pod mit einbauen?