Skip to content
This repository has been archived by the owner on Feb 1, 2024. It is now read-only.

Avoid exception when parsing arrays in raw data #1910

Merged
merged 1 commit into from
Jun 16, 2022

Conversation

jwalgran
Copy link
Contributor

@jwalgran jwalgran commented Jun 15, 2022

Overview

This is the same change as #1904 targeted at ogr/develop

Connects #1903

Testing Instructions

Skip testing because the change is identical in both branches.

Checklist

  • fixup! commits have been squashed
  • CI passes after rebase
  • CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines
  • This PR is targeted at the correct branch (develop vs. ogr/develop)
  • If this PR applies to both OAR and OGR a companion PR has been created

When submitting a facility via the API the raw data for fields like
`product_type` can contain an array value. Attempting to call `fload()` on a
list was raiding a `TypeError` rather than a `ValueError`, resulting in an
unhandled exception in the `try_parse_int_from_float` helper.
@jwalgran jwalgran force-pushed the ogr/bugfix/jcw/embed-array-parsing branch from e7e2945 to fd4d145 Compare June 16, 2022 00:00
@jwalgran
Copy link
Contributor Author

Merging without review because the change is the same across branches.

@jwalgran jwalgran merged commit 85aeb31 into ogr/develop Jun 16, 2022
@jwalgran jwalgran deleted the ogr/bugfix/jcw/embed-array-parsing branch September 20, 2022 19:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant