From: Mike Taylor <mike@indexdata.com>
Date: Tue, 8 Apr 2014 10:19:06 +0000 (+0100)
Subject: Move saved_config code inside $(document).ready(function() {} block.
X-Git-Tag: 1.0.0~997
X-Git-Url: http://sru.miketaylor.org.uk/?a=commitdiff_plain;h=6048ca4e831ca2b9153ba1f363c76f745db3b0c3;hp=e6575aacc629557869f9d0ff13fc5aa9bf9e311d;p=mkws-moved-to-github.git

Move saved_config code inside $(document).ready(function() {} block.
It DOES work there -- I don't know why I thought before it didn't.
(I was probably confused by the difficulty of dealing with a variable
that may or may not have been defined.)
Fixes bug MKWS-149.
---

diff --git a/src/mkws-core.js b/src/mkws-core.js
index 9668cd1..01f82c8 100644
--- a/src/mkws-core.js
+++ b/src/mkws-core.js
@@ -302,21 +302,15 @@ mkws.pagerNext = function(tname) {
 	}
     }
 
-
-    // I don't understand why I need this copy, but I do: mkws_config
-    // is not visible inside the document.ready function, but the
-    // saved copy is.
-    var saved_config;
-    if (typeof mkws_config === 'undefined') {
-	log("setting empty config");
-	saved_config = {};
-    } else {
-	log("using config: " + $.toJSON(mkws_config));
-	saved_config = mkws_config;
-    }
-
-
     $(document).ready(function() {
+	var saved_config;
+	if (typeof mkws_config === 'undefined') {
+	    log("setting empty config");
+	    saved_config = {};
+	} else {
+	    log("using config: " + $.toJSON(mkws_config));
+	    saved_config = mkws_config;
+	}
 	mkws.setMkwsConfig(saved_config);
 
 	for (var key in mkws.config) {