Author: vsiveton
Date: Mon Dec 13 10:47:58 2010
New Revision: 1045058

URL: http://svn.apache.org/viewvc?rev=1045058&view=rev
Log:
MDOAP-30: Update requirement to Maven 2.2 and Java 5

o improved code for java5

Modified:
    
maven/plugins/trunk/maven-doap-plugin/src/main/java/org/apache/maven/plugin/doap/DoapMojo.java
    maven/plugins/trunk/maven-doap-plugin/src/main/mdo/asfextOptions.mdo

Modified: 
maven/plugins/trunk/maven-doap-plugin/src/main/java/org/apache/maven/plugin/doap/DoapMojo.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-doap-plugin/src/main/java/org/apache/maven/plugin/doap/DoapMojo.java?rev=1045058&r1=1045057&r2=1045058&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-doap-plugin/src/main/java/org/apache/maven/plugin/doap/DoapMojo.java
 (original)
+++ 
maven/plugins/trunk/maven-doap-plugin/src/main/java/org/apache/maven/plugin/doap/DoapMojo.java
 Mon Dec 13 10:47:58 2010
@@ -463,9 +463,9 @@ public class DoapMojo
             List<String> errors = DoapUtil.validate( doapFile );
             if ( !errors.isEmpty() )
             {
-                for ( int i = 0; i < errors.size(); i++ )
+                for ( String error : errors )
                 {
-                    getLog().error( errors.get( i ).toString() );
+                    getLog().error( error );
                 }
 
                 throw new MojoExecutionException( "Error parsing the generated 
doap file, see above." );
@@ -591,7 +591,8 @@ public class DoapMojo
             catch ( MalformedURLException e )
             {
                 getLog().error( "The <doapOptions><oldHomepage/></doapOptions> 
parameter "
-                                    + doapOptions.getOldHomepage() + " is not 
a valid URL. Ignored <old-homepage/> tag." );
+                                    + doapOptions.getOldHomepage()
+                                    + " is not a valid URL. Ignored 
<old-homepage/> tag." );
             }
         }
     }
@@ -627,16 +628,15 @@ public class DoapMojo
         if ( StringUtils.isNotEmpty( doapOptions.getProgrammingLanguage() ) )
         {
             String[] languages = StringUtils.split( 
doapOptions.getProgrammingLanguage(), "," );
-            for ( int i = 0; i < languages.length; i++ )
+            for ( String language : languages )
             {
-                if ( asfExtOptions.isIncluded()
-                    && !ASFExtOptions.isProgrammingLanguageSupportedByASF( 
languages[i].trim() ) )
+                if ( asfExtOptions.isIncluded() && 
!ASFExtOptions.isProgrammingLanguageSupportedByASF( language.trim() ) )
                 {
-                    getLog().warn( "The programming language '" + 
languages[i].trim() + "' is not supported by ASF. "
+                    getLog().warn( "The programming language '" + 
language.trim() + "' is not supported by ASF. "
                                        + "Refer you to 
http://projects.apache.org/languages.html"; );
                 }
 
-                DoapUtil.writeElement( writer, "programming-language", 
languages[i].trim() );
+                DoapUtil.writeElement( writer, "programming-language", 
language.trim() );
             }
         }
     }
@@ -679,22 +679,22 @@ public class DoapMojo
         if ( StringUtils.isNotEmpty( doapOptions.getCategory() ) )
         {
             String[] categories = StringUtils.split( 
doapOptions.getCategory(), "," );
-            for ( int i = 0; i < categories.length; i++ )
+            for ( String category : categories )
             {
-                if ( asfExtOptions.isIncluded() && 
!ASFExtOptions.isCategorySupportedByASF( categories[i] ) )
+                if ( asfExtOptions.isIncluded() && 
!ASFExtOptions.isCategorySupportedByASF( category ) )
                 {
-                    getLog().warn( "The given category '" + categories[i] + "' 
is not supported by ASF. "
+                    getLog().warn( "The given category '" + category + "' is 
not supported by ASF. "
                                        + "Refer you to 
http://projects.apache.org/categories.html"; );
                 }
 
                 if ( asfExtOptions.isIncluded() )
                 {
                     DoapUtil.writeRdfResourceElement( writer, "category", 
"http://projects.apache.org/category/";
-                        + categories[i].trim() );
+                        + category.trim() );
                 }
                 else
                 {
-                    DoapUtil.writeRdfResourceElement( writer, "category", 
categories[i].trim() );
+                    DoapUtil.writeRdfResourceElement( writer, "category", 
category.trim() );
                 }
             }
         }
@@ -729,9 +729,9 @@ public class DoapMojo
             XmlWriterUtil.writeLineBreak( writer );
             XmlWriterUtil.writeCommentText( writer, "Mirror of software 
download web page.", 2 );
             String[] downloadMirrors = StringUtils.split( 
doapOptions.getDownloadMirror(), "," );
-            for ( int i = 0; i < downloadMirrors.length; i++ )
+            for ( String downloadMirror : downloadMirrors )
             {
-                DoapUtil.writeRdfResourceElement( writer, "download-mirror", 
downloadMirrors[i].trim() );
+                DoapUtil.writeRdfResourceElement( writer, "download-mirror", 
downloadMirror.trim() );
             }
         }
     }
@@ -753,9 +753,9 @@ public class DoapMojo
         XmlWriterUtil.writeCommentText( writer, "Operating system that a 
project is limited to.", 2 );
 
         String[] oses = StringUtils.split( doapOptions.getOs(), "," );
-        for ( int i = 0; i < oses.length; i++ )
+        for ( String os : oses )
         {
-            DoapUtil.writeElement( writer, "os", oses[i].trim() );
+            DoapUtil.writeElement( writer, "os", os.trim() );
         }
     }
 
@@ -1776,7 +1776,8 @@ public class DoapMojo
         catch ( MalformedURLException e )
         {
             getLog().error( "The <doapOptions><serviceEndpoint/></doapOptions> 
parameter "
-                                + doapOptions.getServiceEndpoint() + " is not 
a valid URL. Ignored <service-endpoint/> tag." );
+                                + doapOptions.getServiceEndpoint()
+                                + " is not a valid URL. Ignored 
<service-endpoint/> tag." );
             return;
         }
 
@@ -1802,9 +1803,9 @@ public class DoapMojo
         XmlWriterUtil.writeLineBreak( writer );
         XmlWriterUtil.writeCommentText( writer, "Implements.", 2 );
         String[] implementations = StringUtils.split( 
doapOptions.getImplementations(), "," );
-        for ( int i = 0; i < implementations.length; i++ )
+        for ( String implementation : implementations )
         {
-            DoapUtil.writeRdfResourceElement( writer, "implements", 
implementations[i].trim() );
+            DoapUtil.writeRdfResourceElement( writer, "implements", 
implementation.trim() );
         }
     }
 

Modified: maven/plugins/trunk/maven-doap-plugin/src/main/mdo/asfextOptions.mdo
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-doap-plugin/src/main/mdo/asfextOptions.mdo?rev=1045058&r1=1045057&r2=1045058&view=diff
==============================================================================
--- maven/plugins/trunk/maven-doap-plugin/src/main/mdo/asfextOptions.mdo 
(original)
+++ maven/plugins/trunk/maven-doap-plugin/src/main/mdo/asfextOptions.mdo Mon 
Dec 13 10:47:58 2010
@@ -193,9 +193,9 @@ http://svn.apache.org/repos/asf/infrastr
      */
     public static boolean isCategorySupportedByASF( String category )
     {
-        for ( int i = 0; i < CATEGORIES.length; i++ )
+        for ( String category_ : CATEGORIES )
         {
-            if ( CATEGORIES[i].equalsIgnoreCase( category ) )
+            if ( category_.equalsIgnoreCase( category ) )
             {
                 return true;
             }
@@ -212,9 +212,9 @@ http://svn.apache.org/repos/asf/infrastr
      */
     public static boolean isProgrammingLanguageSupportedByASF( String 
programmingLanguage )
     {
-        for ( int i = 0; i < PROGRAMMING_LANGUAGES.length; i++ )
+        for ( String programmingLanguage_ : PROGRAMMING_LANGUAGES )
         {
-            if ( PROGRAMMING_LANGUAGES[i].equalsIgnoreCase( 
programmingLanguage ) )
+            if ( programmingLanguage_.equalsIgnoreCase( programmingLanguage ) )
             {
                 return true;
             }


Reply via email to