Skip to content

Commit abca3e1

Browse files
committed
replaced literal string with final static field
1 parent e24b3ef commit abca3e1

File tree

3 files changed

+9
-8
lines changed

3 files changed

+9
-8
lines changed

railo-java/railo-core/src/railo/runtime/config/ConfigImpl.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -115,6 +115,7 @@
115115
import railo.transformer.library.tag.TagLibTag;
116116
import railo.transformer.library.tag.TagLibTagAttr;
117117
import flex.messaging.config.ConfigMap;
118+
import static railo.runtime.db.DatasourceManagerImpl.QOQ_DATASOURCE_NAME;
118119

119120

120121
/**
@@ -2186,7 +2187,7 @@ public Map<String,DataSource> getDataSourcesAsMap() {
21862187
Entry<String, DataSource> entry;
21872188
while(it.hasNext()) {
21882189
entry = it.next();
2189-
if(!entry.getKey().equals("_queryofquerydb"))
2190+
if(!entry.getKey().equals(QOQ_DATASOURCE_NAME))
21902191
map.put(entry.getKey(),entry.getValue());
21912192
}
21922193
return map;

railo-java/railo-core/src/railo/runtime/db/DatasourceManagerImpl.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,9 @@
2020
* this class handle multible db connection, transaction and logging
2121
*/
2222
public final class DatasourceManagerImpl implements DataSourceManager {
23-
23+
24+
public static final String QOQ_DATASOURCE_NAME = "_queryofquerydb";
25+
2426
private ConfigImpl config;
2527

2628
boolean autoCommit=true;
@@ -61,7 +63,7 @@ public DatasourceConnection getConnection(PageContext pc,DataSource ds, String u
6163
transConn=dc;
6264
}
6365
else if(!transConn.equals(dc)) {
64-
if("_queryofquerydb".equalsIgnoreCase(ds.getName())) return dc;
66+
if(QOQ_DATASOURCE_NAME.equalsIgnoreCase(ds.getName())) return dc;
6567
throw new DatabaseException(
6668
"can't use different connections inside a transaction",null,null,dc);
6769
}

railo-java/railo-core/src/railo/runtime/db/HSQLDBHandler.java

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@
3030
import railo.runtime.type.QueryColumn;
3131
import railo.runtime.type.QueryImpl;
3232
import railo.runtime.type.util.CollectionUtil;
33+
import static railo.runtime.db.DatasourceManagerImpl.QOQ_DATASOURCE_NAME;
3334

3435
/**
3536
* class to reexecute queries on the resultset object inside the cfml enviroment
@@ -325,14 +326,11 @@ public static QueryImpl __execute(PageContext pc, SQL sql, int maxrows, int fet
325326
QueryImpl nqr=null;
326327
ConfigImpl config = (ConfigImpl)pc.getConfig();
327328
DatasourceConnectionPool pool = config.getDatasourceConnectionPool();
328-
DatasourceConnection dc=pool.getDatasourceConnection(pc,config.getDataSource("_queryofquerydb"),"sa","");
329+
DatasourceConnection dc=pool.getDatasourceConnection(pc,config.getDataSource(QOQ_DATASOURCE_NAME),"sa","");
329330
Connection conn = dc.getConnection();
330331
try {
331332
DBUtil.setAutoCommitEL(conn,false);
332-
333-
334-
335-
333+
336334
//sql.setSQLString(HSQLUtil.sqlToZQL(sql.getSQLString(),false));
337335
try {
338336
Iterator<String> it = tables.iterator();

0 commit comments

Comments
 (0)