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

Issue detected by HA #102

Closed
popsyann opened this issue Jul 13, 2023 · 6 comments · Fixed by #103
Closed

Issue detected by HA #102

popsyann opened this issue Jul 13, 2023 · 6 comments · Fixed by #103

Comments

@popsyann
Copy link

Here is a log issued by Home Assistant (version 2023.7.1):

Logger: homeassistant.helpers.entity
Source: helpers/entity.py:701
First occurred: 10:40:10 (4 occurrences)
Last logged: 10:54:18

Entity var.living_room_temperature (<class 'custom_components.var.Variable'>) is using self.async_update_ha_state(), without enabling force_update. Instead it should use self.async_write_ha_state(), please report it to the custom integration author.
Entity var.master_bedroom_temperature (<class 'custom_components.var.Variable'>) is using self.async_update_ha_state(), without enabling force_update. Instead it should use self.async_write_ha_state(), please report it to the custom integration author.
Entity var.erwan_bedroom_temperature (<class 'custom_components.var.Variable'>) is using self.async_update_ha_state(), without enabling force_update. Instead it should use self.async_write_ha_state(), please report it to the custom integration author.
Entity var.bitcoin_total_initial_value (<class 'custom_components.var.Variable'>) is using self.async_update_ha_state(), without enabling force_update. Instead it should use self.async_write_ha_state(), please report it to the custom integration author.

@Foguet
Copy link

Foguet commented Jul 14, 2023

Hi! I have the same problem. How I can help to fix it?
Thanks in advance

@ercsey
Copy link

ercsey commented Jul 17, 2023

I made some changes for my own and created a pull request #100 but it was not yet merged. After the the changes I do not had any issues.

@pdwonline
Copy link

pull request #100 but it was not yet merged

Did you see the questions posted by @snarky-snark ? Could you please answer these to get this merged?

@lloyda
Copy link

lloyda commented Oct 11, 2023

@ercsey Be good if you could respond to @snarky-snark's questions. Would love to stop these errors filling my logfile.

@ercsey
Copy link

ercsey commented Oct 12, 2023

Hello. Sorry for late response. First of all I am not @ercsey, but his brother. He is in hospital. He sees the message but unable to respond. I am interpreting what he said: "I am not a programmer. I saw this solution on other github repo and made the changes for my self. After I've seen that the error messages are gone and the integration is working fine I made the changes and the pull request."
Sorry for this long waiting.

@danielbrunt57
Copy link
Contributor

danielbrunt57 commented Oct 16, 2023

I created a new pull request #103 which is slightly different than pull request #100. I also am not a python programmer so I do not know the absolute correct way of fixing this but my PR has eliminated the error message and the var integration is working for me. All I had to change was line 448:
await self.async_update_ha_state() --> await self.async_update_ha_state(True)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
6 participants