From 958a3bcfb96383d4c7997e82d1cd379d4860c398 Mon Sep 17 00:00:00 2001 From: Gunther Brunner Date: Wed, 4 Mar 2015 21:22:43 +0900 Subject: [PATCH] Fix bug where http:// is not being added if there is a "http://" already in the query string. --- .../dashboard/navigation/navigation-controller.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/res/app/control-panes/dashboard/navigation/navigation-controller.js b/res/app/control-panes/dashboard/navigation/navigation-controller.js index 9f6b1234..8a0f0c29 100644 --- a/res/app/control-panes/dashboard/navigation/navigation-controller.js +++ b/res/app/control-panes/dashboard/navigation/navigation-controller.js @@ -26,7 +26,9 @@ module.exports = function NavigationCtrl($scope, $rootScope) { function addHttp(textUrl) { // Check for '://' because a protocol-less URL might include // a username:password combination. - return (textUrl.indexOf('://') === -1 ? 'http://' : '') + textUrl + // Ignores also any query parameter because it may contain a http:// inside. + return (textUrl.replace(/\?.*/, '').indexOf('://') === -1 ? 'http://' : '' + ) + textUrl } $scope.blurUrl = false