Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mehrere Timer definiert #7

Open
elektron-bbs opened this issue Jan 3, 2020 · 5 comments
Open

mehrere Timer definiert #7

elektron-bbs opened this issue Jan 3, 2020 · 5 comments
Labels
bug Something isn't working

Comments

@elektron-bbs
Copy link
Contributor

elektron-bbs commented Jan 3, 2020

@HomeAutoUser
Hast du eine Idee, wie wir das lösen könnten:
https://forum.fhem.de/index.php/topic,103986.msg1007924.html#msg1007924

Eigentlich war dieser Modus ja nie vorgesehen, das mehrere Timer definiert werden. Ich wäre auch nie auf diese Idee gekommen...
Für das Sytem ist es auch nicht gerade zuträglich, wenn mehrere InternalTimer zur gleichen Zeit laufen.

Woran es liegt, ist mir eigentlich klar. Wir verwenden zwar eindeutige $id in der sub Timer_FW_Detail, wenn auf einer Seite allerdings mehrere Timer dargestellt werden, sind diese nicht mehr eindeutig.

Entweder lassen wir uns etwas einfallen mit der Id (z.B. noch den Timernamen mit anhängen), was aber nicht ganz einfach werden dürfte, oder vieleicht in der Raumansicht die Speichern-Buttons ausblenden, oder vieleicht gleich das Definieren mehrerer Timer unterbinden...

@HomeAutoUser
Copy link
Collaborator

So wie du es geschildert hast und so wie ich es auch verstanden habe, so verstehen wir auf jedenfall das Problem beide gleich.

Entweder lassen wir uns etwas einfallen mit der Id (z.B. noch den Timernamen mit anhängen),

Sowas fiel mir auch sofort als Idee ein aber dann wäre was "kurzes eindeutiges" am Besten.
Mir kommt spontan die Idee, das wir vielleicht auch den Timestamp vom definieren des Modules nehmen könnten und den Jeweils an die ID anhängen oder sowas.

Ich habe mal den User hinterfragt um erstmal ein wenig Input zu halten wieso der User sowas überhaupt macht. Letztendlich ist es richtig, das da ggf ne Lösung her sollte, denn bei einem DOIF oder AT kann man ja auch mehrere definieren.

@HomeAutoUser HomeAutoUser added the bug Something isn't working label Jan 3, 2020
@elektron-bbs
Copy link
Contributor Author

Sowas fiel mir auch sofort als Idee ein aber dann wäre was "kurzes eindeutiges" am Besten.
Mir kommt spontan die Idee, das wir vielleicht auch den Timestamp vom definieren des Modules nehmen könnten und den Jeweils an die ID anhängen oder sowas.

Hat nur den Haken, das es im Javascript wieder auseinander genommen werden muss...

@HomeAutoUser
Copy link
Collaborator

Egal was wir in irgendeiner Form dranhängen oder davor setzen, das müssen wir überall wieder "splitten".

@HomeAutoUser
Copy link
Collaborator

Aufgrund das die Userzahlen mit dem Modul steigen, denke ich, wäre es eine gute Variante wenn wir da eine Lösung finden.
Mir kam soben die Idee, das wir anstatt langen Namen oder Zahlen es so einfach aufbauen, das wir beim Define schauen ob ein Timermodul schon definiert ist und dieses einfach +1 bennen. So fangen wir mit 1 an und wenn jemand 5 im Besitz hat und ein weiteres definiert, dieses dann die Zahl 6 erhält. Das ganze können Wir auf zwei stellen zulassen von 01 - 99. Die Zahl wird dann in einem Internal gespeichert oder unsichtbar in einem Helper.

... spontan mal herumgesponnen

@elektron-bbs
Copy link
Contributor Author

Naja, das wird evtl. schwierig, wenn der User mal nen Timer löscht und neu anlegt usw.
Eindeutig müsste das Internal NAME sein. Das ist sowieso immer vorhanden und ändert sich auch automatisch beim Umbenennen.
Da fällt mir auch noch ein, das dieser Name gleich beim Speichern in die Datei mit verwendet werden muss.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants