Skip to content

Conversation

@max65482
Copy link
Contributor

@max65482 max65482 commented Feb 3, 2022

Similar to #3924, this PR fixes an inconsistency issue. This time it is about the additional location information that Apple applications store in the X-APPLE-STRUCTURED-LOCATION property. This is a supplement to the regular LOCATION property. It includes coordinates and a custom map reference. Upon change of LOCATION in a non-Apple application, X-APPLE-STRUCTURED-LOCATION becomes invalid. This PR solves this inconsistency by removing the X-APPLE-STRUCTURED-LOCATION property upon change of the location.

Discussion can be found in #3905.

@codecov
Copy link

codecov bot commented Feb 3, 2022

Codecov Report

Merging #3930 (302aa48) into main (bf456a9) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##               main    #3930      +/-   ##
============================================
- Coverage     29.42%   29.42%   -0.01%     
  Complexity      322      322              
============================================
  Files           221      221              
  Lines          7578     7579       +1     
  Branches       1001     1001              
============================================
  Hits           2230     2230              
- Misses         5348     5349       +1     
Flag Coverage Δ
javascript 20.80% <0.00%> (-0.01%) ⬇️
php 67.35% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/store/calendarObjectInstance.js 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bf456a9...302aa48. Read the comment docs.

Copy link
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense

@ChristophWurst ChristophWurst added this to the v3.1.0 milestone Feb 4, 2022
@ChristophWurst
Copy link
Member

/backport to stable3.0

@backportbot-nextcloud backportbot-nextcloud bot added the backport-request A backport was requested for this pull request label Feb 4, 2022
@ChristophWurst ChristophWurst merged commit 84ea7a2 into nextcloud:main Feb 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-request A backport was requested for this pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants