Skip to content

Commit f48ab20

Browse files
committed
Update QueryBuilder.
1 parent f3c0cce commit f48ab20

File tree

2 files changed

+3
-17
lines changed

2 files changed

+3
-17
lines changed

kilo-client/src/main/java/org/httprpc/kilo/sql/QueryBuilder.java

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -269,10 +269,6 @@ public static QueryBuilder select(Class<?>... types) {
269269
for (var entry : BeanAdapter.getProperties(type).entrySet()) {
270270
var accessor = entry.getValue().getAccessor();
271271

272-
if (!getTableName(accessor.getDeclaringClass()).equals(tableName)) {
273-
continue;
274-
}
275-
276272
var column = accessor.getAnnotation(Column.class);
277273

278274
if (column == null) {
@@ -586,10 +582,6 @@ public static QueryBuilder insert(Class<?> type) {
586582
for (var entry : BeanAdapter.getProperties(type).entrySet()) {
587583
var accessor = entry.getValue().getAccessor();
588584

589-
if (!getTableName(accessor.getDeclaringClass()).equals(tableName)) {
590-
continue;
591-
}
592-
593585
var column = accessor.getAnnotation(Column.class);
594586

595587
if (column == null) {
@@ -682,10 +674,6 @@ public static QueryBuilder update(Class<?> type) {
682674
for (var entry : BeanAdapter.getProperties(type).entrySet()) {
683675
var accessor = entry.getValue().getAccessor();
684676

685-
if (!getTableName(accessor.getDeclaringClass()).equals(tableName)) {
686-
continue;
687-
}
688-
689677
var column = accessor.getAnnotation(Column.class);
690678

691679
if (column == null) {

kilo-client/src/test/java/org/httprpc/kilo/sql/QueryBuilderTest.java

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -240,11 +240,9 @@ public void testSelectA() {
240240

241241
@Test
242242
public void testSelectB() {
243-
var queryBuilder = QueryBuilder.select(A.class, B.class)
244-
.join(B.class, A.class)
245-
.filterByPrimaryKey("a").limit(10);
243+
var queryBuilder = QueryBuilder.select(B.class).filterByPrimaryKey("a").limit(10);
246244

247-
assertEquals("select A.a, A.b, A.c, A.d as x, B.e, B.f as y from A join B on A.a = B.a where A.a = ? limit 10", queryBuilder.toString());
245+
assertEquals("select B.a, B.b, B.c, B.e, B.d as x, B.f as y from B where B.a = ? limit 10", queryBuilder.toString());
248246
assertEquals(listOf("a"), getParameters(queryBuilder));
249247
}
250248

@@ -278,7 +276,7 @@ public void testSelectBtoD() {
278276
.filterByPrimaryKey("a")
279277
.filterByForeignKey(E.class, D.class, "d");
280278

281-
assertEquals("select B.a, B.e, B.f as y, E.z from B "
279+
assertEquals("select B.a, B.b, B.c, B.e, B.d as x, B.f as y, E.z from B "
282280
+ "join E on B.a = E.a "
283281
+ "where B.a = ? "
284282
+ "and E.d = ?", queryBuilder.toString());

0 commit comments

Comments
 (0)