jamesmontemagno / MediaPlugin

Take & Pick Photos and Video Plugin for Xamarin and Windows
MIT License
712 stars 360 forks source link

iOS/Android photo filename discrepancy #460

Open duzenko opened 6 years ago

duzenko commented 6 years ago

Bug Information

Version Number of Plugin: latest stable Device Tested On: Galaxy Tab S2 Simulator Tested On: iPhone 7 Plus Version of VS: latest stable Version of Xamarin: latest stable

Expected Behavior

File naming follows the same logic on iOS and Android

Actual Behavior

iOS file name is device current time: appdir/Documents/IMG_20180208_130142.jpg

Android file name is adhoc. Assume the picked image has already been picked before and it's copy is already in the app/temp directory. Then app is restarted. First pick results in file name appdir/Pictures/Screenshot_20180207-172152_3.png (note the _3 ending that makes this filename unique). Base filename part is the source file name, not device current time as on iOS. Second and consecutive picks of the same source file results in the very same file name: appdir/Pictures/Screenshot_20180207-172152_3.png. After app is restarted it changes to appdir/Pictures/Screenshot_20180207-172152_4.png and stays on it until next app restart.

Please note that all picks here are of the same source file.

Problem: impossible to figure out if this file has been already picked.

Code snippet

            var picture = await CrossMedia.Current.PickPhotoAsync( new PickMediaOptions { CustomPhotoSize = 512 } );

Suggestion

Let android file name be the same as on iOS: device current time. At least it will fix the duplication between app restarts.

MikaTml commented 6 years ago

Not quite sure if we are talking about same problem, but the title of this issue could be same.

When I take picture with iOS, file is saved "appdir/Documents/'directory name'/'picture name'" and with Android it is "appdir/files/Pictures/'directory name'/'picture name'". So the problem is that extra Pictures folder with Android. Why with Android, there is this extra folder? pic_android pic_ios

jamesmontemagno commented 6 years ago

That is expect on android @MikaTml because you specify where you want them to save inside of the file_paths.xml

duzenko commented 6 years ago

What about my original request?

duzenko commented 6 years ago

Any comment would be appreciated, anyone?

duzenko commented 6 years ago

Hello?