Revert "Add edit feature for tasbih entries with counter protection"

This reverts commit f79e90a.

Author: MahmoudAtef999

core_functions/tasbih/controller.py

@@ -145,16 +145,5 @@ def delete_all_entries(self):
                 logger.info("All tasbih entries deleted. Reinitializing default entries.")
             self._initialize_default_entries()
         except Exception as e:
-            logger.error(f"Error deleting all tasbih entries: {e}", exc_info=True) 
-
-
-    def update_entry_name(self, entry_id: int, new_name: str):
-        """update tasbih entrie."""
-        logger.debug(f"Updating name of tasbih entry ID {entry_id} to '{new_name}'")
-        entry = self.get_entry(entry_id)
-        if entry:
-            old_name = entry.name
-            entry.name = new_name
-            self.update_entry(entry)
-            self.entrieUpdated.emit(entry)
-            logger.info(f"Updated tasbih name from '{old_name}' to '{new_name}' (ID: {entry_id})")
+            logger.error(f"Error deleting all tasbih entries: {e}", exc_info=True)
+ 

ui/dialogs/tasbih_dialog.py

@@ -38,12 +38,6 @@ def __init__(self, parent) -> None:
         self.addButton.setToolTip("إضافة تسبيح")
         self.addButton.setAccessibleName("إضافة تسبيح")
 
-        self.editButton = QPushButton()
-        self.editButton.setIcon(qta.icon("fa.edit"))
-        self.editButton.setToolTip("تعديل التسبيح")
-        self.editButton.setAccessibleName("تعديل التسبيح")
-        self.editButton.setEnabled(False)
-
         self.delete_button = QPushButton()
         self.delete_button.setIcon(qta.icon("fa.trash"))
         self.delete_button.setToolTip("حذف تسبيح")
@@ -94,7 +88,6 @@ def __init__(self, parent) -> None:
         gridLayout = QGridLayout()
         gridLayout.addWidget(self.openButton, 0, 0)
         gridLayout.addWidget(self.addButton, 0, 1)
-        gridLayout.addWidget(self.editButton, 0, 3)
         gridLayout.addWidget(self.delete_button, 0, 2)
         gridLayout.addWidget(self.incrementButton, 1, 0)
         gridLayout.addWidget(self.decrementButton, 1, 1)
@@ -109,7 +102,6 @@ def __init__(self, parent) -> None:
         # Connect UI button clicks to slots.
         self.openButton.clicked.connect(self.open_tasbih_entry_dialog)
         self.addButton.clicked.connect(self.handle_add_entry)
-        self.editButton.clicked.connect(self.handle_edit_entry)
         self.delete_button.clicked.connect(self.handle_delete_entry)
         self.incrementButton.clicked.connect(self.handle_increment)
         self.decrementButton.clicked.connect(self.handle_decrement)
@@ -162,20 +154,6 @@ def open_tasbih_entry_dialog(self):
         UniversalSpeech.say(F"مرحبا بك في المِسْبَحَة، التسبيح: {tasbih_entry.name}، العدد: {tasbih_entry.counter}. استخدم المفاتيح التالية لزيادة العداد: Space, Enter, +,أو C. لإنقاص العداد استخدم: D, Ctrl+Space, -, أو Backspace. لإعادة تعيين العداد استخدم: Ctrl+R. للمعلومات استخدم: V للعدد، T للذِكر، I للكل.")
         dialog.exec()
 
-    def update_edit_button_state(self):
-        selected_item = self.listWidget.currentItem()
-        if not selected_item:
-            self.editButton.setEnabled(False)
-            return
-
-        entry_id = selected_item.data(Qt.ItemDataRole.UserRole)
-        entry = self.controller.get_entry(entry_id)
-        if entry and entry.counter > 0:
-            self.editButton.setEnabled(False)
-        else:
-            self.editButton.setEnabled(True)
-
-
     def OnItemSelectionChanged(self):    
         status = bool(self.listWidget.selectedItems())
         self.openButton.setEnabled(status)
@@ -184,10 +162,7 @@ def OnItemSelectionChanged(self):
         self.resetButton.setEnabled(status)
         self.delete_button.setEnabled(status)
         self.resetAllButton.setEnabled(status)
-        self.editButton.setEnabled(status)
         self.deleteAllButton.setEnabled(status)
-        self.update_edit_button_state()
-
         logger.debug(f"List item selection status: {status}. Buttons enabled: {status}.")
 
     def populate_list(self):
@@ -208,21 +183,6 @@ def add_list_item(self, entry):
         self.listWidget.addItem(item)
         logger.debug(f"Added Tasbih entry to list: {entry.name} (ID: {entry.id}, Count: {entry.counter})")
 
-    def handle_edit_entry(self):
-        selected_item = self.listWidget.currentItem()
-        if not selected_item:
-            return
-        entry_id = selected_item.data(Qt.ItemDataRole.UserRole)
-        old_entry = self.controller.get_entry(entry_id)
-
-        new_name, ok = QInputDialog.getText(self, "تعديل التسبيح", "أدخل الاسم الجديد:", text=old_entry.name)
-        if ok and new_name.strip():
-            self.controller.update_entry_name(entry_id, new_name.strip())
-            updated_entry = self.controller.get_entry(entry_id)
-            item_text = f"{updated_entry.name} | {updated_entry.counter}"
-            selected_item.setText(item_text)
-            UniversalSpeech.say(f"تم تعديل التسبيح إلى: {updated_entry.name}")
-
     def handle_add_entry(self):
         """Called when the Add button is clicked.
         
@@ -275,7 +235,6 @@ def handle_increment(self):
         entry_id = selected_item.data(Qt.ItemDataRole.UserRole)
         logger.debug(f"Incrementing counter for Tasbih entry ID: {entry_id}")
         self.controller.increment_entry_counter(entry_id)
-        self.update_edit_button_state()
 
     def handle_decrement(self):
         """Decrement the counter for the selected entry."""
@@ -284,16 +243,14 @@ def handle_decrement(self):
         entry_id = selected_item.data(Qt.ItemDataRole.UserRole)
         logger.debug(f"Decrementing counter for Tasbih entry ID: {entry_id}")
         self.controller.decrement_entry_counter(entry_id)
-        self.update_edit_button_state()        
-
+        
     def handle_reset(self):
         """Reset the counter for the selected entry."""
         logger.debug("Reset button has been clicked")
         selected_item = self.listWidget.currentItem()
         entry_id = selected_item.data(Qt.ItemDataRole.UserRole)
         logger.warning(f"Resetting counter for Tasbih entry ID: {entry_id}")
         self.controller.reset_entry_counter(entry_id)
-        self.update_edit_button_state()
 
     def handle_delete_entry(self):
         """Delete the selected entry."""