Enhance player matching process in find_name_mismatches function

- Improved the user interface for manual player matching by ensuring that all players needing manual matching are processed through a tabbed form layout.
- Streamlined the logic for applying both automatic and manual name matches, ensuring accurate updates to contest, ownership, and points data.
- Added success messages to confirm changes applied, enhancing user feedback during the matching process.
- Included a success message for cases where all players are automatically matched, improving user experience.

global_func/find_name_mismatches.py

@@ -43,58 +43,72 @@ def find_name_mismatches(contest_df, projections_df, ownership_df, fpts_df):
         else:
             players_to_process.append(player)
 
-    st.warning(f"Found {len(players_to_process)} players that need manual matching")
-    
-    # Create a form for batch processing
-    with st.form("name_matching_form"):
-        # Create tabs for each player
-        tabs = st.tabs([f"Player {i+1}" for i in range(len(players_to_process))])
-        
-        # Dictionary to store selections
-        selections = {}
+    if players_to_process:
+        st.warning(f"Found {len(players_to_process)} players that need manual matching")
         
-        # Populate each tab
-        for i, player in enumerate(players_to_process):
-            with tabs[i]:
-                st.write(f"**Projection Name:** {player}")
-                
-                # Find the top 3 closest matches
-                closest_matches = process.extract(player, portfolio_players_list, limit=3)
-                
-                # Create radio buttons for selection
-                options = [f"{match[0]} ({match[1]}%)" for match in closest_matches]
-                options.append("None of these")
-                
-                selections[player] = st.radio(
-                    f"Select correct match:",
-                    options,
-                    key=f"radio_{player}"
-                )
-        
-        # Submit button for the entire form
-        submitted = st.form_submit_button("Apply All Changes")
-        
-        if submitted:
-            # Process automatic matches
-            for projection_name, contest_name in auto_matches.items():
-                for col in name_columns:
-                    contest_raw[col] = contest_raw[col].replace(contest_name, projection_name)
-                ownership_raw['Player'] = ownership_raw['Player'].replace(contest_name, projection_name)
-                fpts_raw['Player'] = fpts_raw['Player'].replace(contest_name, projection_name)
+        # Create a form for batch processing
+        with st.form("name_matching_form"):
+            # Create tabs for each player
+            tabs = st.tabs([f"Player {i+1}" for i in range(len(players_to_process))])
+            
+            # Dictionary to store selections
+            selections = {}
+            
+            # Populate each tab
+            for i, player in enumerate(players_to_process):
+                with tabs[i]:
+                    st.write(f"**Projection Name:** {player}")
                     
-            # Process manual selections
-            for projection_name, selection in selections.items():
-                if selection != "None of these":
-                    selected_name = selection.split(" (")[0]
-                    for col in name_columns:
-                        contest_raw[col] = contest_raw[col].replace(selected_name, projection_name)
-                        ownership_raw['Player'] = ownership_raw['Player'].replace(contest_name, projection_name)
-                        fpts_raw['Player'] = fpts_raw['Player'].replace(contest_name, projection_name)
-
-                    st.success(f"Replaced '{selected_name}' with '{projection_name}'")
+                    # Find the top 3 closest matches
+                    closest_matches = process.extract(player, portfolio_players_list, limit=3)
+                    
+                    # Create radio buttons for selection
+                    options = [f"{match[0]} ({match[1]}%)" for match in closest_matches]
+                    options.append("None of these")
+                    
+                    selections[player] = st.radio(
+                        f"Select correct match:",
+                        options,
+                        key=f"radio_{player}"
+                    )
             
-            ownership_dict = dict(zip(ownership_raw['Player'], ownership_raw['Own']))
-            fpts_dict = dict(zip(fpts_raw['Player'], fpts_raw['FPTS']))
+            # Submit button for the entire form
+            submitted = st.form_submit_button("Apply All Changes")
             
-            st.success("All changes applied successfully!")
-            return contest_raw, projections_raw, ownership_dict, fpts_dict
+            if submitted:
+                # Process automatic matches
+                for projection_name, contest_name in auto_matches.items():
+                    for col in name_columns:
+                        contest_raw[col] = contest_raw[col].replace(contest_name, projection_name)
+                    ownership_raw['Player'] = ownership_raw['Player'].replace(contest_name, projection_name)
+                    fpts_raw['Player'] = fpts_raw['Player'].replace(contest_name, projection_name)
+                    st.write(contest_name + ' ' + projection_name)
+
+                # Process manual selections
+                for projection_name, selection in selections.items():
+                    if selection != "None of these":
+                        selected_name = selection.split(" (")[0]
+                        for col in name_columns:
+                            contest_raw[col] = contest_raw[col].replace(selected_name, projection_name)
+                        ownership_raw['Player'] = ownership_raw['Player'].replace(selected_name, projection_name)
+                        fpts_raw['Player'] = fpts_raw['Player'].replace(selected_name, projection_name)
+                        st.write(contest_name + ' ' + projection_name)
+                        st.success(f"Replaced '{selected_name}' with '{projection_name}'")
+                
+                ownership_dict = dict(zip(ownership_raw['Player'], ownership_raw['Own']))
+                fpts_dict = dict(zip(fpts_raw['Player'], fpts_raw['FPTS']))
+                
+                st.success("All changes applied successfully!")
+                return contest_raw, projections_raw, ownership_dict, fpts_dict
+    else:
+        st.success("All players have been automatically matched!")
+        # Apply automatic matches
+        for projection_name, contest_name in auto_matches.items():
+            for col in name_columns:
+                contest_raw[col] = contest_raw[col].replace(contest_name, projection_name)
+            ownership_raw['Player'] = ownership_raw['Player'].replace(contest_name, projection_name)
+            fpts_raw['Player'] = fpts_raw['Player'].replace(contest_name, projection_name)
+            st.write(contest_name + ' ' + projection_name)
+        ownership_dict = dict(zip(ownership_raw['Player'], ownership_raw['Own']))
+        fpts_dict = dict(zip(fpts_raw['Player'], fpts_raw['FPTS']))
+        return contest_raw, projections_raw, ownership_dict, fpts_dict