You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cordova.apache.org by an...@apache.org on 2015/07/17 16:24:04 UTC

cordova-plugin-file-transfer git commit: CB-9376 Fix FileTransfer plugin manual tests issue - 'undefined' in paths

Repository: cordova-plugin-file-transfer
Updated Branches:
  refs/heads/master f0209a2f7 -> 499ec5c07


CB-9376 Fix FileTransfer plugin manual tests issue - 'undefined' in paths


Project: http://git-wip-us.apache.org/repos/asf/cordova-plugin-file-transfer/repo
Commit: http://git-wip-us.apache.org/repos/asf/cordova-plugin-file-transfer/commit/499ec5c0
Tree: http://git-wip-us.apache.org/repos/asf/cordova-plugin-file-transfer/tree/499ec5c0
Diff: http://git-wip-us.apache.org/repos/asf/cordova-plugin-file-transfer/diff/499ec5c0

Branch: refs/heads/master
Commit: 499ec5c07b4087829623facc07f1b1f3e31e9afc
Parents: f0209a2
Author: daserge <da...@yandex.ru>
Authored: Thu Jul 16 15:40:44 2015 +0300
Committer: daserge <da...@yandex.ru>
Committed: Thu Jul 16 15:40:44 2015 +0300

----------------------------------------------------------------------
 tests/tests.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cordova-plugin-file-transfer/blob/499ec5c0/tests/tests.js
----------------------------------------------------------------------
diff --git a/tests/tests.js b/tests/tests.js
index b7146a0..c56c4f9 100644
--- a/tests/tests.js
+++ b/tests/tests.js
@@ -912,7 +912,7 @@ exports.defineManualTests = function (contentEl, createActionButton) {
 
     function downloadImg(source, urlFn, element, directory) {
         var filename = source.substring(source.lastIndexOf("/") + 1);
-        filename = directory + filename || filename;
+        filename = (directory || '') + filename;
         function download(fileSystem) {
             var ft = new FileTransfer();
             console.log("Starting download");
@@ -928,7 +928,7 @@ exports.defineManualTests = function (contentEl, createActionButton) {
         clearResults();
         window.requestFileSystem(LocalFileSystem.TEMPORARY, 0, function (fileSystem) {
             console.log("Checking for existing file");
-            if (directory !== undefined) {
+            if (typeof directory !== 'undefined') {
                 console.log("Checking for existing directory.");
                 fileSystem.root.getDirectory(directory, {}, function (dirEntry) {
                     dirEntry.removeRecursively(function () {


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org