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 2011/01/06 11:12:41 UTC
svn commit: r1055816 - in /felix/trunk/ipojo:
manipulator/src/main/java/org/apache/felix/ipojo/manipulation/annotations/
tests/core/annotations/src/main/java/org/apache/felix/ipojo/test/scenarios/component/error/
webconsole-plugin/
Author: clement
Date: Thu Jan 6 10:12:41 2011
New Revision: 1055816
URL: http://svn.apache.org/viewvc?rev=1055816&view=rev
Log:
Configure the RAT plugin for the webconsole-plugin project
Add a test about a potential NPE in the Annotation Collector.
Added:
felix/trunk/ipojo/tests/core/annotations/src/main/java/org/apache/felix/ipojo/test/scenarios/component/error/
felix/trunk/ipojo/tests/core/annotations/src/main/java/org/apache/felix/ipojo/test/scenarios/component/error/AbstractClass.java
Modified:
felix/trunk/ipojo/manipulator/src/main/java/org/apache/felix/ipojo/manipulation/annotations/MetadataCollector.java
felix/trunk/ipojo/webconsole-plugin/pom.xml
Modified: felix/trunk/ipojo/manipulator/src/main/java/org/apache/felix/ipojo/manipulation/annotations/MetadataCollector.java
URL: http://svn.apache.org/viewvc/felix/trunk/ipojo/manipulator/src/main/java/org/apache/felix/ipojo/manipulation/annotations/MetadataCollector.java?rev=1055816&r1=1055815&r2=1055816&view=diff
==============================================================================
--- felix/trunk/ipojo/manipulator/src/main/java/org/apache/felix/ipojo/manipulation/annotations/MetadataCollector.java (original)
+++ felix/trunk/ipojo/manipulator/src/main/java/org/apache/felix/ipojo/manipulation/annotations/MetadataCollector.java Thu Jan 6 10:12:41 2011
@@ -219,11 +219,16 @@ public class MetadataCollector extends E
*/
public void visitEnd() {
// If m_elem (Component) is null, print a warning and ignore.
- if (m_elem == null && ! m_elements.isEmpty()) {
+ if (m_elem == null && ! m_elements.isEmpty()) {
m_ignoredBecauseOfMissingComponent = true;
return;
}
+ if (! m_containsComponentAnnotation) {
+ m_ignoredBecauseOfMissingComponent = true;
+ return;
+ }
+
// Recompute the tree
Set elems = getElements().keySet();
Iterator it = elems.iterator();
Added: felix/trunk/ipojo/tests/core/annotations/src/main/java/org/apache/felix/ipojo/test/scenarios/component/error/AbstractClass.java
URL: http://svn.apache.org/viewvc/felix/trunk/ipojo/tests/core/annotations/src/main/java/org/apache/felix/ipojo/test/scenarios/component/error/AbstractClass.java?rev=1055816&view=auto
==============================================================================
--- felix/trunk/ipojo/tests/core/annotations/src/main/java/org/apache/felix/ipojo/test/scenarios/component/error/AbstractClass.java (added)
+++ felix/trunk/ipojo/tests/core/annotations/src/main/java/org/apache/felix/ipojo/test/scenarios/component/error/AbstractClass.java Thu Jan 6 10:12:41 2011
@@ -0,0 +1,10 @@
+package org.apache.felix.ipojo.test.scenarios.component.error;
+
+import org.apache.felix.ipojo.annotations.Requires;
+
+public abstract class AbstractClass {
+
+ @Requires
+ private Runnable run;
+
+}
Modified: felix/trunk/ipojo/webconsole-plugin/pom.xml
URL: http://svn.apache.org/viewvc/felix/trunk/ipojo/webconsole-plugin/pom.xml?rev=1055816&r1=1055815&r2=1055816&view=diff
==============================================================================
--- felix/trunk/ipojo/webconsole-plugin/pom.xml (original)
+++ felix/trunk/ipojo/webconsole-plugin/pom.xml Thu Jan 6 10:12:41 2011
@@ -28,7 +28,7 @@
<artifactId>org.apache.felix.ipojo.webconsole</artifactId>
<version>1.5.0-SNAPSHOT</version>
<name>Apache Felix iPOJO WebConsole Plugins</name>
-
+
<description>
iPOJO plugin for the Apache Felix Web Console. This plugin allows introspecting an iPOJO system with the Apache Felix
Web Console.
@@ -61,7 +61,7 @@
org.apache.felix.ipojo.webconsole
</Private-Package>
<Import-Package>*</Import-Package>
- <Include-Resource>
+ <Include-Resource>
META-INF/LICENSE=LICENSE,
META-INF/LICENSE.json=LICENSE.json,
META-INF/NOTICE=NOTICE,
@@ -99,7 +99,8 @@
<param>maven-eclipse.xml</param>
<param>.checkstyle</param>
<param>.externalToolBuilders/*</param>
- <param>LICENSE.asm</param>
+ <param>LICENSE.*</param>
+ <param>DEPENDENCIES</param>
</excludes>
</configuration>
</plugin>
@@ -118,17 +119,17 @@
<dependency>
<groupId>org.apache.felix</groupId>
<artifactId>org.apache.felix.webconsole</artifactId>
- <version>3.1.0</version>
+ <version>3.1.6</version>
</dependency>
<dependency>
<groupId>org.apache.felix</groupId>
<artifactId>org.apache.felix.ipojo</artifactId>
- <version>1.6.2</version>
+ <version>1.6.4</version>
</dependency>
<dependency>
<groupId>org.apache.felix</groupId>
<artifactId>org.apache.felix.ipojo.annotations</artifactId>
- <version>1.6.2</version>
+ <version>1.6.4</version>
</dependency>
<dependency>
<groupId>org.osgi</groupId>