Skip to content

fix: ignore malformed GEO properties - #7

Merged
niccokunzmann merged 1 commit into
PyCalendar:mainfrom
nightcityblade:fix/issue-5-invalid-geo
Sep 16, 2026
Merged

niccokunzmann merged 1 commit into
PyCalendar:mainfrom
nightcityblade:fix/issue-5-invalid-geo

Conversation

@nightcityblade

Copy link
Copy Markdown

Fixes #5

Summary

  • accept only parsed vGeo values from the event
  • treat the vBroken value produced for malformed GEO data as absent

Tests

  • .venv/bin/pytest -q icalendar_compatibility/test/test_location.py::test_geo_location_is_also_escaped
  • .venv/bin/pytest -q icalendar_compatibility/test/test_location.py (51 passed)
  • .venv/bin/pytest -q (78 passed)
  • .venv/bin/ruff check --ignore UP045 icalendar_compatibility/location.py
  • .venv/bin/black --check icalendar_compatibility/location.py

@niccokunzmann

Copy link
Copy Markdown
Member

Thanks for the PR! Sorry that it took me so long to have a look.

@niccokunzmann
niccokunzmann merged commit 432a9cf into PyCalendar:main Sep 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test_geo_location_is_also_escaped fails with icalendar v7

2 participants