From dcfea9baacc41984f683342e6a3e4949243b7324 Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <noreply@fakecake.org>
Date: Fri, 19 Feb 2021 06:51:15 +0300
Subject: [PATCH] properly validate feed editor dialog

---
 classes/pref/feeds.php | 2 +-
 js/CommonDialogs.js    | 2 ++
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/classes/pref/feeds.php b/classes/pref/feeds.php
index 3d633a378..761a0a087 100755
--- a/classes/pref/feeds.php
+++ b/classes/pref/feeds.php
@@ -803,7 +803,7 @@ class Pref_Feeds extends Handler_Protected {
 				<button style='float : left' class='alt-danger' dojoType='dijit.form.Button'
 					onclick='App.dialogOf(this).unsubscribeFeed($feed_id, \"$title\")'>".
 					__('Unsubscribe')."</button>
-				<button dojoType='dijit.form.Button' class='alt-primary' type='submit'>".__('Save')."</button>
+				<button dojoType='dijit.form.Button' class='alt-primary' onclick='return App.dialogOf(this).execute()' type='submit'>".__('Save')."</button>
 				<button dojoType='dijit.form.Button' onclick='App.dialogOf(this).hide()'>".__('Cancel')."</button>
 				</footer>";
 		}
diff --git a/js/CommonDialogs.js b/js/CommonDialogs.js
index 40313d34f..c62e6893e 100644
--- a/js/CommonDialogs.js
+++ b/js/CommonDialogs.js
@@ -425,7 +425,9 @@ const	CommonDialogs = {
 								Feeds.reload();
 
 						});
+						return true;
 					}
+					return false;
 				},
 				content: __("Loading, please wait...")
 			});