You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by pc...@apache.org on 2006/11/15 01:45:11 UTC
svn commit: r475087 - in /incubator/openjpa/trunk:
openjpa-kernel/src/main/java/org/apache/openjpa/conf/
openjpa-kernel/src/main/resources/org/apache/openjpa/conf/
openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/
Author: pcl
Date: Tue Nov 14 16:45:09 2006
New Revision: 475087
URL: http://svn.apache.org/viewvc?view=rev&rev=475087
Log:
realized immediately after committing that the id belongs at the generic Configuration level, not at the OpenJPAConfiguration level.
Modified:
incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/conf/OpenJPAConfiguration.java
incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/conf/OpenJPAConfigurationImpl.java
incubator/openjpa/trunk/openjpa-kernel/src/main/resources/org/apache/openjpa/conf/localizer.properties
incubator/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/Configuration.java
incubator/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/ConfigurationImpl.java
Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/conf/OpenJPAConfiguration.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/conf/OpenJPAConfiguration.java?view=diff&rev=475087&r1=475086&r2=475087
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/conf/OpenJPAConfiguration.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/conf/OpenJPAConfiguration.java Tue Nov 14 16:45:09 2006
@@ -1383,22 +1383,4 @@
* configuration.
*/
public StoreFacadeTypeRegistry getStoreFacadeTypeRegistry();
-
- /**
- * An environment-specific identifier for this configuration. This
- * might correspond to a JPA persistence-unit name, or to some other
- * more-unique value available in the current environment.
- *
- * @since 0.9.7
- */
- public String getId();
-
- /**
- * An environment-specific identifier for this configuration. This
- * might correspond to a JPA persistence-unit name, or to some other
- * more-unique value available in the current environment.
- *
- * @since 0.9.7
- */
- public void setId(String id);
}
Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/conf/OpenJPAConfigurationImpl.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/conf/OpenJPAConfigurationImpl.java?view=diff&rev=475087&r1=475086&r2=475087
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/conf/OpenJPAConfigurationImpl.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/conf/OpenJPAConfigurationImpl.java Tue Nov 14 16:45:09 2006
@@ -127,7 +127,6 @@
public ObjectValue orphanedKeyPlugin;
public ObjectValue compatibilityPlugin;
public QueryCompilationCacheValue queryCompilationCachePlugin;
- public StringValue id;
// custom values
public BrokerFactoryValue brokerFactoryPlugin;
@@ -467,8 +466,6 @@
"getQueryCompilationCacheInstance");
addValue(queryCompilationCachePlugin);
- addString("Id");
-
// initialize supported options that some runtimes may not support
supportedOptions.add(OPTION_NONTRANS_READ);
supportedOptions.add(OPTION_OPTIMISTIC);
@@ -1408,15 +1405,6 @@
return _storeFacadeRegistry;
}
- public String getId() {
- return id.get();
- }
-
- public void setId(String id) {
- assertNotReadOnly();
- this.id.set(id);
- }
-
public void instantiateAll() {
super.instantiateAll();
getMetaDataRepositoryInstance();
Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/resources/org/apache/openjpa/conf/localizer.properties
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/resources/org/apache/openjpa/conf/localizer.properties?view=diff&rev=475087&r1=475086&r2=475087
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/resources/org/apache/openjpa/conf/localizer.properties (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/resources/org/apache/openjpa/conf/localizer.properties Tue Nov 14 16:45:09 2006
@@ -557,15 +557,6 @@
QueryCompilationCache-expert: true
QueryCompilationCache-interface: java.util.Map
-Id-name: Configuration ID
-Id-desc: An environment-specific identifier for this configuration. This \
- might correspond to a JPA persistence-unit name, or to some other \
- more-unique value available in the current environment.
-Id-type: General
-Id-cat: Persistence
-Id-displayorder: 50
-Id-expert: true
-
no-named-cf: use a DataSource bound to JNDI
diff-specs: Attempt to configure for multiple specifications. Was configured \
for "{0}". Attempt to now configure for "{1}". This attempt will be \
Modified: incubator/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/Configuration.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/Configuration.java?view=diff&rev=475087&r1=475086&r2=475087
==============================================================================
--- incubator/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/Configuration.java (original)
+++ incubator/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/Configuration.java Tue Nov 14 16:45:09 2006
@@ -115,6 +115,25 @@
*/
public Log getConfigurationLog();
+
+ /**
+ * An environment-specific identifier for this configuration. This
+ * might correspond to a JPA persistence-unit name, or to some other
+ * more-unique value available in the current environment.
+ *
+ * @since 0.9.7
+ */
+ public String getId();
+
+ /**
+ * An environment-specific identifier for this configuration. This
+ * might correspond to a JPA persistence-unit name, or to some other
+ * more-unique value available in the current environment.
+ *
+ * @since 0.9.7
+ */
+ public void setId(String id);
+
/**
* Return the {@link Value} for the given property, or null if none.
*/
Modified: incubator/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/ConfigurationImpl.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/ConfigurationImpl.java?view=diff&rev=475087&r1=475086&r2=475087
==============================================================================
--- incubator/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/ConfigurationImpl.java (original)
+++ incubator/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/ConfigurationImpl.java Tue Nov 14 16:45:09 2006
@@ -101,6 +101,7 @@
(ConfigurationImpl.class);
public ObjectValue logFactoryPlugin;
+ public StringValue id;
private String _product = null;
private boolean _readOnly = false;
@@ -146,6 +147,8 @@
logFactoryPlugin.setString(aliases[0]);
logFactoryPlugin.setInstantiatingGetter("getLogFactory");
+ id = addString("Id");
+
if (loadGlobals)
loadGlobals();
}
@@ -207,6 +210,15 @@
public Log getLog(String category) {
return getLogFactory().getLog(category);
+ }
+
+ public String getId() {
+ return id.get();
+ }
+
+ public void setId(String id) {
+ assertNotReadOnly();
+ this.id.set(id);
}
/**