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