Merge pull request #29 from vanioinformatika/br_readall_fix

patoi · web-flow · commit 1eb67d4e1045 · 2021-02-10T13:55:10.000+01:00
readAll method reads records ordered by PK
diff --git a/easydao-maven-plugin/src/main/resources/hu/vanio/easydao/templates/dao.ftl b/easydao-maven-plugin/src/main/resources/hu/vanio/easydao/templates/dao.ftl
@@ -81,7 +81,7 @@ public class ${t.javaName}${e.daoSuffix}Impl implements ${t.javaName}${e.daoSuff
 
     @Override
     public List<${t.javaName}> readAll(<#if t.hasBlobField || t.hasClobField>boolean readLobFields</#if>) {
-        String query = "select " + SELECTED_FIELDS + " from ${t.dbName}";
+        String query = "select " + SELECTED_FIELDS + " from ${t.dbName}<#if t.hasPkField && !t.compositePk> order by ${t.pkField.dbName}</#if>";
         List<${t.javaName}> retVal = this.jdbcTemplate.query(query, new ${t.javaName}RowMapper(<#if t.hasBlobField || t.hasClobField>readLobFields</#if>));
         return retVal;
     }

Original file line number	Diff line number	Diff line change
`@@ -81,7 +81,7 @@ public class ${t.javaName}${e.daoSuffix}Impl implements ${t.javaName}${e.daoSuff`
`81`	`81`
`82`	`82`	`@Override`
`83`	`83`	`public List<${t.javaName}> readAll(<#if t.hasBlobField \|\| t.hasClobField>boolean readLobFields</#if>) {`
`84`		`- String query = "select " + SELECTED_FIELDS + " from ${t.dbName}";`
	`84`	`+ String query = "select " + SELECTED_FIELDS + " from ${t.dbName}<#if t.hasPkField && !t.compositePk> order by ${t.pkField.dbName}</#if>";`
`85`	`85`	`List<${t.javaName}> retVal = this.jdbcTemplate.query(query, new ${t.javaName}RowMapper(<#if t.hasBlobField \|\| t.hasClobField>readLobFields</#if>));`
`86`	`86`	`return retVal;`
`87`	`87`	`}`