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

Crash after retrieved a file (image) #93

Closed
gorbat-o opened this issue May 2, 2018 · 9 comments
Closed

Crash after retrieved a file (image) #93

gorbat-o opened this issue May 2, 2018 · 9 comments
Labels

Comments

@gorbat-o
Copy link

gorbat-o commented May 2, 2018

Hey,

Nice library,
But I have a crash when I am trying to get the content of a file in the iCloud,

image

documentsProvider?.contents(path: content.url.relativeString, completionHandler: { data, error in
    print(error)
})

Maybe I am using the wrong variable for the path ??? :/
Thanks for future help :)

@amosavian
Copy link
Owner

Please remark all calls to private function monitorFile() in problematic methods until I fix the issue.

I didn't retain the object that passed to context variable. I have to change architecture of this part entirely.

@gorbat-o
Copy link
Author

gorbat-o commented May 3, 2018

Thank you for this fast answer,
Do you have a workaround ?

@amosavian
Copy link
Owner

The issue should be fixed entirely in latest commit. You must do a git pull.

@gorbat-o
Copy link
Author

gorbat-o commented May 3, 2018

I tried a pod update, and I am stil at 0.24.0
I am wondering if you didn't forget to update the version ?

@amosavian
Copy link
Owner

amosavian commented May 3, 2018

Unfortunately it's almost impossible to have test unit for iCloud provider, thus there is no way to guarantee as I can do for WebDav, Dropbox or FTP.

I didn't release a new version. I usually update pod when the version is stable. It would be nice if you can switch to git clone to test. Otherwise I have to release new version as my test inside my app is okay.

@gorbat-o
Copy link
Author

gorbat-o commented May 3, 2018

Ok, I tried and it works.
Thank you so much for your reactivity.

@amosavian
Copy link
Owner

Thanks. I will release a new pod version in minutes to allow you to switch back.

@gorbat-o
Copy link
Author

gorbat-o commented May 3, 2018

weird, it didn't update my pod :/

@amosavian
Copy link
Owner

I'm writing release note and waiting to Travis checks to be passed. You can check within an hour.

amosavian added a commit that referenced this issue Sep 20, 2023
- Fixed FileObject comparison
- Fixed image GPS metadata population in ExtendedLocalFileProvider
- More items for image metadata in ExtendedLocalFileProvider
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants