Update delete_tab.py

delete_tab.py

@@ -26,7 +26,7 @@ class DeleteTab:
         """Get all transactions for a farmer with row indices"""
         try:
             if not farmer_name or not farmer_name.strip():
-                return None, None, "❌ Please select a farmer"
+                return None, "❌ Please select a farmer"
             
             farmer_name = farmer_name.strip()
             spreadsheet = self.get_sheet()
@@ -34,17 +34,17 @@ class DeleteTab:
             data = trans_sheet.get_all_values()
             
             if len(data) <= 1:
-                return None, None, f"❌ No transactions found for farmer '{farmer_name}'"
+                return None, f"❌ No transactions found for farmer '{farmer_name}'"
             
             # Create DataFrame with row indices
             df = pd.DataFrame(data[1:], columns=data[0])
-            df['Sheet_Row'] = range(2, len(data) + 1)  # Google Sheets row numbers (starting from 2)
+            df['Sheet_Row'] = range(2, len(data) + 1)  
             
             # Filter by farmer
             farmer_df = df[df['Farmer Name'] == farmer_name].copy()
             
             if farmer_df.empty:
-                return None, None, f"❌ No transactions found for farmer '{farmer_name}'"
+                return None, f"❌ No transactions found for farmer '{farmer_name}'"
             
             # Convert amount and date
             farmer_df['Amount'] = pd.to_numeric(farmer_df['Amount'], errors='coerce')
@@ -57,15 +57,12 @@ class DeleteTab:
             display_df['Date'] = display_df['Date'].apply(lambda x: self.format_date_display(str(x)[:10]))
             display_df['Amount (₹)'] = display_df['Amount (₹)'].apply(self.format_indian_currency)
             
-            # Create row number choices for dropdown
-            row_choices = [str(row) for row in farmer_df['Sheet_Row'].tolist()]
-            
-            return display_df, row_choices, f"✅ Found {len(farmer_df)} transaction(s) for {farmer_name}"
+            return display_df, f"✅ Found {len(farmer_df)} transaction(s) for {farmer_name}"
         
         except Exception as e:
             import traceback
             traceback.print_exc()
-            return None, None, f"❌ Error loading transactions: {str(e)}"
+            return None, f"❌ Error loading transactions: {str(e)}"
     
     def delete_single_transaction(self, row_number):
         """Delete a single transaction by row number"""
@@ -140,10 +137,66 @@ class DeleteTab:
             import traceback
             traceback.print_exc()
             return f"❌ Error deleting farmer data: {str(e)}"
+
+
+    def get_banks(self):
+        """Get list of registered bank accounts"""
+        try:
+            spreadsheet = self.get_sheet()
+            meta_sheet = spreadsheet.worksheet("Metadata")
+            data = meta_sheet.get_all_values()
+            
+            banks = []
+            for row in data:
+                if len(row) >= 2 and row[0] == 'bank' and row[1]:
+                    banks.extend([b.strip() for b in row[1].split(',') if b.strip()])
+            
+            return sorted(list(set(banks)))
+        except Exception as e:
+            print(f"Error getting banks: {str(e)}")
+            return []
+    
+    def refresh_banks(self):
+        """Refresh bank dropdown"""
+        return gr.Dropdown(choices=self.get_banks())
+    
+    def delete_bank(self, bank_name):
+        """Delete a bank from the registration list"""
+        try:
+            if not bank_name or not bank_name.strip():
+                return "❌ Please select a bank account"
+            
+            bank_name = bank_name.strip()
+            spreadsheet = self.get_sheet()
+            meta_sheet = spreadsheet.worksheet("Metadata")
+            meta_data = meta_sheet.get_all_values()
+            
+            for idx, row in enumerate(meta_data):
+                if len(row) >= 2 and row[0] == 'bank':
+                    current_banks = row[1]
+                    banks_list = [b.strip() for b in current_banks.split(',') if b.strip()]
+                    
+                    if bank_name in banks_list:
+                        banks_list.remove(bank_name)
+                        new_banks = ','.join(banks_list)
+                        meta_sheet.update_cell(idx + 1, 2, new_banks)
+                        
+                        return f"✅ Successfully deleted bank account '{bank_name}' from registration list"
+            
+            return f"❌ Bank account '{bank_name}' not found in registration list"
+        
+        except Exception as e:
+            import traceback
+            traceback.print_exc()
+            return f"❌ Error deleting bank: {str(e)}"
+    
+    def clear_delete_bank(self):
+        """Clear bank deletion interface"""
+        return ""
     
     def clear_delete_single(self):
         """Clear single deletion interface"""
-        return None, None, "", None
+        return None, "", ""
     
     def clear_delete_all(self):
         """Clear complete deletion interface"""
@@ -178,13 +231,14 @@ class DeleteTab:
                 )
                 
                 gr.Markdown("### Select Transaction to Delete")
+                gr.Markdown("*Enter the Row # from the table above*")
                 with gr.Row():
-                    delete_row_dropdown = gr.Dropdown(
-                        label="Select Row Number to Delete",
-                        choices=[],
-                        allow_custom_value=False
+                    delete_row_input = gr.Textbox(
+                        label="Enter Row Number to Delete",
+                        placeholder="e.g., 15",
+                        scale=3
                     )
-                    delete_single_btn = gr.Button("Delete Selected Row", variant="stop", size="lg")
+                    delete_single_btn = gr.Button("Delete Selected Row", variant="stop", size="lg", scale=1)
                 
                 delete_result = gr.Textbox(label="Deletion Result", interactive=False)
             
@@ -219,6 +273,35 @@ class DeleteTab:
                 )
                 
                 delete_all_result = gr.Textbox(label="Deletion Result", interactive=False)
+
+
+            with gr.Accordion("🏦 Delete Bank Account", open=False):
+                gr.Markdown("*Remove a bank account from the registration list*")
+                gr.Markdown("⚠️ **Note**: This only removes the bank from the dropdown list. Existing transactions with this bank will remain.")
+                
+                with gr.Row():
+                    delete_bank_dropdown = gr.Dropdown(
+                        label="Select Bank Account to Delete",
+                        choices=[],
+                        allow_custom_value=True,
+                        scale=9
+                    )
+                    refresh_delete_bank_btn = gr.Button("🔄", scale=1, size="sm")
+                
+                delete_bank_confirm = gr.Checkbox(
+                    label="I confirm I want to delete this bank account from the registration list",
+                    value=False
+                )
+                
+                delete_bank_btn = gr.Button(
+                    "Delete Bank Account",
+                    variant="stop",
+                    size="lg"
+                )
+                
+                delete_bank_result = gr.Textbox(label="Deletion Result", interactive=False)
+
+
             
             # Event handlers - Delete Single Transaction
             refresh_delete_single_btn.click(
@@ -229,23 +312,28 @@ class DeleteTab:
             load_delete_trans_btn.click(
                 fn=self.get_farmer_transactions_for_deletion,
                 inputs=[delete_single_farmer],
-                outputs=[delete_trans_df, delete_row_dropdown, delete_single_status]
-            ).then(
-                fn=lambda: None,  # Clear the dropdown selection after loading
-                outputs=[delete_row_dropdown]
+                outputs=[delete_trans_df, delete_single_status]
             )
             
+            # Replace the delete_single_btn.click handler:
             def delete_and_refresh(row_number, farmer_name):
                 """Delete transaction and refresh the list"""
-                result = self.delete_single_transaction(row_number)
-                # Reload transactions after deletion
-                new_df, new_choices, new_status = self.get_farmer_transactions_for_deletion(farmer_name)
-                return result, new_df, new_choices, new_status
+                if not row_number or not row_number.strip():
+                    return "❌ Please enter a row number", None, ""
+                
+                result = self.delete_single_transaction(row_number.strip())
+                
+                # Reload transactions after deletion if deletion was successful
+                if "Successfully deleted" in result:
+                    new_df, new_status = self.get_farmer_transactions_for_deletion(farmer_name)
+                    return result, new_df, new_status
+                else:
+                    return result, None, ""
             
             delete_single_btn.click(
                 fn=delete_and_refresh,
-                inputs=[delete_row_dropdown, delete_single_farmer],
-                outputs=[delete_result, delete_trans_df, delete_row_dropdown, delete_single_status]
+                inputs=[delete_row_input, delete_single_farmer],
+                outputs=[delete_result, delete_trans_df, delete_single_status]
             )
             
             # Event handlers - Delete All Farmer Data
@@ -253,7 +341,28 @@ class DeleteTab:
                 fn=self.refresh_farmers,
                 outputs=[delete_all_farmer]
             )
+
+
+            # Event handlers - Delete Bank Account
+            refresh_delete_bank_btn.click(
+                fn=self.refresh_banks,
+                outputs=[delete_bank_dropdown]
+            )
             
+            def delete_bank_with_confirmation(bank_name, confirmed):
+                if not confirmed:
+                    return "❌ Please check the confirmation box to proceed with deletion"
+                return self.delete_bank(bank_name)
+            
+            delete_bank_btn.click(
+                fn=delete_bank_with_confirmation,
+                inputs=[delete_bank_dropdown, delete_bank_confirm],
+                outputs=[delete_bank_result]
+            ).then(
+                fn=lambda: False,  # Uncheck the confirmation box
+                outputs=[delete_bank_confirm]
+            )
+                        
             def delete_all_with_confirmation(farmer_name, confirmed):
                 if not confirmed:
                     return "❌ Please check the confirmation box to proceed with deletion"