Update updated_word.py

updated_word.py

@@ -599,7 +599,7 @@ def fix_management_summary_details_column(table, flat_json):
     return replacements_made
 
 def fix_operator_declaration_empty_values(table, flat_json):
-    """Fix Operator Declaration table when values are empty"""
+    """Fix Operator Declaration table when values are empty or need updating"""
     replacements_made = 0
     
     print(f"    🎯 FIX: Operator Declaration empty values processing")
@@ -638,67 +638,77 @@ def fix_operator_declaration_empty_values(table, flat_json):
                         
                         print(f"      📋 Current values: Name='{name_text}', Position='{position_text}'")
                         
-                        # Fix Print Name if empty or has red text
-                        if not name_text or has_red_text(name_cell):
-                            print(f"      🔧 Fixing empty/red Print Name")
-                            
-                            # Try multiple sources for operator name
-                            name_sources = [
-                                "Operator Declaration.Print Name",
-                                "Print Name", 
-                                "Operator name (Legal entity)",
-                                "(print accreditation name)"
-                            ]
-                            
-                            for source in name_sources:
-                                name_value = find_matching_json_value(source, flat_json)
-                                if name_value is not None:
-                                    name_replacement = get_value_as_string(name_value)
-                                    if name_replacement.strip():
-                                        # Extract just the name if it's a company name
-                                        if "Pty Ltd" in name_replacement or "Company" in name_replacement:
-                                            continue
-                                        
+                        # ALWAYS update Print Name with new data - check for red text OR existing data
+                        print(f"      🔧 Updating Print Name")
+                        
+                        # Try multiple sources for operator name
+                        name_sources = [
+                            "Operator Declaration.Print Name",
+                            "Print Name", 
+                            "Operator name (Legal entity)",
+                            "(print accreditation name)"
+                        ]
+                        
+                        for source in name_sources:
+                            name_value = find_matching_json_value(source, flat_json)
+                            if name_value is not None:
+                                name_replacement = get_value_as_string(name_value)
+                                if name_replacement.strip():
+                                    # Extract just the name if it's a company name
+                                    if "Pty Ltd" in name_replacement or "Company" in name_replacement:
+                                        continue
+                                    
+                                    # Check if we need to update (different from current)
+                                    if name_replacement.strip() != name_text:
                                         if has_red_text(name_cell):
                                             cell_replacements = replace_red_text_in_cell(name_cell, name_replacement)
                                         else:
+                                            # Replace existing text
                                             name_cell.text = name_replacement
                                             cell_replacements = 1
                                         
                                         replacements_made += cell_replacements
-                                        print(f"      ✅ Fixed Print Name with: '{name_replacement}' (from {source})")
+                                        print(f"      ✅ Updated Print Name: '{name_text}' -> '{name_replacement}' (from {source})")
+                                        break
+                                    else:
+                                        print(f"      ℹ️ Print Name already correct: '{name_replacement}'")
                                         break
                         
-                        # Fix Position Title if empty or has red text  
-                        if not position_text or has_red_text(position_cell):
-                            print(f"      🔧 Fixing empty/red Position Title")
-                            
-                            position_sources = [
-                                "Operator Declaration.Position Title",
-                                "Position Title"
-                            ]
-                            
-                            for source in position_sources:
-                                position_value = find_matching_json_value(source, flat_json)
-                                if position_value is not None:
-                                    position_replacement = get_value_as_string(position_value)
-                                    if position_replacement.strip():
-                                        
+                        # ALWAYS update Position Title with new data
+                        print(f"      🔧 Updating Position Title")
+                        
+                        position_sources = [
+                            "Operator Declaration.Position Title",
+                            "Position Title"
+                        ]
+                        
+                        for source in position_sources:
+                            position_value = find_matching_json_value(source, flat_json)
+                            if position_value is not None:
+                                position_replacement = get_value_as_string(position_value)
+                                if position_replacement.strip():
+                                    
+                                    # Check if we need to update (different from current)
+                                    if position_replacement.strip() != position_text:
                                         if has_red_text(position_cell):
                                             cell_replacements = replace_red_text_in_cell(position_cell, position_replacement)
                                         else:
+                                            # Replace existing text
                                             position_cell.text = position_replacement
                                             cell_replacements = 1
                                         
                                         replacements_made += cell_replacements
-                                        print(f"      ✅ Fixed Position Title with: '{position_replacement}' (from {source})")
+                                        print(f"      ✅ Updated Position Title: '{position_text}' -> '{position_replacement}' (from {source})")
+                                        break
+                                    else:
+                                        print(f"      ℹ️ Position Title already correct: '{position_replacement}'")
                                         break
-                            
-                            # Fallback: use "Manager" if nothing found
-                            if not position_text and not has_red_text(position_cell):
-                                position_cell.text = "Manager"
-                                replacements_made += 1
-                                print(f"      ✅ Used fallback Position Title: 'Manager'")
+                        
+                        # Fallback: use "Manager" if nothing found and cell is empty
+                        if not position_text and not has_red_text(position_cell):
+                            position_cell.text = "Manager"
+                            replacements_made += 1
+                            print(f"      ✅ Used fallback Position Title: 'Manager'")
                 break
     
     return replacements_made