Enhance player ID processing in grab_contest_data function

- Updated the logic for splitting lineup hashes to only include valid player IDs, improving data accuracy.
- Refined the combination of player positions and IDs to handle empty slots more effectively, ensuring clearer output formatting.
- These changes contribute to ongoing efforts to enhance data integrity and improve user experience within the application.

global_func/grab_contest_data.py

@@ -12,7 +12,8 @@ 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
-        player_ids = lineup_hash.lstrip(':').split(':')
+        # Only split on colons that are between numbers
+        player_ids = [pid for pid in lineup_hash.lstrip(':').split(':') if pid.isdigit() or pid == '-1']
         
         # Get the actual positions needed for this lineup
         actual_positions = positions[:len(player_ids)]
@@ -22,8 +23,13 @@ def grab_contest_data(sport, contest_name, contest_id_map, contest_date):
             print(f"Warning: Mismatch for hash {lineup_hash}. IDs: {len(player_ids)}, Positions: {len(actual_positions)}")
             return lineup_hash
 
-        # Combine positions and player IDs
-        combined_parts = [pos + pid_map.get(pid, pid) for pos, pid in zip(actual_positions, player_ids)]
+        # Combine positions and player IDs, only for valid player IDs
+        combined_parts = []
+        for pos, pid in zip(actual_positions, player_ids):
+            if pid == '-1':  # Handle empty slots
+                combined_parts.append(pos + '')
+            else:
+                combined_parts.append(pos + pid_map.get(pid, pid))
         
         # Join them into a single string
         return "".join(combined_parts)