Author: rdonkin
Date: Mon May 11 09:46:30 2009
New Revision: 773513

URL: http://svn.apache.org/viewvc?rev=773513&view=rev
Log:
Default just in case no one read the javadocs. IMAP-72 
https://issues.apache.org/jira/browse/IMAP-72

Modified:
    
james/imap/trunk/decode/src/main/java/org/apache/james/imap/decode/DecodingException.java
    
james/imap/trunk/mailbox/src/main/java/org/apache/james/imap/mailbox/MailboxException.java

Modified: 
james/imap/trunk/decode/src/main/java/org/apache/james/imap/decode/DecodingException.java
URL: 
http://svn.apache.org/viewvc/james/imap/trunk/decode/src/main/java/org/apache/james/imap/decode/DecodingException.java?rev=773513&r1=773512&r2=773513&view=diff
==============================================================================
--- 
james/imap/trunk/decode/src/main/java/org/apache/james/imap/decode/DecodingException.java
 (original)
+++ 
james/imap/trunk/decode/src/main/java/org/apache/james/imap/decode/DecodingException.java
 Mon May 11 09:46:30 2009
@@ -69,6 +69,13 @@
      * @return the key, not null
      */
     public final HumanReadableTextKey getKey() {
+        final HumanReadableTextKey key;
+        if (this.key == null) {
+            // API specifies not null but best to default to generic message 
+            key = HumanReadableTextKey.ILLEGAL_ARGUMENTS;
+        } else {
+            key = this.key;
+        }
         return key;
     }
 

Modified: 
james/imap/trunk/mailbox/src/main/java/org/apache/james/imap/mailbox/MailboxException.java
URL: 
http://svn.apache.org/viewvc/james/imap/trunk/mailbox/src/main/java/org/apache/james/imap/mailbox/MailboxException.java?rev=773513&r1=773512&r2=773513&view=diff
==============================================================================
--- 
james/imap/trunk/mailbox/src/main/java/org/apache/james/imap/mailbox/MailboxException.java
 (original)
+++ 
james/imap/trunk/mailbox/src/main/java/org/apache/james/imap/mailbox/MailboxException.java
 Mon May 11 09:46:30 2009
@@ -52,6 +52,13 @@
      * @return the key, possibly null
      */
     public final HumanReadableTextKey getKey() {
+        final HumanReadableTextKey key;
+        if (this.key == null) {
+            // API specifies not null but best to default to generic message 
+            key = HumanReadableTextKey.GENERIC_FAILURE_DURING_PROCESSING;
+        } else {
+            key = this.key;
+        }
         return key;
     }
 }



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to