You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@roller.apache.org by sn...@apache.org on 2007/07/30 23:43:33 UTC
svn commit: r561138 -
/roller/trunk/apps/weblogger/src/java/org/apache/roller/weblogger/business/jpa/JPAWeblogManagerImpl.java
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) {