new module for Keeloq devices

[HomeAutoUser]

• Please check if the PR fulfills these requirements

• Tests for the changes have been added / modified (needed for for bug fixes / features)
• commandref has been added / updated (needed for bug fixes / features)
• CHANGED has been updated (needed for bug fixes / features)

• What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

• new module SD_Keeloq for devices Jarolift-Geräte #380 (without key: unencrypted data is output | mit Schlüssel: verschlüsselten Daten werden ausgegeben und senden ist möglich)
• remove Roto from 90_SIGNALduino_un.pm module
• revised signalduino_protocols.hash

• What is the current behavior? (You can also link to an open issue here)

• module not exist

[coveralls]

Pull Request Test Coverage Report for Build 1201

• 18 of 691 (2.6%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 3.509%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
FHEM/14_SD_Keeloq.pm	18	691	2.6%

Totals
Change from base Build 1191:	0.0%
Covered Lines:	30
Relevant Lines:	855

---

💛 - Coveralls

[sidey79]

Hast Du das Modul absichtlich in ein Package Main gepackt?

Ich würde emofehlen zu prüfen, ob Du es nicht besser in ein eigenes Package packst.
Ist aber natürlich blöd, jetzt nachdem es erst Mal fertig ist.

[HomeAutoUser]

Hast Du das Modul absichtlich in ein Package Main gepackt?

Hm.... was meinst du damit genau? Stehe gerade auf dem Schlauch.

[sidey79]

Ich meine

package main; im 14_SD_Keeloq.pm Modul.

Wir hatten das mal bei der Entstehung des Moduls SD_Bell diskutiert:

RFFHEM/FHEM/14_SD_BELL.pm

Line 86 in 441a465

package SD_BELL;

[HomeAutoUser]

Dachte ich mir :-)
Ich kann es ja mal noch umbauen gern.

[sidey79]

In diesem PR umbauen oder später. Ich wollte nur fragen ob es Absicht war es nicht in ein eigenes package zu stecken.

Du müsstest auch in dem Modul Syntax Check das neue Modul eintragen. Dann wird automatisch der Syntax geprüft :)

RFFHEM/test/test_modules-definition.txt

Line 3 in 4a2427a

my @modulesToTest=qw/00_SIGNALduino 10_FS10 14_BresserTemeo 14_FLAMINGO 14_Hideki 14_SD_AS 14_SD_BELL 14_SD_UT 14_SD_RSL 14_SD_WS 14_SD_WS_Maverick 14_SD_WS07 14_SD_WS09 41_OREGON 90_SIGNALduino_un 98_Dooya/;

[HomeAutoUser]

Du müsstest auch in dem Modul Syntax Check das neue Modul eintragen. Dann wird automatisch der Syntax geprüft :)

Dann wünsche ich es mir gern in den PR Text, das man spätestens dann dort dran erinnert wird. Du weißt es, aber wir nicht solange du es nicht dokumentierst ;-)
... oder wir müssen irgendwo ne ToDo List anlegen

[sidey79]

@HomeAutoUser

Der Reminder steht oben bereits im PR:

• Tests for the changes have been added / modified (needed for for bug fixes / features)