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

Item properties / item details ignoring fromhook #903

Closed
marc2k3 opened this issue Jun 29, 2024 · 3 comments · Fixed by #912
Closed

Item properties / item details ignoring fromhook #903

marc2k3 opened this issue Jun 29, 2024 · 3 comments · Fixed by #912

Comments

@marc2k3
Copy link
Contributor

marc2k3 commented Jun 29, 2024

if (!p_fromhook && !m_nowplaying_active) {

Is there a reason for this? It means Playback Statistics / 3rd party components using metadb_index_manager::dispatch_refresh don't see real time updates.

If performance is an issue, maybe loosen it for handle counts of one?

@reupen
Copy link
Owner

reupen commented Jun 29, 2024

Hmm, definitely wrong for Item details.

Taking the docs at face value, it looks correct for Item properties (as p_fromhook and metadb_display_field_provider are for title formatting). However, Selection Properties in DUI updates when changing ratings using Playback Statistics, so I assume Selection Properties is ignoring the flag (as I'm not sure how else it would be updating...)

@marc2k3
Copy link
Contributor Author

marc2k3 commented Jun 30, 2024

Hmm, definitely wrong for Item details.

IMHO, Item properties populating Playback Statistics / Other sections using track_property_provider::enumerate gets data from the exact the same place as metadb_display_field_provider so it should respond to on_changed_sorted / p_fromhook as well.

@marc2k3
Copy link
Contributor Author

marc2k3 commented Jul 11, 2024

Thanks for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants