Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

(ios) workaround for Issue #778 #838

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

AdrianV
Copy link

@AdrianV AdrianV commented Jun 21, 2023

Platforms affected

iOS

Motivation and Context

Issue #778

Description

setting data to nil in line 409 crashes the front end app. Therefore I commented it out. This is not a perfect solution for the Issue #778 because we want the location data when CameraUsesGeolocation but better not to crash.

Testing

with CameraUsesGeolocation set to true in the config.xml and taking a photo from the Album does not crash anymore on iOS >= 15

Checklist

  • I've run the tests to see all new and existing tests pass
  • I added automated test coverage as appropriate for this change
  • [ +] Commit is prefixed with (platform) if this change only applies to one platform (e.g. (android))
  • [+ ] If this Pull Request resolves an issue, I linked to the issue in the text above (and used the correct keyword to close issues using keywords)
  • I've updated the documentation if necessary

setting data to nil in line 386 crashes the front end app. Therefore I commented it out.
This is not a perfect solution for the Issue apache#778 because we want the location data when `CameraUsesGeolocation`  but better not to crash.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant