rework a few more dialogs to use App.dialogOf()

4 files changed, 12 insertions, 13 deletions

diff --git a/classes/article.php b/classes/article.php
index 305764ed3..6d3746968 100755
--- a/classes/article.php
+++ b/classes/article.php
@@ -184,7 +184,7 @@ class Article extends Handler_Protected {
 		print "<button dojoType='dijit.form.Button'
 			type='submit' class='alt-primary'>".__('Save')."</button> ";
 		print "<button dojoType='dijit.form.Button'
-			onclick=\"dijit.byId('editTagsDlg').hide()\">".__('Cancel')."</button>";
+			onclick='App.dialogOf(this).hide()'>".__('Cancel')."</button>";
 		print "</footer>";
 
 	}
diff --git a/classes/backend.php b/classes/backend.php
index 6e0fa191e..aa1935f23 100644
--- a/classes/backend.php
+++ b/classes/backend.php
@@ -83,8 +83,7 @@ class Backend extends Handler_Protected {
 		}
 
 		print "<footer class='text-center'>";
-		print "<button dojoType='dijit.form.Button'
-			onclick=\"return dijit.byId('helpDlg').hide()\">".__('Close this window')."</button>";
+		print "<button dojoType='dijit.form.Button' class='alt-primary' type='submit'>".__('Close this window')."</button>";
 		print "</footer>";
 
 	}
diff --git a/classes/feeds.php b/classes/feeds.php
index e8c3bcf2c..d995b4be1 100755
--- a/classes/feeds.php
+++ b/classes/feeds.php
@@ -730,9 +730,9 @@ class Feeds extends Handler_Protected {
 
 		print "<footer>";
 		print "<button dojoType='dijit.form.Button' class='alt-primary' type='submit'
-				onclick=\"return dijit.byId('feedAddDlg').execute()\">".__('Subscribe')."</button>";
+				onclick=\"App.dialogOf(this).execute()\">".__('Subscribe')."</button>";
 
-		print "<button dojoType='dijit.form.Button' onclick=\"return dijit.byId('feedAddDlg').hide()\">".__('Cancel')."</button>";
+		print "<button dojoType='dijit.form.Button' onclick=\"App.dialogOf(this).hide()\">".__('Cancel')."</button>";
 		print "</footer>";
 
 		print "</form>";
@@ -744,7 +744,7 @@ class Feeds extends Handler_Protected {
 		$active_feed_id = sprintf("%d", $this->params[0]);
 		$is_cat = $this->params[1] != "false";
 
-		print "<form onsubmit='return false;'>";
+		print "<form onsubmit='return false'>";
 
 		print "<section>";
 
@@ -772,8 +772,9 @@ class Feeds extends Handler_Protected {
 				<i class='material-icons'>help</i> ".__("Search syntax")."</button>";
 		}
 
-		print "<button dojoType='dijit.form.Button' type='submit' class='alt-primary' onclick=\"dijit.byId('searchDlg').execute()\">".__('Search')."</button>
-			<button dojoType='dijit.form.Button' onclick=\"dijit.byId('searchDlg').hide()\">".__('Cancel')."</button>";
+		print "<button dojoType='dijit.form.Button' class='alt-primary'
+				type='submit' onclick='App.dialogOf(this).execute()'>".__('Search')."</button>
+			<button dojoType='dijit.form.Button' onclick='App.dialogOf(this).hide()'>".__('Cancel')."</button>";
 
 		print "</footer>";
 
diff --git a/classes/pref/feeds.php b/classes/pref/feeds.php
index 5fcafeaae..06e0e4c3d 100755
--- a/classes/pref/feeds.php
+++ b/classes/pref/feeds.php
@@ -809,8 +809,8 @@ class Pref_Feeds extends Handler_Protected {
 			print "<footer>
 				<button style='float : left' class='alt-danger' dojoType='dijit.form.Button' onclick='return CommonDialogs.unsubscribeFeed($feed_id, \"$title\")'>".
 					__('Unsubscribe')."</button>
-				<button dojoType='dijit.form.Button' class='alt-primary' onclick=\"return dijit.byId('feedEditDlg').execute()\">".__('Save')."</button>
-				<button dojoType='dijit.form.Button' onclick=\"return dijit.byId('feedEditDlg').hide()\">".__('Cancel')."</button>
+				<button dojoType='dijit.form.Button' class='alt-primary' type='submit'>".__('Save')."</button>
+				<button dojoType='dijit.form.Button' onclick='App.dialogOf(this).hide()'>".__('Cancel')."</button>
 				</footer>";
 		}
 	}
@@ -972,11 +972,10 @@ class Pref_Feeds extends Handler_Protected {
 		print "</section>";
 
 		print "<footer>
-			<button dojoType='dijit.form.Button' type='submit' class='alt-primary'
-				onclick=\"return dijit.byId('feedEditDlg').execute()\">".
+			<button dojoType='dijit.form.Button' type='submit' class='alt-primary' type='submit'>".
 				__('Save')."</button>
 			<button dojoType='dijit.form.Button'
-			onclick=\"return dijit.byId('feedEditDlg').hide()\">".
+			onclick=\"App.dialogOf(this).hide()\">".
 				__('Cancel')."</button>
 			</footer>";