You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by rd...@apache.org on 2008/12/22 20:56:13 UTC

svn commit: r728757 - /james/jsieve/trunk/mailet/src/main/java/org/apache/jsieve/mailet/SieveMailboxMailet.java

Author: rdonkin
Date: Mon Dec 22 11:56:12 2008
New Revision: 728757

URL: http://svn.apache.org/viewvc?rev=728757&view=rev
Log:
Clearly and cleanly fail on init() to indicate that mailet requires locator.

Modified:
    james/jsieve/trunk/mailet/src/main/java/org/apache/jsieve/mailet/SieveMailboxMailet.java

Modified: james/jsieve/trunk/mailet/src/main/java/org/apache/jsieve/mailet/SieveMailboxMailet.java
URL: http://svn.apache.org/viewvc/james/jsieve/trunk/mailet/src/main/java/org/apache/jsieve/mailet/SieveMailboxMailet.java?rev=728757&r1=728756&r2=728757&view=diff
==============================================================================
--- james/jsieve/trunk/mailet/src/main/java/org/apache/jsieve/mailet/SieveMailboxMailet.java (original)
+++ james/jsieve/trunk/mailet/src/main/java/org/apache/jsieve/mailet/SieveMailboxMailet.java Mon Dec 22 11:56:12 2008
@@ -325,7 +325,7 @@
      */
     public void init() throws MessagingException {
         super.init();
-        if (poster == null) {
+        if (poster == null || locator == null) {
             throw new MailetException("Not initialised. Please ensure that the mailet container supports either" +
                     " setter or constructor injection");
         }



---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org