July 2012 - Maven-helper-plugin-commits - mailman.cloud.codelutin.com

r850 - trunk/src/site
by tchemit＠users.nuiton.org July 14, 2012

July 14, 2012

Author: tchemit Date: 2012-07-14 19:00:02 +0200 (Sat, 14 Jul 2012) New Revision: 850 Url: http://nuiton.org/repositories/revision/maven-helper-plugin/850 Log: Fix site scm url Modified: trunk/src/site/site_en.xml Modified: trunk/src/site/site_en.xml =================================================================== --- trunk/src/site/site_en.xml 2012-07-13 17:10:09 UTC (rev 849) +++ trunk/src/site/site_en.xml 2012-07-14 17:00:02 UTC (rev 850) @@ -126,7 +126,7 @@ projectversion='${project.version}' platform='${project.platform}' projectid='${project.projectId}' - scm='${project.scm.connection}' + scm='${project.scm.developerConnection}' scmwebeditorenabled='${project.scmwebeditorEnabled}' scmwebeditorurl='${project.scmwebeditorUrl}' siteSourcesType='${project.siteSourcesType}'

1 0

r849 - in trunk/src: main/java/org/nuiton/helper/plugin main/java/org/nuiton/io/rest main/java/org/nuiton/io/xpp3 test/java/org/nuiton/io/xpp3
by tchemit＠users.nuiton.org July 13, 2012

July 13, 2012

Author: tchemit Date: 2012-07-13 19:10:09 +0200 (Fri, 13 Jul 2012) New Revision: 849 Url: http://nuiton.org/repositories/revision/maven-helper-plugin/849 Log: update headers Modified: trunk/src/main/java/org/nuiton/helper/plugin/ExecuteRemoteCommandMojo.java trunk/src/main/java/org/nuiton/io/rest/AbstractRequestFactory.java trunk/src/main/java/org/nuiton/io/rest/RequestFactory.java trunk/src/main/java/org/nuiton/io/rest/RestMethod.java trunk/src/main/java/org/nuiton/io/xpp3/AttributeValueToProperty.java trunk/src/main/java/org/nuiton/io/xpp3/TagContentToProperty.java trunk/src/main/java/org/nuiton/io/xpp3/TagTextContentToProperty.java trunk/src/test/java/org/nuiton/io/xpp3/CountryRef.java trunk/src/test/java/org/nuiton/io/xpp3/CountryRefXpp3Reader.java Modified: trunk/src/main/java/org/nuiton/helper/plugin/ExecuteRemoteCommandMojo.java =================================================================== --- trunk/src/main/java/org/nuiton/helper/plugin/ExecuteRemoteCommandMojo.java 2012-07-13 16:25:17 UTC (rev 848) +++ trunk/src/main/java/org/nuiton/helper/plugin/ExecuteRemoteCommandMojo.java 2012-07-13 17:10:09 UTC (rev 849) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2009 - 2012 CodeLutin + * Copyright (C) 2009 - 2012 CodeLutin, Tony Chemit * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU Lesser General Public License as Modified: trunk/src/main/java/org/nuiton/io/rest/AbstractRequestFactory.java =================================================================== --- trunk/src/main/java/org/nuiton/io/rest/AbstractRequestFactory.java 2012-07-13 16:25:17 UTC (rev 848) +++ trunk/src/main/java/org/nuiton/io/rest/AbstractRequestFactory.java 2012-07-13 17:10:09 UTC (rev 849) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2009 - 2012 CodeLutin + * Copyright (C) 2009 - 2012 CodeLutin, Tony Chemit * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU Lesser General Public License as Modified: trunk/src/main/java/org/nuiton/io/rest/RequestFactory.java =================================================================== --- trunk/src/main/java/org/nuiton/io/rest/RequestFactory.java 2012-07-13 16:25:17 UTC (rev 848) +++ trunk/src/main/java/org/nuiton/io/rest/RequestFactory.java 2012-07-13 17:10:09 UTC (rev 849) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2009 - 2012 CodeLutin + * Copyright (C) 2009 - 2012 CodeLutin, Tony Chemit * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU Lesser General Public License as Property changes on: trunk/src/main/java/org/nuiton/io/rest/RestMethod.java ___________________________________________________________________ Added: svn:keywords + Author Date Id Revision HeadURL Added: svn:eol-style + native Modified: trunk/src/main/java/org/nuiton/io/xpp3/AttributeValueToProperty.java =================================================================== --- trunk/src/main/java/org/nuiton/io/xpp3/AttributeValueToProperty.java 2012-07-13 16:25:17 UTC (rev 848) +++ trunk/src/main/java/org/nuiton/io/xpp3/AttributeValueToProperty.java 2012-07-13 17:10:09 UTC (rev 849) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2009 - 2012 CodeLutin + * Copyright (C) 2009 - 2012 CodeLutin, Tony Chemit * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU Lesser General Public License as Modified: trunk/src/main/java/org/nuiton/io/xpp3/TagContentToProperty.java =================================================================== --- trunk/src/main/java/org/nuiton/io/xpp3/TagContentToProperty.java 2012-07-13 16:25:17 UTC (rev 848) +++ trunk/src/main/java/org/nuiton/io/xpp3/TagContentToProperty.java 2012-07-13 17:10:09 UTC (rev 849) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2009 - 2012 CodeLutin + * Copyright (C) 2009 - 2012 CodeLutin, Tony Chemit * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU Lesser General Public License as Modified: trunk/src/main/java/org/nuiton/io/xpp3/TagTextContentToProperty.java =================================================================== --- trunk/src/main/java/org/nuiton/io/xpp3/TagTextContentToProperty.java 2012-07-13 16:25:17 UTC (rev 848) +++ trunk/src/main/java/org/nuiton/io/xpp3/TagTextContentToProperty.java 2012-07-13 17:10:09 UTC (rev 849) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2009 - 2012 CodeLutin + * Copyright (C) 2009 - 2012 CodeLutin, Tony Chemit * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU Lesser General Public License as Modified: trunk/src/test/java/org/nuiton/io/xpp3/CountryRef.java =================================================================== --- trunk/src/test/java/org/nuiton/io/xpp3/CountryRef.java 2012-07-13 16:25:17 UTC (rev 848) +++ trunk/src/test/java/org/nuiton/io/xpp3/CountryRef.java 2012-07-13 17:10:09 UTC (rev 849) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2009 - 2012 CodeLutin + * Copyright (C) 2009 - 2012 CodeLutin, Tony Chemit * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU Lesser General Public License as Modified: trunk/src/test/java/org/nuiton/io/xpp3/CountryRefXpp3Reader.java =================================================================== --- trunk/src/test/java/org/nuiton/io/xpp3/CountryRefXpp3Reader.java 2012-07-13 16:25:17 UTC (rev 848) +++ trunk/src/test/java/org/nuiton/io/xpp3/CountryRefXpp3Reader.java 2012-07-13 17:10:09 UTC (rev 849) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2009 - 2012 CodeLutin + * Copyright (C) 2009 - 2012 CodeLutin, Tony Chemit * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU Lesser General Public License as

1 0

r848 - trunk/src/main/java/org/nuiton/io/rest
by tchemit＠users.nuiton.org July 13, 2012

July 13, 2012

Author: tchemit Date: 2012-07-13 18:25:17 +0200 (Fri, 13 Jul 2012) New Revision: 848 Url: http://nuiton.org/repositories/revision/maven-helper-plugin/848 Log: refs #2183: Introduce a RequestFactory instead of using it inside the rest client (let's get the request builder if needed) Modified: trunk/src/main/java/org/nuiton/io/rest/AbstractRequestFactory.java trunk/src/main/java/org/nuiton/io/rest/RequestFactory.java Modified: trunk/src/main/java/org/nuiton/io/rest/AbstractRequestFactory.java =================================================================== --- trunk/src/main/java/org/nuiton/io/rest/AbstractRequestFactory.java 2012-07-12 22:58:47 UTC (rev 847) +++ trunk/src/main/java/org/nuiton/io/rest/AbstractRequestFactory.java 2012-07-13 16:25:17 UTC (rev 848) @@ -61,7 +61,7 @@ public RestRequest getRequest(String id, Object... args) { RestRequest r; - RestRequestBuilder builder = requestBuilders.get(id); + RestRequestBuilder builder = getRequestBuilder(id); if (builder == null) { throw new IllegalArgumentException( @@ -77,4 +77,11 @@ return r; } + /** {@inheritDoc} */ + public RestRequestBuilder getRequestBuilder(String id) { + RestRequestBuilder builder = requestBuilders.get(id); + + return builder; + } + } Modified: trunk/src/main/java/org/nuiton/io/rest/RequestFactory.java =================================================================== --- trunk/src/main/java/org/nuiton/io/rest/RequestFactory.java 2012-07-12 22:58:47 UTC (rev 847) +++ trunk/src/main/java/org/nuiton/io/rest/RequestFactory.java 2012-07-13 16:25:17 UTC (rev 848) @@ -55,4 +55,11 @@ */ RestRequest getRequest(String id, Object... args); + /** + * Obtain the request builder from a given request id. + * + * @param id the id of request to seek for + * @return the requets builder or {@code null} if not found + */ + RestRequestBuilder getRequestBuilder(String id); }

1 0

r847 - trunk/src/main/java/org/nuiton/io/rest
by tchemit＠users.nuiton.org July 12, 2012

July 12, 2012

Author: tchemit Date: 2012-07-13 00:58:47 +0200 (Fri, 13 Jul 2012) New Revision: 847 Url: http://nuiton.org/repositories/revision/maven-helper-plugin/847 Log: rest request contains now his method Added: trunk/src/main/java/org/nuiton/io/rest/RestMethod.java Modified: trunk/src/main/java/org/nuiton/io/rest/RestClient.java trunk/src/main/java/org/nuiton/io/rest/RestRequest.java trunk/src/main/java/org/nuiton/io/rest/RestSession.java Modified: trunk/src/main/java/org/nuiton/io/rest/RestClient.java =================================================================== --- trunk/src/main/java/org/nuiton/io/rest/RestClient.java 2012-07-12 17:03:55 UTC (rev 846) +++ trunk/src/main/java/org/nuiton/io/rest/RestClient.java 2012-07-12 22:58:47 UTC (rev 847) @@ -25,9 +25,6 @@ package org.nuiton.io.rest; -import org.apache.commons.logging.Log; -import org.apache.commons.logging.LogFactory; - import java.io.IOException; import java.io.InputStream; @@ -39,8 +36,6 @@ */ public abstract class RestClient { - private static final Log log = LogFactory.getLog(RestClient.class); - /** rest client configuration */ protected RestClientConfiguration configuration; @@ -85,15 +80,6 @@ } /** - * Add a request into the client. - * - * @param builder the new request to add - */ - public void addRequestBuilder(RestRequestBuilder builder) { - getRequestFactory().addRequestBuilder(builder); - } - - /** * Obtain a request given his id and the args given. * * @param id id of the request Added: trunk/src/main/java/org/nuiton/io/rest/RestMethod.java =================================================================== --- trunk/src/main/java/org/nuiton/io/rest/RestMethod.java (rev 0) +++ trunk/src/main/java/org/nuiton/io/rest/RestMethod.java 2012-07-12 22:58:47 UTC (rev 847) @@ -0,0 +1,11 @@ +package org.nuiton.io.rest; + +/** + * Possible rest methods. + * + * @author tchemit <chemit(a)codelutin.com> + * @since 1.5 + */ +public enum RestMethod { + GET, POST, PUT, DELETE, HEAD +} Modified: trunk/src/main/java/org/nuiton/io/rest/RestRequest.java =================================================================== --- trunk/src/main/java/org/nuiton/io/rest/RestRequest.java 2012-07-12 17:03:55 UTC (rev 846) +++ trunk/src/main/java/org/nuiton/io/rest/RestRequest.java 2012-07-12 22:58:47 UTC (rev 847) @@ -26,7 +26,6 @@ package org.nuiton.io.rest; import java.io.File; -import java.net.URL; import java.util.Map; /** @@ -36,6 +35,7 @@ * @since 1.0.3 */ public interface RestRequest { + /** @return the splitted path to add to url */ String[] getPath(); @@ -61,5 +61,12 @@ * @return the full url for this request * @since 1.5 */ - String toPath(URL baseUrl); + String toPath(String baseUrl); + + /** + * The method to use for this request. + * + * @return 1.5 + */ + RestMethod getMethod(); } Modified: trunk/src/main/java/org/nuiton/io/rest/RestSession.java =================================================================== --- trunk/src/main/java/org/nuiton/io/rest/RestSession.java 2012-07-12 17:03:55 UTC (rev 846) +++ trunk/src/main/java/org/nuiton/io/rest/RestSession.java 2012-07-12 22:58:47 UTC (rev 847) @@ -30,10 +30,16 @@ import org.apache.commons.httpclient.HttpClient; import org.apache.commons.httpclient.HttpConnection; import org.apache.commons.httpclient.HttpMethod; +import org.apache.commons.httpclient.HttpMethodBase; import org.apache.commons.httpclient.HttpState; import org.apache.commons.httpclient.HttpStatus; +import org.apache.commons.httpclient.NameValuePair; import org.apache.commons.httpclient.StatusLine; +import org.apache.commons.httpclient.methods.DeleteMethod; +import org.apache.commons.httpclient.methods.EntityEnclosingMethod; +import org.apache.commons.httpclient.methods.GetMethod; import org.apache.commons.httpclient.methods.PostMethod; +import org.apache.commons.httpclient.methods.PutMethod; import org.apache.commons.httpclient.methods.multipart.FilePart; import org.apache.commons.httpclient.methods.multipart.MultipartRequestEntity; import org.apache.commons.httpclient.methods.multipart.Part; @@ -183,28 +189,45 @@ ); } - protected String getUri(String... paths) { + protected String getRequestUrl(RestRequest request) { String uri = client.getHostConfiguration().getHostURL(); - for (String path : paths) { - uri += '/' + path; - } - return uri; + String result = request.toPath(uri); + return result; } public HttpMethod doRequest(RestRequest request) throws IOException { - String uri = getUri(request.getPath()); - String[] parameters = request.getParameters(); - Map<String, File> attachments = request.getAttachments(); - if (showRequest) { - log.info("doRequest " + uri); + log.info("doRequest " + getRequestUrl(request)); } if (log.isDebugEnabled()) { log.debug("doRequest with parameters : " + - Arrays.toString(parameters)); + Arrays.toString(request.getParameters())); } - PostMethod gm = new PostMethod(uri); + + HttpMethod gm; + switch (request.getMethod()) { + + case GET: + gm = prepareGetRequest(request); + break; + case POST: + gm = preparePostRequest(request); + + break; + case PUT: + gm = preparePutRequest(request); + + break; + case DELETE: + gm = prepareDeleteRequest(request); + break; + case HEAD: + default: + throw new IllegalStateException("Can not deal with method " + request.getMethod()); + } + + // add cookies if any if (open) { Cookie[] cookies = client.getState().getCookies(); for (Cookie c : cookies) { @@ -216,8 +239,65 @@ } } + client.executeMethod(gm); + + return gm; + } + + public HttpMethod prepareGetRequest(RestRequest request) throws IOException { + + String uri = getRequestUrl(request); + String[] parameters = request.getParameters(); + Map<String, File> attachments = request.getAttachments(); + + + if (attachments != null) { + + // multi-part request + //not possible with a simple Get method + + throw new IllegalStateException("Can not do a GET request with multi-parts, use a POST or UPDATE request"); + } + + GetMethod gm = new GetMethod(uri); + + addParams(gm, parameters); + + return gm; + } + + public HttpMethod prepareDeleteRequest(RestRequest request) throws IOException { + + String uri = getRequestUrl(request); + String[] parameters = request.getParameters(); + Map<String, File> attachments = request.getAttachments(); + + if (attachments != null) { + + // multi-part request + //not possible with a simple Get method + + throw new IllegalStateException("Can not do a DELETE request with multi-parts, use a POST or UPDATE request"); + } + + DeleteMethod gm = new DeleteMethod(uri); + + addParams(gm, parameters); + + return gm; + } + + public HttpMethod preparePostRequest(RestRequest request) throws IOException { + + String uri = getRequestUrl(request); + String[] parameters = request.getParameters(); + Map<String, File> attachments = request.getAttachments(); + + PostMethod gm = new PostMethod(uri); + if (attachments == null) { + // not a multi-part request if (parameters.length > 0) { @@ -234,6 +314,7 @@ } continue; } + gm.addParameter(key, value); } } @@ -241,69 +322,119 @@ // multi-part request - List<Part> paramParts = new ArrayList<Part>(); + prepareMultiPart(gm, attachments, parameters); + } + return gm; + } - if (parameters.length > 0) { + public HttpMethod preparePutRequest(RestRequest request) throws IOException { - // add parameters as part of multi-part + String uri = getRequestUrl(request); + String[] parameters = request.getParameters(); + Map<String, File> attachments = request.getAttachments(); - int nbParams = parameters.length / 2; - for (int i = 0; i < nbParams; i++) { - String key = parameters[2 * i]; - String value = parameters[2 * i + 1]; - if (value == null) { - if (log.isDebugEnabled()) { - log.debug("skip null parameter " + key); - } - continue; - } - paramParts.add(new StringPart(key, value)); + PutMethod gm = new PutMethod(uri); + + if (attachments == null) { + + + // not a multi-part request + + addParams(gm, parameters); + + } else { + + // multi-part request + + prepareMultiPart(gm, attachments, parameters); + } + return gm; + } + + protected void addParams(HttpMethodBase gm, String... parameters) { + if (parameters.length > 0) { + + // add parameters + + int nbParams = parameters.length / 2; + NameValuePair[] params = new NameValuePair[nbParams]; + for (int i = 0; i < nbParams; i++) { + String key = parameters[2 * i]; + String value = parameters[2 * i + 1]; + if (value == null) { if (log.isDebugEnabled()) { - log.debug("add parameter [" + key + "]=" + value); + log.debug("skip null parameter " + key); } + continue; } + params[i] = new NameValuePair(key, value); } + gm.setQueryString(params); + } + } - // add file parts - for (Entry<String, File> entry : attachments.entrySet()) { - String key = entry.getKey(); - File file = entry.getValue(); + protected void prepareMultiPart(EntityEnclosingMethod gm, + Map<String, File> attachments, + String... parameters) throws IOException { + + List<Part> paramParts = new ArrayList<Part>(); + + if (parameters.length > 0) { + + // add parameters as part of multi-part + + int nbParams = parameters.length / 2; + for (int i = 0; i < nbParams; i++) { + String key = parameters[2 * i]; + String value = parameters[2 * i + 1]; + if (value == null) { + if (log.isDebugEnabled()) { + log.debug("skip null parameter " + key); + } + continue; + } + paramParts.add(new StringPart(key, value)); if (log.isDebugEnabled()) { - log.debug("add attachment " + key + "=" + file); + log.debug("add parameter [" + key + "]=" + value); } - paramParts.add(new FilePart( - key, - file.getName(), - file, - FilePart.DEFAULT_CONTENT_TYPE, - configuration.getEncoding()) - ); } - if (attachments.isEmpty()) { - log.warn("no attachment in a multi-part request!"); - } + } - Part[] parts = paramParts.toArray(new Part[paramParts.size()]); - - MultipartRequestEntity entity = - new MultipartRequestEntity(parts, gm.getParams()); + // add file parts + for (Entry<String, File> entry : attachments.entrySet()) { + String key = entry.getKey(); + File file = entry.getValue(); if (log.isDebugEnabled()) { - entity.writeRequest(System.out); + log.debug("add attachment " + key + "=" + file); } - gm.setRequestEntity(entity); -// gm.addRequestHeader("content-type", "multipart/form-data"); - gm.addRequestHeader("content-type", entity.getContentType()); - gm.addRequestHeader("content-length", entity.getContentLength() + - ""); - if (showRequest) { - log.info("content-type : " + entity.getContentType() + - ", content-length : " + entity.getContentLength()); - } + paramParts.add(new FilePart( + key, + file.getName(), + file, + FilePart.DEFAULT_CONTENT_TYPE, + configuration.getEncoding()) + ); + } + if (attachments.isEmpty()) { + log.warn("no attachment in a multi-part request!"); + } + Part[] parts = paramParts.toArray(new Part[paramParts.size()]); + + MultipartRequestEntity entity = + new MultipartRequestEntity(parts, gm.getParams()); + if (log.isDebugEnabled()) { + entity.writeRequest(System.out); } + gm.setRequestEntity(entity); +// gm.addRequestHeader("content-type", "multipart/form-data"); + gm.addRequestHeader("content-type", entity.getContentType()); + gm.addRequestHeader("content-length", entity.getContentLength() + + ""); + if (showRequest) { + log.info("content-type : " + entity.getContentType() + + ", content-length : " + entity.getContentLength()); + } - client.executeMethod(gm); - - return gm; } }

1 0

r846 - in trunk/src/main/java/org/nuiton/io: mail rest
by tchemit＠users.nuiton.org July 12, 2012

July 12, 2012

Author: tchemit Date: 2012-07-12 19:03:55 +0200 (Thu, 12 Jul 2012) New Revision: 846 Url: http://nuiton.org/repositories/revision/maven-helper-plugin/846 Log: add svn properties + fix headers + fix api 1.6 Modified: trunk/src/main/java/org/nuiton/io/mail/ProjectJavamailMailSender.java trunk/src/main/java/org/nuiton/io/rest/AbstractRequestFactory.java trunk/src/main/java/org/nuiton/io/rest/RequestFactory.java Modified: trunk/src/main/java/org/nuiton/io/mail/ProjectJavamailMailSender.java =================================================================== --- trunk/src/main/java/org/nuiton/io/mail/ProjectJavamailMailSender.java 2012-07-12 16:55:57 UTC (rev 845) +++ trunk/src/main/java/org/nuiton/io/mail/ProjectJavamailMailSender.java 2012-07-12 17:03:55 UTC (rev 846) @@ -25,10 +25,11 @@ package org.nuiton.io.mail; -import java.security.Security; -import java.util.Date; -import java.util.Iterator; -import java.util.Properties; +import org.codehaus.plexus.mailsender.AbstractMailSender; +import org.codehaus.plexus.mailsender.MailMessage; +import org.codehaus.plexus.mailsender.MailSenderException; +import org.codehaus.plexus.mailsender.util.DateFormatUtils; +import org.codehaus.plexus.util.StringUtils; import javax.mail.Authenticator; import javax.mail.Message; @@ -38,20 +39,18 @@ import javax.mail.Transport; import javax.mail.internet.InternetAddress; import javax.mail.internet.MimeMessage; +import java.security.Provider; +import java.security.Security; +import java.util.Date; +import java.util.Iterator; +import java.util.Properties; -import com.sun.net.ssl.internal.ssl.Provider; -import org.codehaus.plexus.mailsender.AbstractMailSender; -import org.codehaus.plexus.mailsender.MailMessage; -import org.codehaus.plexus.mailsender.MailSenderException; -import org.codehaus.plexus.mailsender.util.DateFormatUtils; -import org.codehaus.plexus.util.StringUtils; - /** * Helper class for sending email. - * + * * Note : this code was stolen in {@code maven-changes-plugin}, should thanks * them... - * + * * @author tchemit <chemit(a)codelutin.com> * @since 1.0.3 */ @@ -59,10 +58,12 @@ extends AbstractMailSender { private static final String SSL_FACTORY = "javax.net.ssl.SSLSocketFactory"; + // ---------------------------------------------------------------------- // // ---------------------------------------------------------------------- private Properties userProperties; + private Properties props; // ---------------------------------------------------------------------- @@ -90,7 +91,7 @@ props.put("mail.debug", String.valueOf(getLogger().isDebugEnabled())); if (isSslMode()) { - Security.addProvider(new Provider()); + Security.addProvider(getSsllProvider()); props.put("mail.smtp.socketFactory.port", String.valueOf(getSmtpPort())); @@ -100,7 +101,7 @@ } if (userProperties != null) { - for (Iterator<?> i = userProperties.keySet().iterator(); i.hasNext();) { + for (Iterator<?> i = userProperties.keySet().iterator(); i.hasNext(); ) { String key = (String) i.next(); String value = userProperties.getProperty(key); @@ -110,6 +111,17 @@ } } + private Provider getSsllProvider() { + try { + Class<?> aClass = + Class.forName("com.sun.net.ssl.internal.ssl.Provider"); + return (Provider) aClass.newInstance(); + } catch (Exception e) { + + throw new IllegalStateException("Could not obtain ssl provider", e); + } + } + // ---------------------------------------------------------------------- // MailSender Implementation // ---------------------------------------------------------------------- @@ -142,7 +154,7 @@ if (mail.getToAddresses().size() > 0) { InternetAddress[] addressTo = new InternetAddress[mail.getToAddresses().size()]; int count = 0; - for (Iterator<?> i = mail.getToAddresses().iterator(); i.hasNext();) { + for (Iterator<?> i = mail.getToAddresses().iterator(); i.hasNext(); ) { String address = ((MailMessage.Address) i.next()).getRfc2822Address(); addressTo[count++] = new InternetAddress(address); } @@ -152,7 +164,7 @@ if (mail.getCcAddresses().size() > 0) { InternetAddress[] addressCc = new InternetAddress[mail.getCcAddresses().size()]; int count = 0; - for (Iterator<?> i = mail.getCcAddresses().iterator(); i.hasNext();) { + for (Iterator<?> i = mail.getCcAddresses().iterator(); i.hasNext(); ) { String address = ((MailMessage.Address) i.next()).getRfc2822Address(); addressCc[count++] = new InternetAddress(address); } @@ -162,7 +174,7 @@ if (mail.getBccAddresses().size() > 0) { InternetAddress[] addressBcc = new InternetAddress[mail.getBccAddresses().size()]; int count = 0; - for (Iterator<?> i = mail.getBccAddresses().iterator(); i.hasNext();) { + for (Iterator<?> i = mail.getBccAddresses().iterator(); i.hasNext(); ) { String address = ((MailMessage.Address) i.next()).getRfc2822Address(); addressBcc[count++] = new InternetAddress(address); } Modified: trunk/src/main/java/org/nuiton/io/rest/AbstractRequestFactory.java =================================================================== --- trunk/src/main/java/org/nuiton/io/rest/AbstractRequestFactory.java 2012-07-12 16:55:57 UTC (rev 845) +++ trunk/src/main/java/org/nuiton/io/rest/AbstractRequestFactory.java 2012-07-12 17:03:55 UTC (rev 846) @@ -1,4 +1,27 @@ package org.nuiton.io.rest; +/* + * #%L + * Helper Maven Plugin + * $Id$ + * $HeadURL$ + * %% + * Copyright (C) 2009 - 2012 CodeLutin + * %% + * This program is free software: you can redistribute it and/or modify + * it under the terms of the GNU Lesser General Public License as + * published by the Free Software Foundation, either version 3 of the + * License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Lesser Public License for more details. + * + * You should have received a copy of the GNU General Lesser Public + * License along with this program. If not, see + * <http://www.gnu.org/licenses/lgpl-3.0.html>. + * #L% + */ import com.google.common.collect.Maps; Property changes on: trunk/src/main/java/org/nuiton/io/rest/AbstractRequestFactory.java ___________________________________________________________________ Added: svn:keywords + Author Date Id Revision HeadURL Added: svn:eol-style + native Modified: trunk/src/main/java/org/nuiton/io/rest/RequestFactory.java =================================================================== --- trunk/src/main/java/org/nuiton/io/rest/RequestFactory.java 2012-07-12 16:55:57 UTC (rev 845) +++ trunk/src/main/java/org/nuiton/io/rest/RequestFactory.java 2012-07-12 17:03:55 UTC (rev 846) @@ -1,4 +1,27 @@ package org.nuiton.io.rest; +/* + * #%L + * Helper Maven Plugin + * $Id$ + * $HeadURL$ + * %% + * Copyright (C) 2009 - 2012 CodeLutin + * %% + * This program is free software: you can redistribute it and/or modify + * it under the terms of the GNU Lesser General Public License as + * published by the Free Software Foundation, either version 3 of the + * License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Lesser Public License for more details. + * + * You should have received a copy of the GNU General Lesser Public + * License along with this program. If not, see + * <http://www.gnu.org/licenses/lgpl-3.0.html>. + * #L% + */ import java.text.DateFormat; import java.text.SimpleDateFormat; Property changes on: trunk/src/main/java/org/nuiton/io/rest/RequestFactory.java ___________________________________________________________________ Added: svn:keywords + Author Date Id Revision HeadURL Added: svn:eol-style + native

1 0

r845 - in trunk: . src/main/java/org/nuiton/io/rest src/main/java/org/nuiton/io/xpp3
by tchemit＠users.nuiton.org July 12, 2012

July 12, 2012

Author: tchemit Date: 2012-07-12 18:55:57 +0200 (Thu, 12 Jul 2012) New Revision: 845 Url: http://nuiton.org/repositories/revision/maven-helper-plugin/845 Log: fixes #2183: Introduce a RequestFactory instead of using it inside the rest client fixes #2182: updates to commons lang 3 Added: trunk/src/main/java/org/nuiton/io/rest/AbstractRequestFactory.java trunk/src/main/java/org/nuiton/io/rest/RequestFactory.java Modified: trunk/pom.xml trunk/src/main/java/org/nuiton/io/rest/DefaultRestClientConfiguration.java trunk/src/main/java/org/nuiton/io/rest/RestClient.java trunk/src/main/java/org/nuiton/io/rest/RestRequest.java trunk/src/main/java/org/nuiton/io/rest/RestRequestBuilder.java trunk/src/main/java/org/nuiton/io/xpp3/DefaultDataConverter.java trunk/src/main/java/org/nuiton/io/xpp3/PropertyMapper.java Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2012-07-12 16:53:52 UTC (rev 844) +++ trunk/pom.xml 2012-07-12 16:55:57 UTC (rev 845) @@ -1,10 +1,7 @@ <?xml version="1.0" encoding="UTF-8"?>  -<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd"> +<project xmlns="http://maven.apache.org/POM/4.0.0" + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd"> <modelVersion>4.0.0</modelVersion> @@ -56,8 +55,8 @@ </dependency> <dependency> - <groupId>commons-lang</groupId> - <artifactId>commons-lang</artifactId> + <groupId>org.apache.commons</groupId> + <artifactId>commons-lang3</artifactId> </dependency> <dependency> @@ -146,7 +145,7 @@ <version>${mavenVersion}</version> <scope>provided</scope> </dependency> - + <dependency> <groupId>org.apache.maven</groupId> <artifactId>maven-plugin-api</artifactId> @@ -222,7 +221,7 @@ <artifactId>log4j</artifactId> <scope>test</scope> </dependency> - + <dependency> <groupId>velocity</groupId> <artifactId>velocity</artifactId> @@ -245,7 +244,7 @@ <id>tchemit</id> <email>chemit at codelutin dot com</email> <organization>CodeLutin</organization> - <timezone>+2</timezone> + <timezone>Europe/Paris</timezone> <roles> <role>Lead</role> <role>Developer</role> @@ -256,7 +255,7 @@ <id>ymartel</id> <email>martel at codelutin dot com</email> <organization>CodeLutin</organization> - <timezone>+2</timezone> + <timezone>Europe/Paris</timezone> <roles> <role>Developer</role> </roles> @@ -268,9 +267,15 @@  <scm> - <connection>scm:svn:http://svn.nuiton.org/svn/maven-helper-plugin/trunk</connection> - <developerConnection>scm:svn:http://svn.nuiton.org/svn/maven-helper-plugin/trunk</developerConnection> - <url>http://www.nuiton.org/repositories/browse/maven-helper-plugin/trunk</url> + <connection> + scm:svn:http://svn.nuiton.org/svn/maven-helper-plugin/trunk + </connection> + <developerConnection> + scm:svn:http://svn.nuiton.org/svn/maven-helper-plugin/trunk + </developerConnection> + <url> + http://www.nuiton.org/repositories/browse/maven-helper-plugin/trunk + </url> </scm> <distributionManagement> @@ -301,11 +306,6 @@  <siteSourcesType>apt</siteSourcesType> -  - <maven.compiler.source>1.5</maven.compiler.source> - <maven.compiler.target>1.5</maven.compiler.target> - </properties> <build> @@ -487,7 +487,7 @@ </plugins> </build> - + </profile>  @@ -527,7 +527,7 @@ </plugins> </build> - + </profile>  Added: trunk/src/main/java/org/nuiton/io/rest/AbstractRequestFactory.java =================================================================== --- trunk/src/main/java/org/nuiton/io/rest/AbstractRequestFactory.java (rev 0) +++ trunk/src/main/java/org/nuiton/io/rest/AbstractRequestFactory.java 2012-07-12 16:55:57 UTC (rev 845) @@ -0,0 +1,57 @@ +package org.nuiton.io.rest; + +import com.google.common.collect.Maps; + +import java.text.DateFormat; +import java.text.SimpleDateFormat; +import java.util.Map; + +/** + * Abstract implementation of {@link RequestFactory}. + * + * @author tchemit <chemit(a)codelutin.com> + * @since 1.5 + */ +public abstract class AbstractRequestFactory implements RequestFactory { + + public static final DateFormat DATE_FORMAT = + new SimpleDateFormat("yyyy-MM-dd"); + + /** registred requests */ + protected final Map<String, RestRequestBuilder> requestBuilders; + + public AbstractRequestFactory() { + requestBuilders = Maps.newTreeMap(); + } + + /** {@inheritDoc} */ + public void addRequestBuilder(RestRequestBuilder builder) { + String name = builder.getName(); + if (requestBuilders.containsKey(name)) { + throw new IllegalArgumentException("a request builder with name " + + name + " already exists."); + } + requestBuilders.put(name, builder); + } + + /** {@inheritDoc} */ + public RestRequest getRequest(String id, Object... args) { + + RestRequest r; + RestRequestBuilder builder = requestBuilders.get(id); + if (builder == null) { + + throw new IllegalArgumentException( + "Could not find any request builder for request named '" + + id + "'"); + } + r = builder.create(args); + if (r == null) { + throw new IllegalArgumentException( + "Could not find any request named '" + id + + "' using builder " + builder); + } + return r; + } + +} Modified: trunk/src/main/java/org/nuiton/io/rest/DefaultRestClientConfiguration.java =================================================================== --- trunk/src/main/java/org/nuiton/io/rest/DefaultRestClientConfiguration.java 2012-07-12 16:53:52 UTC (rev 844) +++ trunk/src/main/java/org/nuiton/io/rest/DefaultRestClientConfiguration.java 2012-07-12 16:55:57 UTC (rev 845) @@ -25,8 +25,8 @@ package org.nuiton.io.rest; -import org.apache.commons.lang.builder.ToStringBuilder; -import org.apache.commons.lang.builder.ToStringStyle; +import org.apache.commons.lang3.builder.ToStringBuilder; +import org.apache.commons.lang3.builder.ToStringStyle; import java.net.URL; Added: trunk/src/main/java/org/nuiton/io/rest/RequestFactory.java =================================================================== --- trunk/src/main/java/org/nuiton/io/rest/RequestFactory.java (rev 0) +++ trunk/src/main/java/org/nuiton/io/rest/RequestFactory.java 2012-07-12 16:55:57 UTC (rev 845) @@ -0,0 +1,35 @@ +package org.nuiton.io.rest; + +import java.text.DateFormat; +import java.text.SimpleDateFormat; + +/** + * Factory of {@link RestRequest}. + * + * @author tchemit <chemit(a)codelutin.com> + * @since 1.5 + */ +public interface RequestFactory { + + public static final DateFormat DATE_FORMAT = + new SimpleDateFormat("yyyy-MM-dd"); + + void addDefaultRequests(); + + /** + * Add a request into the factory. + * + * @param builder the new request to add + */ + void addRequestBuilder(RestRequestBuilder builder); + + /** + * Obtain a request given his id and the args given. + * + * @param id id of the request + * @param args args passed to build the request + * @return the new request + */ + RestRequest getRequest(String id, Object... args); + +} Modified: trunk/src/main/java/org/nuiton/io/rest/RestClient.java =================================================================== --- trunk/src/main/java/org/nuiton/io/rest/RestClient.java 2012-07-12 16:53:52 UTC (rev 844) +++ trunk/src/main/java/org/nuiton/io/rest/RestClient.java 2012-07-12 16:55:57 UTC (rev 845) @@ -30,8 +30,6 @@ import java.io.IOException; import java.io.InputStream; -import java.util.Map; -import java.util.TreeMap; /** * Abtract REST client. @@ -49,15 +47,13 @@ /** rest session */ protected RestSession session; - /** registred requests */ - protected final Map<String, RestRequestBuilder> requestBuilders; - /** - * Add the default available requests for this client. - * - * Note: This method is invoked in the constructor of the client. + * Obtains the underlying factory of requests. + * + * @return the factory of requests + * @since 1.5 */ - protected abstract void addDefaultRequests(); + public abstract RequestFactory getRequestFactory(); /** * Open the client. @@ -82,12 +78,9 @@ protected abstract void close(RestSession session) throws IOException; public RestClient() { - requestBuilders = new TreeMap<String, RestRequestBuilder>(); - addDefaultRequests(); } public RestClient(RestClientConfiguration configuration) { - this(); this.configuration = configuration; } @@ -97,11 +90,7 @@ * @param builder the new request to add */ public void addRequestBuilder(RestRequestBuilder builder) { - String name = builder.getName(); - if (requestBuilders.containsKey(name)) { - throw new IllegalArgumentException("a request builder with name " + name + " already exists."); - } - requestBuilders.put(name, builder); + getRequestFactory().addRequestBuilder(builder); } /** @@ -112,12 +101,7 @@ * @return the new request */ public RestRequest getRequest(String id, Object... args) { - RestRequestBuilder builder = requestBuilders.get(id); - if (builder == null) { - // pas de builder de requete... - return null; - } - RestRequest r = builder.create(args); + RestRequest r = getRequestFactory().getRequest(id, args); return r; } Modified: trunk/src/main/java/org/nuiton/io/rest/RestRequest.java =================================================================== --- trunk/src/main/java/org/nuiton/io/rest/RestRequest.java 2012-07-12 16:53:52 UTC (rev 844) +++ trunk/src/main/java/org/nuiton/io/rest/RestRequest.java 2012-07-12 16:55:57 UTC (rev 845) @@ -26,6 +26,7 @@ package org.nuiton.io.rest; import java.io.File; +import java.net.URL; import java.util.Map; /** @@ -44,6 +45,21 @@ */ String[] getParameters(); + /** + * @return {@code true} if request has some attachments. + * @since 1.5 + */ + boolean containsAttachments(); + /** @return the map of attachment to pass to request */ Map<String, File> getAttachments(); + + /** + * Obtain the full url of the request. + * + * @param baseUrl the base url to use + * @return the full url for this request + * @since 1.5 + */ + String toPath(URL baseUrl); } Modified: trunk/src/main/java/org/nuiton/io/rest/RestRequestBuilder.java =================================================================== --- trunk/src/main/java/org/nuiton/io/rest/RestRequestBuilder.java 2012-07-12 16:53:52 UTC (rev 844) +++ trunk/src/main/java/org/nuiton/io/rest/RestRequestBuilder.java 2012-07-12 16:55:57 UTC (rev 845) @@ -25,13 +25,15 @@ package org.nuiton.io.rest; +import java.io.Serializable; + /** * The contract of a request builder. * * @author tchemit <chemit(a)codelutin.com> * @since 1.0.3 */ -public interface RestRequestBuilder { +public interface RestRequestBuilder extends Serializable { /** @return the unique name of the request builder */ String getName(); Modified: trunk/src/main/java/org/nuiton/io/xpp3/DefaultDataConverter.java =================================================================== --- trunk/src/main/java/org/nuiton/io/xpp3/DefaultDataConverter.java 2012-07-12 16:53:52 UTC (rev 844) +++ trunk/src/main/java/org/nuiton/io/xpp3/DefaultDataConverter.java 2012-07-12 16:55:57 UTC (rev 845) @@ -35,7 +35,6 @@ public enum DefaultDataConverter implements DataConverter { Boolean { - @Override public Object convert(String t) throws Exception { Object r = java.lang.Boolean.valueOf(t); @@ -43,7 +42,6 @@ } }, Short { - @Override public Object convert(String t) throws Exception { Object r = java.lang.Short.valueOf(t); @@ -51,7 +49,6 @@ } }, Integer { - @Override public Object convert(String t) throws Exception { Object r = java.lang.Integer.valueOf(t); @@ -59,7 +56,6 @@ } }, Long { - @Override public Object convert(String t) throws Exception { Object r = java.lang.Long.valueOf(t); @@ -67,7 +63,6 @@ } }, Float { - @Override public Object convert(String t) throws Exception { Object r = java.lang.Float.valueOf(t); @@ -75,7 +70,6 @@ } }, Double { - @Override public Object convert(String t) throws Exception { Object r = java.lang.Double.valueOf(t); @@ -83,7 +77,6 @@ } }, Text { - @Override public Object convert(String t) throws Exception { return t; Modified: trunk/src/main/java/org/nuiton/io/xpp3/PropertyMapper.java =================================================================== --- trunk/src/main/java/org/nuiton/io/xpp3/PropertyMapper.java 2012-07-12 16:53:52 UTC (rev 844) +++ trunk/src/main/java/org/nuiton/io/xpp3/PropertyMapper.java 2012-07-12 16:55:57 UTC (rev 845) @@ -26,8 +26,8 @@ package org.nuiton.io.xpp3; import com.google.common.base.Function; -import org.apache.commons.lang.StringUtils; -import org.apache.commons.lang.builder.ToStringBuilder; +import org.apache.commons.lang3.StringUtils; +import org.apache.commons.lang3.builder.ToStringBuilder; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.codehaus.plexus.util.xml.pull.XmlPullParser;

1 0

r844 - trunk/src/main/java/org/nuiton/helper/plugin
by tchemit＠users.nuiton.org July 12, 2012

July 12, 2012

Author: tchemit Date: 2012-07-12 18:53:52 +0200 (Thu, 12 Jul 2012) New Revision: 844 Url: http://nuiton.org/repositories/revision/maven-helper-plugin/844 Log: fixes #925: Use maven-plugin-plugin 3 api Modified: trunk/src/main/java/org/nuiton/helper/plugin/CheckAutoContainerPlugin.java trunk/src/main/java/org/nuiton/helper/plugin/CollectFilesMojo.java trunk/src/main/java/org/nuiton/helper/plugin/ExecuteRemoteCommandMojo.java trunk/src/main/java/org/nuiton/helper/plugin/SendEmailMojo.java trunk/src/main/java/org/nuiton/helper/plugin/ShareServerSecretPlugin.java Modified: trunk/src/main/java/org/nuiton/helper/plugin/CheckAutoContainerPlugin.java =================================================================== --- trunk/src/main/java/org/nuiton/helper/plugin/CheckAutoContainerPlugin.java 2012-07-09 10:20:16 UTC (rev 843) +++ trunk/src/main/java/org/nuiton/helper/plugin/CheckAutoContainerPlugin.java 2012-07-12 16:53:52 UTC (rev 844) @@ -37,6 +37,11 @@ import org.apache.maven.artifact.resolver.ArtifactResolver; import org.apache.maven.plugin.MojoFailureException; import org.apache.maven.plugin.logging.Log; +import org.apache.maven.plugins.annotations.Component; +import org.apache.maven.plugins.annotations.LifecyclePhase; +import org.apache.maven.plugins.annotations.Mojo; +import org.apache.maven.plugins.annotations.Parameter; +import org.apache.maven.plugins.annotations.ResolutionScope; import org.apache.maven.project.MavenProject; import org.apache.maven.settings.Proxy; import org.apache.maven.wagon.ConnectionException; @@ -59,13 +64,9 @@ * * @author tchemit <chemit(a)codelutin.com> * @version $Id$ - * @goal check-auto-container - * @phase validate - * @requiresProject true - * @requiresOnline true - * @requiresDependencyResolution runtime * @since 1.2.5 */ +@Mojo(name = "check-auto-container", defaultPhase = LifecyclePhase.VALIDATE, requiresProject = true, requiresOnline = true, requiresDependencyResolution = ResolutionScope.RUNTIME) public class CheckAutoContainerPlugin extends AbstractPlugin { /** @@ -73,34 +74,34 @@ * * Keys are repository id and Values are repository url. * - * @parameter * @since 1.2.5 */ + @Parameter protected Map<String, String> repositories; /** * A flag to add the maven central repository http://repo1.maven.org/maven2 * to {@link #repositories}. * - * @parameter expression="${addMavenCentral}" default-value="false" * @since 1.2.5 */ + @Parameter(property = "addMavenCentral", defaultValue = "false") protected boolean addMavenCentral; /** * A flag to fail if project is not auto container. * - * @parameter expression="${helper.failIfNotSafe}" default-value="false" * @since 1.2.5 */ + @Parameter(property = "helper.failIfNotSafe", defaultValue = "false") protected boolean failIfNotSafe; /** * A flag to activate verbose mode. * - * @parameter expression="${helper.verbose}" default-value="${maven.verbose}" * @since 1.2.5 */ + @Parameter(property = "helper.verbose", defaultValue = "${maven.verbose}") protected boolean verbose; /** @@ -109,88 +110,73 @@ * Note: By default, value is {@code true} since it is not necessary * to check twice for a same artifact. * - * @parameter expression="${helper.runOnce}" default-value="true" * @since 1.2.6 */ + @Parameter(property = "helper.runOnce", defaultValue = "true") protected boolean runOnce; /** * Project. * - * @parameter default-value="${project}" - * @required - * @readonly * @since 1.2.5 */ + @Component protected MavenProject project; /** * The projects in reactor (used to detected sibling dependencies). * - * @parameter expression="${reactorProjects}" - * @readonly * @since 1.2.5 */ + @Parameter(property = "reactorProjects", readonly = true) protected List<?> reactorProjects; /** * Active proxy from settings (if any). * - * @parameter default-value="${settings.activeProxy}" - * @required - * @readonly * @since 1.2.5 */ + @Parameter(property = "settings.activeProxy", required = true, readonly = true) protected Proxy proxy; /** * Local repository. * - * @parameter expression="${localRepository}" - * @required - * @readonly * @since 1.2.5 */ + @Parameter(property = "localRepository", required = true, readonly = true) protected ArtifactRepository localRepository; /** * Artifact repository factory component. * - * @component - * @required - * @readonly * @since 1.2.5 */ + @Component protected ArtifactRepositoryFactory artifactRepositoryFactory; /** * Artifact factory component. * - * @component - * @required - * @readonly * @since 1.2.5 */ + @Component protected ArtifactFactory factory; /** * Artifact resolver component. * - * @component - * @required - * @readonly * @since 1.2.5 */ + @Component protected ArtifactResolver resolver; /** * Wagon manager component. * - * @component - * @required - * @readonly * @since 1.2.5 */ + @Component protected WagonManager wagonManager; /** Modified: trunk/src/main/java/org/nuiton/helper/plugin/CollectFilesMojo.java =================================================================== --- trunk/src/main/java/org/nuiton/helper/plugin/CollectFilesMojo.java 2012-07-09 10:20:16 UTC (rev 843) +++ trunk/src/main/java/org/nuiton/helper/plugin/CollectFilesMojo.java 2012-07-12 16:53:52 UTC (rev 844) @@ -25,11 +25,15 @@ package org.nuiton.helper.plugin; +import org.apache.commons.lang3.StringUtils; import org.apache.maven.artifact.Artifact; import org.apache.maven.artifact.metadata.ArtifactMetadata; import org.apache.maven.artifact.repository.ArtifactRepository; import org.apache.maven.plugin.MojoExecutionException; import org.apache.maven.plugin.MojoFailureException; +import org.apache.maven.plugins.annotations.Component; +import org.apache.maven.plugins.annotations.Mojo; +import org.apache.maven.plugins.annotations.Parameter; import org.apache.maven.project.MavenProject; import org.nuiton.plugin.AbstractPlugin; import org.nuiton.plugin.PluginHelper; @@ -46,69 +50,57 @@ * Collect files some files from a project and copy them into a directory. * * @author tchemit <chemit(a)codelutin.com> - * @goal collect-files - * @requiresProject true * @since 1.1.0 */ +@Mojo(name = "collect-files", requiresProject = true) public class CollectFilesMojo extends AbstractPlugin implements PluginWithEncoding { /** * The projects in the reactor. * - * @parameter expression="${reactorProjects}" - * @readonly * @since 1.2.4 */ + @Parameter(property = "reactorProjects", readonly = true, required = true) protected List<?> reactorProjects; /** * Fallback maven project (for maven 2). * - * @parameter default-value="${project}" - * @required - * @readonly * @since 1.0.3 */ + @Component protected MavenProject project; /** * The artifacts to publish from the project build. * - * @parameter expression="${project.attachedArtifacts}" - * @required - * @readonly * @since 1.0.0 */ + @Parameter(property = "project.attachedArtifacts", readonly = true, required = true) protected List<?> attachedArtifacts; /** * The local repository. * - * @parameter expression="${localRepository}" - * @required - * @readonly * @since 1.2.2 */ + @Parameter(property = "localRepository", readonly = true, required = true) protected ArtifactRepository localRepository; /** * The remote repository. * - * @parameter expression="${project.distributionManagementArtifactRepository}" - * @required - * @readonly * @since 1.2.2 */ + @Parameter(property = "project.distributionManagementArtifactRepository", readonly = true, required = true) protected ArtifactRepository deploymentRepository; /** * Project artifact. * - * @parameter expression="${project.artifact}" - * @required - * @readonly * @since 1.2.2 */ + @Parameter(property = "project.artifact", readonly = true, required = true) protected Artifact artifact; /** @@ -116,9 +108,9 @@ * * Multi values can be used, separated by comma. * - * @parameter expression="${helper.extraFiles}" * @since 1.0.0 */ + @Parameter(property = "helper.extraFiles") protected String extraFiles; /** @@ -126,9 +118,9 @@ * * If no Set - no include filter * - * @parameter expression="${helper.includes}" default-value="" * @since 1.0.0 */ + @Parameter(property = "helper.includes", defaultValue = "") protected String includes; /** @@ -136,9 +128,9 @@ * * If no Set - no exclude filter * - * @parameter expression="${helper.excludes}" default-value="" * @since 1.0.0 */ + @Parameter(property = "helper.excludes", defaultValue = "") protected String excludes; /** @@ -148,9 +140,9 @@ * the property of the root module, because we need to push collected files * to only one place. * - * @parameter expression="${helper.outputDirectory}" default-value="target/collect" * @since 1.0.0 */ + @Parameter(property = "helper.outputDirectory", defaultValue = "target/collect") protected String outputDirectory; /** @@ -158,66 +150,65 @@ * * Note : If not Set, will not generate the description file. * - * @parameter expression="${helper.descriptionFile}" * @since 1.0.0 */ + @Parameter(property = "helper.descriptionFile") protected String descriptionFile; /** * Un flag pour collecter aussi les fichiers attaches au projet. * - * @parameter expression="${helper.includeAttached}" default-value="true" * @since 1.0.0 */ + @Parameter(property = "helper.includeAttached", defaultValue = "true") protected boolean includeAttached; /** * Un flag pour collecter aussi les fichiers des sites attaches au projet. * - * @parameter expression="${helper.includeSiteAttached}" default-value="false" * @since 1.2.2 */ + @Parameter(property = "helper.includeSiteAttached", defaultValue = "false") protected boolean includeSiteAttached; /** * Un flag pour recopier les fichiers collectés. * - * @parameter expression="${helper.copyFiles}" default-value="true" * @since 1.0.0 */ + @Parameter(property = "helper.copyFiles", defaultValue = "true") protected boolean copyFiles; /** * Un flag pour activer le mode verbeux. * - * @parameter expression="${helper.verbose}" default-value="${maven.verbose}" * @since 1.0.0 */ + @Parameter(property = "helper.verbose", defaultValue = "${maven.verbose}") protected boolean verbose; /** * A flag to skip the goal. * - * @parameter expression="${helper.skip}" default-value="false" * @since 1.0.0 */ + @Parameter(property = "helper.skip", defaultValue = "false") protected boolean skip; /** * Un flag pour activer le mode verbeux. * - * @parameter expression="${helper.dryRun}" default-value="false" * @since 1.0.0 */ + @Parameter(property = "helper.dryRun", defaultValue = "false") protected boolean dryRun; /** * Encoding a utiliser pour lire et ecrire les fichiers. * - * @parameter expression="${helper.encoding}" default-value="${project.build.sourceEncoding}" - * @required * @since 1.0.0 */ + @Parameter(property = "helper.encoding", defaultValue = "${project.build.sourceEncoding}", required = true) protected String encoding; /** Files to collect */ @@ -293,7 +284,7 @@ boolean withDescriptionFile = false; - if (descriptionFile != null && !descriptionFile.trim().isEmpty()) { + if (StringUtils.isNotBlank(descriptionFile)) { description = new File(rootProject.getBasedir(), descriptionFile); @@ -389,11 +380,9 @@ protected List<File> getFiles() { - Pattern includePattern = includes == null || - includes.trim().isEmpty() ? null : + Pattern includePattern = StringUtils.isBlank(includes) ? null : Pattern.compile(includes.trim()); - Pattern excludePattern = excludes == null || - excludes.trim().isEmpty() ? null : + Pattern excludePattern = StringUtils.isBlank(excludes) ? null : Pattern.compile(excludes.trim()); List<File> result = new ArrayList<File>(); @@ -423,7 +412,7 @@ // attach extra files - if (extraFiles != null && !extraFiles.trim().isEmpty()) { + if (StringUtils.isNotBlank(extraFiles)) { File basedir = project.getBasedir(); for (String path : extraFiles.split(",")) { Modified: trunk/src/main/java/org/nuiton/helper/plugin/ExecuteRemoteCommandMojo.java =================================================================== --- trunk/src/main/java/org/nuiton/helper/plugin/ExecuteRemoteCommandMojo.java 2012-07-09 10:20:16 UTC (rev 843) +++ trunk/src/main/java/org/nuiton/helper/plugin/ExecuteRemoteCommandMojo.java 2012-07-12 16:53:52 UTC (rev 844) @@ -24,10 +24,14 @@ */ package org.nuiton.helper.plugin; -import org.apache.commons.lang.StringUtils; +import org.apache.commons.lang3.StringUtils; import org.apache.maven.artifact.manager.WagonManager; import org.apache.maven.plugin.MojoExecutionException; import org.apache.maven.plugin.logging.Log; +import org.apache.maven.plugins.annotations.Component; +import org.apache.maven.plugins.annotations.LifecyclePhase; +import org.apache.maven.plugins.annotations.Mojo; +import org.apache.maven.plugins.annotations.Parameter; import org.apache.maven.project.MavenProject; import org.apache.maven.settings.Proxy; import org.apache.maven.settings.Server; @@ -45,82 +49,54 @@ * Mojo to execute on a remote server some scripts. * * @author tchemit <chemit(a)codelutin.com> - * @goal execute-remote-command - * @phase validate - * @requiresProject true - * @requiresOnline true * @since 1.3.1 */ +@Mojo(name = "execute-remote-command", defaultPhase = LifecyclePhase.VALIDATE, + requiresOnline = true, requiresProject = true) public class ExecuteRemoteCommandMojo extends AbstractPlugin { - /** - * A flag to activate verbose mode. - * - * @parameter expression="${helper.verbose}" default-value="${maven.verbose}" - */ + /** A flag to activate verbose mode. */ + @Parameter(property = "helper.verbose", defaultValue = "${maven.verbose}") protected boolean verbose; - /** - * Project. - * - * @parameter default-value="${project}" - * @required - * @readonly - */ + /** Project. */ + @Component protected MavenProject project; - /** - * Active proxy from settings (if any). - * - * @parameter default-value="${settings.activeProxy}" - * @required - * @readonly - */ + /** Active proxy from settings (if any). */ + @Parameter(defaultValue = "${settings.activeProxy}", required = true, readonly = true) protected Proxy proxy; /** * Settings. * - * @parameter default-value="${settings}" - * @required - * @readonly * @since 1.1.0 */ + @Component protected Settings settings; /** * Id of the server to use to obtain user login. * * Must be found in your settings.xml. - * - * @parameter expression="${helper.repositoryId}" - * @required */ + @Parameter(property = "helper.repositoryId", required = true) protected String repositoryId; /** * Url where to execute script. * * The url must contains the wagon protocol (says scp:... or scpexe:...). - * - * @parameter expression="${helper.repositoryUrl}" - * @required */ + @Parameter(property = "helper.repositoryUrl", required = true) protected String repositoryUrl; - /** - * Command to execute on remote server. - * - * @parameter expression="${helper.command}" - * @required - */ + /** Command to execute on remote server. */ + @Parameter(property = "helper.command", required = true) protected String command; - /** - * Wagon manager component. - * - * @component - */ + /** Wagon manager component. */ + @Component protected WagonManager wagonManager; private CommandExecutor commandExecutor; @@ -145,7 +121,7 @@ if (server == null) { throw new MojoExecutionException( "Could not find server with id '" + repositoryId + - "', check your settings.xml file."); + "', check your settings.xml file."); } Repository repository = new Repository(repositoryId, repositoryUrl); @@ -157,7 +133,7 @@ throw new MojoExecutionException( "The wagon " + wagon + " is not a command executor, " + - "will not be able to execute command."); + "will not be able to execute command."); } commandExecutor = (CommandExecutor) wagon; Modified: trunk/src/main/java/org/nuiton/helper/plugin/SendEmailMojo.java =================================================================== --- trunk/src/main/java/org/nuiton/helper/plugin/SendEmailMojo.java 2012-07-09 10:20:16 UTC (rev 843) +++ trunk/src/main/java/org/nuiton/helper/plugin/SendEmailMojo.java 2012-07-12 16:53:52 UTC (rev 844) @@ -25,9 +25,12 @@ package org.nuiton.helper.plugin; -import org.apache.commons.lang.StringUtils; +import org.apache.commons.lang3.StringUtils; import org.apache.maven.model.Developer; import org.apache.maven.plugin.MojoExecutionException; +import org.apache.maven.plugins.annotations.Component; +import org.apache.maven.plugins.annotations.Mojo; +import org.apache.maven.plugins.annotations.Parameter; import org.apache.maven.project.MavenProject; import org.codehaus.plexus.logging.Logger; import org.codehaus.plexus.logging.console.ConsoleLogger; @@ -46,89 +49,81 @@ * Send a email. * * @author tchemit <chemit(a)codelutin.com> - * @goal send-email - * @requiresOnline true - * @requiresProject true * @since 1.0.3 */ +@Mojo(name = "send-email", requiresOnline = true, requiresProject = true) public class SendEmailMojo extends AbstractPlugin implements PluginWithEncoding { /** * Dependance du projet. * - * @parameter default-value="${project}" - * @required - * @readonly * @since 1.0.3 */ + @Component protected MavenProject project; /** * Un flag pour activer le mode verbeux. * - * @parameter expression="${helper.verbose}" default-value="${maven.verbose}" * @since 1.0.3 */ + @Parameter(property = "helper.verbose", defaultValue = "${maven.verbose}") protected boolean verbose; /** * The Velocity template used to format the email announcement. * - * @parameter expression="${helper.emailContentFile}" - * @required * @since 1.0.3 */ + @Parameter(property = "helper.emailContentFile", required = true) protected File emailContentFile; /** * The title of the email to send. * - * @parameter expression="${helper.emailTitle}" - * @required * @since 1.0.3 */ + @Parameter(property = "helper.emailTitle", required = true) protected String emailTitle; /** * Smtp Server. * - * @parameter expression="${helper.smtpHost}" - * @required * @since 1.0.3 */ + @Parameter(property = "helper.smtpHost", required = true) protected String smtpHost; /** * Port. * - * @parameter default-value="25" expression="${helper.smtpPort}" - * @required * @since 1.0.3 */ + @Parameter(property = "helper.smtpPort", defaultValue = "25", required = true) protected int smtpPort; /** * The username used to send the email. * - * @parameter expression="${helper.username}" * @since 1.0.3 */ + @Parameter(property = "helper.username") protected String username; /** * The password used to send the email. * - * @parameter expression="${helper.password}" * @since 1.0.3 */ + @Parameter(property = "helper.password") protected String password; /** * If the email should be sent in SSL mode. * - * @parameter default-value="false" expression="${helper.sslMode}" * @since 1.0.3 */ + @Parameter(property = "helper.sslMode", defaultValue = "false") protected boolean sslMode; /** @@ -138,78 +133,74 @@ * developer is not found, then the first developer in the pom will be * used. * - * @parameter expression="${helper.fromDeveloperId}" * @since 1.0.3 */ + @Parameter(property = "helper.fromDeveloperId") protected String fromDeveloperId; /** * Defines the sender of the announcement if the list of developer is empty * or if the sender is not a member of the development team. * - * @parameter expression="${helper.mailSender}" * @since 1.0.3 */ + @Parameter(property = "helper.mailSender") protected MailSender mailSender; /** * Recipient email address. * - * @parameter - * @required * @since 1.0.3 */ + @Parameter(required = true) protected List<String> toAddresses; /** * Possible senders. * - * @parameter expression="${project.developers}" - * @required - * @readonly * @since 1.0.3 */ + @Parameter(property = "project.developers", required = true, readonly = true) protected List<Developer> from; /** * Mail content type to use. * - * @parameter default-value="text/plain" - * @required * @since 1.0.3 */ + @Parameter(defaultValue = "text/plain", required = true) protected String mailContentType; /** * The encoding used to read and write files. * - * @parameter expression="${helper.encoding}" default-value="${project.build.sourceEncoding}" * @since 1.0.3 */ + @Parameter(property = "helper.encoding", defaultValue = "${project.build.sourceEncoding}") protected String encoding; /** * A flag to skip the goal. * - * @parameter expression="${helper.skipSendEmail}" default-value="false" * @since 1.0.3 */ + @Parameter(property = "helper.skipSendEmail", defaultValue = "false") protected boolean skipSendEmail; /** * A flag to test plugin but send nothing to redmine. * - * @parameter expression="${dryRun}" default-value="false" * @since 1.0.3 */ + @Parameter(property = "dryRun", defaultValue = "false") protected boolean dryRun; /** * A flag to restirct only one run in a build (for multi-module context). * - * @parameter expression="${redmine.runOnce}" default-value="true" * @since 1.0.3 */ + @Parameter(property = "redmine.runOnce", defaultValue = "true") protected boolean runOnce; /** Modified: trunk/src/main/java/org/nuiton/helper/plugin/ShareServerSecretPlugin.java =================================================================== --- trunk/src/main/java/org/nuiton/helper/plugin/ShareServerSecretPlugin.java 2012-07-09 10:20:16 UTC (rev 843) +++ trunk/src/main/java/org/nuiton/helper/plugin/ShareServerSecretPlugin.java 2012-07-12 16:53:52 UTC (rev 844) @@ -25,8 +25,12 @@ package org.nuiton.helper.plugin; -import org.apache.commons.lang.StringUtils; +import org.apache.commons.lang3.StringUtils; import org.apache.maven.plugin.MojoExecutionException; +import org.apache.maven.plugins.annotations.Component; +import org.apache.maven.plugins.annotations.LifecyclePhase; +import org.apache.maven.plugins.annotations.Mojo; +import org.apache.maven.plugins.annotations.Parameter; import org.apache.maven.project.MavenProject; import org.apache.maven.settings.Server; import org.apache.maven.settings.Settings; @@ -51,50 +55,47 @@ * </pre> * * @author tchemit <chemit(a)codelutin.com> - * @goal share-server-secret - * @phase initialize - * @requiresProject true * @since 1.1.0 */ +@Mojo(name = "share-server-secret", defaultPhase = LifecyclePhase.INITIALIZE, requiresProject = true) public class ShareServerSecretPlugin extends AbstractPlugin { /** * Project. * - * @parameter default-value="${project}" - * @required - * @readonly * @since 1.1.0 */ + @Component protected MavenProject project; + /** * Settings. * - * @parameter default-value="${settings}" - * @required - * @readonly * @since 1.1.0 */ + @Component protected Settings settings; + /** * Server id to use for authentication (must be defined in your setting * and use the maven >= 2.1.0 password encryption mecanism). * * - * @parameter expression="${helper.serverId}" - * @required * @since 1.1.0 */ + @Parameter(property = "helper.serverId", required = true) protected String serverId; + /** * The name of the property where to export the username of the server. * * Note: If not set - then no export of the username of the server. * - * @parameter * @since 1.1.0 */ + @Parameter protected String usernameOut; + /** * The name of the property where to export the password of the server. * @@ -103,10 +104,11 @@ * Note: If the password is crypted (since maven 2.1.0) then * decrypt it. * - * @parameter * @since 1.1.0 */ + @Parameter protected String passwordOut; + /** * The name of the property where to export the passphrase of the server. * @@ -115,50 +117,52 @@ * Note: If the passphrase is crypted (since maven 2.1.0) then * decrypt it. * - * @parameter * @since 1.1.0 */ + @Parameter protected String passphraseOut; + /** * The name of the property where to export the private key of the server. * * Note: If not set - then no export of the private key of the server. * - * @parameter * @since 1.1.0 */ + @Parameter protected String privateKeyOut; + /** * A flag to activate verbose mode. * - * @parameter expression="${helper.verbose}" default-value="${maven.verbose}" * @since 1.1.0 */ + @Parameter(property = "helper.verbose", defaultValue = "${maven.verbose}") protected boolean verbose; + /** * A flag to execute only once the mojo. * * Note: By default, value is {@code true} since it is not necessary * to inject twice secrets in session. * - * @parameter expression="${helper.runOnce}" default-value="true" * @since 1.1.0 */ + @Parameter(property = "helper.runOnce", defaultValue = "true") protected boolean runOnce; + /** * password decypher * - * @component roleHint="maven-helper-plugin" * @since 1.1.0 */ + @Component(hint = "maven-helper-plugin") protected SecDispatcher sec; - /** - * the server to use to obtain secrets - */ + + /** the server to use to obtain secrets */ private Server server; - /** - * server secrets to expose - */ + + /** server secrets to expose */ private Map<Property, String> propertiesToTreate; public enum Property {

1 0

r843 - trunk
by tchemit＠users.nuiton.org July 9, 2012

July 9, 2012

Author: tchemit Date: 2012-07-09 12:20:16 +0200 (Mon, 09 Jul 2012) New Revision: 843 Url: http://nuiton.org/repositories/revision/maven-helper-plugin/843 Log: updates to mavenpom 3.3.4 refs #925: Use maven-plugin-plugin 3 api (change pom) Modified: trunk/pom.xml Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2012-05-30 10:15:07 UTC (rev 842) +++ trunk/pom.xml 2012-07-09 10:20:16 UTC (rev 843) @@ -34,7 +34,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>mavenpom4redmineAndCentral</artifactId> - <version>3.3.2</version> + <version>3.3.4</version> </parent> <artifactId>helper-maven-plugin</artifactId> @@ -154,7 +154,15 @@ <scope>provided</scope> </dependency> +  <dependency> + <groupId>org.apache.maven.plugin-tools</groupId> + <artifactId>maven-plugin-annotations</artifactId> + <version>${pluginPluginVersion}</version> + <scope>compile</scope> + </dependency> + + <dependency> <groupId>javax.mail</groupId> <artifactId>mail</artifactId> <version>1.4.4</version> @@ -317,6 +325,21 @@ <plugins> <plugin> + <artifactId>maven-plugin-plugin</artifactId> + <configuration> + <skipErrorNoDescriptorsFound>true</skipErrorNoDescriptorsFound> + </configuration> + <executions> + <execution> + <goals> + <goal>helpmojo</goal> + <goal>descriptor</goal> + </goals> + </execution> + </executions> + </plugin> + + <plugin> <artifactId>maven-jar-plugin</artifactId> <executions> <execution> @@ -378,6 +401,33 @@ </reportSet> </reportSets> </plugin> + <plugin> + <artifactId>maven-javadoc-plugin</artifactId> + <version>${javadocPluginVersion}</version> + <configuration> + <quiet>true</quiet> + <tagletArtifacts> + <tagletArtifact> + <groupId>org.apache.maven.plugin-tools</groupId> + <artifactId>maven-plugin-tools-javadoc</artifactId> + <version>${pluginPluginVersion}</version> + </tagletArtifact> + <tagletArtifact> + <groupId>org.codehaus.plexus</groupId> + <artifactId>plexus-component-javadoc</artifactId> + <version>1.5.5</version> + </tagletArtifact> + </tagletArtifacts> + </configuration> + <reportSets> + <reportSet> + <reports> + <report>javadoc</report> + <report>test-javadoc</report> + </reports> + </reportSet> + </reportSets> + </plugin> </plugins> </reporting> </profile>

1 0