From ff708a9357ec16b4f6a49d242ce62063c9345136 Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <noreply@fakecake.org>
Date: Sat, 1 Dec 2018 18:08:09 +0300
Subject: [PATCH] prefs: initial objectification

---
 js/prefs.js | 178 ++++++++++++++++++++++++++--------------------------
 prefs.php   |   2 +-
 2 files changed, 90 insertions(+), 90 deletions(-)

diff --git a/js/prefs.js b/js/prefs.js
index 59629955d..032098f6f 100755
--- a/js/prefs.js
+++ b/js/prefs.js
@@ -1,6 +1,94 @@
 /* global dijit, __ */
 
-let seq = "";
+const App = {
+	init: function() {
+		window.onerror = function (message, filename, lineno, colno, error) {
+			report_error(message, filename, lineno, colno, error);
+		};
+
+		require(["dojo/_base/kernel",
+			"dojo/ready",
+			"dojo/parser",
+			"dojo/_base/loader",
+			"dojo/_base/html",
+			"dijit/ColorPalette",
+			"dijit/Dialog",
+			"dijit/form/Button",
+			"dijit/form/CheckBox",
+			"dijit/form/DropDownButton",
+			"dijit/form/FilteringSelect",
+			"dijit/form/MultiSelect",
+			"dijit/form/Form",
+			"dijit/form/RadioButton",
+			"dijit/form/ComboButton",
+			"dijit/form/Select",
+			"dijit/form/SimpleTextarea",
+			"dijit/form/TextBox",
+			"dijit/form/ValidationTextBox",
+			"dijit/InlineEditBox",
+			"dijit/layout/AccordionContainer",
+			"dijit/layout/AccordionPane",
+			"dijit/layout/BorderContainer",
+			"dijit/layout/ContentPane",
+			"dijit/layout/TabContainer",
+			"dijit/Menu",
+			"dijit/ProgressBar",
+			"dijit/Toolbar",
+			"dijit/Tree",
+			"dijit/tree/dndSource",
+			"dojo/data/ItemFileWriteStore",
+			"lib/CheckBoxStoreModel",
+			"lib/CheckBoxTree",
+			"fox/PrefFeedStore",
+			"fox/PrefFilterStore",
+			"fox/PrefFeedTree",
+			"fox/PrefFilterTree",
+			"fox/PrefLabelTree"], function (dojo, ready, parser) {
+
+			ready(function () {
+				try {
+					parser.parse();
+
+					setLoadingProgress(50);
+
+					const clientTzOffset = new Date().getTimezoneOffset() * 60;
+					const params = {op: "rpc", method: "sanityCheck", clientTzOffset: clientTzOffset};
+
+					xhrPost("backend.php", params, (transport) => {
+						Utils.backendSanityCallback(transport);
+					});
+
+				} catch (e) {
+					exception_error(e);
+				}
+			});
+		});
+	},
+	initSecondStage: function() {
+		document.onkeydown = pref_hotkey_handler;
+		setLoadingProgress(50);
+		notify("");
+
+		let tab = getURLParam('tab');
+
+		if (tab) {
+			tab = dijit.byId(tab + "Tab");
+			if (tab) dijit.byId("pref-tabs").selectChild(tab);
+		}
+
+		const method = getURLParam('method');
+
+		if (method == 'editFeed') {
+			const param = getURLParam('methodparam');
+
+			window.setTimeout(function () {
+				editFeed(param)
+			}, 100);
+		}
+
+		setInterval(hotkeyPrefixTimeout, 5 * 1000);
+	}
+}
 
 function notify_callback2(transport, sticky) {
 	notify_info(transport.responseText, sticky);
@@ -645,94 +733,6 @@ function selectTab(id, noupdate) {
 	}
 }
 
-function init_second_stage() {
-	document.onkeydown = pref_hotkey_handler;
-	setLoadingProgress(50);
-	notify("");
-
-	let tab = getURLParam('tab');
-
-	if (tab) {
-		tab = dijit.byId(tab + "Tab");
-		if (tab) dijit.byId("pref-tabs").selectChild(tab);
-	}
-
-	const method = getURLParam('method');
-
-	if (method == 'editFeed') {
-		const param = getURLParam('methodparam');
-
-		window.setTimeout(function() { editFeed(param) }, 100);
-	}
-
-	setInterval(hotkeyPrefixTimeout, 5*1000);
-}
-
-function init() {
-	window.onerror = function (message, filename, lineno, colno, error) {
-		report_error(message, filename, lineno, colno, error);
-	};
-
-	require(["dojo/_base/kernel",
-		"dojo/ready",
-		"dojo/parser",
-		"dojo/_base/loader",
-		"dojo/_base/html",
-		"dijit/ColorPalette",
-		"dijit/Dialog",
-		"dijit/form/Button",
-		"dijit/form/CheckBox",
-		"dijit/form/DropDownButton",
-		"dijit/form/FilteringSelect",
-		"dijit/form/MultiSelect",
-		"dijit/form/Form",
-		"dijit/form/RadioButton",
-		"dijit/form/ComboButton",
-		"dijit/form/Select",
-		"dijit/form/SimpleTextarea",
-		"dijit/form/TextBox",
-		"dijit/form/ValidationTextBox",
-		"dijit/InlineEditBox",
-		"dijit/layout/AccordionContainer",
-		"dijit/layout/AccordionPane",
-		"dijit/layout/BorderContainer",
-		"dijit/layout/ContentPane",
-		"dijit/layout/TabContainer",
-		"dijit/Menu",
-		"dijit/ProgressBar",
-		"dijit/Toolbar",
-		"dijit/Tree",
-		"dijit/tree/dndSource",
-		"dojo/data/ItemFileWriteStore",
-		"lib/CheckBoxStoreModel",
-		"lib/CheckBoxTree",
-		"fox/PrefFeedStore",
-		"fox/PrefFilterStore",
-		"fox/PrefFeedTree",
-		"fox/PrefFilterTree",
-		"fox/PrefLabelTree"], function (dojo, ready, parser) {
-
-		ready(function () {
-			try {
-				parser.parse();
-
-				setLoadingProgress(50);
-
-				const clientTzOffset = new Date().getTimezoneOffset() * 60;
-				const params = { op: "rpc", method: "sanityCheck", clientTzOffset: clientTzOffset };
-
-				xhrPost("backend.php", params, (transport) => {
-					backend_sanity_check_callback(transport);
-				});
-
-			} catch (e) {
-				exception_error(e);
-			}
-		});
-	});
-}
-
-
 function validatePrefsReset() {
 	if (confirm(__("Reset to defaults?"))) {
 
diff --git a/prefs.php b/prefs.php
index 9f2b2e43b..6ff81af74 100644
--- a/prefs.php
+++ b/prefs.php
@@ -104,7 +104,7 @@
 
 	<script type="text/javascript">
 		Event.observe(window, 'load', function() {
-			init();
+			App.init();
 		});
 	</script>