search

Antidote-for-Tox / objcTox

No longer maintained
Mozilla Public License 2.0
37 stars 12 forks source link

Add error handling for setAvatar #5

Closed Chuongv closed 9 years ago

Chuongv commented 9 years ago

@dvor Let me know your thoughts. Added error handling.

Also I'll send another pull request later this week once I get more unit tests added for 'hasAvatar' and 'avatar'

dvor commented 9 years ago

We want to handle this error too. :) In case if old image cannot be removed it makes sense to return from method.

[fileManager removeItemAtPath:path error:nil];
Chuongv commented 9 years ago

Closing pull request, will reopen once tests are updated.