Merge pull request #5221 from jlembeck/master

Fix for issue 4882

Author: gseguin

js/jquery.mobile.navigation.js

@@ -47,7 +47,7 @@ define( [
 			//    [16]: ?msg=1234&type=unread
 			//    [17]: #msg-content
 			//
-			urlParseRE: /^(((([^:\/#\?]+:)?(?:(\/\/)((?:(([^:@\/#\?]+)(?:\:([^:@\/#\?]+))?)@)?(([^:\/#\?\]\[]+|\[[^\/\]@#?]+\])(?:\:([0-9]+))?))?)?)?((\/?(?:[^\/\?#]+\/+)*)([^\?#]*)))?(\?[^#]+)?)(#.*)?/,
+			urlParseRE: /^\s*(((([^:\/#\?]+:)?(?:(\/\/)((?:(([^:@\/#\?]+)(?:\:([^:@\/#\?]+))?)@)?(([^:\/#\?\]\[]+|\[[^\/\]@#?]+\])(?:\:([0-9]+))?))?)?)?((\/?(?:[^\/\?#]+\/+)*)([^\?#]*)))?(\?[^#]+)?)(#.*)?/,
 
 			// Abstraction to address xss (Issue #4787) by removing the authority in
 			// browsers that auto	decode it. All references to location.href should be

tests/unit/navigation/navigation_core.js

@@ -1307,4 +1307,18 @@
 			}
 		]);
 	});
+
+	asyncTest( "external page is accessed correctly even if it has a space in the url", function(){
+		$.testHelper.pageSequence([
+			function(){
+				$.mobile.changePage( " external.html" );
+			},
+			function(){
+				equal( $.mobile.activePage.attr( "id" ), "external-test", "the correct page is loaded" );
+				start();
+			}
+
+		]);
+	});
+	
 })(jQuery);