abelabbesnabi / cordova-plugin-image-picker

Forked from wymsee/cordova-imagePicker. Fixes permission issue on Android that causes the app to crash
MIT License
13 stars 17 forks source link

Some of the Images selected, the images URI is undefined #1

Closed clarklight closed 7 years ago

clarklight commented 7 years ago

Hey man

Thanks for adding in the permission part, but i just experienced that some images selected it doesn't return a URI, it is shown as undefined at the moment.

Not sure why at the moment, i have tested on Android 5.0, android 7.0 on LG, Samsung, on Phonegap 6.4.0.

abelabbesnabi commented 7 years ago

I'm not sure why. I only forked the main plugin to fix the permission problem. The rest of the code is identical. Can you attach those images that are causing the issue? I will add them to my phone gallery and see if I can replicate. Thanks.

clarklight commented 7 years ago

Hi, i guess its not the specific images, i just took a picture on my camera, and tried to select that, it doesnt show up either. I am just taking a look again to see the URI results etc

But 1 thing i see is, for the images that doesn't show up, the app will flash then back to normal. But the URI would be undefined.

It seems like any picture from the Camera folder won't get uploaded. But when i tried like pictures i got from whatsapp, then that had no problem.

Thanks i will keep digging and update you if i find any more info.

abelabbesnabi commented 7 years ago

Ok. Seems like this problem might not be with the plugin itself. If so, maybe we should consider closing this issue. Let me know how it goes.

clarklight commented 7 years ago

I just gave it a go again the results[i] is undefined. I simply took a picture with the camera and just select that image through the plugin. Not sure if this would be the problem but i included your version within my config.xml like this

Do you have a npm version?

Thanks

abelabbesnabi commented 7 years ago

No, I don't :(

clarklight commented 7 years ago

Spent hours.. found the reason finally, nothing to do with the plugin. Closing the issue, Thanks for the help/response.

abelabbesnabi commented 7 years ago

Excellent!