Fixed issue with new UmbracoModule trying to strip the legacy umbPage query string stuff too early (still need to ask Niels about what this does)

Fixed issue with page update with Document ctor to try catch the Parent property. Removed a bunch of erroneous tracing and replaced with LogHelper.

src/Umbraco.Web/UmbracoModule.cs

@@ -69,12 +69,12 @@ namespace Umbraco.Web
 			var uri = umbracoContext.UmbracoUrl;
 			var lpath = umbracoContext.UmbracoUrl.AbsolutePath.ToLowerInvariant();
 
-			// legacy - no idea what this is
-			LegacyCleanUmbPageFromQueryString(ref uri, ref lpath);
-
 			//do not continue if this request is not a front-end routable page
 			if (EnsureUmbracoRoutablePage(uri, lpath, httpContext))
 			{
+				// legacy - no idea what this is
+				LegacyCleanUmbPageFromQueryString(ref uri, ref lpath);
+
 				//Create a document request since we are rendering a document on the front-end
 
 				// create the new document request