Merge branch 'fix/status-not-updating' into development

Fixed the document status to properly change when the user edit something thus warning the document is not saved.nijthen

CHANGELOG.md

@@ -1,51 +0,0 @@
-# Changelog
-
-All notable changes to this project will be documented in this file.
-
-The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
-and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
-
-## [Unreleased]
-
-### Planned
-
-- Installation Method 1 (repository compilation)
-- Organization of trips by date, location, or theme
-- Enhanced photo management features
-- Search functionality
-- Export features
-- Testing implementation
-
-## [0.0.3] - 2025-07-07
-
-### Changed
-- Removed the dependency on textual-dev from pyproject.toml
-
-## [0.0.2] - 2025-07-07
-
-### Changed
-- Changed the license in pyproject.toml to BSD
-
-## [0.0.1] - 2025-07-06
-
-### Added
-
-- Initial alpha release of Pilgrim travel diary application
-- Create and edit travel diaries
-- Create and edit diary entries
-- Photo ingestion system
-- Photo addition and reference via sidebar
-- Text User Interface (TUI) built with Textual framework
-- Pre-compiled binary installation method (Method 2)
-- Support for Linux operating systems
-- Basic project documentation (README)
-
-### Known Issues
-
-- Installation Method 1 not yet implemented
-- No testing suite implemented yet
-- Some features may be unstable in an alpha version
-
-[Unreleased]: https://github.com/username/pilgrim/compare/v0.0.1...HEAD
-
-[0.0.1]: https://github.com/username/pilgrim/releases/tag/v0.0.1

README.md

@@ -1,17 +1,16 @@
-# Pilgrim
+# Python_Pilgrim
 
 ## Overview
 
-**Pilgrim** is a Python-based travel diary application designed to help users document and manage their travel
-experiences. The project provides tools for recording trips, organizing travel notes, and storing memories in a
-structured and accessible format.
+**Python_Pilgrim** is a Python-based travel diary application designed to help users document and manage their travel experiences. The project provides tools for recording trips, organizing travel notes, and storing memories in a structured and accessible format.
 
 ## Features
 
 - Create and manage travel diaries
 - Add, edit, and delete travel entries
 - Organize trips by date, location, or theme
-- Store photos and add them to the diary entry
+- Store photos, notes, and other media
+- Export and share travel logs
 
 ## Requirements
 - Python 3.8 or higher
@@ -22,25 +21,30 @@ structured and accessible format.
 
 ## Installation
 
-To install the application you must issue the command:
-
-```bash
- pip install --index-url https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple Pilgrim
-```
+1. Clone the repository:
+    ```bash
+    git clone https://github.com/gmbrax/Pilgrim.git
+    ```
+2. Navigate to the project directory:
+    ```bash
+    cd Pilgrim
+    ```
+3. Create a virtual environment and, then, activate it:
+    ```bash
+    python -m venv .venv
+    source .venv/bin/activate
+    ```
+4. Install the required dependencies:
+    ```bash
+    pip install -r requirements.txt
+    ```
 
 ## Usage
 
-To use application you just issue:
-```bash 
-pilgrim
+To run the main application, execute:
+
+```bash
+python <???>.py
 ```
 
-This will start the Pilgrim application. Follow the on-screen instructions to create and manage your travel diaries.
-
-## Changelog
-
-To see all the changes in the current version, please refer to the [CHANGELOG](CHANGELOG.md)
-
-## License
-
-This project is licensed under the BSD 3-Clause License — see the [LICENSE](LICENSE) file for details.
+This will start the Python_Pilgrim application. Follow the on-screen instructions to create and manage your travel diaries.

pyproject.toml

@@ -5,7 +5,7 @@
 
     [project]
     name = "Pilgrim"
-    version = "0.0.3"
+    version = "0.0.1"
     authors = [
       { name="Gustavo Henrique Santos Souza de Miranda", email="gustavohssmiranda@gmail.com" }
     ]

requirements.txt

@@ -1,6 +1,6 @@
 greenlet==3.2.3
 SQLAlchemy==2.0.41
-typing_extensions==4.14.1
+typing_extensions==4.14.0
 
 
-textual~=4.0.0
+textual~=3.3.0

src/pilgrim/ui/screens/diary_list_screen.py

@@ -203,36 +203,35 @@ class DiaryListScreen(Screen):
             self.action_open_diary()
 
     def action_new_diary(self):
-        """
-        Opens the modal dialog to create a new diary and registers a callback for submission.
-        """
+        """Action to create new diary"""
         self.app.push_screen(NewDiaryModal(),self._on_new_diary_submitted)
 
-    def _on_new_diary_submitted(self, result):
-        """
-        Handles the callback after a new diary is submitted.
-        
-        If a diary was created (non-empty result), notifies the user and refreshes the diary list; otherwise, notifies that creation was canceled.
-        """
-        if result:  # Se result não é string vazia, o diário foi criado
-            self.notify(f"Returning to diary list...")
-            # Atualiza a lista de diários
-            self.refresh_diaries()
+    def _on_new_diary_submitted(self,result):
+        self.notify(str(result))
+        if result:
+            self.notify(f"Creating Diary:{result}'...")
+            self.call_later(self._async_create_diary,result)
         else:
-            self.notify(f"Creation canceled...")
+            self.notify(f"Canceled...")
+
+    async def _async_create_diary(self,name: str):
+
+        try:
+            service = self.app.service_manager.get_travel_diary_service()
+            created_diary = await service.async_create(name)
+            if created_diary:
+                self.diary_id_map[created_diary.id] = created_diary.id
+                await self.async_refresh_diaries()
+                self.notify(f"Diary: '{name}' created!")
+            else:
+                self.notify("Error Creating the diary")
+        except Exception as e:
+            self.notify(f"Exception on creating the diary: {str(e)}")
+
 
-    def _on_screen_resume(self) -> None:
-        """
-        Refreshes the diary list when the screen resumes.
-        """
-        self.refresh_diaries()
 
     def action_edit_selected_diary(self):
-        """
-        Opens the edit dialog for the currently selected diary.
-        
-        If a diary is selected, displays the edit modal and registers a callback for when the edit is submitted. Notifies the user if no diary is selected.
-        """
+        """Action to edit selected diary"""
         if self.selected_diary_index is not None:
             diary_id = self.diary_id_map.get(self.selected_diary_index)
             if diary_id:

src/pilgrim/ui/screens/edit_diary_modal.py

@@ -8,13 +8,9 @@ from textual.widgets import Label, Input, Button
 class EditDiaryModal(ModalScreen[tuple[int,str]]):
     BINDINGS = [
         Binding("escape", "cancel", "Cancel"),
-        Binding("enter", "edit_diary", "Save",priority=True),
     ]
 
     def __init__(self, diary_id: int):
-        """
-        Initialize the EditDiaryModal with the specified diary ID, pre-filling the input field with the current diary name.
-        """
         super().__init__()
         self.diary_id = diary_id
         self.current_diary_name = self.app.service_manager.get_travel_diary_service().read_by_id(self.diary_id).name
@@ -35,48 +31,18 @@ class EditDiaryModal(ModalScreen[tuple[int,str]]):
         self.name_input.cursor_position = len(self.name_input.value)
 
     def on_button_pressed(self, event: Button.Pressed) -> None:
-        """
-        Handle button press events in the modal.
-        
-        Triggers the save action if the "Save" button is pressed, or cancels and dismisses the modal if the "Cancel" button is pressed.
-        """
         if event.button.id == "save_diary_button":
-            self.action_edit_diary()
+            new_diary_name = self.name_input.value.strip()
+            if new_diary_name and new_diary_name != self.current_diary_name:
+                self.dismiss((self.diary_id, new_diary_name))
+            elif new_diary_name == self.current_diary_name:
+                self.notify("No changes made.", severity="warning")
+                self.dismiss(None)
+            else:
+                self.notify("Diary name cannot be empty.", severity="warning")
+                self.name_input.focus()
         elif event.button.id == "cancel_button":
             self.dismiss(None)
-    def on_key(self, event):
-        """
-        Handles the Enter key press event by triggering the diary save action and preventing the default behavior.
-        """
-        if event.key == "enter":
-            self.action_edit_diary()
-            event.prevent_default()
-
-    def action_edit_diary(self) -> None:
-        """
-        Attempts to save the edited diary name, validating input and providing user feedback.
-        
-        If the new name is non-empty and different from the current name, dismisses the modal and returns the diary ID and new name. If the name is unchanged, notifies the user and dismisses without changes. If the input is empty, notifies the user and refocuses the input field.
-        """
-        new_diary_name = self.name_input.value.strip()
-        if new_diary_name and new_diary_name != self.current_diary_name:
-            self.dismiss((self.diary_id, new_diary_name))
-        elif new_diary_name == self.current_diary_name:
-            self.notify("No changes made.", severity="warning")
-            self.dismiss(None)
-        else:
-            self.notify("Diary name cannot be empty.", severity="warning")
-            self.name_input.focus()
-
-    def on_input_submitted(self, event: Input.Submitted) -> None:
-        """
-        Handles submission of the diary name input field by triggering the save action if the relevant input is submitted.
-        """
-        if event.input.id == "edit_diary_name_input":
-            self.action_edit_diary()
 
     def action_cancel(self) -> None:
-        """
-        Dismisses the modal without saving changes or returning any data.
-        """
         self.dismiss(None)

src/pilgrim/ui/screens/edit_entry_screen.py

@@ -24,33 +24,26 @@ class EditEntryScreen(Screen):
     BINDINGS = [
         Binding("ctrl+q", "quit", "Quit"),
         Binding("ctrl+s", "save", "Save"),
-        Binding("shift+f5", "new_entry", "New Entry"),
-        Binding("f5", "next_entry", "Next Entry"),
-        Binding("f4", "prev_entry", "Previous Entry"),
+        Binding("ctrl+n", "new_entry", "New Entry"),
+        Binding("ctrl+shift+n", "next_entry", "Next Entry"),
+        Binding("ctrl+shift+p", "prev_entry", "Previous Entry"),
         Binding("ctrl+r", "rename_entry", "Rename Entry"),
         Binding("f8", "toggle_sidebar", "Toggle Photos"),
         Binding("f9", "toggle_focus", "Toggle Focus"),
         Binding("escape", "back_to_list", "Back to List"),
     ]
 
-    def __init__(self, diary_id: int = 1,create_new: bool = True):
+    def __init__(self, diary_id: int = 1):
         super().__init__()
-
-        if create_new:
-            self.current_entry_index = -1
-            self.is_new_entry = True
-            self.next_entry_id = None
-
-        else:
-            self.is_new_entry = False
-            self.current_entry_index = 0
-            self.next_entry_id = 1
-        self.new_entry_title = ""
-        self.new_entry_content = ""
         self.diary_id = diary_id
         self.diary_name = f"Diary {diary_id}"
+        self.current_entry_index = 0
         self.entries: List[Entry] = []
+        self.is_new_entry = False
         self.has_unsaved_changes = False
+        self.new_entry_content = ""
+        self.new_entry_title = "New Entry"
+        self.next_entry_id = 1
         self._updating_display = False
         self._original_content = ""
         self.is_refreshing = False
@@ -913,22 +906,11 @@ class EditEntryScreen(Screen):
                 self.notify("Empty entry cannot be saved")
                 return
             # Passe a lista de fotos para o método de criação
-            if self.new_entry_title == "":
-               self.app.push_screen(RenameEntryModal(current_name=""), lambda result: self._handle_save_after_rename(result,content,
-                                                                                               photos_to_link))
-            else:
-                self.call_later(self._async_create_entry, content, photos_to_link)
+            self.call_later(self._async_create_entry, content, photos_to_link)
         else:
             # Passe a lista de fotos para o método de atualização
             self.call_later(self._async_update_entry, content, photos_to_link)
 
-    def _handle_save_after_rename(self, result: str | None, content: str, photos_to_link: List[Photo]) -> None:
-        if result is None:
-            self.notify("Save cancelled")
-            return
-        self.new_entry_title = result
-        self.call_later(self._async_create_entry, content, photos_to_link)
-
     async def _async_create_entry(self, content: str, photos_to_link: List[Photo]):
         """Creates a new entry and links the referenced photos."""
         try:
@@ -955,7 +937,7 @@ class EditEntryScreen(Screen):
                 self.is_new_entry = False
                 self.has_unsaved_changes = False
                 self._original_content = new_entry.text
-                self.new_entry_title = ""
+                self.new_entry_title = "New Entry"
                 self.next_entry_id = max(entry.id for entry in self.entries) + 1
 
                 self._update_entry_display()
@@ -1000,66 +982,49 @@ class EditEntryScreen(Screen):
         except Exception as e:
             self.notify(f"Error updating entry: {str(e)}")
 
-    def check_key(self, event):
-        """Check for custom key handling before bindings are processed"""
-
-        # Sidebar shortcuts
-        if self.sidebar_focused and self.sidebar_visible:
-            sidebar_keys = ["i", "n", "d", "e"]
-            if event.key in sidebar_keys:
-                if event.key == "i":
-                    self.action_insert_photo()
-                elif event.key == "n":
-                    self.action_ingest_new_photo()
-                elif event.key == "d":
-                    self.action_delete_photo()
-                elif event.key == "e":
-                    self.action_edit_photo()
-                return True  # Indica que o evento foi processado
-
-        # Text area shortcuts
-        elif self.focused is self.text_entry:
-            if event.key in ["tab", "shift+tab"]:
-                if event.key == "shift+tab":
-                    self._handle_shift_tab()
-                elif event.key == "tab":
-                    self.text_entry.insert('\t')
-                return True  # Indica que o evento foi processado
-
-        return False  # Não foi processado, continuar com bindings
-
-    def _handle_shift_tab(self):
-        """Handle shift+tab for removing indentation"""
-        textarea = self.text_entry
-        row, col = textarea.cursor_location
-        lines = textarea.text.splitlines()
-        if row < len(lines):
-            line = lines[row]
-            if line.startswith('\t'):
-                lines[row] = line[1:]
-                textarea.text = '\n'.join(lines)
-                textarea.cursor_location = (row, max(col - 1, 0))
-            elif line.startswith('    '):  # 4 spaces
-                lines[row] = line[4:]
-                textarea.text = '\n'.join(lines)
-                textarea.cursor_location = (row, max(col - 4, 0))
-            elif line.startswith(' '):
-                n = len(line) - len(line.lstrip(' '))
-                to_remove = min(n, 4)
-                lines[row] = line[to_remove:]
-                textarea.text = '\n'.join(lines)
-                textarea.cursor_location = (row, max(col - to_remove, 0))
-
     def on_key(self, event):
-        if self.check_key(event):
-            event.stop()
-            return
+        # Sidebar contextual shortcuts
+        if self.sidebar_focused and self.sidebar_visible:
 
-    def on_footer_action(self, event) -> None:
-        """Handle clicks on footer actions (Textual 3.x)."""
-        action = event.action
-        method = getattr(self, f"action_{action}", None)
-        if method:
-            method()
-        else:
-            self.notify(f"No action found for: {action}", severity="warning")
+            if event.key == "i":
+
+                self.action_insert_photo()
+                event.stop()
+            elif event.key == "n":
+
+                self.action_ingest_new_photo()
+                event.stop()
+            elif event.key == "d":
+
+                self.action_delete_photo()
+                event.stop()
+            elif event.key == "e":
+
+                self.action_edit_photo()
+                event.stop()
+        # Shift+Tab: remove indent
+        elif self.focused is self.text_entry and event.key == "shift+tab":
+            textarea = self.text_entry
+            row, col = textarea.cursor_location
+            lines = textarea.text.splitlines()
+            if row < len(lines):
+                line = lines[row]
+                if line.startswith('\t'):
+                    lines[row] = line[1:]
+                    textarea.text = '\n'.join(lines)
+                    textarea.cursor_location = (row, max(col - 1, 0))
+                elif line.startswith('    '):  # 4 spaces
+                    lines[row] = line[4:]
+                    textarea.text = '\n'.join(lines)
+                    textarea.cursor_location = (row, max(col - 4, 0))
+                elif line.startswith(' '):
+                    n = len(line) - len(line.lstrip(' '))
+                    to_remove = min(n, 4)
+                    lines[row] = line[to_remove:]
+                    textarea.text = '\n'.join(lines)
+                    textarea.cursor_location = (row, max(col - to_remove, 0))
+            event.stop()
+        # Tab: insert tab
+        elif self.focused is self.text_entry and event.key == "tab":
+            self.text_entry.insert('\t')
+            event.stop()

src/pilgrim/ui/screens/new_diary_modal.py

@@ -4,33 +4,17 @@ from textual.containers import Vertical, Horizontal
 from textual.screen import ModalScreen
 from textual.widgets import Label, Input, Button
 
-from pilgrim.ui.screens.edit_entry_screen import EditEntryScreen
-
 
 class NewDiaryModal(ModalScreen[str]):
     BINDINGS = [
         Binding("escape", "cancel", "Cancel"),
-        Binding("enter", "create_diary", "Create",priority=True),
     ]
-    def __init__(self,autoopen=True):
-        """
-        Initialize the NewDiaryModal with optional automatic opening of the created diary.
-        
-        Parameters:
-            autoopen (bool): If True, the newly created diary will open automatically after creation. Defaults to True.
-        """
+    def __init__(self):
         super().__init__()
-        self.auto_open = autoopen
         self.name_input = Input(id="NewDiaryModal-NameInput",classes="NewDiaryModal-NameInput") # This ID is fine, it's specific to the input
 
     def compose(self) -> ComposeResult:
 
-        """
-        Constructs and yields the UI layout for the new diary modal dialog.
-        
-        Returns:
-            ComposeResult: An iterable of widgets forming the modal, including title, input field, and action buttons.
-        """
         with Vertical(id="new_diary_dialog",classes="NewDiaryModal-Dialog"):
             yield Label("Create a new diary", classes="NewDiaryModal-Title")
             yield Label("Diary Name:")
@@ -45,67 +29,17 @@ class NewDiaryModal(ModalScreen[str]):
           self.name_input.focus()
 
     def on_button_pressed(self, event: Button.Pressed) -> None:
-        """
-        Handle button press events for the modal.
-        
-        Triggers diary creation when the "Create" button is pressed, or dismisses the modal when the "Cancel" button is pressed.
-        """
+        """Handles button clicks."""
         if event.button.id == "create_diary_button":
-            self.action_create_diary()
+            diary_name = self.name_input.value.strip()
+            if diary_name:
+                self.dismiss(diary_name)
+            else:
+                self.notify("Diary name cannot be empty.", severity="warning")
+                self.name_input.focus()
         elif event.button.id == "cancel_button":
-            self.dismiss()
+            self.dismiss("")
 
     def action_cancel(self) -> None:
-        """
-        Cancels the modal dialog and dismisses it without returning a diary name.
-        """
-        self.dismiss("")
-
-    def action_create_diary(self) -> None:
-        """
-        Validates the diary name input and initiates asynchronous diary creation if valid.
-        
-        If the input is empty, displays a warning notification and refocuses the input field.
-        """
-        diary_name = self.name_input.value.strip()
-        if diary_name:
-
-            self.call_later(self._async_create_diary, diary_name)
-
-        else:
-            self.notify("Diary name cannot be empty.", severity="warning")
-            self.name_input.focus()
-
-    def on_input_submitted(self, event: Input.Submitted) -> None:
-        """
-        Handles submission of the diary name input field and initiates diary creation if triggered from the correct input.
-        """
-        if event.input.id == "NewDiaryModal-NameInput":
-            self.action_create_diary()
-
-    async def _async_create_diary(self, name: str):
-
-        """
-        Asynchronously creates a new diary entry with the given name and handles post-creation actions.
-        
-        If creation succeeds, dismisses the modal with the diary name, optionally opens the diary for editing, and shows a success notification. If creation fails or an exception occurs, displays an error notification.
-        """
-        try:
-            service = self.app.service_manager.get_travel_diary_service()
-            created_diary = await service.async_create(name)
-            if created_diary:
-                self.dismiss(name)
-
-                if self.auto_open:
-                    self.app.push_screen(EditEntryScreen(diary_id=created_diary.id))
-
-                self.notify(f"Diary: '{name}' created!")
-            else:
-                self.notify("Error Creating the diary")
-        except Exception as e:
-            self.notify(f"Exception on creating the diary: {str(e)}")
-
-
-
-
-
+        """Action to cancel the modal."""
+        self.dismiss("")