Combine States and Objects #203
Comments
|
delState -> OK While testing this i found out that setState create's no Object related to the state. |
|
setState cannot create object. It has no info about state
ср, 13 июн. 2018 г., 9:34 Jey-Cee <notifications@github.com>:
… delState -> OK
delObject/delForeignObject -> OK
While testing this i found out that setState create's no Object related to
the state.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#203 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AEXqgAMX7H4TL_4_sbRG7Pkq3Vvq6I9qks5t8MB0gaJpZM4UPPmD>
.
|
|
Done? |
|
Yes |
### Breaking changes: * (bluefox) Breaking changes: "][*,;'"`<>?" are no more allowed in IDs ### Feature changes: * (bluefox) the user by changing of states and objects is logged * (bluefox) install specific version of missing adapter * (bluefox) add disk info * (bluefox) add memAvailable state * (bluefox) add noChmod experimental settings * (bluefox) stop instances only if they belongs to current host * (bluefox) delete adapter only on current host and leave it on others * (bluefox) Combine States and Objects (#203) * (bluefox) Implement adapter.terminate('Because I need it') (#197) * (alcalzone) add restart, disable, updateConfig methods to Adapter class * (bluefox) add reinstall.js * (bluefox) add "iob" as shortcut * (bluefox) add getInterfaces to sendToHost * (bluefox) implement backup of data folders via dataFolder flag * (alcalzone) Add a wrapper method to safely expose ES6 classes to legacy code * (bluefox) add information if instance is alive in "list instances" * (stabilostick) added 'pidusage' for information about cpu and new states per adapter instance * (bluefox) support of new mime types: pdf, doc, xls, ppt * (bluefox) implement flag common.eraseOnUpload * (bluefox) set quality codes on connection lost and adapter disconnect * (alcalzone) Give adapters a chance to handle their own uncaught errors * (alcalzone) Add call stack information to hard to trace error logs * (bluefox) add pattern2RegEx to tools ### Fixes and Optimizations: * (apollon77) the node10 testing was added * (bluefox) the read file function was protected * (bluefox) possible access rights problem was closed * (bluefox) refactoring of memory calculations * (bluefox) set default values of states by creation of new instances * (alcalzone) Fix "install npm" messages, upgrade node typings to v6 * (bluefox) check if mem file could be read * (buzzy1337) fix calculation of kilobyte to megabyte and fix reading /proc/meminfo * (bluefox) Fix: Access to log files through admin does not work * (bluefox) fix redis disconnect * (bluefox) Update acme to V2 (#199) * (bluefox) Improve deleteDevice (#186) * (bluefox) fix cookie expiration * (alcalzone) several async and testing fixes * (bluefox) timeout for npm check added * (bluefox) fix "object chmod 777 777 javascript.0.*" command * (bluefox) do not send ready second time on reconnect db * (Apollon77, Bluefox) prevent defaultObj to be overwritten with normal common details * (bluefox) fix error if ID is empty * (bluefox) check id by setObject * (apollon77) Add amazon-dash to list of adapter to be installed with unsafe-perm * (bluefox) leave backitup enabled after restore
|
I just found out this causes the issue I describe here #268 . The reason for that is that I use The OP describes that States can not exists without an related Object. However this change is not entirely in line with this description. I think it is fair that deleting an object deletes the corresponding state. But deleting the state should not delete the object IMO. Is it really necessary to delete the object when deleting the state? Or can we have the separation like this:
|
|
After thinking I would see it like @AlCalzone ... delState should only clear the value (as it did before). I also see this as a needed functionality e.g. when restructuring states in an adapter and adding a new substructure under the nbame of a former state ... sometime you want to remove the value from such a new channel object or so ... |
|
Also the method name contains the scope:
(same for the Foreign methods) I think this way it is understandable |
Problem with node-version check
As in this thread written, States can not exists without an related Object. So it is uneccessary to handle them sepperate when deleting Object or State.
ToDo:
The text was updated successfully, but these errors were encountered: