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 Harmeet Bedi <hb...@yahoo.com> on 2001/03/26 01:27:44 UTC

bug in JamesSpoolManager.

line 197 of JamesSpoolManager sets the
mail.setState("Mail.ERROR");
This seems to be bug, it should be
mail.setState(Mail.ERROR);

Still waiting for cvs access. Someone else would need to checkin this
fix.:-(.
Sending it around in the hope that someone checks this in and noone gets
bitten by this bug.

Harmeet



_________________________________________________________
Do You Yahoo!?
Get your free @yahoo.com address at http://mail.yahoo.com


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


Re: bug in JamesSpoolManager.

Posted by Harmeet Bedi <hb...@yahoo.com>.
Found the string "Mail.ERROR" in one more place, again n JamesSpoolManager.
These changes seem to fix the bug.

@@ -188,13 +188,13 @@
                 //This is a strange error situation that shouldn't
ordinarily happen
                 System.err.println("Exception in processor <" +
processorName + ">");
                 e.printStackTrace();
-                if (processorName.equals("Mail.ERROR")) {
+                if (processorName.equals(Mail.ERROR)) {
                     //We got an error on the error processor... kill the
message
                     mail.setState(Mail.GHOST);
                     mail.setErrorMessage(e.getMessage());
                 } else {
                     //We got an error... send it to the error processor
-                    mail.setState("Mail.ERROR");
+                    mail.setState(Mail.ERROR);
                     mail.setErrorMessage(e.getMessage());
                 }
             }

Harmeet
----- Original Message -----
From: "Harmeet Bedi" <hb...@yahoo.com>
To: <ja...@jakarta.apache.org>
Sent: Sunday, March 25, 2001 3:27 PM
Subject: bug in JamesSpoolManager.


> line 197 of JamesSpoolManager sets the
> mail.setState("Mail.ERROR");
> This seems to be bug, it should be
> mail.setState(Mail.ERROR);
>
> Still waiting for cvs access. Someone else would need to checkin this
> fix.:-(.
> Sending it around in the hope that someone checks this in and noone gets
> bitten by this bug.
>
> Harmeet
>
>
>
> _________________________________________________________
> Do You Yahoo!?
> Get your free @yahoo.com address at http://mail.yahoo.com
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: james-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: james-dev-help@jakarta.apache.org


_________________________________________________________
Do You Yahoo!?
Get your free @yahoo.com address at http://mail.yahoo.com


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