home-assistant / frontend

:lollipop: Frontend for Home Assistant
https://demo.home-assistant.io
Other
3.95k stars 2.68k forks source link

Image variable in picture-entity card overridden by `person` image #21738

Open damiano75 opened 3 weeks ago

damiano75 commented 3 weeks ago

Checklist

Describe the issue you are experiencing

I want to change the entity image with a local one, but I still have the entity default image.

Describe the behavior you expected

Before 2024.8 it worked fine

Steps to reproduce the issue

n/a

What version of Home Assistant Core has the issue?

core-2024.8.x

What was the last working version of Home Assistant Core?

core-2024.7.x

In which browser are you experiencing the issue with?

Many

Which operating system are you using to run this browser?

No response

State of relevant entities

No response

Problem-relevant frontend configuration

show_state: true
show_name: false
camera_view: auto
type: picture-entity
entity: person.damiano
image: /local/io_round.png

Javascript errors shown in your browser console/inspector

No response

Additional information

No response

karwosts commented 3 weeks ago

This would have regressed from this PR: https://github.com/home-assistant/frontend/pull/20593

They added support to get the image from person, but I suspect if a person entity and a specific image is specified, the given URL should probably have priority, and only use the person's picture if image: is not specified.

@Quentame does that sound right to you?

jschollenberger commented 2 weeks ago

I have this issue as well. I use a picture entity with a picture of a vehicle to fire off some actions when pressed (open garage door, start car, etc). The picture entity has a state filter which dims the photo when the driver is not home. This has worked well until the change above where now the person photo overrides the “image:” tag. Ultimately the photo of the car has been replaced by the photo of the person.

https://community.home-assistant.io/t/picture-entity-change-in-2024-8-broke-dashboard/763826