Closed elektron-bbs closed 1 year ago
Müsste ich mir mal genauer ansehen. Ich habe auch schon an devspec2array gedacht. Ich habe es erst mal so übernommen. Was mir bei diesem Pull Request aufgefallen ist: Es wurden beim Erstellen gar keine Tests durchgeführt. Der letzte Testdurchlauf war wohl gestern beim "Merge branch 'master' into master_SD_GT_feature".
Kann sein, dass das eröffnen einen PRs keine Jobs erzeugt. Ist mir vielleicht noch nie aufgefallen, weil ich noch mal anschließend was gepusht habe. Hmm.
devspec2array durchsucht auch jedes Element in %defs ;) $modules{defptr} müsste ggf. halt beim define explizit mit gesetzt werden. Aktuell wird es glaube ich nur in parse gesetzt.
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
[ ] Bugfix (please link issue)
[x] Feature enhancement
[ ] Documentation update
[ ] Unittest enhancement
[ ] other
What is the current behavior? (You can also link to an open issue here, if this describes the current behavior) https://github.com/RFD-FHEM/RFFHEM/issues/1138
What is the new behavior (if this is a feature change)? add attributes repeats and disableSetAllFunction in commandref, change format "a name="SD_GT"" to "a id="SD_GT""
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?) no
Other information: