diff --git a/mailnews/base/src/nsMsgAccountManagerDS.cpp b/mailnews/base/src/nsMsgAccountManagerDS.cpp
index feb41d71c1a..b7a12ad45d5 100644
--- a/mailnews/base/src/nsMsgAccountManagerDS.cpp
+++ b/mailnews/base/src/nsMsgAccountManagerDS.cpp
@@ -801,14 +801,14 @@ nsMsgAccountManagerDataSource::createSettingsResources(nsIRDFResource *aSource,
       aNodeArray->AppendElement(kNC_PageTitleServer);
       aNodeArray->AppendElement(kNC_PageTitleCopies);
       aNodeArray->AppendElement(kNC_PageTitleAddressing);
-      // junk settings apply for all server types except for news
-      // Should this be moved outside of the hasIdentities check ??
-      nsXPIDLCString serverType;
-      server->GetType(getter_Copies(serverType));
-      if (nsCRT::strcasecmp(serverType, "nntp"))
-        aNodeArray->AppendElement(kNC_PageTitleJunk);
     }
 
+    // junk settings apply for all server types except for news
+    nsXPIDLCString serverType;
+    server->GetType(getter_Copies(serverType));
+    if (!serverType.LowerCaseEqualsLiteral("nntp"))
+      aNodeArray->AppendElement(kNC_PageTitleJunk);
+
     // Check the offline capability before adding
     // offline item
     PRInt32 offlineSupportLevel = 0;