fix for #22345. we were unescaping a message id when we shouldn't be.

a=sdagley

mailnews/news/public/nsINntpUrl.idl

@@ -63,4 +63,5 @@ interface nsINntpUrl : nsISupports {
   const long ActionSaveMessageToDisk = 2;
   const long ActionCancelArticle = 3;
   const long ActionPostArticle = 4;
+  const long ActionSearch = 5;
 };

mailnews/news/src/nsNNTPProtocol.cpp

@@ -928,7 +928,11 @@ nsresult nsNNTPProtocol::ParseURL(nsIURI * aURL, PRBool * bValP, char ** aGroup,
 
 	nsAllocator::Free(fullPath);
 
-	nsUnescape (group);
+	// more to do here, but for now, this works.
+	// only escape if we are doing a search
+	if (m_newsAction == nsINntpUrl::ActionSearch) { 
+		nsUnescape (group);
+	}
 
     /* "group" now holds the part after the host name:
 	 "message@id?search" or "/group/xxx?search" or "/message?id@xx?search"