cleanup markup in some plugins, make nsfw generate dijit widgets

2 files changed, 46 insertions, 51 deletions

diff --git a/plugins/nsfw/init.js b/plugins/nsfw/init.js
index adb6d43c0..4bc2443e8 100644
--- a/plugins/nsfw/init.js
+++ b/plugins/nsfw/init.js
@@ -1,7 +1,12 @@
-function nsfwShow(elem) {
-	let content = elem.parentNode.getElementsBySelector("div.nswf.content")[0];
+/* global Plugins */
 
-	if (content) {
-		Element.toggle(content);
+Plugins.NSFW = {
+	toggle: function(elem) {
+		const content = elem.domNode.parentNode.querySelector(".nswf.content");
+
+		if (content) {
+			Element.toggle(content);
+		}
 	}
 }
+
diff --git a/plugins/nsfw/init.php b/plugins/nsfw/init.php
index 1b27cc017..fecbc62af 100644
--- a/plugins/nsfw/init.php
+++ b/plugins/nsfw/init.php
@@ -27,74 +27,64 @@ class NSFW extends Plugin {
 		$a_tags = array_map("trim", explode(",", $article["tag_cache"]));
 
 		if (count(array_intersect($tags, $a_tags)) > 0) {
-			$article["content"] = "<div class='nswf wrapper'><button onclick=\"nsfwShow(this)\">".__("Not work safe (click to toggle)")."</button>
-				<div class='nswf content' style='display : none'>".$article["content"]."</div></div>";
+			$article["content"] = "<div class='nswf wrapper'>".
+					\Controls\button_tag(__("Not work safe (click to toggle)"), '', ['onclick' => 'Plugins.NSFW.toggle(this)']).
+					"<div class='nswf content' style='display : none'>".$article["content"]."</div>
+				</div>";
 		}
 
 		return $article;
 	}
 
 	function hook_render_article_cdm($article) {
-		$tags = array_map("trim", explode(",", $this->host->get($this, "tags")));
-		$a_tags = array_map("trim", explode(",", $article["tag_cache"]));
-
-		if (count(array_intersect($tags, $a_tags)) > 0) {
-			$article["content"] = "<div class='nswf wrapper'><button onclick=\"nsfwShow(this)\">".__("Not work safe (click to toggle)")."</button>
-				<div class='nswf content' style='display : none'>".$article["content"]."</div></div>";
-		}
-
-		return $article;
+		return $this->hook_render_article($article);
 	}
 
 	function hook_prefs_tab($args) {
 		if ($args != "prefPrefs") return;
 
-		print "<div dojoType=\"dijit.layout.AccordionPane\" 
-			title=\"<i class='material-icons'>extension</i> ".__("NSFW Plugin")."\">";
-
-		print "<br/>";
-
 		$tags = $this->host->get($this, "tags");
 
-		print "<form dojoType=\"dijit.form.Form\">";
-
-		print "<script type=\"dojo/method\" event=\"onSubmit\" args=\"evt\">
-			evt.preventDefault();
-			if (this.validate()) {
-				new Ajax.Request('backend.php', {
-					parameters: dojo.objectToQuery(this.getValues()),
-					onComplete: function(transport) {
-						Notify.info(transport.responseText);
+		?>
+		<div dojoType="dijit.layout.AccordionPane"
+			title="<i class='material-icons'>extension</i> <?= __("NSFW Plugin") ?>">
+			<form dojoType="dijit.form.Form">
+
+				<?= \Controls\hidden_tag("op", "pluginhandler") ?>
+				<?= \Controls\hidden_tag("method", "save") ?>
+				<?= \Controls\hidden_tag("plugin", "nsfw") ?>
+
+				<script type="dojo/method" event="onSubmit" args="evt">
+					evt.preventDefault();
+					if (this.validate()) {
+						new Ajax.Request('backend.php', {
+							parameters: dojo.objectToQuery(this.getValues()),
+							onComplete: function(transport) {
+								Notify.info(transport.responseText);
+							}
+						});
 					}
-				});
-				//this.reset();
-			}
-			</script>";
-
-			print \Controls\hidden_tag("op", "pluginhandler");
-			print \Controls\hidden_tag("method", "save");
-			print \Controls\hidden_tag("plugin", "nsfw");
-
-			print "<table width=\"100%\" class=\"prefPrefsList\">";
-
-			print "<tr><td width=\"40%\">".__("Tags to consider NSFW (comma-separated)")."</td>";
-			print "<td class=\"prefValue\"><input dojoType=\"dijit.form.ValidationTextBox\" required=\"1\" name=\"tags\" value=\"$tags\"></td></tr>";
+				</script>
 
-			print "</table>";
+				<header><?= __("Tags to consider NSFW (comma-separated):") ?></header>
 
-			print "<p><button dojoType=\"dijit.form.Button\" type=\"submit\">".
-				__("Save")."</button>";
+				<fieldset>
+					<textarea dojoType='dijit.form.SimpleTextarea' rows='4'
+							style='width: 500px; font-size : 12px;'
+							name='tags'><?= $tags ?></textarea>
+				</fieldset>
 
-			print "</form>";
+				<hr/>
 
-			print "</div>"; #pane
+				<?= \Controls\submit_tag(__("Save")) ?>
+			</form>
+		</div>
+		<?php
 	}
 
 	function save() {
-		$tags = explode(",", $_POST["tags"]);
-		$tags = array_map("trim", $tags);
-		$tags = array_map("mb_strtolower", $tags);
-		$tags = join(", ", $tags);
+		$tags = implode(", ",
+			FeedItem_Common::normalize_categories(explode(",", $_POST["tags"] ?? "")));
 
 		$this->host->set($this, "tags", $tags);