From 3a015eb8e390f776bb4b2f1ae71930f0a6b4b6f3 Mon Sep 17 00:00:00 2001
From: Julian Lam <julian@nodebb.org>
Date: Wed, 15 Jun 2022 13:37:54 -0400
Subject: [PATCH] fix: no need to pass in empty Set in deprecated hooks

---
 src/plugins/hooks.js | 7 +++----
 src/plugins/index.js | 2 +-
 2 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/src/plugins/hooks.js b/src/plugins/hooks.js
index b240652a84..de00a6755c 100644
--- a/src/plugins/hooks.js
+++ b/src/plugins/hooks.js
@@ -12,25 +12,21 @@ Hooks._deprecated = new Map([
 		new: 'static:email.send',
 		since: 'v1.17.0',
 		until: 'v2.0.0',
-		affected: new Set(),
 	}],
 	['filter:router.page', {
 		new: 'response:router.page',
 		since: 'v1.15.3',
 		until: 'v2.1.0',
-		affected: new Set(),
 	}],
 	['filter:post.purge', {
 		new: 'filter:posts.purge',
 		since: 'v1.19.6',
 		until: 'v2.1.0',
-		affected: new Set(),
 	}],
 	['action:post.purge', {
 		new: 'action:posts.purge',
 		since: 'v1.19.6',
 		until: 'v2.1.0',
-		affected: new Set(),
 	}],
 ]);
 
@@ -63,6 +59,9 @@ Hooks.register = function (id, data) {
 	// `hasOwnProperty` needed for hooks with no alternative (set to null)
 	if (Hooks._deprecated.has(data.hook)) {
 		const deprecation = Hooks._deprecated.get(data.hook);
+		if (!deprecation.hasOwnProperty('affected')) {
+			deprecation.affected = new Set();
+		}
 		deprecation.affected.add(id);
 		Hooks._deprecated.set(data.hook, deprecation);
 	}
diff --git a/src/plugins/index.js b/src/plugins/index.js
index a84eb40ddc..d1a3bd92bf 100644
--- a/src/plugins/index.js
+++ b/src/plugins/index.js
@@ -129,7 +129,7 @@ Plugins.reload = async function () {
 
 	// Deprecation notices
 	Plugins.hooks._deprecated.forEach((deprecation, hook) => {
-		if (!deprecation.affected.size) {
+		if (!deprecation.affected || !deprecation.affected.size) {
 			return;
 		}