Skip to content
This repository has been archived by the owner on May 31, 2023. It is now read-only.

getImageProperties should include date taken #36

Open
kinex opened this issue Jan 4, 2019 · 7 comments
Open

getImageProperties should include date taken #36

kinex opened this issue Jan 4, 2019 · 7 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@kinex
Copy link

kinex commented Jan 4, 2019

It would be nice to get also image date/time information in ImageProperties (specifically the date image was taken). This is needed e.g. for displaying images in date/time order.

@thedejifab
Copy link

Feature seconded

@btastic
Copy link
Owner

btastic commented Jan 6, 2019

I think this feature depends on having the correct exif information. You can grab these by using the https://github.com/brendan-duncan/image library. I am not sure how I'd add this feature without reading out exif information, which basically is already done by the library mentioned.

edit: There might be some issues as mentioned in #31

@kinex
Copy link
Author

kinex commented Jan 6, 2019

Well, your library is reading exif information too :) See FlutterNativeImagePlugin.java, line 115. It would be very easy to add there much more exif data, including "date taken".

The only reason for using this library is to avoid using https://github.com/brendan-duncan/image. It uses all too much memory to be usable in Android/iOS.

@btastic
Copy link
Owner

btastic commented Jan 6, 2019

Oh right, this was even a recent addition. Thanks for pointing this out.

@btastic btastic added help wanted Extra attention is needed good first issue Good for newcomers enhancement New feature or request labels Jan 6, 2019
@btastic
Copy link
Owner

btastic commented May 15, 2019

Added with 18369e6

There is now a "datetime" property on the ImageProperties which will have the TAG_DATETIME_ORIGINAL as a string. You can then parse it the way you want.

@btastic btastic closed this as completed May 15, 2019
@btastic
Copy link
Owner

btastic commented May 17, 2019

I had to undo the changes cause of issues on iOS. Will have to investigate further.

@btastic btastic reopened this May 17, 2019
@aloisdeniel
Copy link

Any news on a potential reintroduction?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants