fix(v0.2.1): update release notes and profile stability behavior

Author: pntech-dev

README.md

@@ -35,15 +35,16 @@ Allows authorized users to create, edit, and manage document pages. Support for
 
 ---
 
-## 💡 What's New (v0.2.1) - Stability Fixes
+## 💡 What's New (v0.2.1) - Stability & Profile Improvements
 
-This patch release focuses on reliability fixes.
+This patch release improves reliability and profile editing behavior.
 
-### 🛠 Fixed
+### 🛠 Improved
 
-- Fixed a crash when opening the profile dialog if `username` is empty/null (`NoneType` error on `split`).
-- Hardened tag generation in the document editor for cases where document/page names are empty or null.
-- Added safer handling for malformed virtual category IDs in search/loading flows.
+- Fixed a crash when opening the profile window for accounts without a filled display name.
+- Improved reliability of profile data updates after backend deployment updates.
+- Improved handling of empty values in document and page names.
+- Improved stability of search and document loading in edge cases.
 
 ---
 

README_RU.md

@@ -35,15 +35,16 @@
 
 ---
 
-## 💡 Что нового (v0.2.1) - Исправления стабильности
+## 💡 Что нового (v0.2.1) - Стабильность и профиль
 
-Это патч-обновление сфокусировано на исправлениях надежности.
+Это патч-обновление повышает надежность и улучшает поведение при работе с профилем.
 
-### 🛠 Исправлено
+### 🛠 Улучшено
 
-- Исправлено падение при открытии окна профиля, если `username` пустой/null (ошибка `NoneType` при вызове `split`).
-- Усилена генерация тегов в редакторе документов для случаев, когда имена документа/страниц пустые или null.
-- Добавлена безопасная обработка некорректных виртуальных ID категорий в поиске и загрузке.
+- Исправлен сбой при открытии профиля для аккаунтов без заполненного отображаемого имени.
+- Повышена надежность обновления данных профиля после обновлений backend-сервиса.
+- Улучшена обработка пустых значений в названиях документов и страниц.
+- Повышена стабильность поиска и загрузки документов в пограничных сценариях.
 
 ---
 

modules/main/mvc/main_controller.py

@@ -400,9 +400,13 @@ def _show_profile_dialog(self) -> None:
             )
 
             if updated_data:
+                current_user_id = user_data.get("id") or user_data.get("user_id")
+                if current_user_id is None:
+                    current_user_id = self.model.get_current_user_id()
+
                 worker = APIWorker(
                     self.model.update_user_profile,
-                    user_id=user_data['id'],
+                    user_id=current_user_id,
                     data=updated_data
                 )
                 self.active_workers.add(worker)
@@ -446,13 +450,16 @@ def _on_department_selected(self, selected, deselected) -> None:
             if dept_id is not None and dept_id != self.model.current_department_id:
                 self.model.current_department_id = dept_id
                 self.model.current_category_id = None
-                
-                # Explicitly clear documents to avoid showing stale data from previous department
-                self._update_documents_list()                
+
                 self._update_categories_list()
-                
+
+                # With active search keep previous table until new results arrive.
+                # This avoids a blank table if the first request after idle fails.
                 if self.view.get_search_text():
                     self._on_search_lineedit_text_changed()
+                else:
+                    # No search active: clear stale documents and load current category data.
+                    self._update_documents_list()
 
                 self._update_create_category_state()
                 self._update_create_document_state()

modules/main/mvc/main_model.py

@@ -44,6 +44,7 @@ def get_user_data(self) -> dict | None:
             return None
 
         data = self.api.get_user_data(token)
+        data = self._normalize_user_data(data)
 
         return data
 
@@ -79,15 +80,38 @@ def get_full_user_data(self) -> dict | None:
             return None
 
         # This might be the same as get_user_data, but let's assume it could be different
-        return self.api.get_user_data(token)
+        data = self.api.get_user_data(token)
+        return self._normalize_user_data(data)
+
+    def get_current_user_id(self) -> int | None:
+        """Returns the currently logged-in user id from local session storage."""
+        if self.mode == "guest":
+            return None
+
+        last_logged = read_json(self.LOCAL_DIR_LAST_LOGGED)
+        if not isinstance(last_logged, dict):
+            return None
+
+        user_id = last_logged.get("user_id")
+        if user_id is None:
+            return None
+
+        try:
+            return int(user_id)
+        except (TypeError, ValueError):
+            return None
 
 
-    def update_user_profile(self, user_id: int, data: dict) -> dict:
+    def update_user_profile(self, user_id: int | None, data: dict) -> dict:
         """Updates the user's profile data."""
+        resolved_user_id = user_id if user_id is not None else self.get_current_user_id()
+        if resolved_user_id is None:
+            raise ValueError("Cannot update profile: missing current user id")
+
         # The data dict should contain 'username', 'department' (ID)
         return self._make_authorized_request(
             self.api.update_user_data,
-            user_id=user_id,
+            user_id=resolved_user_id,
             data=data
         )
 
@@ -330,6 +354,18 @@ def _get_user_token(self) -> str | None:
         )
 
         return access_token
+
+    @staticmethod
+    def _normalize_user_data(data: dict | None) -> dict | None:
+        """Normalizes API user payload to a flat user dict."""
+        if not isinstance(data, dict):
+            return None
+
+        nested_user = data.get("user")
+        if isinstance(nested_user, dict):
+            return nested_user
+
+        return data
 
 
     def _get_departments(self) -> list[dict]:

tests/test_main.py

@@ -88,6 +88,21 @@ def test_department_selection(self, controller):
         assert controller.model.current_category_id is None
         controller.view.update_categories.assert_called()
 
+    def test_department_selection_with_search_does_not_clear_table(self, controller):
+        """With active search, department switch should not clear table before search response."""
+        mock_index = Mock()
+        mock_index.data.return_value = 2
+        mock_selection = Mock()
+        mock_selection.indexes.return_value = [mock_index]
+        controller.view.get_search_text.return_value = "query"
+
+        with patch.object(controller, "_update_documents_list") as mock_update_docs, \
+             patch.object(controller, "_on_search_lineedit_text_changed") as mock_search_changed:
+            controller._on_department_selected(mock_selection, None)
+
+        mock_update_docs.assert_not_called()
+        mock_search_changed.assert_called_once()
+
 
     def test_category_selection(self, controller):
         """Test selecting a category triggers document loading."""

tests/test_main_model.py

@@ -0,0 +1,34 @@
+from unittest.mock import patch
+
+from modules.main.mvc.main_model import MainModel
+
+
+class TestMainModel:
+    def _build_model(self):
+        with patch("modules.main.mvc.main_model.load_config", return_value={"base_url": "http://localhost"}), \
+             patch("modules.main.mvc.main_model.get_app_data_dir"), \
+             patch("modules.main.mvc.main_model.get_local_data_dir"):
+            return MainModel(mode="auth")
+
+    def test_get_user_data_normalizes_nested_payload(self):
+        model = self._build_model()
+        model._get_user_token = lambda: "token"
+        model.api.get_user_data = lambda token: {"user": {"id": 7, "email": "u@x.test"}}
+
+        data = model.get_user_data()
+
+        assert data == {"id": 7, "email": "u@x.test"}
+
+    def test_get_full_user_data_normalizes_nested_payload(self):
+        model = self._build_model()
+        model._get_user_token = lambda: "token"
+        model.api.get_user_data = lambda token: {"user": {"id": 3, "username": "Test User"}}
+
+        data = model.get_full_user_data()
+
+        assert data == {"id": 3, "username": "Test User"}
+
+    def test_normalize_user_data_keeps_flat_payload(self):
+        flat = {"id": 1, "email": "flat@x.test"}
+        assert MainModel._normalize_user_data(flat) == flat
+

utils/whats_new_modal.py

@@ -19,17 +19,12 @@
 RELEASE_NOTES = {
     "0.2.1": [
         {
-            "title": "Исправления стабильности",
+            "title": "Стабильность и профиль",
             "items": [
-                "Исправлено падение при открытии профиля, если username отсутствует (ошибка NoneType при split).",
-                "Усилена обработка данных в генерации тегов: пустые/null имена документа и страниц больше не приводят к ошибкам.",
-                "Добавлена безопасная обработка некорректных virtual ID категорий в поиске и загрузке документов.",
-            ],
-        },
-        {
-            "title": "Качество и проверка",
-            "items": [
-                "Добавлены регрессионные тесты для случаев с null/некорректными значениями в критичных сценариях.",
+                "Исправлен сбой при открытии профиля для аккаунтов без заполненного отображаемого имени.",
+                "Повышена надежность обновления данных профиля после обновлений backend-сервиса.",
+                "Улучшена обработка пустых значений в названиях документов и страниц.",
+                "Повышена стабильность поиска и загрузки документов в пограничных сценариях.",
             ],
         },
     ],