From 5c68551800e477db41ae6fe3a318b2ff2d7a9cb1 Mon Sep 17 00:00:00 2001
From: Wojtek Kosior <koszko@koszko.org>
Date: Wed, 21 Jul 2021 17:42:21 +0200
Subject: store repository URLs in settings

---
 background/storage_server.js | 12 ++++++------
 common/storage_client.js     | 17 +++--------------
 common/stored_types.js       |  3 +++
 html/options.html            | 18 ++++++++++++++++++
 html/options_main.js         | 33 ++++++++++++++++++++++++++++-----
 5 files changed, 58 insertions(+), 25 deletions(-)

diff --git a/background/storage_server.js b/background/storage_server.js
index 554aff2..2252eb5 100644
--- a/background/storage_server.js
+++ b/background/storage_server.js
@@ -9,7 +9,7 @@
  * IMPORTS_START
  * IMPORT listen_for_connection
  * IMPORT get_storage
- * IMPORT TYPE_PREFIX
+ * IMPORT list_prefixes
  * IMPORT CONNECTION_TYPE
  * IMPORTS_END
  */
@@ -38,11 +38,11 @@ function new_connection(port)
 {
     console.log("new remote storage connection!");
 
-    port.postMessage({
-	[TYPE_PREFIX.SCRIPT] : storage.get_all(TYPE_PREFIX.SCRIPT),
-	[TYPE_PREFIX.BAG] : storage.get_all(TYPE_PREFIX.BAG),
-	[TYPE_PREFIX.PAGE] : storage.get_all(TYPE_PREFIX.PAGE)
-    });
+    const message = {};
+    for (const prefix of list_prefixes)
+	message[prefix] = storage.get_all(prefix);
+
+    port.postMessage(message);
 
     let handle_change = change => port.postMessage(change);
 
diff --git a/common/storage_client.js b/common/storage_client.js
index 4849a65..2b2f495 100644
--- a/common/storage_client.js
+++ b/common/storage_client.js
@@ -8,7 +8,6 @@
 /*
  * IMPORTS_START
  * IMPORT CONNECTION_TYPE
- * IMPORT TYPE_PREFIX
  * IMPORT list_prefixes
  * IMPORT make_once
  * IMPORT browser
@@ -47,20 +46,10 @@ function handle_message(message)
 	setTimeout(resolve, 0, message.result);
 }
 
-function list(name, prefix)
-{
-    return {prefix, name, listeners : new Set()};
-}
-
-var scripts = list("scripts", TYPE_PREFIX.SCRIPT);
-var bags = list("bags", TYPE_PREFIX.BAG);
-var pages = list("pages", TYPE_PREFIX.PAGE);
+const list_by_prefix = {};
 
-const list_by_prefix = {
-    [TYPE_PREFIX.SCRIPT] : scripts,
-    [TYPE_PREFIX.BAG] : bags,
-    [TYPE_PREFIX.PAGE] : pages
-};
+for (const prefix of list_prefixes)
+    list_by_prefix[prefix] = {prefix, listeners : new Set()};
 
 var resolve_init;
 
diff --git a/common/stored_types.js b/common/stored_types.js
index a6f1f2f..304842b 100644
--- a/common/stored_types.js
+++ b/common/stored_types.js
@@ -14,6 +14,7 @@
  */
 
 const TYPE_PREFIX = {
+    REPO: "r",
     PAGE : "p",
     BAG : "b",
     SCRIPT : "s",
@@ -21,12 +22,14 @@ const TYPE_PREFIX = {
 };
 
 const TYPE_NAME = {
+    [TYPE_PREFIX.REPO] : "repo",
     [TYPE_PREFIX.PAGE] : "page",
     [TYPE_PREFIX.BAG] : "bag",
     [TYPE_PREFIX.SCRIPT] : "script"
 }
 
 const list_prefixes = [
+    TYPE_PREFIX.REPO,
     TYPE_PREFIX.PAGE,
     TYPE_PREFIX.BAG,
     TYPE_PREFIX.SCRIPT
diff --git a/html/options.html b/html/options.html
index bfb9e52..a10b919 100644
--- a/html/options.html
+++ b/html/options.html
@@ -33,12 +33,14 @@
       }
 
       /* tabbed view */
+      #show_repos:not(:checked) ~ #repos,
       #show_pages:not(:checked) ~ #pages,
       #show_bags:not(:checked) ~ #bags,
       #show_scripts:not(:checked) ~ #scripts {
 	  display: none;
       }
 
+      #show_repos:checked ~ #repos_lbl,
       #show_pages:checked ~ #pages_lbl,
       #show_bags:checked ~ #bags_lbl,
       #show_scripts:checked ~ #scripts_lbl {
@@ -124,9 +126,12 @@
     </div>
 
     <!-- Mind the show_*s ids below - their format is assumed in js code -->
+    <input type="radio" name="tabs" id="show_repos"></input>
     <input type="radio" name="tabs" id="show_pages" checked></input>
     <input type="radio" name="tabs" id="show_bags"></input>
     <input type="radio" name="tabs" id="show_scripts"></input>
+    <label for="show_repos" id="repos_lbl"
+	   class="tab_head"> Repos </label>
     <label for="show_pages" id="pages_lbl"
 	   class="tab_head"> Pages </label>
     <label for="show_bags" id="bags_lbl"
@@ -134,6 +139,19 @@
     <label for="show_scripts" id="scripts_lbl"
 	   class="tab_head"> Scripts </label>
 
+    <div id="repos">
+      <ul id="repos_ul">
+	<li id="work_repo_li" class="hide">
+	  <label for="repo_url_field">URL: </label>
+	  <input id="repo_url_field"></input>
+	  <br/>
+	  <button id="save_repo_but" type="button"> Save </button>
+	  <button id="discard_repo_but" type="button"> Cancel </button>
+	</li>
+      </ul>
+      <button id="add_repo_but" type="button"> Add repository </button>
+    </div>
+
     <div id="pages">
       <ul id="pages_ul">
 	<li id="work_page_li" class="hide">
diff --git a/html/options_main.js b/html/options_main.js
index e1e6cbe..026b9ba 100644
--- a/html/options_main.js
+++ b/html/options_main.js
@@ -38,7 +38,7 @@ function item_li_id(prefix, item)
     return `li_${prefix}_${item}`;
 }
 
-/* Insert into list of bags/pages/scripts */
+/* Insert into list of bags/pages/scripts/repos */
 function add_li(prefix, item, at_the_end=false)
 {
     let ul = ul_by_prefix[prefix];
@@ -58,6 +58,8 @@ function add_li(prefix, item, at_the_end=false)
     let export_button = remove_button.nextElementSibling;
     export_button.addEventListener("click",
 				   () => export_item(prefix, item));
+    if (prefix === TYPE_PREFIX.REPO)
+	export_button.remove();
 
     if (!at_the_end) {
 	for (let element of ul.ul.children) {
@@ -89,7 +91,7 @@ function radio_li_id(prefix, item)
 
 function add_chbx_li(prefix, name)
 {
-    if (prefix === TYPE_PREFIX.PAGE)
+    if (![TYPE_PREFIX.BAG, TYPE_PREFIX.SCRIPT].includes(prefix))
 	return;
 
     let li = chbx_component_li_template.cloneNode(true);
@@ -109,7 +111,7 @@ var radio_component_none_li = by_id("radio_component_none_li");
 
 function add_radio_li(prefix, name)
 {
-    if (prefix === TYPE_PREFIX.PAGE)
+    if (![TYPE_PREFIX.BAG, TYPE_PREFIX.SCRIPT].includes(prefix))
 	return;
 
     let li = radio_component_li_template.cloneNode(true);
@@ -125,6 +127,18 @@ function add_radio_li(prefix, name)
     radio_components_ul.insertBefore(li, radio_component_none_li);
 }
 
+/* Used to reset edited repo. */
+function reset_work_repo_li(ul, item, _)
+{
+    ul.work_name_input.value = maybe_string(item);
+}
+
+/* Used to get repo data for saving */
+function work_repo_li_data(ul)
+{
+    return [ul.work_name_input.value, {}];
+}
+
 const page_payload_span = by_id("page_payload");
 
 function set_page_components(components)
@@ -461,6 +475,15 @@ const UL_STATE = {
 };
 
 const ul_by_prefix = {
+    [TYPE_PREFIX.REPO] : {
+	ul : by_id("repos_ul"),
+	work_li : by_id("work_repo_li"),
+	work_name_input : by_id("repo_url_field"),
+	reset_work_li : reset_work_repo_li,
+	get_work_li_data : work_repo_li_data,
+	state : UL_STATE.IDLE,
+	edited_item : undefined,
+    },
     [TYPE_PREFIX.PAGE] : {
 	ul : by_id("pages_ul"),
 	work_li : by_id("work_page_li"),
@@ -727,7 +750,7 @@ async function main()
 	discard_but.addEventListener("click", () => cancel_work(prefix));
 	save_but.addEventListener("click", () => save_work(prefix));
 
-	if (prefix === TYPE_PREFIX.SCRIPT)
+	if ([TYPE_PREFIX.REPO, TYPE_PREFIX.SCRIPT].includes(prefix))
 	    continue;
 
 	let ul = ul_by_prefix[prefix];
@@ -772,7 +795,7 @@ function handle_change(change)
 
     let uls_creators = [[ul.ul, item_li_id]];
 
-    if (change.prefix !== TYPE_PREFIX.PAGE) {
+    if ([TYPE_PREFIX.BAG, TYPE_PREFIX.SCRIPT].includes(change.prefix)) {
 	uls_creators.push([chbx_components_ul, chbx_li_id]);
 	uls_creators.push([radio_components_ul, radio_li_id]);
     }
-- 
cgit v1.2.3