Remove some IS_SERVER() checks

- InspIRCd::FindUUID() and FindNick() no longer return FakeUsers so checking the user returned from those methods is needless
- m_dccallow is calling FindNickOnly() so it had the check needlessly

3 files changed, 3 insertions, 3 deletions

diff --git a/src/modules/m_spanningtree/idle.cpp b/src/modules/m_spanningtree/idle.cpp
index 06af4d0fd..ad58e52f0 100644
--- a/src/modules/m_spanningtree/idle.cpp
+++ b/src/modules/m_spanningtree/idle.cpp
@@ -35,7 +35,7 @@ CmdResult CommandIdle::HandleRemote(RemoteUser* issuer, std::vector<std::string>
 	 */
 
 	User* target = ServerInstance->FindUUID(params[0]);
-	if ((!target) || (IS_SERVER(target) || (target->registered != REG_ALL)))
+	if ((!target) || (target->registered != REG_ALL))
 		return CMD_FAILURE;
 
 	LocalUser* localtarget = IS_LOCAL(target);
diff --git a/src/modules/m_spanningtree/metadata.cpp b/src/modules/m_spanningtree/metadata.cpp
index f758754b4..47c2f8bc5 100644
--- a/src/modules/m_spanningtree/metadata.cpp
+++ b/src/modules/m_spanningtree/metadata.cpp
@@ -56,7 +56,7 @@ CmdResult CommandMetadata::Handle(User* srcuser, std::vector<std::string>& param
 	else
 	{
 		User* u = ServerInstance->FindUUID(params[0]);
-		if ((u) && (!IS_SERVER(u)))
+		if (u)
 		{
 			ExtensionItem* item = ServerInstance->Extensions.GetItem(params[1]);
 			std::string value = params.size() < 3 ? "" : params[2];
diff --git a/src/modules/m_spanningtree/save.cpp b/src/modules/m_spanningtree/save.cpp
index a382b8d66..7131b49fe 100644
--- a/src/modules/m_spanningtree/save.cpp
+++ b/src/modules/m_spanningtree/save.cpp
@@ -29,7 +29,7 @@
 CmdResult CommandSave::Handle(User* user, std::vector<std::string>& params)
 {
 	User* u = ServerInstance->FindUUID(params[0]);
-	if ((!u) || (IS_SERVER(u)))
+	if (!u)
 		return CMD_FAILURE;
 
 	time_t ts = atol(params[1].c_str());