incubator-callback-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Andrew Grieve (JIRA)" <>
Subject [jira] [Commented] (CB-1385) 2.1.0rc2 breaks certain plugins on iOS due to added "null" argument using FORMAT TWO in iOSExec
Date Mon, 10 Sep 2012 13:22:07 GMT


Andrew Grieve commented on CB-1385:

Hmm, yes, thanks Matthew for figuring out what went wrong here. I think what we could do to
fix this is to pop off the null callbackId in CDVInvokedUrlCommand::legacyArguments if it
is null/nil.

Shaz - We're away today at a company golf off-site. I will fix this tomorrow, but since it's
a 2.1 blocker, feel free to take it as well.
> 2.1.0rc2 breaks certain plugins on iOS due to added "null" argument using FORMAT TWO
in iOSExec
> -----------------------------------------------------------------------------------------------
>                 Key: CB-1385
>                 URL:
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: CordovaJS, iOS
>    Affects Versions: 2.1.0
>         Environment: iOS
>            Reporter: Matthew Windwer
>            Assignee: Andrew Grieve
>             Fix For: 2.1.0
> This was brought to my attention when the InAppPurchaseManager plugin no longer worked
for me on 2.1.0rc2, but was working in 2.0.
> The issue happens when calling cordova.exec using "FORMAT TWO", e.g:
> cordova.exec('InAppPurchaseManager.requestProductData', productId, callback + '.success',
callback + '.fail');
> When the arguments get to the native side there is an extra null value preceding the
rest of the arguments. This is only a problem when the plugin expects the arguments to be
in a certain order, like for InAppPurchaseManager. A workaround for InAppPurchaseManager is
to increase the index by one for each call in the .m file (e.g. [arguments objectAtIndex:0]
becomes [arguments objectAtIndex:1]).
> The extra null value is the callbackId, which is always going to be null using "FORMAT
2". The code that handles this was refactored since 2.0, which may be the source of the problem,
in particular around line 959 of the current cordova.js:
>  var command = [callbackId, service, action, actionArgs];

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see:

View raw message