Enhance position handling in grab_contest_data function for MLB lineups

- Updated the logic to correctly map player positions for 9-player MLB lineups, ensuring accurate output formatting.
- Simplified the handling of player IDs by removing unnecessary checks, improving data processing efficiency.
- These changes contribute to ongoing efforts to enhance data integrity and improve user experience within the application.

global_func/grab_contest_data.py

@@ -12,18 +12,29 @@ def grab_contest_data(sport, contest_name, contest_id_map, contest_date):
     def format_lineup_string(lineup_hash, positions):
         """Replaces colons in a lineup hash with sequential positions."""
         # Remove the leading colon and split by the remaining colons
-        # Only split on colons that are between numbers
-        player_ids = [pid for pid in lineup_hash.lstrip(':').split(':') if pid.isdigit() or pid == '-1']
+        player_ids = lineup_hash.lstrip(':').split(':')
         
-        # Get the actual positions needed for this lineup
-        actual_positions = positions[:len(player_ids)]
-        
-        # Check if the number of IDs matches the number of positions
-        if len(player_ids) != len(actual_positions):
-            print(f"Warning: Mismatch for hash {lineup_hash}. IDs: {len(player_ids)}, Positions: {len(actual_positions)}")
-            return lineup_hash
+        # For MLB, we need to handle the specific position order
+        if len(player_ids) == 9:  # 9-player lineup
+            # Map positions to the correct order for a 9-player lineup
+            position_map = {
+                0: '1B ',
+                1: ' 2B ',
+                2: ' 3B ',
+                3: ' C ',
+                4: ' OF ',
+                5: ' OF ',
+                6: ' P ',
+                7: ' P ',
+                8: ' SS '
+            }
+            # Use the mapped positions instead of the full position list
+            actual_positions = [position_map[i] for i in range(len(player_ids))]
+        else:
+            # For other cases, use the standard position list
+            actual_positions = positions[:len(player_ids)]
 
-        # Combine positions and player IDs, only for valid player IDs
+        # Combine positions and player IDs
         combined_parts = []
         for pos, pid in zip(actual_positions, player_ids):
             if pid == '-1':  # Handle empty slots