Change integration configuration page (#3372)

I've refactored the user query code to be able to use it also on the
integration configuration page. Now, items can be re-organized as in
the user query list.

3 files changed, 82 insertions, 70 deletions

diff --git a/p/scripts/user.query.js b/p/scripts/draggable.js
index 17bf041a6..d9d1d1cc0 100644
--- a/p/scripts/user.query.js
+++ b/p/scripts/draggable.js
@@ -2,96 +2,96 @@
 "use strict";
 /* jshint esversion:6, strict:global */
 
-const init_draggable = function() {
+const init_draggable_list = function() {
 	if (!window.context) {
 		if (window.console) {
-			console.log('FreshRSS user query waiting for JS…');
+			console.log('FreshRSS draggable list waiting for JS…');
 		}
-		setTimeout(init_draggable, 50);
+		setTimeout(init_draggable_list, 50);
 		return;
 	}
-	
+
 	let source;
-	const configureQueries = document.querySelector('#configureQueries');
+	const draggableList = document.querySelector('.draggableList');
 	const addMarker = (position, element) => {
-		const hr = configureQueries.querySelector('hr.drag-drop-marker');
+		const hr = draggableList.querySelector('hr.drag-drop-marker');
 		if (null === hr) {
 			element.insertAdjacentHTML(position, '<hr class="drag-drop-marker" />');
 		}
 	};
 	const removeMarker = () => {
-		const hr = configureQueries.querySelector('hr.drag-drop-marker');
+		const hr = draggableList.querySelector('hr.drag-drop-marker');
 		if (null !== hr) {
 			hr.remove();
 		}
 	};
 
-	configureQueries.addEventListener('dragstart', event => {
+	draggableList.addEventListener('dragstart', event => {
 		source = event.target.closest('[draggable="true"]');
 		event.dataTransfer.setData('text/html', source.outerHTML);
 		event.dataTransfer.effectAllowed = 'move';
 	});
-	configureQueries.addEventListener('dragover', event => {
+	draggableList.addEventListener('dragover', event => {
 		event.preventDefault();
 		if (!event.target || !event.target.closest) {
 			return;
 		}
 
-		const dropQuery = event.target.closest('[draggable="true"]');
-		if (null === dropQuery || source === dropQuery) {
+		const draggableItem = event.target.closest('[draggable="true"]');
+		if (null === draggableItem || source === draggableItem) {
 			return;
 		}
 
-		const rect = dropQuery.getBoundingClientRect();
+		const rect = draggableItem.getBoundingClientRect();
 		if (event.clientY < (rect.top + rect.height / 2)) {
-			addMarker('beforebegin', dropQuery);
+			addMarker('beforebegin', draggableItem);
 		} else {
-			addMarker('afterend', dropQuery);
+			addMarker('afterend', draggableItem);
 		}
 	});
-	configureQueries.addEventListener('dragleave', event => {
+	draggableList.addEventListener('dragleave', event => {
 		event.preventDefault();
 		removeMarker();
 	});
-	configureQueries.addEventListener('drop', event => {
+	draggableList.addEventListener('drop', event => {
 		event.preventDefault();
 		event.stopPropagation();
 		if (!event.target || !event.target.closest) {
 			return;
 		}
 
-		const dropQuery = event.target.closest('[draggable="true"]');
-		if (null === dropQuery || source === dropQuery) {
+		const draggableItem = event.target.closest('[draggable="true"]');
+		if (null === draggableItem || source === draggableItem) {
 			return;
 		}
 
-		const rect = dropQuery.getBoundingClientRect();
+		const rect = draggableItem.getBoundingClientRect();
 		if (event.clientY < (rect.top + rect.height / 2)) {
-			dropQuery.insertAdjacentHTML('beforebegin', event.dataTransfer.getData('text/html'));
+			draggableItem.insertAdjacentHTML('beforebegin', event.dataTransfer.getData('text/html'));
 		} else {
-			dropQuery.insertAdjacentHTML('afterend', event.dataTransfer.getData('text/html'));
+			draggableItem.insertAdjacentHTML('afterend', event.dataTransfer.getData('text/html'));
 		}
 		source.remove();
 		removeMarker();
-		configureQueries.submit();
+		draggableList.submit();
 	});
 
 	// This is needed to work around a Firefox bug → https://bugzilla.mozilla.org/show_bug.cgi?id=800050
-	configureQueries.addEventListener('focusin', event => {
+	draggableList.addEventListener('focusin', event => {
 		if (!event.target || !event.target.closest) {
 			return;
 		}
 
-		const queryName = event.target.closest('input[id^="queries_"][id$="_name"]');
-		if (null !== queryName) {
-			queryName.select();
+		const itemName = event.target.closest('input[type="text"]');
+		if (null !== itemName) {
+			itemName.select();
 		}
 	});
 };
 
 if (document.readyState && document.readyState !== 'loading') {
-	init_draggable();
+	init_draggable_list();
 } else if (document.addEventListener) {
-	document.addEventListener('DOMContentLoaded', event => init_draggable(), false);
+	document.addEventListener('DOMContentLoaded', event => init_draggable_list(), false);
 }
 // @license-end
diff --git a/p/scripts/extra.js b/p/scripts/extra.js
index 3cb7b93aa..a78b5b10d 100644
--- a/p/scripts/extra.js
+++ b/p/scripts/extra.js
@@ -93,46 +93,6 @@ function init_crypto_form() {
 }
 //</crypto form (Web login)>
 
-function init_share_observers() {
-	let shares = document.querySelectorAll('.group-share').length;
-	const shareAdd = document.querySelector('.share.add');
-	if (shareAdd) {
-		shareAdd.onclick = function (ev) {
-				const s = this.parentElement.querySelector('select'),
-					opt = s.options[s.selectedIndex];
-				let row = this.closest('form').getAttribute('data-' + opt.getAttribute('data-form'));
-				row = row.replace(/##label##/g, opt.text);
-				row = row.replace(/##type##/g, opt.value);
-				row = row.replace(/##help##/g, opt.getAttribute('data-help'));
-				row = row.replace(/##key##/g, shares);
-				row = row.replace(/##method##/g, opt.getAttribute('data-method'));
-				row = row.replace(/##field##/g, opt.getAttribute('data-field'));
-				this.closest('.form-group').insertAdjacentHTML('beforebegin', row);
-				shares++;
-				return false;
-			};
-	}
-}
-
-
-function init_remove_observers() {
-	document.querySelectorAll('.post').forEach(function (div) {
-			div.onclick = function (ev) {
-					const a = ev.target.closest('a.remove');
-					if (a) {
-						const remove_what = a.getAttribute('data-remove');
-						if (remove_what !== undefined) {
-							const d = document.getElementById(remove_what);
-							if (d) {
-								d.remove();
-							}
-						}
-						return false;
-					}
-				};
-		});
-}
-
 function init_password_observers() {
 	document.querySelectorAll('.toggle-password').forEach(function (a) {
 			a.onmousedown = function (ev) {
@@ -248,8 +208,6 @@ function init_extra() {
 		return;
 	}
 	init_crypto_form();
-	init_share_observers();
-	init_remove_observers();
 	init_password_observers();
 	init_select_observers();
 	init_slider_observers();
diff --git a/p/scripts/integration.js b/p/scripts/integration.js
new file mode 100644
index 000000000..076f379ec
--- /dev/null
+++ b/p/scripts/integration.js
@@ -0,0 +1,54 @@
+// @license magnet:?xt=urn:btih:0b31508aeb0634b347b8270c7bee4d411b5d4109&dn=agpl-3.0.txt AGPL-3.0
+"use strict";
+/* jshint esversion:6, strict:global */
+
+const init_integration = function() {
+	if (!window.context) {
+		if (window.console) {
+			console.log('FreshRSS integration waiting for JS…');
+		}
+		setTimeout(init_integration, 50);
+		return;
+    }
+
+    let shares = document.querySelectorAll('.group-share').length;
+    document.querySelector('.share.add').addEventListener('click', event => {
+        const shareTypes = event.target.closest('.group-controls').querySelector('select');
+        const shareType = shareTypes.options[shareTypes.selectedIndex];
+        let newShare = event.target.closest('form').getAttribute(`data-${shareType.getAttribute('data-form')}`);
+        newShare = newShare.replace(/##label##/g, shareType.text);
+        newShare = newShare.replace(/##type##/g, shareType.value);
+        newShare = newShare.replace(/##help##/g, shareType.getAttribute('data-help'));
+        newShare = newShare.replace(/##key##/g, shares);
+        newShare = newShare.replace(/##method##/g, shareType.getAttribute('data-method'));
+        newShare = newShare.replace(/##field##/g, shareType.getAttribute('data-field'));
+        event.target.closest('.form-group').insertAdjacentHTML('beforebegin', newShare);
+        shares++;
+    });
+
+    document.querySelector('.post').addEventListener('click', event => {
+        if (!event.target || !event.target.closest) {
+            return;
+        }
+
+        const deleteButton = event.target.closest('a.remove');
+        if (null === deleteButton || !deleteButton.closest) {
+            return;
+        }
+
+        const share = deleteButton.closest('.group-share');
+        const form = deleteButton.closest('form');
+        if (!share.remove || !form.submit) {
+            return;
+        }
+        share.remove();
+        form.submit();
+    });
+};
+
+if (document.readyState && document.readyState !== 'loading') {
+	init_integration();
+} else if (document.addEventListener) {
+	document.addEventListener('DOMContentLoaded', event => init_integration(), false);
+}
+// @license-end