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;
}