Modul 98_Matrix zur Kopplung von FHEM an den Matrix-Chat

[mBielemeier]

Hallo CoolTux,

danke für die ausführlichen Anregungen im Github. Ich habe alles übernommen.

Jetzt bin ich am Grübeln mit den packages: kommen 98_Matrix.pl und /lib/FHEM/Devices/Matrix/Matrix.pl in das gleiche package oder ist vorgesehen, dass jede Datei ein eigenes package bekommt?

Viele Grüße
Manfred

[CoolTux]

Hallo Manfred,

Ich habe schon gelesen. Nicht jede Datei bekommt ihr eigenes Package, sondern jedes Modul bekommt im normal Fall sein eigenes package. Das ist keine pflicht, aber in der Perl Welt seit Jahren Gang und Gebe.
Ausserdem sagt man das ein Modul immer eine Datei ist. Also steht dem zu Folge dann am Ende doch wieder jede Datei bekommt ein package. Dein package package FHEM::Matrix; für die Datei lib/FHEM/Devices/Matrix/Matrix.pm past also nicht. Hier muss tatsächlich package FHEM::Devices::Matrix; hin

[CoolTux]

@CoolTux: Hast du Kontakt zu Loredo? Evtl. würde es sich anbieten, das Modul gleich für die Verwendung im MSG-Kontext vorzubereiten bzw. die angrenzenden Arbeiten auch mit zu erledigen. (https://forum.fhem.de/index.php/topic,128285.0.html)

Bin ich dran.

[CoolTux]

Ich habe das Matrix etwas überarbeitet und meine Arbeiten Manfred zukommen lassen. Er wird sicherlich etwas brauchen bis er es in seinen master branch führt.

[mBielemeier]

Moin,
ich war leider 3 Wochen ausgefallen und musste jetzt erst mal die Arbeit im Job erledigen. Danke dass du dich so intensiv damit beschäftigst und ich dadurch das Gefühl bekomme, wie FHEM-Module intern arbeiten.

Jetzt habe ich die Änderungen eingelesen. Mein Merge sagte allerdings 497 Änderungen in client.pm die ich jetzt nicht einzeln kontrolliert habe. Ich aktiviere die neue Version in meinem Testsystem und werde sie testen.

Viele Grüße, Manfred

[CoolTux]

Dadurch das ich viel von Deinem Code in einzelne kleine Funktionen ausgelagert habe kommen wohl die vielen Änderungen zu stande.