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

Shazron Abdullah edited comment on CB-7287 at 11/27/15 8:41 AM:
----------------------------------------------------------------

Unfortunately I found another problem while trying to implement my solution 
above.
While cordova.exec points to the new bridge, doing a:
{code}
cordova.require('cordova/exec')
{code}

... still points to the old bridge. This is obviously problematic since that's 
how plugins grab the exec function, and they will still use the old bridge.

What we need to do is a "cordova.define.remove('cordova/exec')" and redefine 
'cordova/exec' to the new bridge. A definition would look like this:
https://github.com/apache/cordova-ios/blob/e3ae7e3ca0395988a087884f9ba18a683160dbde/CordovaLib/cordova.js#L821


was (Author: shazron):
Unfortunately I found another problem while trying to implement my solution 
above.
While cordova.exec points to the new bridge, doing a:
{code}
cordova.require('cordova/exec')
{code}

... still points to the old bridge. This is obviously problematic since that's 
how plugins grab the exec function, and they will still use the old bridge.

> [WKWebView] Failing tests
> -------------------------
>
>                 Key: CB-7287
>                 URL: https://issues.apache.org/jira/browse/CB-7287
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: iOS
>         Environment: iOS 8
>            Reporter: Shazron Abdullah
>            Assignee: Anis Kadri
>              Labels: cordova-ios-4.0.x
>
> This is only for the wkwebview branch, and using these plugins, and 3.5.0 
> mobile-spec:
> {code}
>     "org.apache.cordova.battery-status": "0.2.10",
>     "org.apache.cordova.camera": "0.3.1",
>     "org.apache.cordova.console": "0.2.10",
>     "org.apache.cordova.contacts": "0.2.12",
>     "org.apache.cordova.device": "0.2.11",
>     "org.apache.cordova.device-motion": "0.2.9",
>     "org.apache.cordova.device-orientation": "0.3.8",
>     "org.apache.cordova.dialogs": "0.2.9",
>     "org.apache.cordova.file": "1.3.0",
>     "org.apache.cordova.file-transfer": "0.4.5",
>     "org.apache.cordova.geolocation": "0.3.9",
>     "org.apache.cordova.globalization": "0.3.0",
>     "org.apache.cordova.inappbrowser": "0.5.1",
>     "org.apache.cordova.media": "0.2.12",
>     "org.apache.cordova.media-capture": "0.3.2",
>     "org.apache.cordova.network-information": "0.2.11",
>     "org.apache.cordova.splashscreen": "0.3.2",
>     "org.apache.cordova.vibration": "0.3.10"
> {code}
> The 3 failing tests:
> {code}
> FileTransfer upload method filetransfer.spec.18 should be able to upload a 
> file.
> Expected 
> '{"fields":{"value2":"param","value1":"test"},"files":{"file":{"size":24,"path":"/opt/run/snapshot/.tmp/ed60ec667664b454d72b193bf8841e5c","name":"upload.txt","type":"text/plain","mtime":"2014-08-12T21:53:28.862Z"}}}
>  ' to match /fields:\s*{\s*value1.*/.
> FileTransfer upload method filetransfer.spec.19 should be able to upload a 
> file with http basic auth.
> Expected 
> '{"fields":{"value2":"param","value1":"test"},"files":{"file":{"size":24,"path":"/opt/run/snapshot/.tmp/94aa407aca7bd9c53081f0da5ab33d54","name":"upload.txt","type":"text/plain","mtime":"2014-08-12T21:53:29.322Z"}}}
>  ' to match /fields:\s*{\s*value1.*/.
> FileTransfer Backwards compatibility filetransfer.spec.29 should be able to 
> upload a file using local paths.
> Expected 
> '{"fields":{"value2":"param","value1":"test"},"files":{"file":{"size":24,"path":"/opt/run/snapshot/.tmp/9bf857a3d64ef9beaf4eae4e22876ecc","name":"upload.txt","type":"text/plain","mtime":"2014-08-12T21:53:31.307Z"}}}
>  ' to match /fields:\s*{\s*value1.*/.
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to