You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by cl...@apache.org on 2008/04/15 01:03:54 UTC
svn commit: r648046 - in /felix/trunk/ipojo:
composite/src/main/java/org/apache/felix/ipojo/composite/service/provides/
core/src/main/java/org/apache/felix/ipojo/
tests/tests.composite/src/main/java/org/apache/felix/ipojo/test/composite/provides/
tests...
Author: clement
Date: Mon Apr 14 16:03:32 2008
New Revision: 648046
URL: http://svn.apache.org/viewvc?rev=648046&view=rev
Log:
Fix some minor formatting issue
Improve a log message in composite service providing
Change the log level of a useless message
Modify external handler tests to check when the namespace (o the external handler) is not the class name.
Modified:
felix/trunk/ipojo/composite/src/main/java/org/apache/felix/ipojo/composite/service/provides/CompositionMetadata.java
felix/trunk/ipojo/composite/src/main/java/org/apache/felix/ipojo/composite/service/provides/ProvidedService.java
felix/trunk/ipojo/core/src/main/java/org/apache/felix/ipojo/ComponentFactory.java
felix/trunk/ipojo/core/src/main/java/org/apache/felix/ipojo/Extender.java
felix/trunk/ipojo/core/src/main/java/org/apache/felix/ipojo/InstanceManager.java
felix/trunk/ipojo/tests/tests.composite/src/main/java/org/apache/felix/ipojo/test/composite/provides/TestComp0.java
felix/trunk/ipojo/tests/tests.core/src/main/java/org/apache/felix/ipojo/test/handler/CheckServiceHandler.java
felix/trunk/ipojo/tests/tests.core/src/main/resources/metadata.xml
Modified: felix/trunk/ipojo/composite/src/main/java/org/apache/felix/ipojo/composite/service/provides/CompositionMetadata.java
URL: http://svn.apache.org/viewvc/felix/trunk/ipojo/composite/src/main/java/org/apache/felix/ipojo/composite/service/provides/CompositionMetadata.java?rev=648046&r1=648045&r2=648046&view=diff
==============================================================================
--- felix/trunk/ipojo/composite/src/main/java/org/apache/felix/ipojo/composite/service/provides/CompositionMetadata.java (original)
+++ felix/trunk/ipojo/composite/src/main/java/org/apache/felix/ipojo/composite/service/provides/CompositionMetadata.java Mon Apr 14 16:03:32 2008
@@ -252,7 +252,7 @@
*/
protected Element buildMetadata(String name) {
Element elem = new Element("component", "");
- Attribute className = new Attribute("className", m_name);
+ Attribute className = new Attribute("classname", m_name);
Attribute factory = new Attribute("factory", "false");
elem.addAttribute(className);
elem.addAttribute(factory);
Modified: felix/trunk/ipojo/composite/src/main/java/org/apache/felix/ipojo/composite/service/provides/ProvidedService.java
URL: http://svn.apache.org/viewvc/felix/trunk/ipojo/composite/src/main/java/org/apache/felix/ipojo/composite/service/provides/ProvidedService.java?rev=648046&r1=648045&r2=648046&view=diff
==============================================================================
--- felix/trunk/ipojo/composite/src/main/java/org/apache/felix/ipojo/composite/service/provides/ProvidedService.java (original)
+++ felix/trunk/ipojo/composite/src/main/java/org/apache/felix/ipojo/composite/service/provides/ProvidedService.java Mon Apr 14 16:03:32 2008
@@ -112,10 +112,11 @@
// Create the factory
try {
m_factory = new ComponentFactory(m_context, clazz, metadata);
+ m_factory.start();
} catch (ConfigurationException e) {
// Should not happen.
+ m_manager.getFactory().getLogger().log(Logger.ERROR, "A factory cannot be created", e);
}
- m_factory.start();
try {
Class spec = DependencyModel.loadSpecification(m_composition.getSpecificationMetadata().getName(), m_context);
Modified: felix/trunk/ipojo/core/src/main/java/org/apache/felix/ipojo/ComponentFactory.java
URL: http://svn.apache.org/viewvc/felix/trunk/ipojo/core/src/main/java/org/apache/felix/ipojo/ComponentFactory.java?rev=648046&r1=648045&r2=648046&view=diff
==============================================================================
--- felix/trunk/ipojo/core/src/main/java/org/apache/felix/ipojo/ComponentFactory.java (original)
+++ felix/trunk/ipojo/core/src/main/java/org/apache/felix/ipojo/ComponentFactory.java Mon Apr 14 16:03:32 2008
@@ -107,7 +107,7 @@
* @throws ConfigurationException occurs when the element describing the factory is malformed.
*/
public void check(Element element) throws ConfigurationException {
- m_classname = element.getAttribute("className");
+ m_classname = element.getAttribute("classname");
if (m_classname == null) { throw new ConfigurationException("A component needs a class name : " + element); }
}
@@ -215,7 +215,7 @@
if (name == null) { // No factory name, try with factory attribute
name = m_componentMetadata.getAttribute("factory");
if (name == null || name.equalsIgnoreCase("true") || name.equalsIgnoreCase("false")) { // Avoid boolean case
- name = m_componentMetadata.getAttribute("className");
+ name = m_componentMetadata.getAttribute("classname");
}
}
return name;
Modified: felix/trunk/ipojo/core/src/main/java/org/apache/felix/ipojo/Extender.java
URL: http://svn.apache.org/viewvc/felix/trunk/ipojo/core/src/main/java/org/apache/felix/ipojo/Extender.java?rev=648046&r1=648045&r2=648046&view=diff
==============================================================================
--- felix/trunk/ipojo/core/src/main/java/org/apache/felix/ipojo/Extender.java (original)
+++ felix/trunk/ipojo/core/src/main/java/org/apache/felix/ipojo/Extender.java Mon Apr 14 16:03:32 2008
@@ -152,7 +152,7 @@
for (int i = 0; i < toRemove.size(); i++) {
ManagedAbstractFactoryType mft = (ManagedAbstractFactoryType) toRemove.get(i);
- m_logger.log(Logger.WARNING, "The factory type available: " + mft.m_type + " is no more available");
+ m_logger.log(Logger.INFO, "The factory type: " + mft.m_type + " is no more available");
mft.m_bundle = null;
mft.m_clazz = null;
mft.m_created = null;
Modified: felix/trunk/ipojo/core/src/main/java/org/apache/felix/ipojo/InstanceManager.java
URL: http://svn.apache.org/viewvc/felix/trunk/ipojo/core/src/main/java/org/apache/felix/ipojo/InstanceManager.java?rev=648046&r1=648045&r2=648046&view=diff
==============================================================================
--- felix/trunk/ipojo/core/src/main/java/org/apache/felix/ipojo/InstanceManager.java (original)
+++ felix/trunk/ipojo/core/src/main/java/org/apache/felix/ipojo/InstanceManager.java Mon Apr 14 16:03:32 2008
@@ -140,7 +140,7 @@
* @throws ConfigurationException : occurs if the metadata are not correct
*/
public void configure(Element metadata, Dictionary configuration) throws ConfigurationException {
- m_className = metadata.getAttribute("className");
+ m_className = metadata.getAttribute("classname");
// Add the name
m_name = (String) configuration.get("name");
Modified: felix/trunk/ipojo/tests/tests.composite/src/main/java/org/apache/felix/ipojo/test/composite/provides/TestComp0.java
URL: http://svn.apache.org/viewvc/felix/trunk/ipojo/tests/tests.composite/src/main/java/org/apache/felix/ipojo/test/composite/provides/TestComp0.java?rev=648046&r1=648045&r2=648046&view=diff
==============================================================================
--- felix/trunk/ipojo/tests/tests.composite/src/main/java/org/apache/felix/ipojo/test/composite/provides/TestComp0.java (original)
+++ felix/trunk/ipojo/tests/tests.composite/src/main/java/org/apache/felix/ipojo/test/composite/provides/TestComp0.java Mon Apr 14 16:03:32 2008
@@ -72,6 +72,7 @@
try {
under = factory.createComponentInstance(props2);
} catch(Exception e) {
+ e.printStackTrace();
fail("Cannot create an instance : " + e.getMessage());
}
}
Modified: felix/trunk/ipojo/tests/tests.core/src/main/java/org/apache/felix/ipojo/test/handler/CheckServiceHandler.java
URL: http://svn.apache.org/viewvc/felix/trunk/ipojo/tests/tests.core/src/main/java/org/apache/felix/ipojo/test/handler/CheckServiceHandler.java?rev=648046&r1=648045&r2=648046&view=diff
==============================================================================
--- felix/trunk/ipojo/tests/tests.core/src/main/java/org/apache/felix/ipojo/test/handler/CheckServiceHandler.java (original)
+++ felix/trunk/ipojo/tests/tests.core/src/main/java/org/apache/felix/ipojo/test/handler/CheckServiceHandler.java Mon Apr 14 16:03:32 2008
@@ -34,13 +34,13 @@
ServiceRegistration sr;
boolean isValid;
int changes = 0;
- static final String NAMESPACE = org.apache.felix.ipojo.test.handler.CheckServiceHandler.class.getName();
+ static final String NAMESPACE = "org.apache.felix.ipojo.test.handler.checkservice";
Properties props = new Properties();
public void configure(Element metadata, Dictionary configuration) {
- Element[] meta = metadata.getElements("Check", NAMESPACE);
- if(meta.length == 0) { return; }
+ Element[] meta = metadata.getElements("check", NAMESPACE);
+ if(meta == null) { return; }
// Get handler props
props.put("instance.name", configuration.get("name"));
if(configuration.get("csh.simple") != null) { props.put("Simple", configuration.get("csh.simple")); }
Modified: felix/trunk/ipojo/tests/tests.core/src/main/resources/metadata.xml
URL: http://svn.apache.org/viewvc/felix/trunk/ipojo/tests/tests.core/src/main/resources/metadata.xml?rev=648046&r1=648045&r2=648046&view=diff
==============================================================================
--- felix/trunk/ipojo/tests/tests.core/src/main/resources/metadata.xml (original)
+++ felix/trunk/ipojo/tests/tests.core/src/main/resources/metadata.xml Mon Apr 14 16:03:32 2008
@@ -1,5 +1,5 @@
<ipojo
- xmlns:cs="org.apache.felix.ipojo.test.handler.CheckServiceHandler">
+ xmlns:cs="org.apache.felix.ipojo.test.handler.checkservice">
<component classname="org.apache.felix.ipojo.test.log.LogImpl"
factory="true" name="log">
<provides />
@@ -1077,7 +1077,7 @@
<handler
classname="org.apache.felix.ipojo.test.handler.CheckServiceHandler"
name="check"
- namespace="org.apache.felix.ipojo.test.handler.CheckServiceHandler"
+ namespace="org.apache.felix.ipojo.test.handler.checkservice"
architecture="false">
<controller field="isValid" />
</handler>