[ 
https://issues.apache.org/jira/browse/CB-6165?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13990057#comment-13990057
 ] 

Shazron Abdullah commented on CB-6165:
--------------------------------------

Perplexed  how the "OK" is added, it could be in the JS.

Here's where it processes the arguments in native:
https://github.com/apache/cordova-ios/blob/87c751c54ba7aa3c49a03e7953e18f94c2ce57c4/CordovaLib/Classes/CDVPluginResult.m#L154-L164

There doesn't seem to be a unit-test for this function, I'll add it

> Removing the "OK" String from success callback 
> -----------------------------------------------
>
>                 Key: CB-6165
>                 URL: https://issues.apache.org/jira/browse/CB-6165
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: iOS
>            Reporter: Ally Ogilvie
>            Priority: Minor
>              Labels: core
>             Fix For: 3.5.0
>
>
> [Brief]
> If no "message" (e.g. messageWithString, messageWithInt) is given to the 
> resultWithStatus success callback, an "OK" is added.
> [About]
> Since the very beginning oh PhoneGap a success callback without "message" 
> (native args) has returned the string "OK" to the success function in 
> JavaScript.
> There is no reason for this to be. The reason it's there is because it's been 
> there from the beginning. But we should get rid of this now.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to