Author: snoopdave
Date: Mon Jul 30 14:43:32 2007
New Revision: 561138
URL: http://svn.apache.org/viewvc?view=rev&rev=561138
Log:
Fixes http://opensource.atlassian.com/projects/roller/browse/ROL-1506
We were not adding index to parameter substitution points in JPQL query for
comments
Modified:
roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/jpa/JPAWeblogManagerImpl.java
Modified:
roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/jpa/JPAWeblogManagerImpl.java
URL:
http://svn.apache.org/viewvc/roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/jpa/JPAWeblogManagerImpl.java?view=diff&rev=561138&r1=561137&r2=561138
==============================================================================
---
roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/jpa/JPAWeblogManagerImpl.java
(original)
+++
roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/jpa/JPAWeblogManagerImpl.java
Mon Jul 30 14:43:32 2007
@@ -859,19 +859,19 @@
if (searchString != null) {
params.add(size++, "%" + searchString + "%");
appendConjuctionToWhereclause(whereClause, "(c.url LIKE ?")
- .append(size).append(" OR c.content LIKE
?").append(size).append(")");
+ .append(size).append(" OR c.content LIKE
?").append(size).append(")");
}
if (startDate != null) {
Timestamp start = new Timestamp(startDate.getTime());
params.add(size++, start);
- appendConjuctionToWhereclause(whereClause, "c.postTime >= ?");
+ appendConjuctionToWhereclause(whereClause, "c.postTime >=
?").append(size);
}
if (endDate != null) {
Timestamp end = new Timestamp(endDate.getTime());
params.add(size++, end);
- appendConjuctionToWhereclause(whereClause, "c.postTime <= ?");
+ appendConjuctionToWhereclause(whereClause, "c.postTime <=
?").append(size);
}
if (status != null) {
@@ -886,7 +886,7 @@
}
params.add(size++, status);
appendConjuctionToWhereclause(whereClause, "c.status ")
- .append(comparisionOperator).append('?').append(size);
+ .append(comparisionOperator).append('?').append(size);
}
if(whereClause.length() != 0) {
