Author: clement
Date: Sat Jul 11 16:42:55 2009
New Revision: 793198
URL: http://svn.apache.org/viewvc?rev=793198&view=rev
Log:
Fix issue FELIX-1319.
The overriding detection badly interpret the 'name' attribute of instance
declaration. However, instance declaration must not impact the overriding
detected. So, now instances are skipped.
Modified:
felix/trunk/ipojo/manipulator/src/main/java/org/apache/felix/ipojo/manipulator/Pojoization.java
Modified:
felix/trunk/ipojo/manipulator/src/main/java/org/apache/felix/ipojo/manipulator/Pojoization.java
URL:
http://svn.apache.org/viewvc/felix/trunk/ipojo/manipulator/src/main/java/org/apache/felix/ipojo/manipulator/Pojoization.java?rev=793198&r1=793197&r2=793198&view=diff
==============================================================================
---
felix/trunk/ipojo/manipulator/src/main/java/org/apache/felix/ipojo/manipulator/Pojoization.java
(original)
+++
felix/trunk/ipojo/manipulator/src/main/java/org/apache/felix/ipojo/manipulator/Pojoization.java
Sat Jul 11 16:42:55 2009
@@ -294,10 +294,12 @@
if (xml.isAnnotated()) {
boolean toskip = false;
for (int i = 0; !toskip && i < m_metadata.length; i++) {
- if (m_metadata[i].containsAttribute("name")
+ if (! m_metadata[i].getName().equals("instance") // Only if
its a component type definition,
+ // so skip
instance declaration
+ && m_metadata[i].containsAttribute("name")
&&
m_metadata[i].getAttribute("name").equalsIgnoreCase(xml.getElem().getAttribute("name")))
{
toskip = true;
- warn("The component " + xml.getElem().getAttribute("name")
+ " is overriden by the metadata file");
+ warn("The component type " +
xml.getElem().getAttribute("name") + " is overriden by the metadata file");
}
}
if (!toskip) {
