Card configuration change #6

thomasloven · 2020-02-13T15:41:13Z

thomasloven · 2020-02-13T15:41:19Z

⚠️ Information: Future card configuration changes.

Hi.

A change in how lovelace cards are set up in Home Assistant 0.105 increases performance, but enables cards to accidentally modify their own configuration in the loaded lovelace configuration.

I'm going through the cards in the HACS default repository, and noticed that your card may be susceptible to this. Looking through your code it seems you may modify parts of the config object passed to your card in setConfig.

The result could be that your card does not work well with the GUI editors or that parts of the configuration start showing up multiple times.

At some point in the future, it is likely that the configuration will be frozen before being passed to setConfig. At this point, your card will fail entirely when it tries to modify the configuration object.

There are several ways to fix/protect agains this problem.

The best is to restructure setConfig such that the configuration is never modified.
Other alternatives are to make a copy of the configuration and work on that instead.

setConfig(config) {
  config = { ...config }; // This works for simple configurations not containing arrays or objects
...

import { deepClone } from "deep-clone-simple";
// https://github.com/balloob/deep-clone-simple

setConfig(config) {
  config = deepClone(config); // This is a safe and fast method
...

or

setConfig(config) {
  config = JSON.parse(JSON.stringify(config)); // This uses built-in functions, but may be slower than deepClone
  ...

Please note that I have not tested your card agains Home Assistant 0.105 or later, but just quickly looked through the code. If I'm mistaken in my assessment, I appologize for taking your time.

See #6 for more info.

thomasloven · 2020-02-13T16:10:41Z

FAQ

Is my card ok if it's checked off on this list and I didn't get an issue?

Not necessarily. I only looked at your source very quickly.

Is `Object.assign({ }, config)` ok?

That's the same thing as {...config}. It will work for most simple configurations.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Card configuration change #6

Card configuration change #6

thomasloven commented Feb 13, 2020 •

edited

thomasloven commented Feb 13, 2020 •

edited

thomasloven commented Feb 13, 2020 •

edited

Card configuration change #6

Card configuration change #6

Comments

thomasloven commented Feb 13, 2020 • edited

thomasloven commented Feb 13, 2020 • edited

thomasloven commented Feb 13, 2020 • edited

FAQ

Is my card ok if it's checked off on this list and I didn't get an issue?

Is Object.assign({ }, config) ok?

thomasloven commented Feb 13, 2020 •

edited

thomasloven commented Feb 13, 2020 •

edited

thomasloven commented Feb 13, 2020 •

edited

Is `Object.assign({ }, config)` ok?