Fixed problem with the query builder and caches

Author: Gray Watson

src/main/java/com/j256/ormlite/stmt/QueryBuilder.java

@@ -98,7 +98,7 @@ String getSelectColumnsAsString() {
 	 */
 	public PreparedQuery<T> prepare() throws SQLException {
 		// we only store things in the cache if there was not selects specified
-		boolean cacheStore = (resultFieldTypes == tableInfo.getFieldTypes());
+		boolean cacheStore = (selectList == null);
 		return super.prepareStatement(limit, cacheStore);
 	}
 

src/test/java/com/j256/ormlite/dao/BaseObjectCacheTest.java

@@ -164,12 +164,13 @@ public void testQueryAll() throws Exception {
 
 		List<Foo> results = dao.queryForAll();
 		assertEquals(1, results.size());
+		assertEquals(foo.id, results.get(0).id);
 		assertNotSame(foo, results.get(0));
 
 		Foo foo2 = dao.queryForId(foo.id);
 		assertNotNull(foo2);
+		assertEquals(foo.id, results.get(0).id);
 		assertSame(results.get(0), foo2);
-
 	}
 
 	@Test
@@ -270,6 +271,7 @@ protected static class NoId {
 		int notId;
 		@DatabaseField
 		String stuff;
+
 		public NoId() {
 		}
 	}
@@ -279,6 +281,7 @@ protected static class WithId {
 		int id;
 		@DatabaseField
 		String stuff;
+
 		public WithId() {
 		}
 	}
@@ -288,6 +291,7 @@ protected static class Parent {
 		int id;
 		@DatabaseField(foreign = true, foreignAutoRefresh = true)
 		Child child;
+
 		public Parent() {
 		}
 	}
@@ -297,6 +301,7 @@ protected static class Child {
 		int id;
 		@ForeignCollectionField
 		ForeignCollection<Parent> parents;
+
 		public Child() {
 		}
 	}