Skip to content

Commit 443acac

Browse files
committed
Fix functional issue introduced by some findbug changes.
1 parent 97bad4f commit 443acac

1 file changed

Lines changed: 14 additions & 14 deletions

File tree

framework/db/src/com/cloud/utils/db/GenericDaoBase.java

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -554,7 +554,7 @@ protected void setField(Object entity, Field field, ResultSet rs, int index) thr
554554
final Enum<?>[] enums = (Enum<?>[])field.getType().getEnumConstants();
555555
for (final Enum<?> e : enums) {
556556
if ((enumType == EnumType.STRING && e.name().equalsIgnoreCase(rs.getString(index))) ||
557-
(enumType == EnumType.ORDINAL && e.ordinal() == rs.getInt(index))) {
557+
(enumType == EnumType.ORDINAL && e.ordinal() == rs.getInt(index))) {
558558
field.set(entity, e);
559559
return;
560560
}
@@ -768,7 +768,7 @@ protected static <M> M getObject(Class<M> type, ResultSet rs, int index) throws
768768
protected int addJoinAttributes(int count, PreparedStatement pstmt, Collection<JoinBuilder<SearchCriteria<?>>> joins) throws SQLException {
769769
for (JoinBuilder<SearchCriteria<?>> join : joins) {
770770
for (final Pair<Attribute, Object> value : join.getT().getValues()) {
771-
prepareAttribute(++count, pstmt, value.first(), value.second());
771+
prepareAttribute(count++, pstmt, value.first(), value.second());
772772
}
773773
}
774774

@@ -1255,18 +1255,18 @@ protected void addJoins(StringBuilder str, Collection<JoinBuilder<SearchCriteria
12551255
for (JoinBuilder<SearchCriteria<?>> join : joins) {
12561256
StringBuilder onClause = new StringBuilder();
12571257
onClause.append(" ")
1258-
.append(join.getType().getName())
1259-
.append(" ")
1260-
.append(join.getSecondAttribute().table)
1261-
.append(" ON ")
1262-
.append(join.getFirstAttribute().table)
1263-
.append(".")
1264-
.append(join.getFirstAttribute().columnName)
1265-
.append("=")
1266-
.append(join.getSecondAttribute().table)
1267-
.append(".")
1268-
.append(join.getSecondAttribute().columnName)
1269-
.append(" ");
1258+
.append(join.getType().getName())
1259+
.append(" ")
1260+
.append(join.getSecondAttribute().table)
1261+
.append(" ON ")
1262+
.append(join.getFirstAttribute().table)
1263+
.append(".")
1264+
.append(join.getFirstAttribute().columnName)
1265+
.append("=")
1266+
.append(join.getSecondAttribute().table)
1267+
.append(".")
1268+
.append(join.getSecondAttribute().columnName)
1269+
.append(" ");
12701270
str.insert(fromIndex, onClause);
12711271
String whereClause = join.getT().getWhereClause();
12721272
if ((whereClause != null) && !"".equals(whereClause)) {

0 commit comments

Comments
 (0)