You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cordova.apache.org by "Andrew Grieve (JIRA)" <ji...@apache.org> on 2013/05/29 16:33:20 UTC
[jira] [Closed] (CB-1936) Backward compatibility on
FileTransfer.upload header support is broken
[ https://issues.apache.org/jira/browse/CB-1936?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Andrew Grieve closed CB-1936.
-----------------------------
Resolution: Cannot Reproduce
Fix Version/s: (was: 2.8.0)
Had a look at this as well and found it to be doing the right thing.
> Backward compatibility on FileTransfer.upload header support is broken
> ----------------------------------------------------------------------
>
> Key: CB-1936
> URL: https://issues.apache.org/jira/browse/CB-1936
> Project: Apache Cordova
> Issue Type: Bug
> Components: Android
> Affects Versions: 2.2.0, 2.3.0, 2.4.0
> Reporter: Simon MacDonald
> Assignee: Simon MacDonald
> Priority: Minor
>
> There is a bug in the Android code for backward compatibility. See if you can spot it:
> // Look for headers on the params map for backwards compatibility with older Cordova versions.
> final JSONObject headers = args.optJSONObject(8) == null ? params.optJSONObject("headers") : args.optJSONObject(8);
> I bet you didn't find it. The first comparison against null will fail. You see when the arguments are stringified on the JS side the null will become a "null". So there needs to be a comparison against the string "null" and not the null value.
--
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: http://www.atlassian.com/software/jira