00_CUL.pm: maxPairForSec entfernen

[Matthias Gehre]

Hallo,

könnten wir maxPairForSec aus 00_CUL entfernen? Das wird alles
in 14_CUL_MAX gemacht (dort set pairmode). maxPairForSec ist also ohne Funktion und verwirrt User.
https://groups.google.com/forum/?fromgroups=#!topic/fhem-users/wlhByZrvQPs

Viele Grüße,
Matthias

[Hausautomat]

Hallo Matthias,

brauchst Du die Basisfunktionen in 00_CUL dazu? Ich würde sonst nur
maxPairForSec aus dem Commandset (%sets) entfernen, dann wird es nicht
mehr am CUL-Device als set option angezeigt, steht aber für Eingeweihte
noch zur Verfügung.

Gruß
Jens

> <https://groups.google.com/forum/?fromgroups=#%21topic/fhem-users/wlhByZrvQPs>
>
> Viele Grüße,
> Matthias
> --
>
>

[Matthias Gehre]

Das kann komplett weg. Im Moment macht set maxPairForSec eh nur
  $hash->{maxPair} = 1;
  InternalTimer(gettimeofday()+$arg, "CUL_RemoveMAXPair", $hash, 1);
und dann in CUL_RemoveMAXPair:
  delete($hash->{maxPair});
Also keine Funktionalität.

--

[Hausautomat]

Am 15.12.2012 15:24, schrieb Matthias Gehre:

Das kann komplett weg. Im Moment macht set maxPairForSec eh nur
  $hash->{maxPair} = 1;
  InternalTimer(gettimeofday()+$arg, "CUL_RemoveMAXPair", $hash, 1);
und dann in CUL_RemoveMAXPair:
  delete($hash->{maxPair});
Also keine Funktionalität.

Definitionsfrage. Die Funktionalität bestand ja darin, eben diesen Wert von einer Stelle aus analog zu HM zu setzen bzw. zu löschen. Damit andere Module darauf aufbauend Pairing machen oder nicht.

Auch in 14_CUL_MAX wäre die Pairing-Auswahl anhand des $hash->{maxPair} möglich gewesen. Dann würde das pairing in MAX genauso funktionieren wie bei HM, nämlich auf dem CUL selbst.

Ist mir aber egal. Ich habe es jetzt komplett in 00_CUL gelöscht, lokal fehlerfrei getestet und committed. Ist im svn drin. Damit werden die User nicht durch zweierlei "Pairing" verwirrt. Gerne weitere Diskussion, welcher Weg der "richtige" oder "gewollte" ist :-).

Gruß
 Jens

Am 15. Dezember 2012 14:56 schrieb Hausautomat <hausa...@googlemail.com>:

Hallo Matthias,

brauchst Du die Basisfunktionen in 00_CUL dazu? Ich würde sonst nur maxPairForSec aus dem Commandset (%sets) entfernen, dann wird es nicht mehr am CUL-Device als set option angezeigt, steht aber für Eingeweihte noch zur Verfügung.

Gruß
 Jens


Am 15.12.2012 12:54, schrieb Matthias Gehre:

Hallo,

könnten wir maxPairForSec aus 00_CUL entfernen? Das wird alles
in 14_CUL_MAX gemacht (dort set pairmode). maxPairForSec ist also ohne Funktion und verwirrt User.

https://groups.google.com/forum/?fromgroups=#!topic/fhem-users/wlhByZrvQPs <https://groups.google.com/forum/?fromgroups=#%21topic/fhem-users/wlhByZrvQPs>

Viele Grüße,
Matthias
--

--

--
 
 

[Matthias Gehre]

Ja, du hast Recht. Habe ich gar nicht so betrachtet. Jedenfalls ist das jetzt erst mal eine gute Lösung.

--