You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@roller.apache.org by sn...@apache.org on 2008/07/20 22:07:56 UTC

svn commit: r678318 - /roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/webservices/xmlrpc/MetaWeblogAPIHandler.java

Author: snoopdave
Date: Sun Jul 20 13:07:56 2008
New Revision: 678318

URL: http://svn.apache.org/viewvc?rev=678318&view=rev
Log:
Testing with ROME v1.0RC1. Improved error message for invalid post in getpost method.

Modified:
    roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/webservices/xmlrpc/MetaWeblogAPIHandler.java

Modified: roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/webservices/xmlrpc/MetaWeblogAPIHandler.java
URL: http://svn.apache.org/viewvc/roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/webservices/xmlrpc/MetaWeblogAPIHandler.java?rev=678318&r1=678317&r2=678318&view=diff
==============================================================================
--- roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/webservices/xmlrpc/MetaWeblogAPIHandler.java (original)
+++ roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/webservices/xmlrpc/MetaWeblogAPIHandler.java Sun Jul 20 13:07:56 2008
@@ -323,13 +323,16 @@
         WeblogEntryManager weblogMgr = roller.getWeblogEntryManager();
         WeblogEntry entry = weblogMgr.getWeblogEntry(postid);
         
-        validate(entry.getWebsite().getHandle(), userid,password);
+        if (entry == null) {
+            throw new XmlRpcException(INVALID_POSTID, INVALID_POSTID_MSG);
+        }
+        validate(entry.getWebsite().getHandle(), userid, password);
         
         try {
             return createPostStruct(entry, userid);
         } catch (Exception e) {
             String msg = "ERROR in MetaWeblogAPIHandler.getPost";
-            mLogger.error(msg,e);
+            mLogger.error(msg, e);
             throw new XmlRpcException(UNKNOWN_EXCEPTION, msg);
         }
     }