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 no...@apache.org on 2009/12/04 17:28:37 UTC

svn commit: r887256 - in /james/hupa/trunk/server/src: main/java/org/apache/hupa/server/handler/FetchMessagesHandler.java test/java/org/apache/hupa/server/handler/FetchMessagesHandlerTest.java

Author: norman
Date: Fri Dec  4 16:28:36 2009
New Revision: 887256

URL: http://svn.apache.org/viewvc?rev=887256&view=rev
Log:
fix bug in index caculation when searching in messages

Modified:
    james/hupa/trunk/server/src/main/java/org/apache/hupa/server/handler/FetchMessagesHandler.java
    james/hupa/trunk/server/src/test/java/org/apache/hupa/server/handler/FetchMessagesHandlerTest.java

Modified: james/hupa/trunk/server/src/main/java/org/apache/hupa/server/handler/FetchMessagesHandler.java
URL: http://svn.apache.org/viewvc/james/hupa/trunk/server/src/main/java/org/apache/hupa/server/handler/FetchMessagesHandler.java?rev=887256&r1=887255&r2=887256&view=diff
==============================================================================
--- james/hupa/trunk/server/src/main/java/org/apache/hupa/server/handler/FetchMessagesHandler.java (original)
+++ james/hupa/trunk/server/src/main/java/org/apache/hupa/server/handler/FetchMessagesHandler.java Fri Dec  4 16:28:36 2009
@@ -99,10 +99,7 @@
             }
             exists = tmpMessages.length;
 
-            int firstIndex = exists - end;
-            if (firstIndex < 1) {
-                firstIndex = 1;
-            }            
+            int firstIndex = exists - end;        
             
             if (tmpMessages.length > firstIndex) {
                 List<Message> mList = new ArrayList<Message>();

Modified: james/hupa/trunk/server/src/test/java/org/apache/hupa/server/handler/FetchMessagesHandlerTest.java
URL: http://svn.apache.org/viewvc/james/hupa/trunk/server/src/test/java/org/apache/hupa/server/handler/FetchMessagesHandlerTest.java?rev=887256&r1=887255&r2=887256&view=diff
==============================================================================
--- james/hupa/trunk/server/src/test/java/org/apache/hupa/server/handler/FetchMessagesHandlerTest.java (original)
+++ james/hupa/trunk/server/src/test/java/org/apache/hupa/server/handler/FetchMessagesHandlerTest.java Fri Dec  4 16:28:36 2009
@@ -82,11 +82,11 @@
         ByteArrayInputStream is = new ByteArrayInputStream("From: a@foo.com\nTo: b@foo.com\nSubject: something\n\ndata".getBytes());
         MimeMessage msg = new MimeMessage(session, is);
         serverfolder.addMessages(new Message[]{msg});
-        
-        result = handler.execute(new FetchMessages(clientfolder, 0, 10, "*"), null);
+        result = handler.execute(new FetchMessages(clientfolder, 0, 10, "something"), null);
         assertEquals(1, result.getRealCount());
         assertEquals(1, result.getMessages().size());
         
+        
         result = handler.execute(new FetchMessages(clientfolder, 0, 10, null), null);
         assertEquals(1, result.getRealCount());
         assertEquals(1, result.getMessages().size());



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