From 1b7d42c5691ffa21246022690f7688dda534c2dc Mon Sep 17 00:00:00 2001
From: Vincent Petry <pvince81@owncloud.com>
Date: Mon, 13 Jul 2015 10:57:52 +0200
Subject: [PATCH] Fix OC.joinPaths with empty arguments

When empty arguments are given, the leading or trailing slash was not
detected properly.
---
 core/js/js.js                   | 18 ++++++++++++++----
 core/js/tests/specs/coreSpec.js |  4 ++++
 2 files changed, 18 insertions(+), 4 deletions(-)

diff --git a/core/js/js.js b/core/js/js.js
index ff52c8f1f3..d701a37acb 100644
--- a/core/js/js.js
+++ b/core/js/js.js
@@ -340,13 +340,23 @@ var OC={
 			return '';
 		}
 		var path = '';
-		var lastArg = arguments[arguments.length - 1];
-		var leadingSlash = arguments[0].charAt(0) === '/';
+		// convert to array
+		var args = Array.prototype.slice.call(arguments);
+		// discard empty arguments
+		args = _.filter(args, function(arg) {
+			return arg.length > 0;
+		});
+		if (args.length < 1) {
+			return '';
+		}
+
+		var lastArg = args[args.length - 1];
+		var leadingSlash = args[0].charAt(0) === '/';
 		var trailingSlash = lastArg.charAt(lastArg.length - 1) === '/';
 		var sections = [];
 		var i;
-		for (i = 0; i < arguments.length; i++) {
-			sections = sections.concat(arguments[i].split('/'));
+		for (i = 0; i < args.length; i++) {
+			sections = sections.concat(args[i].split('/'));
 		}
 		var first = !leadingSlash;
 		for (i = 0; i < sections.length; i++) {
diff --git a/core/js/tests/specs/coreSpec.js b/core/js/tests/specs/coreSpec.js
index 7a09f362ee..3ebe55305f 100644
--- a/core/js/tests/specs/coreSpec.js
+++ b/core/js/tests/specs/coreSpec.js
@@ -147,11 +147,15 @@ describe('Core base tests', function() {
 		});
 		it('keeps leading slashes', function() {
 			expect(OC.joinPaths('/abc')).toEqual('/abc');
+			expect(OC.joinPaths('/abc', '')).toEqual('/abc');
+			expect(OC.joinPaths('', '/abc')).toEqual('/abc');
 			expect(OC.joinPaths('/abc', 'def')).toEqual('/abc/def');
 			expect(OC.joinPaths('/abc', 'def', 'ghi')).toEqual('/abc/def/ghi');
 		});
 		it('keeps trailing slashes', function() {
+			expect(OC.joinPaths('', 'abc/')).toEqual('abc/');
 			expect(OC.joinPaths('abc/')).toEqual('abc/');
+			expect(OC.joinPaths('abc/', '')).toEqual('abc/');
 			expect(OC.joinPaths('abc', 'def/')).toEqual('abc/def/');
 			expect(OC.joinPaths('abc', 'def', 'ghi/')).toEqual('abc/def/ghi/');
 		});