opendev/gerrit
Code Issues Proposed changes

Migration v57 setInt parameter trivial fix

Browse Source 
The first parameter for setInt in sql API starts from 1.
See: http://docs.oracle.com/javase/6/docs/api/java/sql/PreparedStatement.html#setInt(int,int)

Change-Id: I725e9ea30e78a5b0adaa1b3b58bf30e02cd3d759
This commit is contained in:
Levente Tamas
2013-04-08 17:40:04 +03:00
committed by Shawn Pearce
parent ca4db4eea0
commit fab4ab6153
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-server/src/main/java/com/google/gerrit/server/schema/Schema_57.java
View File

@@ -125,7 +125,7 @@ public class Schema_57 extends SchemaVersion {
} }
AccountGroup batch = db.accountGroups().get(sc.batchUsersGroupId); AccountGroup batch = db.accountGroups().get(sc.batchUsersGroupId);
stmt.setInt(0, sc.batchUsersGroupId.get()); stmt.setInt(1, sc.batchUsersGroupId.get());
if (batch != null if (batch != null
&& db.accountGroupMembers().byGroup(sc.batchUsersGroupId).toList().isEmpty() && db.accountGroupMembers().byGroup(sc.batchUsersGroupId).toList().isEmpty()
&& stmt.executeQuery().first() != false) { && stmt.executeQuery().first() != false) {