Simplify checkbox handling for mirror active field

Removed unnecessary hidden input fields and simplified the form parsing
logic for the active checkbox in mirror creation and editing forms.

Changes:
- Replaced complex list indexing with simple membership check
- Removed hidden input fields from both Jinja templates
- Updated comments to reflect simpler approach

The functionality remains identical, but the code is more readable
and maintainable.

Author: rockythorn

apollo/server/routes/admin_supported_products.py

@@ -497,10 +497,8 @@ async def admin_supported_product_mirror_new_post(
 
     # Parse form data to get the active field
     form_data_raw = await request.form()
-    # When checkbox is checked: active will be ["false", "true"], take the last value
-    # When checkbox is unchecked: active will be ["false"], take that value
-    active_values = form_data_raw.getlist("active")
-    active_value = active_values[-1] if active_values else "false"
+    # Checkbox sends "true" when checked, nothing when unchecked
+    active_value = "true" if "true" in form_data_raw.getlist("active") else "false"
 
     # Validation using centralized validation utility
     form_data = {
@@ -602,10 +600,8 @@ async def admin_supported_product_mirror_post(
 
     # Parse form data to get the active field
     form_data = await request.form()
-    # When checkbox is checked: active will be ["false", "true"], take the last value
-    # When checkbox is unchecked: active will be ["false"], take that value
-    active_values = form_data.getlist("active")
-    active_value = active_values[-1] if active_values else "false"
+    # Checkbox sends "true" when checked, nothing when unchecked
+    active_value = "true" if "true" in form_data.getlist("active") else "false"
 
     # Validation using centralized validation utility
     try:

apollo/server/templates/admin_supported_product_mirror.jinja

@@ -123,7 +123,6 @@
               <fieldset class="bx--fieldset">
                 <legend class="bx--label">Status</legend>
                 <div class="bx--form-item bx--checkbox-wrapper">
-                  <input type="hidden" name="active" value="false">
                   <input id="active" name="active" type="checkbox" value="true" class="bx--checkbox"
                          {% if mirror.active %}checked{% endif %}>
                   <label for="active" class="bx--checkbox-label">

apollo/server/templates/admin_supported_product_mirror_new.jinja

@@ -112,7 +112,6 @@
             <fieldset class="bx--fieldset">
               <legend class="bx--label">Status</legend>
               <div class="bx--form-item bx--checkbox-wrapper">
-                <input type="hidden" name="active" value="false">
                 <input id="active" name="active" type="checkbox" value="true" class="bx--checkbox"
                        {% if not form_data or form_data.active == 'true' %}checked{% endif %}>
                 <label for="active" class="bx--checkbox-label">