[Dotify71]

Summary of Changes

Corrupted or non-standard image files containing EXIF GPS metadata with rational denominators equal to zero caused _convert_to_degrees to raise an unhandled ZeroDivisionError. Because ZeroDivisionError inherits from ArithmeticError rather than ValueError, the exception was not caught by the except block in _extract_gps_coordinates, causing image_util_extract_metadata to fail and aborting photo scans.

Implementation Details:

1. Mathematical Edge-Case Handling in _convert_to_degrees:
   • Explicitly checked v.denominator == 0.0 when processing rational EXIF structures and raised ZeroDivisionError with an informative error message.
2. Exception Handling in _extract_gps_coordinates:
   • Added ZeroDivisionError to the exception tuple in _extract_gps_coordinates so malformed GPS tags fallback gracefully to (None, None) without terminating the broader metadata extraction or image scan.
3. Unit Tests:
   • Added comprehensive tests in backend/tests/test_extract_location_metadata.py covering zero-denominator rational tuples, valid rationals, and non-rational (plain float/int) coordinates.

Fixes #1535

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

🗂️ Base branches to auto review: 1

• dev

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration

• Source mode: Auto
• Repository: AOSSIE-Org/PictoPy
• Branch: main
• File: .coderabbit.yaml
• Commit: fc7958e
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: 7100b3d9-52e8-4ebd-8291-51088b99e691

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• Trigger review

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

• X
• Mastodon
• Reddit
• LinkedIn