14_SD_BELL.pm - Adjusted little things

[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, ...)
  revised code for PerlCritic
  revised comments
  remove fixed room allocation

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

• What is the new behavior (if this is a feature change)?
  everything adjusted

• Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

• Other information:
  none

[codecov]

Codecov Report

Merging #937 (f462452) into master (ddafac0) will increase coverage by 0.01%.
The diff coverage is 30.76%.

@@            Coverage Diff             @@
##           master     #937      +/-   ##
==========================================
+ Coverage   58.11%   58.13%   +0.01%     
==========================================
  Files         105      105              
  Lines        8409     8412       +3     
  Branches     1314     1314              
==========================================
+ Hits         4887     4890       +3     
  Misses       2637     2637              
  Partials      885      885              

Flag	Coverage Δ
fhem	48.42% <30.76%> (+0.02%)	⬆️
modules	58.13% <30.76%> (+0.01%)	⬆️
perl	91.42% <ø> (ø)
unittests	58.13% <30.76%> (+0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
FHEM/14_SD_BELL.pm	40.09% <30.76%> (+0.90%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ddafac0...f462452. Read the comment docs.

[HomeAutoUser]

@sidey79
habe finish mit Anpassungen :-D

[sidey79]

Würde ich nicht nicht als constant ablegen.

Du kannst es ganz gut aus den Metadaten holen oder wenns PERL Style sein soll dann als Variable deklarieren

[sidey79]

Ich genehmige, das mit der Version kannst Du dir ja noch überlegen.
Da gibt es auch eine Hilfsfunktion um Internals aus den Metadaten zu erzeugen

[HomeAutoUser]

Wo kann ich das mit der Hilfsfunktion entnehmen oder nachlesen? Ich möchte nicht dumm bleiben :-D

Variable anstatt Constante sehe ich ein, da die Variable sich ändern könnte und nicht ständig gleich = Constant bleibt.

[sidey79]

Hier ein Beispiel wie es in initialize und define eingebunden wird:

################################################## FHEM API
sub initialize {
    my $device_definition = shift;

    $device_definition->{DefFn}         = \&handle_define;
    $device_definition->{UndefFn}       = \&handle_undefine;
    $device_definition->{SetFn}         = \&handle_set;
    $device_definition->{GetFn}         = \&handle_get;
    $device_definition->{AttrFn}        = \&handle_attributes;
    $device_definition->{AttrList}      = join(
        q{ },
        (
            q{disable:0,1},
        )
    ) . qq[ $::readingFnAttributes ];

    return FHEM::Meta::InitMod( __FILE__, $device_definition );
}

sub handle_define {
    my $global_definition   = shift;
    my $define              = shift;

    if ( !FHEM::Meta::SetInternals($global_definition) ) {
        return $EVAL_ERROR;
    }

[sidey79]

Passt für mich Jetzt :)

[HomeAutoUser]

@elektron-bbs ,@sidey79 ich habe fertig.

Für die Version in Github erfolgt dann diese Ausgabe.

FVERSION	14_SD_BELL.pm:v1.0.0-s0/2021-03-22

Für die SVN Variante erfolgt dann dies Ausgabe.

FVERSION	14_SD_BELL.pm:v1.0.0-s22345/2021-03-22

Ich passe bei jeder Änderung die "erste Zeile das Datum im Code an" und somit ist eine Unterscheidung genau möglich.
0 = Github | 22345 = SVN Revision und die zusätzliche Angabe das Datum zeigt mit den Stand.

[HomeAutoUser]

Wieso generiert ab diesem PR 0b99c0d dies hier fehler? Ich sehe da nicht durch.

Dieser PR 60bc67e davor lief noch ohne Fehler durch. Wo ist der Zusammenhang?

[sidey79]

Hmm, irgendwo ist vermutlich ein Syntax Fehler.

Gefunden habe ich ihn aber nicht. :-(

[sidey79]

Der Test der nicht zufrieden ist, ist folgender:

    # Failed test 'No Warnings in logfile'
Error:     # at /home/runner/work/RFFHEM/RFFHEM/t/FHEM/00_SIGNALduino/01_SIGNALduino_Parse_MC.t line 120.
    # +-----+----+-------+
    # | GOT | OP | CHECK |
    # +-----+----+-------+
    # | 2   | eq | 0     |
    # +-----+----+-------+

Der hat ein Warning im Logfile gefunden, dass er aber nicht mag.

Das ist jetzt natürlich irgendwie schade, dass wir einen Fehler aus dem META Modul gefunden haben und er uns zum Verhängnis wird:

Error: 2021.03.22 21:30:18 1: PERL WARNING: Use of uninitialized value in pattern match (m//) at FHEM/Meta.pm line 2435.
Error: 2021.03.22 21:30:18 1: PERL WARNING: Use of uninitialized value $prefix in regexp compilation at FHEM/Meta.pm line 2474.

[HomeAutoUser]

Da sehen wir, das auch andere Fehler uns ins stolpern bringen können. Ich mache nun in einem anderen Branch step bei Step Schritte um die Stelle zu finden.

[sidey79]

https://github.com/fhem/fhem-mirror/blob/32a3a8baf77280b720cf68c77cf1f0e1dbbdddf6/fhem/FHEM/Meta.pm#L2435

[sidey79]

Bugs sind angezeigt:
fhem/pkg-Meta#8
fhem/pkg-Meta#7

[HomeAutoUser]

An die Stelle kam ich auch. Nun sollte alles wieder in Ordnung sein nach dem Durchlauf.

Danke für die Buganzeige gleich :-)