incubator-callback-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Marcus Wu <>
Subject Re: [jira] [Resolved] (CB-442) FileTransfer download should not block the UI
Date Tue, 10 Apr 2012 14:19:44 GMT
I've noticed some problems with my fix for this since I committed it. I noticed two things.

First, my error callbacks were never being called. This was because I didn't realize the first
time around that there is a different call for converting the error callback string.
Second, the success callbacks were failing. I tracked this down to errors in the success callback
string due to the cast now being integrated into the success callback code.

I have some changes to properly call the error callbacks and to eliminate the cast in calling
the success callback for the download. I was about to create another pull request when I realized
that the upload code is also using a cast which appears to no longer be valid (although I
am not as familiar with this code as I am sure others are). Should I go ahead and eliminate
that upload cast before creating another pull request?

Another question is does a ticket need to be created (or the original ticket reopened) prior
to creating a pull request?


On Apr 6, 2012, at 6:16 PM, Shazron Abdullah (Resolved) (JIRA) wrote:

>     [
> Shazron Abdullah resolved CB-442.
> ---------------------------------
>       Resolution: Fixed
>    Fix Version/s: 1.6.0
> Fix commit merged in (from Marcus Wu):;a=commit;h=2bf695b0bc7ec47e2ec1f40842035106811e35f2
>> FileTransfer download should not block the UI
>> ---------------------------------------------
>>                Key: CB-442
>>                URL:
>>            Project: Apache Callback
>>         Issue Type: Bug
>>         Components: iOS
>>   Affects Versions: Master
>>           Reporter: Shazron Abdullah
>>           Assignee: Shazron Abdullah
>>            Fix For: 1.6.0
>>        Attachments: CB-442-test.html
>> Right now the way it downloads items, it blocks the UI:
>> NSData dataWithContentsOfUrl:
>> Change to an async download method.
> --
> This message is automatically generated by JIRA.
> If you think it was sent incorrectly, please contact your JIRA administrators:!default.jspa
> For more information on JIRA, see:

View raw message