[prev in list] [next in list] [prev in thread] [next in thread]
List: avalon-dev
Subject: cvs commit: jakarta-avalon-excalibur/src/scratchpad/org/apache/avalon/excalibur/i18n/test DefaultBun
From: bloritsch () apache ! org
Date: 2001-10-31 20:46:06
[Download RAW message or body]
bloritsch 01/10/31 12:46:06
Modified: src/scratchpad/org/apache/avalon/excalibur/i18n
BundleSelector.java
src/scratchpad/org/apache/avalon/excalibur/i18n/test
DefaultBundleLoaderTestCase.java
XmlBundleTestCase.java
Log:
remove logging dependencies
Revision Changes Path
1.3 +16 -30 \
jakarta-avalon-excalibur/src/scratchpad/org/apache/avalon/excalibur/i18n/BundleSelector.java
Index: BundleSelector.java
===================================================================
RCS file: /home/cvs/jakarta-avalon-excalibur/src/scratchpad/org/apache/avalon/excalibur/i18n/BundleSelector.java,v
retrieving revision 1.2
retrieving revision 1.3
diff -u -r1.2 -r1.3
--- BundleSelector.java 2001/10/31 19:56:47 1.2
+++ BundleSelector.java 2001/10/31 20:46:05 1.3
@@ -13,12 +13,11 @@
import java.util.LinkedList;
import java.util.Locale;
-import org.apache.avalon.framework.component.Composable;
+import org.apache.avalon.excalibur.component.ExcaliburComponentSelector;
import org.apache.avalon.framework.component.Component;
+import org.apache.avalon.framework.component.Composable;
import org.apache.avalon.framework.component.ComponentManager;
import org.apache.avalon.framework.component.ComponentException;
-import org.apache.avalon.framework.component.DefaultComponentSelector;
-import org.apache.avalon.framework.configuration.Configurable;
import org.apache.avalon.framework.configuration.Configuration;
import org.apache.avalon.framework.configuration.ConfigurationException;
import org.apache.avalon.framework.logger.Loggable;
@@ -30,12 +29,11 @@
*
* @author <a href="mailto:neeme@one.lv">Neeme Praks</a>
* @author <a href="mailto:mengelhart@earthtrip.com">Mike Engelhart</a>
- * @version $Id: BundleSelector.java,v 1.2 2001/10/31 19:56:47 neeme Exp $
+ * @version $Id: BundleSelector.java,v 1.3 2001/10/31 20:46:05 bloritsch Exp $
*/
public class BundleSelector
- extends DefaultComponentSelector
- implements Configurable, Loggable, ThreadSafe, Composable
+ extends ExcaliburComponentSelector
{
/**
@@ -62,18 +60,6 @@
/** the default loader */
private BundleLoader defaultLoader;
- /** The logger */
- protected Logger logger;
-
- /**
- * Set the logger.
- *
- * @param logger the logger
- */
- public void setLogger(final Logger logger) {
- this.logger = logger;
- }
-
/**
* Configure the component.
*
@@ -83,7 +69,7 @@
Configuration[] loaderConfs = \
configuration.getChild("loaders").getChildren("loader"); for (int i = 0; i < \
loaderConfs.length; i++) {
BundleLoader loader = (BundleLoader) \
getInstance(loaderConfs[i].getAttribute("class"));
- if (loader instanceof Loggable) ((Loggable)loader).setLogger(logger);
+ if (loader instanceof Loggable) \
((Loggable)loader).setLogger(getLogger()); try {
if (loader instanceof Composable) \
((Composable)loader).compose(this.manager); }
@@ -151,18 +137,18 @@
* @exception ComponentException if a bundle is not found
*/
private Component select(BundleInfo bundleInfo) {
- if (logger.isDebugEnabled()) logger.debug("_getBundle: " + bundleInfo);
+ if (getLogger().isDebugEnabled()) getLogger().debug("_getBundle: " + \
bundleInfo); Bundle bundle = (Bundle) selectCached(bundleInfo);
if (bundle == null && !isNotFoundBundle(bundleInfo)) {
- if (logger.isDebugEnabled()) logger.debug("not found in cache, \
loading: " + bundleInfo); + if (getLogger().isDebugEnabled()) \
getLogger().debug("not found in cache, loading: " + bundleInfo); synchronized(this) \
{ bundle = (Bundle) selectCached(bundleInfo);
if (bundle == null && !isNotFoundBundle(bundleInfo)) {
- if (logger.isDebugEnabled()) logger.debug("synchronized: not \
found in cache, loading: " + bundleInfo); + if \
(getLogger().isDebugEnabled()) getLogger().debug("synchronized: not found in cache, \
loading: " + bundleInfo); bundle = loadBundle(bundleInfo);
BundleInfo parentBundleInfo = bundleInfo.getParent();
while (bundle == null && parentBundleInfo != null) {
- if (logger.isDebugEnabled()) logger.debug("synchronized: \
still not found, trying parent: " + parentBundleInfo); + if \
(getLogger().isDebugEnabled()) getLogger().debug("synchronized: still not found, \
trying parent: " + parentBundleInfo); bundle = loadBundle(parentBundleInfo);
updateCache(parentBundleInfo, bundle);
parentBundleInfo = parentBundleInfo.getParent();
@@ -198,10 +184,10 @@
Component bundle = null;
try {
bundle = super.select(bundleInfo);
- if (logger.isDebugEnabled()) logger.debug("Returning from cache: " + \
bundleInfo); + if (getLogger().isDebugEnabled()) \
getLogger().debug("Returning from cache: " + bundleInfo); }
catch (ComponentException e) {
- if (logger.isDebugEnabled()) logger.debug("Not found in cache: " + \
bundleInfo); + if (getLogger().isDebugEnabled()) getLogger().debug("Not \
found in cache: " + bundleInfo); }
return bundle;
}
@@ -216,10 +202,10 @@
protected boolean isNotFoundBundle(BundleInfo bundleInfo) {
BundleInfo result = (BundleInfo)(cacheNotFound.get(bundleInfo));
if (result != null) {
- if (logger.isDebugEnabled()) logger.debug("Returning from \
not_found_cache: " + bundleInfo); + if (getLogger().isDebugEnabled()) \
getLogger().debug("Returning from not_found_cache: " + bundleInfo); }
else {
- if (logger.isDebugEnabled()) logger.debug("Not found in \
not_found_cache: " + bundleInfo); + if (getLogger().isDebugEnabled()) \
getLogger().debug("Not found in not_found_cache: " + bundleInfo); }
return result != null;
}
@@ -233,12 +219,12 @@
*/
protected void updateCache(BundleInfo bundleInfo, Bundle bundle) {
if (bundle == null) {
- if (logger.isDebugEnabled()) logger.debug("Updating not_found_cache: " \
+ bundleInfo); + if (getLogger().isDebugEnabled()) \
getLogger().debug("Updating not_found_cache: " + bundleInfo); \
cacheNotFound.put(bundleInfo, bundleInfo); }
else {
- if (logger.isDebugEnabled()) logger.debug("Updating cache: " + \
bundleInfo);
- super.put((Object) bundleInfo, (Component) bundle);
+ if (getLogger().isDebugEnabled()) getLogger().debug("Updating cache: " \
+ bundleInfo); + super.addComponentInstance(bundleInfo, (Component) \
bundle); }
}
1.2 +6 -1 \
jakarta-avalon-excalibur/src/scratchpad/org/apache/avalon/excalibur/i18n/test/DefaultBundleLoaderTestCase.java
Index: DefaultBundleLoaderTestCase.java
===================================================================
RCS file: /home/cvs/jakarta-avalon-excalibur/src/scratchpad/org/apache/avalon/excalibur/i18n/test/DefaultBundleLoaderTestCase.java,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -r1.1 -r1.2
--- DefaultBundleLoaderTestCase.java 2001/10/31 19:56:47 1.1
+++ DefaultBundleLoaderTestCase.java 2001/10/31 20:46:05 1.2
@@ -19,7 +19,7 @@
/**
* @author <a href="mailto:neeme@one.lv">Neeme Praks</a>
- * @version $Id: DefaultBundleLoaderTestCase.java,v 1.1 2001/10/31 19:56:47 neeme \
Exp $ + * @version $Id: DefaultBundleLoaderTestCase.java,v 1.2 2001/10/31 20:46:05 \
bloritsch Exp $
*/
public class DefaultBundleLoaderTestCase extends ExcaliburTestCase {
@@ -31,6 +31,11 @@
public void setUp() throws Exception {
this.bundleSelector = (BundleSelector) \
manager.lookup(BundleSelector.ROLE); + }
+
+ public void tearDown() throws Exception {
+ manager.release(this.bundleSelector);
+ this.bundleSelector = null;
}
public void testLoading() throws Exception {
1.2 +2 -2 \
jakarta-avalon-excalibur/src/scratchpad/org/apache/avalon/excalibur/i18n/test/XmlBundleTestCase.java
Index: XmlBundleTestCase.java
===================================================================
RCS file: /home/cvs/jakarta-avalon-excalibur/src/scratchpad/org/apache/avalon/excalibur/i18n/test/XmlBundleTestCase.java,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -r1.1 -r1.2
--- XmlBundleTestCase.java 2001/10/31 19:56:47 1.1
+++ XmlBundleTestCase.java 2001/10/31 20:46:06 1.2
@@ -16,7 +16,7 @@
/**
* @author <a href="mailto:neeme@one.lv">Neeme Praks</a>
- * @version $Id: XmlBundleTestCase.java,v 1.1 2001/10/31 19:56:47 neeme Exp $
+ * @version $Id: XmlBundleTestCase.java,v 1.2 2001/10/31 20:46:06 bloritsch Exp $
*/
public class XmlBundleTestCase extends ExcaliburTestCase {
@@ -36,7 +36,7 @@
this.bundle.setBundleInfo(new BundleInfo("test", null));
this.bundle.setLogger(getLogger());
- this.bundle.compose(super.manager);
+ this.bundle.compose(this.manager);
this.bundle.setLoadOnInit(true);
this.bundleWithRoot.setUseRootElement(false);
this.bundle.init(this.getClass().getClassLoader().getResource(bundleFileName).openStream());
--
To unsubscribe, e-mail: <mailto:avalon-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:avalon-dev-help@jakarta.apache.org>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic