From 82fcbaf22adbc3e42912db496979aab81b25eb3f Mon Sep 17 00:00:00 2001
From: xiaji <xiaji@example.com>
Date: Mon, 18 May 2026 11:29:09 +0800
Subject: [PATCH] Remove battery field from maintenance record form

---
 .../templates/ups_manager/maintenance_form.html   | 14 --------------
 ups_management/ups_manager/views.py               | 15 +++++++++++++--
 2 files changed, 13 insertions(+), 16 deletions(-)

diff --git a/ups_management/ups_manager/templates/ups_manager/maintenance_form.html b/ups_management/ups_manager/templates/ups_manager/maintenance_form.html
index 1914d14..f1e753c 100644
--- a/ups_management/ups_manager/templates/ups_manager/maintenance_form.html
+++ b/ups_management/ups_manager/templates/ups_manager/maintenance_form.html
@@ -26,20 +26,6 @@
                             </div>
                         </div>
                         
-                        <div class="col-md-6">
-                            <div class="form-group">
-                                <label for="{{ form.battery.id_for_label }}" class="form-label font-weight-semibold">
-                                    {{ form.battery.label }}
-                                </label>
-                                {{ form.battery }}
-                                {% if form.battery.errors %}
-                                    {% for error in form.battery.errors %}
-                                        <div class="invalid-feedback d-block">{{ error }}</div>
-                                    {% endfor %}
-                                {% endif %}
-                            </div>
-                        </div>
-                        
                         <div class="col-md-6">
                             <div class="form-group">
                                 <label for="{{ form.supplier.id_for_label }}" class="form-label font-weight-semibold">
diff --git a/ups_management/ups_manager/views.py b/ups_management/ups_manager/views.py
index 3143639..468ea55 100644
--- a/ups_management/ups_manager/views.py
+++ b/ups_management/ups_manager/views.py
@@ -1,3 +1,4 @@
+import logging
 from django.shortcuts import render, redirect
 from django.views.generic import ListView, CreateView, UpdateView, DeleteView
 from django.urls import reverse_lazy
@@ -5,6 +6,8 @@ from django.contrib import messages
 from .models import UPSHost, Battery, Contact, Supplier, MaintenanceRecord
 from .forms import BatteryFormSet, UPSHostForm
 
+logger = logging.getLogger(__name__)
+
 
 class DashboardView(ListView):
     template_name = 'ups_manager/index.html'
@@ -118,6 +121,10 @@ class UPSHostUpdateView(UpdateView):
         return context
     
     def form_valid(self, form):
+        logger.info(f"POST data: {self.request.POST}")
+        logger.info(f"Form data: {form.cleaned_data}")
+        logger.info(f"last_maintenance_date value: {form.cleaned_data.get('last_maintenance_date')}")
+        
         self.object = form.save()
         
         if self.request.POST:
@@ -127,6 +134,10 @@ class UPSHostUpdateView(UpdateView):
         
         messages.success(self.request, 'UPS主机和电池信息已更新')
         return redirect(self.success_url)
+    
+    def form_invalid(self, form):
+        logger.error(f"Form errors: {form.errors}")
+        return super().form_invalid(form)
 
 
 class UPSHostDeleteView(DeleteView):
@@ -277,14 +288,14 @@ class MaintenanceListView(ListView):
 class MaintenanceCreateView(CreateView):
     model = MaintenanceRecord
     template_name = 'ups_manager/maintenance_form.html'
-    fields = ['ups_host', 'battery', 'supplier', 'maintenance_date', 'technician', 'content']
+    fields = ['ups_host', 'supplier', 'maintenance_date', 'technician', 'content']
     success_url = reverse_lazy('maintenance_list')
 
 
 class MaintenanceUpdateView(UpdateView):
     model = MaintenanceRecord
     template_name = 'ups_manager/maintenance_form.html'
-    fields = ['ups_host', 'battery', 'supplier', 'maintenance_date', 'technician', 'content']
+    fields = ['ups_host', 'supplier', 'maintenance_date', 'technician', 'content']
     success_url = reverse_lazy('maintenance_list')