fix(gui): refresh current provider data on user change (#1000)

Author: cosven

changelog.rst

@@ -1,5 +1,11 @@
 版本变更历史
 ----------------------
+5.1a0 (in development)
+""""""""""""""""""""""
+- 功能增强
+
+  - 当 provider 当前用户变化时，自动刷新头像与当前用户歌单（覆盖账号切换场景）
+
 5.0.0 (2026-01-24)
 """"""""""""""""""""""
 - 缺陷修复

feeluown/__init__.py

@@ -6,7 +6,7 @@
 from .consts import LOG_FILE
 
 
-__version__ = '5.0'
+__version__ = '5.1a0'
 
 
 dict_config = {

feeluown/gui/components/avatar.py

@@ -75,7 +75,9 @@ def on_clicked(self):
     def on_provider_added(self, provider: Provider):
         if isinstance(provider, SupportsCurrentUserChanged):
             provider.current_user_changed.connect(
-                self.create_provider_current_user_changed_cb(provider), weak=False
+                self.create_provider_current_user_changed_cb(provider),
+                weak=False,
+                aioqueue=True,
             )
 
     def create_provider_current_user_changed_cb(self, provider: Provider):
@@ -85,6 +87,16 @@ def cb(user: UserModel):
                 self._logging_state[provider.identifier] = user.name
             else:
                 self._logging_state.pop(provider.identifier, None)
+
+            current_pvd_ui = self._app.current_pvd_ui_mgr.get()
+            if user is not None and current_pvd_ui is not None:
+                if current_pvd_ui.provider is provider:
+                    run_afn(self.show_pvd_ui_current_user)
+                    run_afn(
+                        self._app.ui.sidebar.show_provider_current_user_playlists,
+                        provider,
+                    )
+
             if not self._app.current_pvd_ui_mgr.get():
                 if self._logging_state:
                     self._text = t("select-profile")

tests/gui/components/test_avatar.py

@@ -0,0 +1,68 @@
+from types import SimpleNamespace
+from unittest.mock import MagicMock
+
+from feeluown.gui.components.avatar import Avatar
+
+
+class DummyProvider:
+    def __init__(self, identifier="dummy"):
+        self.identifier = identifier
+        self.current_user_changed = SimpleNamespace(connect=MagicMock())
+
+
+def test_on_provider_added_connects_with_aioqueue(qtbot, app_mock):
+    avatar = Avatar(app_mock)
+    qtbot.addWidget(avatar)
+
+    provider = DummyProvider()
+    avatar.on_provider_added(provider)
+
+    assert provider.current_user_changed.connect.call_count == 1
+    _, kwargs = provider.current_user_changed.connect.call_args
+    assert kwargs["weak"] is False
+    assert kwargs["aioqueue"] is True
+
+
+def test_current_user_changed_refreshes_when_provider_is_current(
+    qtbot, app_mock, monkeypatch
+):
+    avatar = Avatar(app_mock)
+    qtbot.addWidget(avatar)
+
+    provider = DummyProvider()
+    app_mock.current_pvd_ui_mgr.get.return_value = SimpleNamespace(provider=provider)
+
+    run_afn_mock = MagicMock()
+    monkeypatch.setattr("feeluown.gui.components.avatar.run_afn", run_afn_mock)
+
+    user = SimpleNamespace(name="tester")
+    cb = avatar.create_provider_current_user_changed_cb(provider)
+    cb(user)
+
+    assert avatar._logging_state[provider.identifier] == "tester"
+    run_afn_mock.assert_any_call(avatar.show_pvd_ui_current_user)
+    run_afn_mock.assert_any_call(
+        app_mock.ui.sidebar.show_provider_current_user_playlists,
+        provider,
+    )
+
+
+def test_current_user_changed_skip_refresh_when_provider_not_current(
+    qtbot, app_mock, monkeypatch
+):
+    avatar = Avatar(app_mock)
+    qtbot.addWidget(avatar)
+
+    provider = DummyProvider("dummy")
+    other_provider = DummyProvider("other")
+    app_mock.current_pvd_ui_mgr.get.return_value = SimpleNamespace(
+        provider=other_provider
+    )
+
+    run_afn_mock = MagicMock()
+    monkeypatch.setattr("feeluown.gui.components.avatar.run_afn", run_afn_mock)
+
+    cb = avatar.create_provider_current_user_changed_cb(provider)
+    cb(SimpleNamespace(name="tester"))
+
+    run_afn_mock.assert_not_called()