• Stefy Lanza (nextime / spora )'s avatar
    Fix fixture parser fighter column mapping bug (v1.2.2) · f6514b18
    Stefy Lanza (nextime / spora ) authored
    - Fixed critical bug where both fighter1 and fighter2 were incorrectly sourced from fighter1 column during XLSX uploads
    - Enhanced FixtureParser.detect_required_columns() with fighter-specific matching logic in app/upload/fixture_parser.py
    - Added proper fighter number detection (1 or 2) to prevent cross-mapping during partial column matching
    - Fighter1 columns now correctly map to fighter1 database field, Fighter2 to fighter2 field
    - Prevents false positives where 'fighter2' column incorrectly matched 'fighter1' field
    - Root cause: Original partial matching used split()[0] causing both 'fighter 1' and 'fighter 2' to become 'fighter'
    - Solution: Implemented specific logic checking for both 'fighter' keyword AND number in column names
    - Updated documentation (README.md, CHANGELOG.md) with bug fix details and version bump to v1.2.2
    - Maintains backward compatibility with existing column naming conventions
    f6514b18
CHANGELOG.md 8.77 KB