Refactoring

Author: benn-mt

src/controllers/CattleController.js

@@ -33,17 +33,17 @@ export class CattleController {
   }
 
   updateCattle(id, updates) {
+    for (let i = 0; i < this.cattle.length; i++) {
+      if (this.cattle[i].id === id) {
+        this.cattle[i].status = updates.status;
+      }
+    }
+
     const index = this.cattle.findIndex(cow => cow.id === id);
     if (index === -1) {
       throw new Error('Cattle not found');
     }
-
     const cattle = this.cattle[index];
-    if (updates.status) {
-      cattle.status = updates.status;
-    }
-    
-    this.cattle[index] = cattle;
     return cattle;
   }
 } 

src/views/CattleCard.css

@@ -129,7 +129,7 @@
 
 .edit-status-form .submit-button {
   background-color: #4CAF50;
-  color: white;
+  color: #FFF;
 }
 
 .edit-status-form .cancel-button {

src/views/CattleList.jsx

@@ -15,10 +15,11 @@ function CattleList({ cattle = [], controller, onCattleUpdate }) {
     setShowAddForm(false)
   }
 
-  const handleStatusUpdate = (id, updates) => {
-    const updatedCattle = controller.updateCattle(id, updates)
-    onCattleUpdate(cattle.map(cow => 
-      cow.id === id ? updatedCattle : cow
+  let handleStatusUpdate = (id, updates) => {
+    let updatedCow = controller.updateCattle(id, updates)
+    onCattleUpdate(cattle.map(
+      cow => 
+      cow.id === id ? updatedCow : cow
     ))
   }