Merge pull request 'Invoice Add Edit and Delete Done' (#17) from swapnil-dev into main

Reviewed-on: #17

controllers/hold_types_controller.py

@@ -10,21 +10,21 @@ hold_bp = Blueprint("hold_types", __name__)
 @hold_bp.route('/add_hold_type', methods=['GET','POST'])
 @login_required
 def add_hold_type():
-
-    hold = HoldTypes()   
+    hold = HoldTypes()
 
     if request.method == 'POST':
-        hold.AddHoldType(request)   # ✅
-        return hold.resultMessage
+        hold.AddHoldType(request)
+        # ✅ Always redirect to same page (NO JSON)
+        return redirect(url_for("hold_types.add_hold_type"))
 
-    hold_types = hold.GetAllHoldTypes()   # ✅
+    # GET request → show data
+    hold_types = hold.GetAllHoldTypes()
 
     return render_template(
         "add_hold_type.html",
         Hold_Types_data=hold_types
     )
 
-
 # ---------------- CHECK HOLD TYPE (OPTIONAL LIKE BLOCK) ----------------
 @hold_bp.route('/check_hold_type', methods=['POST'])
 @login_required

controllers/village_controller.py

@@ -83,17 +83,29 @@ def check_village():
     return village.CheckVillage(request=request)
 
 
-# ------------------------- Delete Village -------------------------
 @village_bp.route('/delete_village/<int:village_id>')
 @login_required
 def delete_village(village_id):
-
     village = Village()
     village.DeleteVillage(request=request, village_id=village_id)
 
-    flash(village.resultMessage, "success" if village.isSuccess else "error")
-    return redirect(url_for('village.add_village'))
+    # ✅ Convert resultMessage to string if it's a Response or tuple
+    raw_msg = village.resultMessage
 
+    if isinstance(raw_msg, tuple):
+        # e.g., (<Response ...>, 200)
+        msg = "Village deleted successfully!"
+    elif hasattr(raw_msg, 'get_data'):
+        # Flask Response object
+        msg = raw_msg.get_data(as_text=True)  # get raw text
+    else:
+        # fallback
+        msg = str(raw_msg) if raw_msg else "Village deleted successfully!"
+
+    return jsonify({
+        "status": "success" if village.isSuccess else "error",
+        "message": msg
+    })
 
 # ------------------------- Edit Village -------------------------
 @village_bp.route('/edit_village/<int:village_id>', methods=['GET', 'POST'])

model/Invoice.py

@@ -66,7 +66,7 @@ def get_all_villages():
         return fetch_all(cursor)
     return execute_db_operation(operation)
 
-
+ 
 # ------------------- Invoice Functions -------------------
 def insert_invoice(data, village_id):
     def operation(cursor):

model/ItemCRUD.py

@@ -167,7 +167,7 @@ class ItemCRUD:
             # ======================================================
             # NORMAL SINGLE-FIELD (Village / Block / State)
             # ======================================================
-            if not re.match(RegEx.patternAlphabetOnly, childname):
+            if not re.match(RegEx.allPattern, childname):
                 self.isSuccess = False
                 self.resultMessage = HtmlHelper.json_response(
                     ResponseHandler.invalid_name(self.itemCRUDMapping.name), 400
@@ -276,7 +276,7 @@ class ItemCRUD:
             # ======================================================
             # NORMAL SINGLE-FIELD
             # ======================================================
-            if not re.match(RegEx.patternAlphabetOnly, childname):
+            if not re.match(RegEx.allPattern, childname):
                 self.isSuccess = False
                 self.resultMessage = ResponseHandler.update_failure(self.itemCRUDMapping.name)['message']
                 return
@@ -364,7 +364,7 @@ class ItemCRUD:
             f"User {current_user.id} checked '{childname}'"
         )
 
-        if not re.match(RegEx.patternAlphabetOnly, childname):
+        if not re.match(RegEx.allPattern, childname):
             return HtmlHelper.json_response(
                 ResponseHandler.invalid_name(self.itemCRUDMapping.name), 400
             )

model/Utilities.py

@@ -12,6 +12,7 @@ class ItemCRUDType(Enum):
 
 class RegEx:
     patternAlphabetOnly = "^[A-Za-z ]+$"
+    allPattern = "^(?!\s*$).+"
 
 
 class ResponseHandler:

static/js/hold_types.js

@@ -3,7 +3,7 @@ $(document).ready(function () {
         let holdType = $(this).val().replace(/^\s+/, "");
         $(this).val(holdType);
 
-         let reg = /^[A-Za-z]/;
+        let reg = /^.+$/; // all pattern allow
 
         if (!reg.test(holdType)) {
             $("#holdTypeMessage").text("Hold Type must start with a letter.").css("color", "red");

static/js/invoice.js

@@ -1,62 +1,271 @@
-// Subcontractor autocomplete functionality
-    $(document).ready(function () {
-      $("#subcontractor").keyup(function () {
+// $(document).ready(function () {
+//     // ===============================
+//     // FORM / TABLE TOGGLE
+//     // ===============================
+//     if ($('#addForm').length && $('#addTable').length) {
+//         $('#addForm').show();
+//         $('#addTable').hide();
+
+//         $('#addButton').click(function () {
+//             $('#addForm').show();
+//             $('#addTable').hide();
+//         });
+
+//         $('#displayButton').click(function () {
+//             $('#addForm').hide();
+//             $('#addTable').show();
+//         });
+//     }
+
+//     // ===============================
+//     // Subcontractor autocomplete
+//     // ===============================
+//     $("#subcontractor").keyup(function () {
+//         let query = $(this).val();
+//         if (query !== "") {
+//             $.ajax({
+//                 url: "/search_subcontractor",
+//                 method: "POST",
+//                 data: { query: query },
+//                 success: function (data) {
+//                     $("#subcontractor_list").fadeIn().html(data);
+//                 }
+//             });
+//         } else {
+//             $("#subcontractor_list").fadeOut();
+//         }
+//     });
+
+//     $(document).on("click", "li", function () {
+//         $("#subcontractor").val($(this).text());
+//         $("#subcontractor_id").val($(this).attr("data-id"));
+//         $("#subcontractor_list").fadeOut();
+//     });
+
+//     // Focus
+//     if (document.getElementById('subcontractor')) {
+//         document.getElementById('subcontractor').focus();
+//     }
+
+//     // ===============================
+//     // ADD INVOICE
+//     // ===============================
+//     if ($('#invoiceForm').length && window.location.href.includes('add_invoice')) {
+//         $("#invoiceForm").on("submit", function (e) {
+//             e.preventDefault();
+//             let formData = $(this).serialize();
+
+//             $.ajax({
+//                 url: '/add_invoice',
+//                 method: 'POST',
+//                 data: formData,
+//                 dataType: 'json',
+//                 success: function (response) {
+//                     if (response.status === "success") {
+//                         alert(response.message || "Invoice added successfully!");
+//                         $('#invoiceForm')[0].reset(); // clear form
+//                         $('#addForm').hide();
+//                         $('#addTable').show(); // switch to table
+//                         location.reload(); // optional refresh
+//                     } else {
+//                         alert(response.message || "Error adding invoice.");
+//                     }
+//                 },
+//                 error: function (xhr) {
+//                     alert(xhr.responseJSON?.message || "Submission failed. Please try again.");
+//                 }
+//             });
+//         });
+//     }
+// // Example AJAX update function
+// function updateInvoice(invoiceId, formElement) {
+//     const formData = $(formElement).serialize();
+
+//     $.ajax({
+//         url: '/update_invoice/' + invoiceId,
+//         method: 'POST',
+//         data: formData,
+//         dataType: 'json',
+//         success: function(response) {
+//             if(response.status === "success") {
+//                 alert(response.message || "Invoice updated successfully!");
+                
+//                 // ✅ Hide Add Form, Show Table
+//                 $('#addForm').hide();
+//                 $('#addTable').show();
+
+//                 // Optional: reload table or refresh page
+//                 location.reload();
+//             } else {
+//                 alert(response.message || "Update failed. Please try again.");
+//             }
+//         },
+//         error: function(xhr) {
+//             alert(xhr.responseJSON?.message || "Error updating invoice.");
+//         }
+//     });
+// }
+
+// // ===============================
+// // DELETE INVOICE
+// // ===============================
+// function deleteInvoice(invoiceId, element) {
+//     if (!confirm("Are you sure you want to delete this invoice?")) return;
+
+//     $.ajax({
+//         url: '/delete_invoice/' + invoiceId,
+//         method: 'GET',
+//         dataType: 'json',
+//         success: function (response) {
+//             alert(response.message || "Invoice deleted successfully!");
+//             if (element) $(element).closest("tr").remove();
+//         },
+//         error: function (xhr) {
+//             alert(xhr.responseJSON?.message || "Error deleting invoice. Please try again.");
+//         }
+//     });
+// }
+
+$(document).ready(function () {
+    // ===============================
+    // FORM / TABLE TOGGLE
+    // ===============================
+    if ($('#addForm').length && $('#addTable').length) {
+        // Default: show form, hide table
+        $('#addForm').show();
+        $('#addTable').hide();
+
+        // ✅ Check URL hash to show table instead
+        if (window.location.hash === "#addTable") {
+            $('#addForm').hide();
+            $('#addTable').show();
+        }
+
+        $('#addButton').click(function () {
+            $('#addForm').show();
+            $('#addTable').hide();
+        });
+
+        $('#displayButton').click(function () {
+            $('#addForm').hide();
+            $('#addTable').show();
+        });
+    }
+
+    // ===============================
+    // Subcontractor autocomplete
+    // ===============================
+    $("#subcontractor").keyup(function () {
         let query = $(this).val();
         if (query !== "") {
-          $.ajax({
-            url: "/search_subcontractor",
-            method: "POST",
-            data: { query: query },
-            success: function (data) {
-              $("#subcontractor_list").fadeIn().html(data);
-            }
-          });
+            $.ajax({
+                url: "/search_subcontractor",
+                method: "POST",
+                data: { query: query },
+                success: function (data) {
+                    $("#subcontractor_list").fadeIn().html(data);
+                }
+            });
         } else {
-          $("#subcontractor_list").fadeOut();
+            $("#subcontractor_list").fadeOut();
         }
-      });
+    });
 
-      $(document).on("click", "li", function () {
+    $(document).on("click", "li", function () {
         $("#subcontractor").val($(this).text());
         $("#subcontractor_id").val($(this).attr("data-id"));
         $("#subcontractor_list").fadeOut();
-      });
     });
 
-    // Success Alert: show alert and reload after 3 seconds
-    function showSuccessAlert() {
-      const alertBox = document.getElementById("invoiceSuccessAlert");
-      alertBox.classList.add("show");
-      setTimeout(() => {
-        alertBox.classList.remove("show");
-        // Reload page or redirect after alert hides
-        window.location.href = '/add_invoice';
-      }, 3000);
+    // Focus
+    if (document.getElementById('subcontractor')) {
+        document.getElementById('subcontractor').focus();
     }
 
-    // Submit form via AJAX
-    $("#invoiceForm").on("submit", function (e) {
-      e.preventDefault();
-      let formData = $(this).serialize();
-      $.ajax({
-        url: '/add_invoice',
-        method: 'POST',
-        data: formData,
-        success: function (response) {
-          if(response.status === "success") {
-            showSuccessAlert();
-          } else {
-            alert(response.message);
-          }
-        },
-        error: function (xhr, status, error) {
-          alert("Submission failed: " + error);
-        }
-      });
-    });
+    // ===============================
+    // ADD INVOICE
+    // ===============================
+    if ($('#invoiceForm').length && window.location.href.includes('add_invoice')) {
+        $("#invoiceForm").on("submit", function (e) {
+            e.preventDefault();
+            let formData = $(this).serialize();
 
+            $.ajax({
+                url: '/add_invoice',
+                method: 'POST',
+                data: formData,
+                dataType: 'json',
+                success: function (response) {
+                    if (response.status === "success") {
+                        alert(response.message || "Invoice added successfully!");
+                        $('#invoiceForm')[0].reset(); // clear form
+                        $('#addForm').hide();
+                        $('#addTable').show(); // switch to table
+                        location.reload(); // optional refresh
+                    } else {
+                        alert(response.message || "Error adding invoice.");
+                    }
+                },
+                error: function (xhr) {
+                    let msg = xhr.responseJSON?.message || "Submission failed. Please try again.";
+                    alert(msg);
+                }
+            });
+        });
+    }
 
+    // ===============================
+    // UPDATE INVOICE
+    // ===============================
+    function updateInvoice(invoiceId, formElement) {
+        const formData = $(formElement).serialize();
 
-window.onload = function () {
-        document.getElementById('subcontractor').focus();
-    };
+        $.ajax({
+            url: '/update_invoice/' + invoiceId,
+            method: 'POST',
+            data: formData,
+            dataType: 'json',
+            success: function(response) {
+                if(response.status === "success") {
+                    alert(response.message || "Invoice updated successfully!");
+
+                    // Redirect to Add Invoice page's table part
+                    window.location.href = "/add_invoice#addTable";
+                } else {
+                    alert(response.message || "Update failed. Please try again.");
+                }
+            },
+            error: function(xhr) {
+                let msg = xhr.responseJSON?.message || "Error updating invoice.";
+                alert(msg);
+            }
+        });
+    }
+    window.updateInvoice = updateInvoice; // make globally accessible
+
+    // ===============================
+    // DELETE INVOICE
+    // ===============================
+    function deleteInvoice(invoiceId, element) {
+        if (!confirm("Are you sure you want to delete this invoice?")) return;
+
+        $.ajax({
+            url: '/delete_invoice/' + invoiceId,
+            method: 'GET',
+            dataType: 'json',
+            success: function (response) {
+                if (response.status === "success") {
+                    alert(response.message || "Invoice deleted successfully!");
+                    if (element) $(element).closest("tr").remove();
+                } else {
+                    alert(response.message || "Error deleting invoice.");
+                }
+            },
+            error: function (xhr) {
+                let msg = xhr.responseJSON?.message || "Error deleting invoice. Please try again.";
+                alert(msg);
+            }
+        });
+    }
+    window.deleteInvoice = deleteInvoice; // make globally accessible
+});

static/js/village.js

@@ -1,264 +1,250 @@
-
-
 window.onload = function () {
-    document.getElementById('Village_Name').focus();
+    if (document.getElementById('Village_Name')) {
+        document.getElementById('Village_Name').focus();
+    }
 };
  
 $(document).ready(function () {
 
-    // 🔥 RESTORE VIEW MODE AFTER RELOAD
-    var viewMode = localStorage.getItem("viewMode");
+    // ✅ RUN ONLY IF THIS PAGE HAS FORM/TABLE
+    if ($('#addForm').length && $('#addTable').length) {
 
-    if (viewMode === "table") {
-        $('#addForm').hide();
-        $('#addTable').show();
-    } else {
+        // ✅ DEFAULT VIEW → SHOW FORM
         $('#addForm').show();
         $('#addTable').hide();
+
+
+        // 🔥 BUTTON TOGGLE
+        $('#addButton').click(function () {
+            $('#addForm').show();
+            $('#addTable').hide();
+        });
+
+        $('#displayButton').click(function () {
+            $('#addForm').hide();
+            $('#addTable').show();
+        });
     }
 
 
-    // 🔥 BUTTON TOGGLE LOGIC
-
-    $('#addButton').click(function () {
-        $('#addForm').show();
-        $('#addTable').hide();
-
-        localStorage.setItem("viewMode", "form");
-    });
-
-    $('#displayButton').click(function () {
-        $('#addForm').hide();
-        $('#addTable').show();
-
-        localStorage.setItem("viewMode", "table");
-    });
-
-
+    // ===============================
     // STATE → DISTRICT
-    $('#state_Id').change(function () {
+    // ===============================
+    if ($('#state_Id').length) {
 
-        var stateId = $(this).val();
+        $('#state_Id').change(function () {
 
-        if (stateId) {
+            var stateId = $(this).val();
 
-            $.ajax({
-                url: '/get_districts/' + stateId,
-                type: 'GET',
+            if (stateId) {
 
-                success: function (data) {
+                $.ajax({
+                    url: '/get_districts/' + stateId,
+                    type: 'GET',
 
-                    var districtDropdown = $('#district_Id');
+                    success: function (data) {
 
-                    districtDropdown.empty();
-                    districtDropdown.append('<option value="" disabled selected>Select District</option>');
+                        var districtDropdown = $('#district_Id');
 
-                    data.forEach(function (district) {
+                        districtDropdown.empty();
+                        districtDropdown.append('<option value="" disabled selected>Select District</option>');
 
-                        districtDropdown.append(
-                            '<option value="' + district.id + '">' + district.name + '</option>'
-                        );
+                        data.forEach(function (district) {
+                            districtDropdown.append(
+                                '<option value="' + district.id + '">' + district.name + '</option>'
+                            );
+                        });
 
-                    });
-
-                    districtDropdown.prop('disabled', false);
-
-                }
-
-            });
-
-        }
-
-    });
+                        districtDropdown.prop('disabled', false);
+                    }
+                });
+            }
+        });
+    }
 
 
+    // ===============================
     // DISTRICT → BLOCK
-    $('#district_Id').change(function () {
+    // ===============================
+    if ($('#district_Id').length) {
 
-        var districtId = $(this).val();
+        $('#district_Id').change(function () {
 
-        if (districtId) {
+            var districtId = $(this).val();
 
-            $.ajax({
-                url: '/get_blocks/' + districtId,
-                type: 'GET',
+            if (districtId) {
 
-                success: function (data) {
+                $.ajax({
+                    url: '/get_blocks/' + districtId,
+                    type: 'GET',
 
-                    var blockDropdown = $('#block_Id');
+                    success: function (data) {
 
-                    blockDropdown.empty();
-                    blockDropdown.append('<option value="" disabled selected>Select Block</option>');
+                        var blockDropdown = $('#block_Id');
 
-                    data.forEach(function (block) {
+                        blockDropdown.empty();
+                        blockDropdown.append('<option value="" disabled selected>Select Block</option>');
 
-                        blockDropdown.append(
-                            '<option value="' + block.id + '">' + block.name + '</option>'
-                        );
+                        data.forEach(function (block) {
+                            blockDropdown.append(
+                                '<option value="' + block.id + '">' + block.name + '</option>'
+                            );
+                        });
 
-                    });
-
-                    blockDropdown.prop('disabled', false);
-
-                }
-
-            });
-
-        }
-
-    });
+                        blockDropdown.prop('disabled', false);
+                    }
+                });
+            }
+        });
+    }
 
 
+    // ===============================
     // VILLAGE NAME VALIDATION
-    $('#Village_Name').on('input', function () {
+    // ===============================
+    if ($('#Village_Name').length) {
 
-        var villageName = $(this).val();
-        var validPattern = /^[A-Za-z ]*$/;
+        $('#Village_Name').on('input', function () {
 
-        if (!validPattern.test(villageName)) {
+            var villageName = $(this).val();
+            var validPattern = /^[A-Za-z ]*$/;
 
-            $('#villageMessage')
-                .text('Only letters and spaces are allowed!')
-                .css('color', 'red');
+            if (!validPattern.test(villageName)) {
 
-            $('#submitVillage').prop('disabled', true);
+                $('#villageMessage')
+                    .text('Only letters and spaces are allowed!')
+                    .css('color', 'red');
 
-        } else {
+                $('#submitVillage').prop('disabled', true);
 
-            $('#villageMessage').text('');
-            $('#submitVillage').prop('disabled', false);
+            } else {
 
-        }
-
-    });
+                $('#villageMessage').text('');
+                $('#submitVillage').prop('disabled', false);
+            }
+        });
+    }
 
 
+    // ===============================
     // CHECK DUPLICATE VILLAGE
-    $('#Village_Name, #block_Id').on('change keyup', function () {
+    // ===============================
+    if ($('#Village_Name').length && $('#block_Id').length) {
 
-        var blockId = $('#block_Id').val();
-        var villageName = $('#Village_Name').val().trim();
+        $('#Village_Name, #block_Id').on('change keyup', function () {
 
-        if (blockId && villageName) {
+            var blockId = $('#block_Id').val();
+            var villageName = $('#Village_Name').val().trim();
 
-            $.ajax({
+            if (blockId && villageName) {
 
-                url: '/check_village',
-                type: 'POST',
+                $.ajax({
+                    url: '/check_village',
+                    type: 'POST',
 
-                data: {
-                    block_Id: blockId,
-                    Village_Name: villageName
-                },
+                    data: {
+                        block_Id: blockId,
+                        Village_Name: villageName
+                    },
 
-                success: function (response) {
+                    success: function (response) {
 
-                    if (response.status === 'exists') {
+                        if (response.status === 'exists') {
+
+                            $('#villageMessage')
+                                .text(response.message)
+                                .css('color', 'red');
+
+                            $('#submitVillage').prop('disabled', true);
+
+                        } else {
+
+                            $('#villageMessage')
+                                .text(response.message)
+                                .css('color', 'green');
+
+                            $('#submitVillage').prop('disabled', false);
+                        }
+                    },
+
+                    error: function () {
 
                         $('#villageMessage')
-                            .text(response.message)
+                            .text('Error checking village name')
                             .css('color', 'red');
 
                         $('#submitVillage').prop('disabled', true);
+                    }
+                });
+            }
+        });
+    }
+
+
+    // ===============================
+    // ADD VILLAGE (SAFE SCOPE FIX)
+    // ===============================
+    if ($('#villageForm').length) {
+
+        $('#villageForm').submit(function (event) {
+
+            event.preventDefault();
+
+            $.ajax({
+                url: '/add_village',
+                type: 'POST',
+                data: $(this).serialize(),
+
+                success: function (response) {
+
+                    if (response && response.status === 'success') {
+
+                        alert(response.message || 'Village added successfully!');
+
+                        // ✅ clear form
+                        $('#villageForm')[0].reset();
+
+                        // ✅ switch to table
+                        $('#addForm').hide();
+                        $('#addTable').show();
+
+                        // optional refresh
+                        location.reload();
 
                     } else {
-
-                        $('#villageMessage')
-                            .text(response.message)
-                            .css('color', 'green');
-
-                        $('#submitVillage').prop('disabled', false);
-
+                        alert(response.message || 'Error adding village. Please try again.');
                     }
-
                 },
 
                 error: function () {
-
-                    $('#villageMessage')
-                        .text('Error checking village name')
-                        .css('color', 'red');
-
-                    $('#submitVillage').prop('disabled', true);
-
+                    alert('An error occurred. Please try again.');
                 }
-
             });
 
-        }
-
-    });
-
-
-    // ADD VILLAGE
-    $('#villageForm').submit(function (event) {
-
-        event.preventDefault();
-
-        $.ajax({
-
-            url: '/add_village',
-            type: 'POST',
-            data: $(this).serialize(),
-
-            success: function (response) {
-
-                if (response.status === 'success') {
-
-                    alert('Village added successfully!');
-                    location.reload();
-
-                } else {
-
-                    alert(response.message || 'Error adding village. Please try again.');
-
-                }
-
-            },
-
-            error: function () {
-
-                alert('An error occurred. Please try again.');
-
-            }
-
         });
-
-    });
+    }
 
 });
 
 
-// 🔥 DELETE FUNCTION (UPDATED)
-function deleteVillage(villageId) {
 
-    if (!confirm("Are you sure you want to delete this village?")) {
-        return;
-    }
 
-    // ✅ save that user is on table
-    localStorage.setItem("viewMode", "table");
+// ===============================
+// DELETE FUNCTION (SAFE)
+// ===============================
+function deleteVillage(villageId, element) {
+    if (!confirm("Are you sure you want to delete this village?")) return;
 
     $.ajax({
         url: '/delete_village/' + villageId,
         type: 'GET',
-
-        success: function () {
-
-            setTimeout(function () {
-
-                alert("Village deleted successfully!");
-
-                // reload but stay on table
-                location.reload();
-
-            }, 1000);
-
+        dataType: 'json',
+        success: function (response) {
+            alert(response.message); // ✅ now shows "Village deleted successfully!"
+            if (element) $(element).closest("tr").remove();
         },
-
         error: function () {
             alert("Error deleting village. Please try again.");
         }
-
     });
 }

templates/add_hold_type.html

@@ -1,59 +1,73 @@
 {% extends 'base.html' %}
 {% block content %}
+
 <head>
     <title>Manage Hold Types</title>
     <script src="https://code.jquery.com/jquery-3.6.0.min.js"></script>
     <!-- <script src="{{ url_for('static', filename='js/hold_types.js') }}"></script> -->
     <link rel="stylesheet" type="text/css" href="{{ url_for('static', filename='css/style.css') }}">
-       <script src="{{ url_for('static', filename='js/search_on_table.js') }}"></script>
+    <script src="{{ url_for('static', filename='js/search_on_table.js') }}"></script>
 </head>
+
 <body>
 
-<div class="button-container">
-    <button id="addButton" class="action-button">Add</button>
-    <button id="displayButton" class="action-button">Display</button>
-</div>
+    <div class="button-container">
+        <button id="addButton" class="action-button">Add</button>
+        <button id="displayButton" class="action-button">Display</button>
+    </div>
 
-<div id="addForm">
-    <h2>Add Hold Types</h2>
-    <form id="holdTypeForm" method="POST" action="{{ url_for('hold_types.add_hold_type') }}">
-        <label>Enter Hold Amount Type:</label>
-        <input type="text" id="hold_type" name="hold_type" placeholder="Enter Type" required>
-        <span id="holdTypeMessage"></span>
-        <button type="submit" value="Add Hold Type" id="successPopup">Add Hold Type</button>
-    </form>
-</div>
-  
-<div id="addTable" style="display: none;">
-  <div class="search-container">
-        <h2>Hold Type List</h2>
-        <input type="text" id="searchBar" placeholder="Searching..." onkeyup="searchTable()">
-            </div>
-    <table id="sortableTable" border="1">
-        <tr>
-            <th>ID</th>
-            <th class="sortable-header">
-                Hold Type
-                <span class="sort-buttons">
-                    <span class="sort-asc">⬆️</span>
-                    <span class="sort-desc">⬇️</span>
-                </span>
-            </th>
-            <th>Update</th>
-            <th>Delete</th>
-        </tr>
-        {% for htd in Hold_Types_data %}
-        <tr>
-            <td>{{ htd['hold_type_id'] }}</td>
-            <td>{{ htd['hold_type'] }}</td>
-            <td><a href="{{ url_for('hold_types.edit_hold_type', id=htd['hold_type_id']) }}">Edit</a></td>
-            <td><button class="delete-button" data-id="{{ htd['hold_type_id'] }}">Delete</button></td>
-        </tr>
-        {% endfor %}
-    </table>
+    <div id="addForm">
+        <h2>Add Hold Types</h2>
+        <form id="holdTypeForm" method="POST" action="{{ url_for('hold_types.add_hold_type') }}">
+            <label>Enter Hold Amount Type:</label>
+            <input type="text" id="hold_type" name="hold_type" placeholder="Enter Type" required>
+            <span id="holdTypeMessage"></span>
+            <button type="submit" value="Add Hold Type" id="successPopup">Add Hold Type</button>
+        </form>
+    </div>
 
-    <a href="/">Back to Dashboard</a>
-</div>
+    <div id="addTable" style="display: none;">
+        <div class="search-container">
+            <h2>Hold Type List</h2>
+            <input type="text" id="searchBar" placeholder="Searching..." onkeyup="searchTable()">
+        </div>
+        <table id="sortableTable" border="1">
+            <tr>
+                <th>ID</th>
+                <th class="sortable-header">
+                    Hold Type
+                    <span class="sort-buttons">
+                        <span class="sort-asc">⬆️</span>
+                        <span class="sort-desc">⬇️</span>
+                    </span>
+                </th>
+                <th>Update</th>
+                <th>Delete</th>
+            </tr>
+            {% for htd in Hold_Types_data %}
+            <tr>
+                <td>{{ htd['hold_type_id'] }}</td>
+                <td>{{ htd['hold_type'] }}</td>
+                <td style="text-align:center;">
+                    <a href="{{ url_for('hold_types.edit_hold_type', id=htd['hold_type_id']) }}">
+                        <img src="{{ url_for('static', filename='images/icons/pen_blue_icon.png') }}" alt="Edit"
+                            class="icon">
+                    </a>
+                </td>
+
+                <td style="text-align:center;">
+                    <a href="{{ url_for('hold_types.delete_hold_type', id=htd['hold_type_id']) }}"
+                        onclick="return confirm('Are you sure you want to delete this hold type?');">
+
+                        <img src="{{ url_for('static', filename='images/icons/bin_red_icon.png') }}" alt="Delete"
+                            class="icon">
+                    </a>
+                </td>
+            </tr>
+            {% endfor %}
+        </table>
+
+        <a href="/">Back to Dashboard</a>
+    </div>
 </body>
-{% endblock %}
-
+{% endblock %}

templates/add_invoice.html

@@ -1,176 +1,322 @@
-{% extends 'base.html' %}
-{% block content %}
+{% extends 'base.html' %} {% block content %}
 <head xmlns="http://www.w3.org/1999/html">
-    <meta charset="UTF-8"/>
-    <meta name="viewport" content="width=device-width, initial-scale=1.0"/>
-    <title>Add Invoice</title>
-    <link rel="stylesheet" type="text/css" href="{{ url_for('static', filename='css/invoice.css') }}">
-    <script src="{{ url_for('static', filename='js/invoice.js') }}"></script>
-    <script src="{{ url_for('static', filename='js/holdAmount.js') }}"></script>
-    <script src="{{ url_for('static', filename='js/search_on_table.js') }}"></script>
+  <meta charset="UTF-8" />
+  <meta name="viewport" content="width=device-width, initial-scale=1.0" />
+  <title>Add Invoice</title>
+  <link
+    rel="stylesheet"
+    type="text/css"
+    href="{{ url_for('static', filename='css/invoice.css') }}"
+  />
+  <script src="{{ url_for('static', filename='js/invoice.js') }}"></script>
+  <script src="{{ url_for('static', filename='js/holdAmount.js') }}"></script>
+  <script src="{{ url_for('static', filename='js/search_on_table.js') }}"></script>
 </head>
 <body>
-{% if success == 'true' %}
-<div class="alert alert-success alert-dismissible fade show mt-3" role="alert">
+  {% if success == 'true' %}
+  <div
+    class="alert alert-success alert-dismissible fade show mt-3"
+    role="alert"
+  >
     ✅ Invoice added successfully!
-    <button type="button" class="btn-close" data-bs-dismiss="alert" aria-label="Close"></button>
-</div>
-{% endif %}
-
-
-<!-- Flash Messages -->
-{% with messages = get_flashed_messages(with_categories=true) %}
-  {% if messages %}
-    <div class="flash-messages">
-      {% for category, message in messages %}
-        <div class="alert {{ category }}">{{ message }}</div>
-      {% endfor %}
-    </div>
+    <button
+      type="button"
+      class="btn-close"
+      data-bs-dismiss="alert"
+      aria-label="Close"
+    ></button>
+  </div>
   {% endif %}
-{% endwith %}
 
-<div class="button-container">
+  <!-- Flash Messages -->
+  {% with messages = get_flashed_messages(with_categories=true) %} {% if
+  messages %}
+  <div class="flash-messages">
+    {% for category, message in messages %}
+    <div class="alert {{ category }}">{{ message }}</div>
+    {% endfor %}
+  </div>
+  {% endif %} {% endwith %}
+
+  <div class="button-container">
     <button id="addButton" class="action-button">Add</button>
     <button id="displayButton" class="action-button">Display</button>
-</div>
+  </div>
 
-<div id="addForm" style="display: none;">
+  <div id="addForm" style="display: none">
     <h2>Add Invoice</h2>
 
-    <form id="invoiceForm" action="{{ url_for('invoice.add_invoice') }}" method="POST">
-        <div class="row1">
-            <div>
-                <label for="subcontractor">Subcontractor Name:</label>
-                <input type="text" id="subcontractor" name="subcontractor" required autocomplete="off"/>
-                <input type="hidden" id="subcontractor_id" name="subcontractor_id"/>
-                <div id="subcontractor_list"></div>
-            </div>
+    <form
+      id="invoiceForm"
+      action="{{ url_for('invoice.add_invoice') }}"
+      method="POST"
+    >
+      <div class="row1">
+        <div>
+          <label for="subcontractor">Subcontractor Name:</label>
+          <input
+            type="text"
+            id="subcontractor"
+            name="subcontractor"
+            required
+            autocomplete="off"
+          />
+          <input type="hidden" id="subcontractor_id" name="subcontractor_id" />
+          <div id="subcontractor_list"></div>
         </div>
+      </div>
 
-        <div class="row2">
-            <div>
-                <label for="village">Village Name:</label>
-                <select id="village" name="village" required>
-                    <option value="">-- Select Village --</option>
-                    {% for village in villages %}
-                        <option value="{{ village.Village_Name }}">{{ village.Village_Name }}</option>
-                    {% endfor %}
-                </select>
-            </div>
-            <div>
-                <label for="pmc_no">PMC No:</label>
-                <input type="text" id="pmc_no" name="pmc_no" required/>
-                <div id="pmc_no_list" class="autocomplete-list"></div>
-            </div>
+      <div class="row2">
+        <div>
+          <label for="village">Village Name:</label>
+          <select id="village" name="village" required>
+            <option value="">-- Select Village --</option>
+            {% for village in villages %}
+            <option value="{{ village.Village_Name }}">
+              {{ village.Village_Name }}
+            </option>
+            {% endfor %}
+          </select>
         </div>
-
-        <div class="row2">
-            <div>
-                <label for="work_type">Work Type:</label>
-                <input type="text" id="work_type" name="work_type" required/>
-            </div>
-            <div>
-                <label for="invoice_details">Invoice Details:</label>
-                <textarea id="invoice_details" name="invoice_details" required></textarea>
-            </div>
+        <div>
+          <label for="pmc_no">PMC No:</label>
+          <input type="text" id="pmc_no" name="pmc_no" required />
+          <div id="pmc_no_list" class="autocomplete-list"></div>
         </div>
+      </div>
 
-        <div class="row2">
-            <div>
-                <label for="invoice_no">Invoice No:</label>
-                <input type="text" id="invoice_no" name="invoice_no" required/>
-            </div>
-            <div>
-                <label for="invoice_date">Invoice Date:</label>
-                <input type="date" id="invoice_date" name="invoice_date" required/>
-            </div>
+      <div class="row2">
+        <div>
+          <label for="work_type">Work Type:</label>
+          <input type="text" id="work_type" name="work_type" required />
         </div>
-
-        <div class="row3">
-    <div>
-        <label for="basic_amount">Basic Amount:</label>
-        <input type="number" step="0.01" id="basic_amount" name="basic_amount" placeholder="₹ - 00.00" required/>
-    </div>
-    <div>
-        <label for="debit_amount">Debit Amount:</label>
-        <input type="number" step="0.01" id="debit_amount" name="debit_amount" placeholder="₹ - 00.00" required/>
-    </div>
-    <div>
-        <label for="after_debit_amount">After Debit Amount:</label>
-        <input type="number" step="0.01" id="after_debit_amount" name="after_debit_amount" placeholder="₹ - 00.00" readonly required/>
-    </div>
-</div>
-
-<div class="row3">
-    <div class="percentage-field">
-        <label for="gst_percentage">GST %:</label>
-        <input type="number" step="0.01" id="gst_percentage" name="gst_percentage" placeholder="%"/>
-        <label for="gst_amount">GST Amount:</label>
-        <input type="number" step="0.01" id="gst_amount" name="gst_amount" placeholder="₹ - 00.00" readonly required/>
-    </div>
-    <div>
-        <label for="amount">Amount:</label>
-        <input type="number" step="0.01" id="amount" name="amount" placeholder="₹ - 00.00" readonly required/>
-    </div>
-    <div class="percentage-field">
-        <label for="tds_percentage">TDS %:</label>
-        <input type="number" step="0.01" id="tds_percentage" name="tds_percentage" placeholder="%"/>
-        <label for="tds_amount">TDS Amount:</label>
-        <input type="number" step="0.01" id="tds_amount" name="tds_amount" placeholder="₹ - 00.00" readonly required/>
-    </div>
-</div>
-
-<div class="row3">
-    <div class="percentage-field">
-        <label for="sd_percentage">SD %:</label>
-        <input type="number" step="0.01" id="sd_percentage" name="sd_percentage" placeholder="%"/>
-        <label for="sd_amount">SD Amount:</label>
-        <input type="number" step="0.01" id="sd_amount" name="sd_amount" placeholder="₹ - 00.00" readonly required>
-    </div>
-    <div class="percentage-field">
-        <label for="commission_percentage">On Commission %:</label>
-        <input type="number" step="0.01" id="commission_percentage" name="commission_percentage" placeholder="%"/>
-        <label for="on_commission">On Commission:</label>
-        <input type="number" step="0.01" id="on_commission" name="on_commission" placeholder="₹ - 00.00" readonly required>
-    </div>
-    <div class="percentage-field">
-        <label for="hydro_percentage">Hydro Testing %:</label>
-        <input type="number" step="0.01" id="hydro_percentage" name="hydro_percentage" placeholder="%"/>
-        <label for="hydro_testing">Hydro Testing:</label>
-        <input type="number" step="0.01" id="hydro_testing" name="hydro_testing" placeholder="₹ - 00.00" readonly required>
-    </div>
-</div>
-        <div class="hold-row">
-            <button type="button" id="add_hold_amount" class="button">+ Add Hold Amount</button>
+        <div>
+          <label for="invoice_details">Invoice Details:</label>
+          <textarea
+            id="invoice_details"
+            name="invoice_details"
+            required
+          ></textarea>
         </div>
+      </div>
 
-        <!-- Dynamically added hold amount fields -->
-        <div id="hold_amount_container"></div>
-
-        <div class="row2">
-            <div>
-                <label for="gst_sd_amount">GST SD Amount:</label>
-                <input type="number" step="0.01" id="gst_sd_amount" name="gst_sd_amount" placeholder="₹ - 00.00" required/>
-            </div>
-            <div>
-                <label for="final_amount">Final Amount:</label>
-                <input type="number" step="0.01" id="final_amount" name="final_amount" placeholder="₹ - 00.00" required/>
-            </div>
+      <div class="row2">
+        <div>
+          <label for="invoice_no">Invoice No:</label>
+          <input type="text" id="invoice_no" name="invoice_no" required />
         </div>
+        <div>
+          <label for="invoice_date">Invoice Date:</label>
+          <input type="date" id="invoice_date" name="invoice_date" required />
+        </div>
+      </div>
 
-        <button type="submit" class="button">Submit</button>
+      <div class="row3">
+        <div>
+          <label for="basic_amount">Basic Amount:</label>
+          <input
+            type="number"
+            step="0.01"
+            id="basic_amount"
+            name="basic_amount"
+            placeholder="₹ - 00.00"
+            required
+          />
+        </div>
+        <div>
+          <label for="debit_amount">Debit Amount:</label>
+          <input
+            type="number"
+            step="0.01"
+            id="debit_amount"
+            name="debit_amount"
+            placeholder="₹ - 00.00"
+            required
+          />
+        </div>
+        <div>
+          <label for="after_debit_amount">After Debit Amount:</label>
+          <input
+            type="number"
+            step="0.01"
+            id="after_debit_amount"
+            name="after_debit_amount"
+            placeholder="₹ - 00.00"
+            readonly
+            required
+          />
+        </div>
+      </div>
+
+      <div class="row3">
+        <div class="percentage-field">
+          <label for="gst_percentage">GST %:</label>
+          <input
+            type="number"
+            step="0.01"
+            id="gst_percentage"
+            name="gst_percentage"
+            placeholder="%"
+          />
+          <label for="gst_amount">GST Amount:</label>
+          <input
+            type="number"
+            step="0.01"
+            id="gst_amount"
+            name="gst_amount"
+            placeholder="₹ - 00.00"
+            readonly
+            required
+          />
+        </div>
+        <div>
+          <label for="amount">Amount:</label>
+          <input
+            type="number"
+            step="0.01"
+            id="amount"
+            name="amount"
+            placeholder="₹ - 00.00"
+            readonly
+            required
+          />
+        </div>
+        <div class="percentage-field">
+          <label for="tds_percentage">TDS %:</label>
+          <input
+            type="number"
+            step="0.01"
+            id="tds_percentage"
+            name="tds_percentage"
+            placeholder="%"
+          />
+          <label for="tds_amount">TDS Amount:</label>
+          <input
+            type="number"
+            step="0.01"
+            id="tds_amount"
+            name="tds_amount"
+            placeholder="₹ - 00.00"
+            readonly
+            required
+          />
+        </div>
+      </div>
+
+      <div class="row3">
+        <div class="percentage-field">
+          <label for="sd_percentage">SD %:</label>
+          <input
+            type="number"
+            step="0.01"
+            id="sd_percentage"
+            name="sd_percentage"
+            placeholder="%"
+          />
+          <label for="sd_amount">SD Amount:</label>
+          <input
+            type="number"
+            step="0.01"
+            id="sd_amount"
+            name="sd_amount"
+            placeholder="₹ - 00.00"
+            readonly
+            required
+          />
+        </div>
+        <div class="percentage-field">
+          <label for="commission_percentage">On Commission %:</label>
+          <input
+            type="number"
+            step="0.01"
+            id="commission_percentage"
+            name="commission_percentage"
+            placeholder="%"
+          />
+          <label for="on_commission">On Commission:</label>
+          <input
+            type="number"
+            step="0.01"
+            id="on_commission"
+            name="on_commission"
+            placeholder="₹ - 00.00"
+            readonly
+            required
+          />
+        </div>
+        <div class="percentage-field">
+          <label for="hydro_percentage">Hydro Testing %:</label>
+          <input
+            type="number"
+            step="0.01"
+            id="hydro_percentage"
+            name="hydro_percentage"
+            placeholder="%"
+          />
+          <label for="hydro_testing">Hydro Testing:</label>
+          <input
+            type="number"
+            step="0.01"
+            id="hydro_testing"
+            name="hydro_testing"
+            placeholder="₹ - 00.00"
+            readonly
+            required
+          />
+        </div>
+      </div>
+      <div class="hold-row">
+        <button type="button" id="add_hold_amount" class="button">
+          + Add Hold Amount
+        </button>
+      </div>
+
+      <!-- Dynamically added hold amount fields -->
+      <div id="hold_amount_container"></div>
+
+      <div class="row2">
+        <div>
+          <label for="gst_sd_amount">GST SD Amount:</label>
+          <input
+            type="number"
+            step="0.01"
+            id="gst_sd_amount"
+            name="gst_sd_amount"
+            placeholder="₹ - 00.00"
+            required
+          />
+        </div>
+        <div>
+          <label for="final_amount">Final Amount:</label>
+          <input
+            type="number"
+            step="0.01"
+            id="final_amount"
+            name="final_amount"
+            placeholder="₹ - 00.00"
+            required
+          />
+        </div>
+      </div>
+
+      <button type="submit" class="button">Submit</button>
     </form>
-</div>
+  </div>
 
-<div id="addTable" style="display: none;">
+  <div id="addTable" style="display: none">
     <!-- Invoice Table Section -->
     <div class="search-container">
-        <h2>Invoice List</h2>
-        <input type="text" id="searchBar" placeholder="Searching..." onkeyup="searchTable()">
-    {% if invoices %}
-    <table class="invoice-table">
+      <h2>Invoice List</h2>
+      <input
+        type="text"
+        id="searchBar"
+        placeholder="Searching..."
+        onkeyup="searchTable()"
+      />
+      {% if invoices %}
+      <table class="invoice-table">
         <thead>
-        <tr>
+          <tr>
             <th>Invoice Id</th>
             <th>SubContractor Name</th>
             <th>PMC No</th>
@@ -192,11 +338,11 @@
             <th>Final Amount</th>
             <th>Update</th>
             <th>Delete</th>
-        </tr>
+          </tr>
         </thead>
         <tbody>
-        {% for invoice in invoices %}
-        <tr>
+          {% for invoice in invoices %}
+          <tr>
             <td>{{ invoice.Invoice_Id }}</td>
             <td>{{ invoice.Contractor_Name }}</td>
             <td>{{ invoice.PMC_No }}</td>
@@ -217,62 +363,77 @@
             <td>{{ invoice.GST_SD_Amount }}</td>
             <td>{{ invoice.Final_Amount }}</td>
             <td>
-           <!-- Edit -->
-              <a href="{{ url_for('invoice.edit_invoice', invoice_id=invoice.Invoice_Id) }}">
-                    <img src="{{ url_for('static', filename='images/icons/pen_blue_icon.png') }}" alt="Edit" class="icon">
-                </a>
+              <!-- Edit -->
+              <a
+                href="{{ url_for('invoice.edit_invoice', invoice_id=invoice.Invoice_Id) }}"
+              >
+                <img
+                  src="{{ url_for('static', filename='images/icons/pen_blue_icon.png') }}"
+                  alt="Edit"
+                  class="icon edit-btn"
+                  data-id="{{ invoice.Invoice_Id }}"
+                />
+              </a>
             </td>
             <td>
-                <!-- Delete -->
-                <a href="{{ url_for('invoice.delete_invoice_route', invoice_id=invoice.Invoice_Id) }}" onclick="return confirm('Are you sure?')">
-                    <img src="{{ url_for('static', filename='images/icons/bin_red_icon.png') }}" alt="Delete" class="icon">
-                </a>
+              <a
+                href="javascript:void(0);"
+                onclick="deleteInvoice({{ invoice.Invoice_Id }}, this)"
+              >
+                <img
+                  src="{{ url_for('static', filename='images/icons/bin_red_icon.png') }}"
+                  alt="Delete"
+                  class="icon"
+                />
+              </a>
             </td>
-        </tr>
-        {% endfor %}
+          </tr>
+          {% endfor %}
         </tbody>
-    </table>
-    {% else %}
-    <p>No invoices found.</p>
-    {% endif %}
-</div>
+      </table>
+      {% else %}
+      <p>No invoices found.</p>
+      {% endif %}
+    </div>
 
     <script>
-document.addEventListener('DOMContentLoaded', function() {
-    // Get all the input fields
-    const basicAmount = document.getElementById('basic_amount');
-    const debitAmount = document.getElementById('debit_amount');
-    const afterDebitAmount = document.getElementById('after_debit_amount');
-    const amount = document.getElementById('amount');
+      document.addEventListener("DOMContentLoaded", function () {
+        // Get all the input fields
+        const basicAmount = document.getElementById("basic_amount");
+        const debitAmount = document.getElementById("debit_amount");
+        const afterDebitAmount = document.getElementById("after_debit_amount");
+        const amount = document.getElementById("amount");
 
-    // Percentage fields
-    const gstPercentage = document.getElementById('gst_percentage');
-    const gstAmount = document.getElementById('gst_amount');
-    const tdsPercentage = document.getElementById('tds_percentage');
-    const tdsAmount = document.getElementById('tds_amount');
-    const sdPercentage = document.getElementById('sd_percentage');
-    const sdAmountInput = document.getElementById('sd_amount');
-    const commissionPercentage = document.getElementById('commission_percentage');
-    const onCommission = document.getElementById('on_commission');
-    const hydroPercentage = document.getElementById('hydro_percentage');
-    const hydroTesting = document.getElementById('hydro_testing');
-    const gstSdAmount = document.getElementById('gst_sd_amount');
-    const finalAmount = document.getElementById('final_amount');
+        // Percentage fields
+        const gstPercentage = document.getElementById("gst_percentage");
+        const gstAmount = document.getElementById("gst_amount");
+        const tdsPercentage = document.getElementById("tds_percentage");
+        const tdsAmount = document.getElementById("tds_amount");
+        const sdPercentage = document.getElementById("sd_percentage");
+        const sdAmountInput = document.getElementById("sd_amount");
+        const commissionPercentage = document.getElementById(
+          "commission_percentage",
+        );
+        const onCommission = document.getElementById("on_commission");
+        const hydroPercentage = document.getElementById("hydro_percentage");
+        const hydroTesting = document.getElementById("hydro_testing");
+        const gstSdAmount = document.getElementById("gst_sd_amount");
+        const finalAmount = document.getElementById("final_amount");
 
-    // Calculate after debit amount when basic or debit amount changes
-    function calculateAfterDebitAmount() {
-        const basic = parseFloat(basicAmount.value) || 0;
-        const debit = parseFloat(debitAmount.value) || 0;
-        const afterDebit = basic - debit;
-        afterDebitAmount.value = afterDebit.toFixed(2);
-        calculateGST();
-    }
+        // Calculate after debit amount when basic or debit amount changes
+        function calculateAfterDebitAmount() {
+          const basic = parseFloat(basicAmount.value) || 0;
+          const debit = parseFloat(debitAmount.value) || 0;
+          const afterDebit = basic - debit;
+          afterDebitAmount.value = afterDebit.toFixed(2);
+          calculateGST();
+        }
 
-    // Calculate GST and Amount
-    function calculateGST() {
-        const baseAmount = parseFloat(afterDebitAmount.value) || 0;
+        // Calculate GST and Amount
+        function calculateGST() {
+          const baseAmount = parseFloat(afterDebitAmount.value) || 0;
 
-        if (gstPercentage.value) {
+          if (gstPercentage.value) {
             const gstPerc = parseFloat(gstPercentage.value) || 0;
             const gstAmt = (baseAmount * gstPerc) / 100;
             gstAmount.value = gstAmt.toFixed(2);
@@ -280,91 +441,96 @@ document.addEventListener('DOMContentLoaded', function() {
 
             // Calculate Amount (After Debit + GST)
             amount.value = (baseAmount + gstAmt).toFixed(2);
-        } else {
+          } else {
             amount.value = baseAmount.toFixed(2);
+          }
+
+          calculateOtherDeductions();
         }
 
-        calculateOtherDeductions();
-    }
+        // Calculate other deductions (TDS, SD, Commission, Hydro)
+        function calculateOtherDeductions() {
+          const baseAmount = parseFloat(afterDebitAmount.value) || 0;
 
-    // Calculate other deductions (TDS, SD, Commission, Hydro)
-    function calculateOtherDeductions() {
-        const baseAmount = parseFloat(afterDebitAmount.value) || 0;
-
-        // Calculate TDS
-        if (tdsPercentage.value) {
+          // Calculate TDS
+          if (tdsPercentage.value) {
             const tdsPerc = parseFloat(tdsPercentage.value) || 0;
             const tdsAmt = (baseAmount * tdsPerc) / 100;
             tdsAmount.value = tdsAmt.toFixed(2);
-        }
+          }
 
-        // Calculate SD
-        if (sdPercentage.value) {
+          // Calculate SD
+          if (sdPercentage.value) {
             const sdPerc = parseFloat(sdPercentage.value) || 0;
             const sdAmt = (baseAmount * sdPerc) / 100;
             sdAmountInput.value = sdAmt.toFixed(2);
-        }
+          }
 
-        // Calculate Commission
-        if (commissionPercentage.value) {
+          // Calculate Commission
+          if (commissionPercentage.value) {
             const commPerc = parseFloat(commissionPercentage.value) || 0;
             const commAmt = (baseAmount * commPerc) / 100;
             onCommission.value = commAmt.toFixed(2);
-        }
+          }
 
-        // Calculate Hydro Testing
-        if (hydroPercentage.value) {
+          // Calculate Hydro Testing
+          if (hydroPercentage.value) {
             const hydroPerc = parseFloat(hydroPercentage.value) || 0;
             const hydroAmt = (baseAmount * hydroPerc) / 100;
             hydroTesting.value = hydroAmt.toFixed(2);
+          }
+
+          calculateFinalAmount();
         }
 
-        calculateFinalAmount();
-    }
+        // Calculate final amount
+        function calculateFinalAmount() {
+          const amt = parseFloat(amount.value) || 0;
+          const tds = parseFloat(tdsAmount.value) || 0;
+          const sd = parseFloat(sdAmountInput.value) || 0;
+          const commission = parseFloat(onCommission.value) || 0;
+          const hydro = parseFloat(hydroTesting.value) || 0;
+          const gstSd = parseFloat(gstSdAmount.value) || 0;
 
-    // Calculate final amount
-    function calculateFinalAmount() {
-        const amt = parseFloat(amount.value) || 0;
-        const tds = parseFloat(tdsAmount.value) || 0;
-        const sd = parseFloat(sdAmountInput.value) || 0;
-        const commission = parseFloat(onCommission.value) || 0;
-        const hydro = parseFloat(hydroTesting.value) || 0;
-        const gstSd = parseFloat(gstSdAmount.value) || 0;
+          // Get hold amounts
+          let totalHold = 0;
+          document
+            .querySelectorAll('input[name="hold_amount[]"]')
+            .forEach((input) => {
+              totalHold += parseFloat(input.value) || 0;
+            });
 
-        // Get hold amounts
-        let totalHold = 0;
-        document.querySelectorAll('input[name="hold_amount[]"]').forEach(input => {
-            totalHold += parseFloat(input.value) || 0;
-        });
+          // Final Amount = Amount - TDS - SD - Commission - Hydro - GST SD - Hold Amounts
+          const final = amt - tds - sd - commission - hydro - gstSd - totalHold;
+          finalAmount.value = final.toFixed(2);
+        }
 
-        // Final Amount = Amount - TDS - SD - Commission - Hydro - GST SD - Hold Amounts
-        const final = amt - tds - sd - commission - hydro - gstSd - totalHold;
-        finalAmount.value = final.toFixed(2);
-    }
+        // Add event listeners
+        basicAmount.addEventListener("input", calculateAfterDebitAmount);
+        debitAmount.addEventListener("input", calculateAfterDebitAmount);
 
-    // Add event listeners
-    basicAmount.addEventListener('input', calculateAfterDebitAmount);
-    debitAmount.addEventListener('input', calculateAfterDebitAmount);
+        // Percentage fields
+        gstPercentage.addEventListener("input", calculateGST);
+        tdsPercentage.addEventListener("input", calculateOtherDeductions);
+        sdPercentage.addEventListener("input", calculateOtherDeductions);
+        commissionPercentage.addEventListener(
+          "input",
+          calculateOtherDeductions,
+        );
+        hydroPercentage.addEventListener("input", calculateOtherDeductions);
 
-    // Percentage fields
-    gstPercentage.addEventListener('input', calculateGST);
-    tdsPercentage.addEventListener('input', calculateOtherDeductions);
-    sdPercentage.addEventListener('input', calculateOtherDeductions);
-    commissionPercentage.addEventListener('input', calculateOtherDeductions);
-    hydroPercentage.addEventListener('input', calculateOtherDeductions);
+        // Listen for changes in hold amounts
+        document.addEventListener("holdAmountChanged", calculateFinalAmount);
+      });
 
-    // Listen for changes in hold amounts
-    document.addEventListener('holdAmountChanged', calculateFinalAmount);
-});
-
-// Optional JS for auto-hiding flash
-setTimeout(() => {
-    document.querySelectorAll('.alert').forEach(el => el.style.display = 'none');
-}, 5000);
-</script>
-
-
-</div>
+      // Optional JS for auto-hiding flash
+      setTimeout(() => {
+        document
+          .querySelectorAll(".alert")
+          .forEach((el) => (el.style.display = "none"));
+      }, 5000);
+    </script>
+  </div>
 </body>
 
 {% endblock %}

templates/add_village.html

@@ -1,99 +1,129 @@
-{% extends 'base.html' %}
-{% block content %}
+{% extends 'base.html' %} {% block content %}
 <head>
-    <meta charset="UTF-8">
-    <meta name="viewport" content="width=device-width, initial-scale=1.0">
-    <title>Village Management</title>
-    <link rel="stylesheet" type="text/css" href="{{ url_for('static', filename='css/style.css') }}">
-    <script src="{{ url_for('static', filename='js/village.js') }}"></script>
-    <script src="{{ url_for('static', filename='js/search_on_table.js') }}"></script>
+  <meta charset="UTF-8" />
+  <meta name="viewport" content="width=device-width, initial-scale=1.0" />
+  <title>Village Management</title>
+  <link
+    rel="stylesheet"
+    type="text/css"
+    href="{{ url_for('static', filename='css/style.css') }}"
+  />
+  <script src="{{ url_for('static', filename='js/village.js') }}"></script>
+  <script src="{{ url_for('static', filename='js/search_on_table.js') }}"></script>
 </head>
 <body>
-
-<!-- Button Container to Center Buttons -->
-<div class="button-container">
+  <!-- Button Container to Center Buttons -->
+  <div class="button-container">
     <button id="addButton" class="action-button">Add</button>
     <button id="displayButton" class="action-button">Display</button>
-</div>
+  </div>
 
-<div id="addForm" style="display: none;">
+  <div id="addForm" style="display: none">
     <div class="container">
-        <div class="form-block">
-            <h2>Add a New Village</h2>
-            <form id="villageForm" method="POST">
-                <label for="state_Id">State:</label>
-                <select id="state_Id" name="state_Id" required>
-                    <option value="" disabled selected>Select State</option>
-                    {% for state in states %}
-                    <option value="{{ state[0] }}">{{ state[1] }}</option>
-                    {% endfor %}
-                </select>
- 
-                <label for="district_Id">District:</label>
-                <select id="district_Id" name="district_Id" required disabled>
-                    <option value="" disabled selected>Select District</option>
-                </select>
+      <div class="form-block">
+        <h2>Add a New Village</h2>
+        <form id="villageForm" method="POST">
+          <label for="state_Id">State:</label>
+          <select id="state_Id" name="state_Id" required>
+            <option value="" disabled selected>Select State</option>
+            {% for state in states %}
+            <option value="{{ state[0] }}">{{ state[1] }}</option>
+            {% endfor %}
+          </select>
 
-                <label for="block_Id">Block:</label>
-                <select id="block_Id" name="block_Id" required disabled>
-                    <option value="" disabled selected>Select Block</option>
-                </select>
+          <label for="district_Id">District:</label>
+          <select id="district_Id" name="district_Id" required disabled>
+            <option value="" disabled selected>Select District</option>
+          </select>
 
-                <label for="Village_Name">Village Name:</label>
-                <input type="text" id="Village_Name" name="Village_Name" placeholder="Enter Village Name" required>
-                <span id="villageMessage"></span>
+          <label for="block_Id">Block:</label>
+          <select id="block_Id" name="block_Id" required disabled>
+            <option value="" disabled selected>Select Block</option>
+          </select>
 
-                <button type="submit" id="submitVillage" disabled>Add Village</button>
-            </form>
-        </div>
+          <label for="Village_Name">Village Name:</label>
+          <input
+            type="text"
+            id="Village_Name"
+            name="Village_Name"
+            placeholder="Enter Village Name"
+            required
+          />
+          <span id="villageMessage"></span>
+
+          <button type="submit" id="submitVillage" disabled>Add Village</button>
+        </form>
+      </div>
     </div>
-</div>
+  </div>
 
-<div id="addTable" style="display: none;">
+  <div id="addTable" style="display: none">
     <div class="search-container">
-        <h2>Display Villages</h2>
-        <input type="text" id="searchBar" placeholder="Searching..." onkeyup="searchTable()">
+      <h2>Display Villages</h2>
+      <input
+        type="text"
+        id="searchBar"
+        placeholder="Searching..."
+        onkeyup="searchTable()"
+      />
     </div>
 
     <table id="sortableTable" border="1">
-        <tr>
-            <th>Village Sr No</th>
-            <th class="sortable-header">
-                Village Name
-                <span class="sort-buttons">
-                        <span class="sort-asc">⬆️</span>
-                        <span class="sort-desc">⬇️</span>
-                    </span></th>
-            <th class="sortable-header">Block Name
-                <span class="sort-buttons">
-                        <span class="sort-asc">⬆️</span>
-                        <span class="sort-desc">⬇️</span>
-                    </span></th>
-            <th>Update</th>
-            <th>Delete</th>
-        </tr>
-        {% for village in villages %}
-        <tr>
-            <td>{{ village[0] }}</td>
-            <td>{{ village[1] }}</td>
-            <td>{{ village[2] }}</td>
-            <td>
-                <a href="{{ url_for('village.edit_village', village_id=village[0]) }}">
-                    <img src="{{ url_for('static', filename='images/icons/pen_blue_icon.png') }}" alt="Edit"
-                         class="icon">
-                </a>
-            </td>
-            <td>
-                <a href="#"
-   onclick="deleteVillage({{ village[0] }}); return false;">
+      <tr>
+        <th>Village Sr No</th>
+        <th class="sortable-header">
+          Village Name
+          <span class="sort-buttons">
+            <span class="sort-asc">⬆️</span>
+            <span class="sort-desc">⬇️</span>
+          </span>
+        </th>
+        <th class="sortable-header">
+          Block Name
+          <span class="sort-buttons">
+            <span class="sort-asc">⬆️</span>
+            <span class="sort-desc">⬇️</span>
+          </span>
+        </th>
+        <th>Update</th>
+        <th>Delete</th>
+      </tr>
+      {% for village in villages %}
+      <tr>
+        <td>{{ village[0] }}</td>
+        <td>{{ village[1] }}</td>
+        <td>{{ village[2] }}</td>
+        <td>
+          <a
+            href="{{ url_for('village.edit_village', village_id=village[0]) }}"
+          >
+            <img
+              src="{{ url_for('static', filename='images/icons/pen_blue_icon.png') }}"
+              alt="Edit"
+              class="icon"
+            />
+          </a>
+        </td>
+        <td>
+       <a href="javascript:void(0);" 
+   onclick="deleteVillage({{ village[0] }}, this)">
     <img src="{{ url_for('static', filename='images/icons/bin_red_icon.png') }}" 
-         alt="Delete" class="icon">
+         class="icon">
 </a>
-            </td>
-        </tr>
-        {% endfor %}
+        </td>
+      </tr>
+      {% endfor %}
     </table>
-</div>
-
+  </div>
+  <!-- Flash Alerts -->
+{% with messages = get_flashed_messages(with_categories=true) %}
+    {% if messages %}
+        <script>
+            {% for category, message in messages %}
+                alert("{{ message }}");
+            {% endfor %}
+        </script>
+    {% endif %}
+{% endwith %}
 </body>
 {% endblock %}

templates/edit_hold_type.html

@@ -1,7 +1,7 @@
-
 {% extends 'base.html' %}
 
 {% block content %}
+
 <head>
     <title>Edit Hold Type</title>
     <link rel="stylesheet" type="text/css" href="{{ url_for('static', filename='css/style.css') }}">
@@ -10,10 +10,10 @@
 
 <body>
     <h2>Edit Hold Type</h2>
-    <form id="updateHoldTypeForm">
-        <input type="hidden" id="hold_type_id" value="{{ hold_type[0] }}">
+    <form id="updateHoldTypeForm" method="POST">
+        <input type="hidden" id="hold_type_id" value="{{ hold_type.hold_type_id }}">
         <label for="edit_hold_type">Hold Type:</label>
-        <input type="text" id="edit_hold_type" name="hold_type" value="{{ hold_type[1] }}" required>
+        <input type="text" id="edit_hold_type" name="hold_type" value="{{ hold_type.hold_type }}" required>
         <button type="submit">Update</button>
         <a href="{{ url_for('hold_types.add_hold_type') }}"></a>
     </form>
@@ -37,7 +37,7 @@
             formData.append('hold_type', newHoldType);
 
             $.ajax({
-                url: '/update_hold_type/' + holdTypeId,
+                url: '/edit_hold_type/' + holdTypeId,
                 method: 'POST',
                 data: formData,
                 processData: false,
@@ -54,9 +54,4 @@
         });
     });
 </script>
-{% endblock %}
-
-
-
-
-
+{% endblock %}

templates/edit_invoice.html

@@ -2,8 +2,8 @@
 {% block content %}
 
 <head>
-    <meta charset="UTF-8"/>
-    <meta name="viewport" content="width=device-width, initial-scale=1.0"/>
+    <meta charset="UTF-8" />
+    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
     <title>Edit Invoice</title>
     <link rel="stylesheet" type="text/css" href="{{ url_for('static', filename='css/invoice.css') }}">
     <link rel="stylesheet" type="text/css" href="{{ url_for('static', filename='css/style1.css') }}">
@@ -12,158 +12,174 @@
 
 <body>
 
-<div id="editForm">
-    <h2>Edit Invoice</h2>
+    <div id="editForm">
+        <h2>Edit Invoice</h2>
 
-    <!-- Success Alert Box -->
-    <div id="invoiceSuccessAlert" class="success-alert" style="display:none;">
-        Invoice successfully updated!
+        <!-- Success Alert Box -->
+        <div id="invoiceSuccessAlert" class="success-alert" style="display:none;">
+            Invoice successfully updated!
+        </div>
+
+        <form id="invoiceForm" action="{{ url_for('invoice.edit_invoice', invoice_id=invoice.Invoice_Id) }}"
+            method="POST">
+
+            <!-- Subcontractor Field -->
+            <div class="row1">
+                <div>
+                    <label for="subcontractor">Subcontractor Name:</label>
+                    <input class="form-control" list="subcontractor_list" id="subcontractor" name="subcontractor"
+                        value="{{ invoice.Contractor_Name }}" placeholder="Type to search..." required>
+                    <input type="hidden" id="subcontractor_id" name="subcontractor_id"
+                        value="{{ invoice.Subcontractor_Id }}">
+                    <datalist id="subcontractor_list"></datalist>
+                </div>
+            </div>
+
+            <!-- Village and PMC No Fields -->
+            <div class="row2">
+                <div>
+                    <label for="village">Village Name:</label>
+                    <input type="text" id="village" name="village" value="{{ invoice.Village_Name }}" required />
+                    <datalist id="village_list"></datalist>
+                </div>
+                <div>
+                    <label for="pmc_no">PMC No:</label>
+                    <input type="text" id="pmc_no" name="pmc_no" value="{{ invoice.PMC_No }}" required />
+                </div>
+            </div>
+
+            <!-- Work Type and Invoice Details -->
+            <div class="row2">
+                <div>
+                    <label for="work_type">Work Type:</label>
+                    <input type="text" id="work_type" name="work_type" value="{{ invoice.Work_Type }}" required />
+                </div>
+                <div>
+                    <label for="invoice_details">Invoice Details:</label>
+                    <textarea id="invoice_details" name="invoice_details"
+                        required>{{ invoice.Invoice_Details }}</textarea>
+                </div>
+            </div>
+
+            <!-- Invoice No and Invoice Date -->
+            <div class="row2">
+                <div>
+                    <label for="invoice_no">Invoice No:</label>
+                    <input type="text" id="invoice_no" name="invoice_no" value="{{ invoice.invoice_no }}" required />
+                </div>
+                <div>
+                    <label for="invoice_date">Invoice Date:</label>
+                    <input type="date" id="invoice_date" name="invoice_date" value="{{ invoice.Invoice_Date }}"
+                        required />
+                </div>
+            </div>
+
+            <!-- Amount Fields -->
+            <div class="row3">
+                <div>
+                    <label for="basic_amount">Basic Amount:</label>
+                    <input type="number" step="0.01" id="basic_amount" name="basic_amount"
+                        value="{{ invoice.Basic_Amount }}" required />
+                </div>
+                <div>
+                    <label for="debit_amount">Debit Amount:</label>
+                    <input type="number" step="0.01" id="debit_amount" name="debit_amount"
+                        value="{{ invoice.Debit_Amount }}" required />
+                </div>
+                <div>
+                    <label for="after_debit_amount">After Debit Amount:</label>
+                    <input type="number" step="0.01" id="after_debit_amount" name="after_debit_amount"
+                        value="{{ invoice.After_Debit_Amount }}" required />
+                </div>
+            </div>
+
+            <!-- GST, TDS, and Other Amounts -->
+            <div class="row3">
+                <div>
+                    <label for="amount">Amount:</label>
+                    <input type="number" step="0.01" id="amount" name="amount" value="{{ invoice.Amount }}" required />
+                </div>
+                <div>
+                    <label for="gst_amount">GST Amount:</label>
+                    <input type="number" step="0.01" id="gst_amount" name="gst_amount" value="{{ invoice.GST_Amount }}"
+                        required />
+                </div>
+                <div>
+                    <label for="tds_amount">TDS Amount:</label>
+                    <input type="number" step="0.01" id="tds_amount" name="tds_amount" value="{{ invoice.TDS_Amount }}"
+                        required />
+                </div>
+            </div>
+
+            <!-- SD, On Commission, Hydro Testing -->
+            <div class="row3">
+                <div>
+                    <label for="sd_amount">SD Amount:</label>
+                    <input type="number" step="0.01" id="sd_amount" name="sd_amount" value="{{ invoice.SD_Amount }}"
+                        required />
+                </div>
+                <div>
+                    <label for="on_commission">On Commission:</label>
+                    <input type="number" step="0.01" id="on_commission" name="on_commission"
+                        value="{{ invoice.On_Commission }}" required />
+                </div>
+                <div>
+                    <label for="hydro_testing">Hydro Testing:</label>
+                    <input type="number" step="0.01" id="hydro_testing" name="hydro_testing"
+                        value="{{ invoice.Hydro_Testing }}" required />
+                </div>
+            </div>
+
+            <!-- Hold Amount Section -->
+            <div id="hold_amount_container">
+                {% set seen_types = [] %}
+                {% for hold in invoice.hold_amounts %}
+                {% if hold.hold_type not in seen_types %}
+                {% set _ = seen_types.append(hold.hold_type) %}
+                <div class="hold-amount-row">
+                    <label for="hold_type_{{ loop.index }}">Hold Type:</label>
+                    <input type="text" id="hold_type_{{ loop.index }}" name="hold_type[]" value="{{ hold.hold_type }}"
+                        required />
+                    <label for="hold_amount_{{ loop.index }}">Hold Amount:</label>
+                    <input type="number" step="0.01" id="hold_amount_{{ loop.index }}" name="hold_amount[]"
+                        value="{{ hold.hold_amount }}" required />
+                    <button type="button" class="remove-hold-amount">Remove</button>
+                </div>
+                {% endif %}
+                {% endfor %}
+            </div>
+
+
+            <div class="hold-row">
+                <button type="button" id="add_hold_amount" class="button">+ Add Hold Amount</button>
+            </div>
+
+            <!-- Final Amounts -->
+            <div class="row2">
+                <div>
+                    <label for="gst_sd_amount">GST SD Amount:</label>
+                    <input type="number" step="0.01" id="gst_sd_amount" name="gst_sd_amount"
+                        value="{{ invoice.GST_SD_Amount }}" required />
+                </div>
+                <div>
+                    <label for="final_amount">Final Amount:</label>
+                    <input type="number" step="0.01" id="final_amount" name="final_amount"
+                        value="{{ invoice.Final_Amount }}" required />
+                </div>
+            </div>
+
+            <!-- Submit Button -->
+            <button type="submit" class="button">Update</button>
+        </form>
     </div>
 
-    <form id="invoiceForm" action="{{ url_for('invoice.edit_invoice', invoice_id=invoice.Invoice_Id) }}" method="POST">
-
-        <!-- Subcontractor Field -->
-        <div class="row1">
-            <div>
-                <label for="subcontractor">Subcontractor Name:</label>
-                <input class="form-control" list="subcontractor_list" id="subcontractor" name="subcontractor"
-                       value="{{ invoice.Contractor_Name }}" placeholder="Type to search..." required>
-                <input type="hidden" id="subcontractor_id" name="subcontractor_id" value="{{ invoice.Subcontractor_Id }}">
-                <datalist id="subcontractor_list"></datalist>
-            </div>
-        </div>
-
-        <!-- Village and PMC No Fields -->
-        <div class="row2">
-            <div>
-                <label for="village">Village Name:</label>
-                <input type="text" id="village" name="village" value="{{ invoice.Village_Name }}" required/>
-                <datalist id="village_list"></datalist>
-            </div>
-            <div>
-                <label for="pmc_no">PMC No:</label>
-                <input type="text" id="pmc_no" name="pmc_no" value="{{ invoice.PMC_No }}" required/>
-            </div>
-        </div>
-
-        <!-- Work Type and Invoice Details -->
-        <div class="row2">
-            <div>
-                <label for="work_type">Work Type:</label>
-                <input type="text" id="work_type" name="work_type" value="{{ invoice.Work_Type }}" required/>
-            </div>
-            <div>
-                <label for="invoice_details">Invoice Details:</label>
-                <textarea id="invoice_details" name="invoice_details" required>{{ invoice.Invoice_Details }}</textarea>
-            </div>
-        </div>
-
-        <!-- Invoice No and Invoice Date -->
-        <div class="row2">
-            <div>
-                <label for="invoice_no">Invoice No:</label>
-                <input type="text" id="invoice_no" name="invoice_no" value="{{ invoice.Invoice_No }}" required/>
-            </div>
-            <div>
-                <label for="invoice_date">Invoice Date:</label>
-                <input type="date" id="invoice_date" name="invoice_date" value="{{ invoice.Invoice_Date }}" required/>
-            </div>
-        </div>
-
-        <!-- Amount Fields -->
-        <div class="row3">
-            <div>
-                <label for="basic_amount">Basic Amount:</label>
-                <input type="number" step="0.01" id="basic_amount" name="basic_amount" value="{{ invoice.Basic_Amount }}" required/>
-            </div>
-            <div>
-                <label for="debit_amount">Debit Amount:</label>
-                <input type="number" step="0.01" id="debit_amount" name="debit_amount" value="{{ invoice.Debit_Amount }}" required/>
-            </div>
-            <div>
-                <label for="after_debit_amount">After Debit Amount:</label>
-                <input type="number" step="0.01" id="after_debit_amount" name="after_debit_amount" value="{{ invoice.After_Debit_Amount }}" required/>
-            </div>
-        </div>
-
-        <!-- GST, TDS, and Other Amounts -->
-        <div class="row3">
-            <div>
-                <label for="amount">Amount:</label>
-                <input type="number" step="0.01" id="amount" name="amount" value="{{ invoice.Amount }}" required/>
-            </div>
-            <div>
-                <label for="gst_amount">GST Amount:</label>
-                <input type="number" step="0.01" id="gst_amount" name="gst_amount" value="{{ invoice.GST_Amount }}" required/>
-            </div>
-            <div>
-                <label for="tds_amount">TDS Amount:</label>
-                <input type="number" step="0.01" id="tds_amount" name="tds_amount" value="{{ invoice.TDS_Amount }}" required/>
-            </div>
-        </div>
-
-        <!-- SD, On Commission, Hydro Testing -->
-        <div class="row3">
-            <div>
-                <label for="sd_amount">SD Amount:</label>
-                <input type="number" step="0.01" id="sd_amount" name="sd_amount" value="{{ invoice.SD_Amount }}" required/>
-            </div>
-            <div>
-                <label for="on_commission">On Commission:</label>
-                <input type="number" step="0.01" id="on_commission" name="on_commission" value="{{ invoice.On_Commission }}" required/>
-            </div>
-            <div>
-                <label for="hydro_testing">Hydro Testing:</label>
-                <input type="number" step="0.01" id="hydro_testing" name="hydro_testing" value="{{ invoice.Hydro_Testing }}" required/>
-            </div>
-        </div>
-
-       <!-- Hold Amount Section -->
-<div id="hold_amount_container">
-    {% set seen_types = [] %}
-    {% for hold in invoice.hold_amounts %}
-        {% if hold.hold_type not in seen_types %}
-            {% set _ = seen_types.append(hold.hold_type) %}
-            <div class="hold-amount-row">
-                <label for="hold_type_{{ loop.index }}">Hold Type:</label>
-                <input type="text" id="hold_type_{{ loop.index }}" name="hold_type[]" value="{{ hold.hold_type }}" required/>
-                <label for="hold_amount_{{ loop.index }}">Hold Amount:</label>
-                <input type="number" step="0.01" id="hold_amount_{{ loop.index }}" name="hold_amount[]" value="{{ hold.hold_amount }}" required/>
-                <button type="button" class="remove-hold-amount">Remove</button>
-            </div>
-        {% endif %}
-    {% endfor %}
-</div>
-
-
-        <div class="hold-row">
-            <button type="button" id="add_hold_amount" class="button">+ Add Hold Amount</button>
-        </div>
-
-        <!-- Final Amounts -->
-        <div class="row2">
-            <div>
-                <label for="gst_sd_amount">GST SD Amount:</label>
-                <input type="number" step="0.01" id="gst_sd_amount" name="gst_sd_amount" value="{{ invoice.GST_SD_Amount }}" required/>
-            </div>
-            <div>
-                <label for="final_amount">Final Amount:</label>
-                <input type="number" step="0.01" id="final_amount" name="final_amount" value="{{ invoice.Final_Amount }}" required/>
-            </div>
-        </div>
-
-        <!-- Submit Button -->
-        <button type="submit" class="button">Update</button>
-    </form>
-</div>
-
-<!-- JS for dynamic hold amount rows -->
-<script>
-$(document).ready(function() {
-    // Add new hold amount row
-    $("#add_hold_amount").click(function() {
-        const index = $("#hold_amount_container .hold-amount-row").length;
-        const newRow = `
+    <!-- JS for dynamic hold amount rows -->
+    <script>
+        $(document).ready(function () {
+            // Add new hold amount row
+            $("#add_hold_amount").click(function () {
+                const index = $("#hold_amount_container .hold-amount-row").length;
+                const newRow = `
             <div class="hold-amount-row">
                 <label for="hold_type_${index}">Hold Type:</label>
                 <input type="text" id="hold_type_${index}" name="hold_type[]" required/>
@@ -172,36 +188,36 @@ $(document).ready(function() {
                 <button type="button" class="remove-hold-amount">Remove</button>
             </div>
         `;
-        $("#hold_amount_container").append(newRow);
-    });
+                $("#hold_amount_container").append(newRow);
+            });
 
-    // Remove hold amount row
-    $(document).on("click", ".remove-hold-amount", function() {
-        $(this).closest(".hold-amount-row").remove();
-    });
+            // Remove hold amount row
+            $(document).on("click", ".remove-hold-amount", function () {
+                $(this).closest(".hold-amount-row").remove();
+            });
 
-    // Submit form via AJAX
-    $("#invoiceForm").submit(function(e) {
-        e.preventDefault();
+            // Submit form via AJAX
+            $("#invoiceForm").submit(function (e) {
+                e.preventDefault();
 
-        $.ajax({
-            type: "POST",
-            url: $(this).attr("action"),
-            data: $(this).serialize(),
-            success: function(response) {
-                if(response.status === "success") {
-                $("#invoiceSuccessAlert").fadeIn().delay(3000).fadeOut();
-                alert("Invoice updated successfully!");  // <-- Popup alert
-                }
+                $.ajax({
+                    type: "POST",
+                    url: $(this).attr("action"),
+                    data: $(this).serialize(),
+                    success: function (response) {
+                        if (response.status === "success") {
+                            $("#invoiceSuccessAlert").fadeIn().delay(3000).fadeOut();
+                            alert("Invoice updated successfully!");  // <-- Popup alert
+                        }
 
-            },
-            error: function(xhr) {
-                alert("Error: " + xhr.responseJSON.message);
-            }
+                    },
+                    error: function (xhr) {
+                        alert("Error: " + xhr.responseJSON.message);
+                    }
+                });
+            });
         });
-    });
-});
-</script>
+    </script>
 
 </body>
 {% endblock %}

templates/edit_village.html

@@ -28,25 +28,17 @@
 
             <button type="submit">Update Village</button>
         </form>
-    </div>
-
-    <!-- Flash Message (Hidden, used for JS popup) -->
+  <!-- Flash Messages (hidden, used for JS popup) -->
+<div id="flash-messages-container" style="display:none;">
     {% with messages = get_flashed_messages(with_categories=true) %}
-      {% if messages %}
-        {% for category, message in messages %}
-          <div id="flash-message" data-category="{{ category }}" style="display:none;">{{ message }}</div>
-        {% endfor %}
-      {% endif %}
+        {% if messages %}
+            {% for category, message in messages %}
+                <div class="flash-message" data-category="{{ category }}">{{ message }}</div>
+            {% endfor %}
+        {% endif %}
     {% endwith %}
 </div>
 
-<script>
-    window.onload = function () {
-        const flash = document.getElementById('flash-message');
-        if (flash && flash.innerText.trim() !== "") {
-            alert(flash.innerText);
-        }
-    };
-</script>
+
 </body>
 {% endblock %}