Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Map interaction inconsistencies when items have multi valued location field #2083

Open
patrickdalla opened this issue Feb 14, 2024 · 2 comments
Assignees
Labels
Projects

Comments

@patrickdalla
Copy link
Collaborator

While implementing a solution for #43, it was found that the maps Aleapp plugin extracted 2 locations per item that represents google maps direction (to and from).
The implementation of map does not expect this, and item selection and highlighting is performing inconsistently.

@lfcnassif
Copy link
Member

Hi @patrickdalla, thanks. Should we classify this as bug or enhancement? Does it happen with currently supported GPS artifacts or just with the new ALeapp integration?

@patrickdalla
Copy link
Collaborator Author

I checked the code for ExtraProperties.LOCATIONS references. In GeofileParser, the method set of Metadata is called, with single lat and long passed with ExtraProperties.LOCATIONS, indicating that only one location is added per extracted subitem.

But the chat parsers (whatsapp, threema) call the method add of Metadata with ExtraProperties.LOCATIONS of the chat correspondent item, when extractMessages is false to extract individual messages as items.

So, there is a possible situation when this issue can occur in past releases. Maybe is better classify it as BUG.

@lfcnassif lfcnassif added the bug label Feb 14, 2024
@lfcnassif lfcnassif added this to To do in 4.2 via automation Apr 27, 2024
@lfcnassif lfcnassif moved this from To do to In progress in 4.2 Apr 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
4.2
In progress
Development

Successfully merging a pull request may close this issue.

2 participants