Fixed peer-reveiewed comments, such as:
in GetUserBlogs(), I had a return statement on the inner loop, not after all blogs were added to list. Verified that the proper XML RPC URL was saved, so as to not confuse blogs using the same multisite installation. Adjusted the exception thrown to mark the proper call method.
This commit is contained in:
parent
5a7ba1d8f3
commit
7a0976d3f8
|
@ -84,18 +84,17 @@ namespace OpenLiveWriter.BlogClient.Clients
|
|||
// add to our list of blogs
|
||||
blogs.Add(new BlogInfo(idNode.InnerText, HttpUtility.HtmlDecode(NodeToText(nameNode)), urlNode.InnerText));
|
||||
}
|
||||
|
||||
// return list of blogs
|
||||
return (BlogInfo[])blogs.ToArray(typeof(BlogInfo));
|
||||
}
|
||||
|
||||
// return list of blogs
|
||||
return (BlogInfo[])blogs.ToArray(typeof(BlogInfo));
|
||||
}
|
||||
catch (Exception ex)
|
||||
{
|
||||
string response = result != null ? result.OuterXml : "(empty response)";
|
||||
Trace.Fail("Exception occurred while parsing GetUsersBlogs response: " + response + "\r\n" + ex.ToString());
|
||||
throw new BlogClientInvalidServerResponseException("blogger.getUsersBlogs", ex.Message, response);
|
||||
throw new BlogClientInvalidServerResponseException("wp.getUsersBlogs", ex.Message, response);
|
||||
}
|
||||
return null;
|
||||
}
|
||||
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue