Author: norman
Date: Tue Dec  7 10:18:13 2010
New Revision: 1042959

URL: http://svn.apache.org/viewvc?rev=1042959&view=rev
Log:
Correctly calculate the last uid when messages already exist in the mailbox

Modified:
    
james/mailbox/trunk/jpa/src/main/java/org/apache/james/mailbox/jpa/mail/JPACachingUidProvider.java
    
james/mailbox/trunk/jpa/src/main/java/org/apache/james/mailbox/jpa/mail/model/openjpa/AbstractJPAMailboxMembership.java

Modified: 
james/mailbox/trunk/jpa/src/main/java/org/apache/james/mailbox/jpa/mail/JPACachingUidProvider.java
URL: 
http://svn.apache.org/viewvc/james/mailbox/trunk/jpa/src/main/java/org/apache/james/mailbox/jpa/mail/JPACachingUidProvider.java?rev=1042959&r1=1042958&r2=1042959&view=diff
==============================================================================
--- 
james/mailbox/trunk/jpa/src/main/java/org/apache/james/mailbox/jpa/mail/JPACachingUidProvider.java
 (original)
+++ 
james/mailbox/trunk/jpa/src/main/java/org/apache/james/mailbox/jpa/mail/JPACachingUidProvider.java
 Tue Dec  7 10:18:13 2010
@@ -50,7 +50,7 @@ public class JPACachingUidProvider exten
         EntityManager em = factory.createEntityManager();
         try {
             em.getTransaction().begin();
-            final long uid = (Long) 
em.createNamedQuery("findLastUidInMailbox").setParameter("idParam", 
mailbox.getMailboxId()).getSingleResult();
+            final long uid = (Long) 
em.createNamedQuery("findLastUidInMailbox").setParameter("idParam", 
mailbox.getMailboxId()).setMaxResults(1).getSingleResult();
             em.getTransaction().commit();
             return uid;
         } catch (NoResultException e) {

Modified: 
james/mailbox/trunk/jpa/src/main/java/org/apache/james/mailbox/jpa/mail/model/openjpa/AbstractJPAMailboxMembership.java
URL: 
http://svn.apache.org/viewvc/james/mailbox/trunk/jpa/src/main/java/org/apache/james/mailbox/jpa/mail/model/openjpa/AbstractJPAMailboxMembership.java?rev=1042959&r1=1042958&r2=1042959&view=diff
==============================================================================
--- 
james/mailbox/trunk/jpa/src/main/java/org/apache/james/mailbox/jpa/mail/model/openjpa/AbstractJPAMailboxMembership.java
 (original)
+++ 
james/mailbox/trunk/jpa/src/main/java/org/apache/james/mailbox/jpa/mail/model/openjpa/AbstractJPAMailboxMembership.java
 Tue Dec  7 10:18:13 2010
@@ -68,7 +68,7 @@ import org.apache.openjpa.persistence.jd
     @NamedQuery(name="deleteMessages",
             query="DELETE FROM Membership membership WHERE 
membership.mailboxId = :idParam"),
     @NamedQuery(name="findLastUidInMailbox",
-            query="SELECT membership.uid FROM Membership membership WHERE 
membership.mailboxId = :idParam ORDER BY membership.uid ASC")
+            query="SELECT membership.uid FROM Membership membership WHERE 
membership.mailboxId = :idParam ORDER BY membership.uid DESC")
 })
 public abstract class AbstractJPAMailboxMembership extends 
AbstractMailboxMembership<Long> {
 



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

Reply via email to