- Jredmine-commits - mailman.cloud.codelutin.com

r454 - tags/jredmine-1.8/jredmine-maven-plugin
by maven-release＠users.nuiton.org April 16, 2014

April 16, 2014

Author: maven-release Date: 2014-04-16 15:25:11 +0200 (Wed, 16 Apr 2014) New Revision: 454 Url: http://forge.nuiton.org/projects/jredmine/repository/revisions/454 Log: fix release Modified: tags/jredmine-1.8/jredmine-maven-plugin/pom.xml Modified: tags/jredmine-1.8/jredmine-maven-plugin/pom.xml =================================================================== --- tags/jredmine-1.8/jredmine-maven-plugin/pom.xml 2014-04-16 13:18:40 UTC (rev 453) +++ tags/jredmine-1.8/jredmine-maven-plugin/pom.xml 2014-04-16 13:25:11 UTC (rev 454) @@ -287,5 +287,110 @@ </build> </profile> +  + <profile> + <id>post-release-profile</id> + + <build> + + <defaultGoal>validate</defaultGoal> + + <plugins> + +  + <plugin> + <groupId>org.nuiton.jredmine</groupId> + <artifactId>jredmine-maven-plugin</artifactId> + <executions> + <execution> + <id>publish-attachments</id> + <goals> + <goal>publish-attachments</goal> + </goals> + <phase>verify</phase> + <configuration> + <filesFromProperties> + ${redmine.collectedAttachments} + </filesFromProperties> + + <skipPublishAttachments> + ${skipReleasePublishAttachments} + </skipPublishAttachments> + </configuration> + </execution> + <execution> + <id>generate-announcements</id> + <goals> + <goal>generate-news-announcement</goal> + <goal>generate-email-announcement</goal> + </goals> + <phase>verify</phase> + <configuration> + <announceParameters> + <releaseProjectGroupId> + ${session.topLevelProject.groupId} + </releaseProjectGroupId> + <releaseProjectArtifactId> + ${session.topLevelProject.artifactId} + </releaseProjectArtifactId> + <releaseProjectVersion> + ${session.topLevelProject.version} + </releaseProjectVersion> + </announceParameters> + <skipGenerateEmailAnnouncement> + ${skipReleaseSendEmail} + </skipGenerateEmailAnnouncement> + <skipGenerateNewsAnnouncement> + ${skipReleasePublishNews} + </skipGenerateNewsAnnouncement> + <urlDownload> + ${redmine.url}/projects/${projectId}/files + </urlDownload> + <artifactsFile>${redmine.collectedArtifacts}</artifactsFile> + </configuration> + </execution> + + <execution> + <id>jredmine-release</id> + <goals> + <goal>update-version</goal> + <goal>publish-news</goal> + </goals> + <phase>install</phase> + <configuration> +  + <runOnce>true</runOnce> + <skipUpdateVersion> + ${skipReleaseUpdateVersion} + </skipUpdateVersion> + <skipPublishNews>${skipReleasePublishNews}</skipPublishNews> + +  + <closeVersion>true</closeVersion> + +  + <newsContentFile> + target/generated-sources/announcement/release-news-announcement.vm + </newsContentFile> + <newsSummary>${project.description}</newsSummary> + <newsTitle> + ${releaseAnnouncementTitle} + </newsTitle> + </configuration> + </execution> + </executions> + + </plugin> + + </plugins> + + </build> + </profile> </profiles> </project>

1 0

r453 - in trunk: . jredmine-client jredmine-maven-plugin
by maven-release＠users.nuiton.org April 16, 2014

April 16, 2014

Author: maven-release Date: 2014-04-16 15:18:40 +0200 (Wed, 16 Apr 2014) New Revision: 453 Url: http://forge.nuiton.org/projects/jredmine/repository/revisions/453 Log: [maven-release-plugin] prepare for next development iteration Modified: trunk/jredmine-client/pom.xml trunk/jredmine-maven-plugin/pom.xml trunk/pom.xml Modified: trunk/jredmine-client/pom.xml =================================================================== --- trunk/jredmine-client/pom.xml 2014-04-16 13:18:35 UTC (rev 452) +++ trunk/jredmine-client/pom.xml 2014-04-16 13:18:40 UTC (rev 453) @@ -27,7 +27,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>jredmine</artifactId> - <version>1.8</version> + <version>2.0-SNAPSHOT</version> </parent> <groupId>org.nuiton.jredmine</groupId> Modified: trunk/jredmine-maven-plugin/pom.xml =================================================================== --- trunk/jredmine-maven-plugin/pom.xml 2014-04-16 13:18:35 UTC (rev 452) +++ trunk/jredmine-maven-plugin/pom.xml 2014-04-16 13:18:40 UTC (rev 453) @@ -27,7 +27,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>jredmine</artifactId> - <version>1.8</version> + <version>2.0-SNAPSHOT</version> </parent> <groupId>org.nuiton.jredmine</groupId> Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2014-04-16 13:18:35 UTC (rev 452) +++ trunk/pom.xml 2014-04-16 13:18:40 UTC (rev 453) @@ -30,7 +30,7 @@ </parent> <artifactId>jredmine</artifactId> - <version>1.8</version> + <version>2.0-SNAPSHOT</version> <packaging>pom</packaging> <name>JRedmine</name> @@ -63,11 +63,11 @@ </modules> <scm> - <connection>scm:svn:http://svn.nuiton.org/jredmine/tags/jredmine-1.8</connection> + <connection>scm:svn:http://svn.nuiton.org/jredmine/trunk</connection> <developerConnection> - scm:svn:https://svn.nuiton.org/jredmine/tags/jredmine-1.8 + scm:svn:https://svn.nuiton.org/jredmine/trunk </developerConnection> - <url>https://forge.nuiton.org/projects/jredmine/repository/show/tags/jredmine-1.8</url> + <url>https://forge.nuiton.org/projects/jredmine/repository/show/trunk</url> </scm> <distributionManagement> <site>

1 0

r452 - tags
by maven-release＠users.nuiton.org April 16, 2014

April 16, 2014

Author: maven-release Date: 2014-04-16 15:18:35 +0200 (Wed, 16 Apr 2014) New Revision: 452 Url: http://forge.nuiton.org/projects/jredmine/repository/revisions/452 Log: [maven-release-plugin] copy for tag jredmine-1.8 Added: tags/jredmine-1.8/

1 0

r451 - in trunk: . jredmine-client jredmine-maven-plugin
by maven-release＠users.nuiton.org April 16, 2014

April 16, 2014

Author: maven-release Date: 2014-04-16 15:18:25 +0200 (Wed, 16 Apr 2014) New Revision: 451 Url: http://forge.nuiton.org/projects/jredmine/repository/revisions/451 Log: [maven-release-plugin] prepare release jredmine-1.8 Modified: trunk/jredmine-client/pom.xml trunk/jredmine-maven-plugin/pom.xml trunk/pom.xml Modified: trunk/jredmine-client/pom.xml =================================================================== --- trunk/jredmine-client/pom.xml 2014-04-16 13:16:11 UTC (rev 450) +++ trunk/jredmine-client/pom.xml 2014-04-16 13:18:25 UTC (rev 451) @@ -27,7 +27,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>jredmine</artifactId> - <version>2.0-SNAPSHOT</version> + <version>1.8</version> </parent> <groupId>org.nuiton.jredmine</groupId> Modified: trunk/jredmine-maven-plugin/pom.xml =================================================================== --- trunk/jredmine-maven-plugin/pom.xml 2014-04-16 13:16:11 UTC (rev 450) +++ trunk/jredmine-maven-plugin/pom.xml 2014-04-16 13:18:25 UTC (rev 451) @@ -27,7 +27,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>jredmine</artifactId> - <version>2.0-SNAPSHOT</version> + <version>1.8</version> </parent> <groupId>org.nuiton.jredmine</groupId> Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2014-04-16 13:16:11 UTC (rev 450) +++ trunk/pom.xml 2014-04-16 13:18:25 UTC (rev 451) @@ -30,7 +30,7 @@ </parent> <artifactId>jredmine</artifactId> - <version>2.0-SNAPSHOT</version> + <version>1.8</version> <packaging>pom</packaging> <name>JRedmine</name> @@ -63,11 +63,11 @@ </modules> <scm> - <connection>scm:svn:http://svn.nuiton.org/jredmine/trunk</connection> + <connection>scm:svn:http://svn.nuiton.org/jredmine/tags/jredmine-1.8</connection> <developerConnection> - scm:svn:https://svn.nuiton.org/jredmine/trunk + scm:svn:https://svn.nuiton.org/jredmine/tags/jredmine-1.8 </developerConnection> - <url>https://forge.nuiton.org/projects/jredmine/repository/show/trunk</url> + <url>https://forge.nuiton.org/projects/jredmine/repository/show/tags/jredmine-1.8</url> </scm> <distributionManagement> <site>

1 0

r450 - in trunk: . jredmine-maven-plugin
by tchemit＠users.nuiton.org April 16, 2014

April 16, 2014

Author: tchemit Date: 2014-04-16 15:16:11 +0200 (Wed, 16 Apr 2014) New Revision: 450 Url: http://forge.nuiton.org/projects/jredmine/repository/revisions/450 Log: optimize dependencies Modified: trunk/jredmine-maven-plugin/pom.xml trunk/pom.xml Modified: trunk/jredmine-maven-plugin/pom.xml =================================================================== --- trunk/jredmine-maven-plugin/pom.xml 2014-04-16 13:10:04 UTC (rev 449) +++ trunk/jredmine-maven-plugin/pom.xml 2014-04-16 13:16:11 UTC (rev 450) @@ -89,11 +89,6 @@ </dependency> <dependency> - <groupId>org.sonatype.plexus</groupId> - <artifactId>plexus-sec-dispatcher</artifactId> - </dependency> - - <dependency> <groupId>org.apache.maven</groupId> <artifactId>maven-project</artifactId> </dependency> Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2014-04-16 13:10:04 UTC (rev 449) +++ trunk/pom.xml 2014-04-16 13:16:11 UTC (rev 450) @@ -366,12 +366,6 @@ </dependency> <dependency> - <groupId>org.sonatype.plexus</groupId> - <artifactId>plexus-sec-dispatcher</artifactId> - <version>1.4</version> - </dependency> - - <dependency> <groupId>org.apache.maven.plugins</groupId> <artifactId>maven-changes-plugin</artifactId> <version>2.9</version>

1 0

r449 - trunk
by tchemit＠users.nuiton.org April 16, 2014

April 16, 2014

Author: tchemit Date: 2014-04-16 15:10:04 +0200 (Wed, 16 Apr 2014) New Revision: 449 Url: http://forge.nuiton.org/projects/jredmine/repository/revisions/449 Log: can't use last jredmine version for release (til mavenpom use it) Modified: trunk/pom.xml Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2014-04-16 13:09:00 UTC (rev 448) +++ trunk/pom.xml 2014-04-16 13:10:04 UTC (rev 449) @@ -81,7 +81,7 @@ <projectId>jredmine</projectId>  - <jredminePluginVersion>1.7</jredminePluginVersion> + <jredminePluginVersion>1.6</jredminePluginVersion> <doxiaVersion>1.4</doxiaVersion>

1 0

r448 - trunk
by tchemit＠users.nuiton.org April 16, 2014

April 16, 2014

Author: tchemit Date: 2014-04-16 15:09:00 +0200 (Wed, 16 Apr 2014) New Revision: 448 Url: http://forge.nuiton.org/projects/jredmine/repository/revisions/448 Log: add last jredmine version for release Modified: trunk/pom.xml Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2014-04-16 12:53:14 UTC (rev 447) +++ trunk/pom.xml 2014-04-16 13:09:00 UTC (rev 448) @@ -81,7 +81,7 @@ <projectId>jredmine</projectId>  - <jredminePluginVersion>1.6</jredminePluginVersion> + <jredminePluginVersion>1.7</jredminePluginVersion> <doxiaVersion>1.4</doxiaVersion>

1 0

r447 - in trunk/jredmine-maven-plugin/src: main/java/org/nuiton/jredmine/plugin main/java/org/nuiton/jredmine/plugin/announcement test/java/org/nuiton/jredmine
by tchemit＠users.nuiton.org April 16, 2014

April 16, 2014

Author: tchemit Date: 2014-04-16 14:53:14 +0200 (Wed, 16 Apr 2014) New Revision: 447 Url: http://forge.nuiton.org/projects/jredmine/repository/revisions/447 Log: fixes #3157 Modified: trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/AbstractRedmineMojo.java trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/DisplayDataMojo.java trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/NextVersionMojo.java trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/PublishAttachmentsMojo.java trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/PublishNewsMojo.java trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/UpdateVersionMojo.java trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/announcement/AbstractAnnouncementMojo.java trunk/jredmine-maven-plugin/src/test/java/org/nuiton/jredmine/RedmineFixtures.java trunk/jredmine-maven-plugin/src/test/java/org/nuiton/jredmine/RedmineLogguedFixtureClassRule.java Modified: trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/AbstractRedmineMojo.java =================================================================== --- trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/AbstractRedmineMojo.java 2014-04-16 12:52:49 UTC (rev 446) +++ trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/AbstractRedmineMojo.java 2014-04-16 12:53:14 UTC (rev 447) @@ -268,20 +268,18 @@ ); if (verbose) { - if (isAnonymous()) { + + if (authConfiguration.isUseApiKey()) { + getLog().info("Redmine configuration :\n>> host : " + + getUrl() + "\n>> apiKey: " + + apiKey); + } else if (authConfiguration.isUseUsername()) { + getLog().info("Redmine configuration :\n>> host : " + + getUrl() + "\n>> username : " + + username); + } else { getLog().info("Redmine anonymous configuration :\n>> host : " + getUrl()); - } else { - - if (authConfiguration.isUseApiKey()) { - getLog().info("Redmine configuration :\n>> host : " + - getUrl() + "\n>> apiKey: " + - apiKey); - } else { - getLog().info("Redmine configuration :\n>> host : " + - getUrl() + "\n>> username : " + - username); - } } } Modified: trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/DisplayDataMojo.java =================================================================== --- trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/DisplayDataMojo.java 2014-04-16 12:52:49 UTC (rev 446) +++ trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/DisplayDataMojo.java 2014-04-16 12:53:14 UTC (rev 447) @@ -110,19 +110,6 @@ @Parameter(property = "sortById", defaultValue = "true", required = true) protected boolean sortById; - /** - * Flag to know if anonymous connexion to redmine server is required. - * - * For this goal, the default value is {@code true} - * - * Note: If set to {@code false}, you should fill {@link #username} - * and {@link #password} properties. - * - * @since 1.1.3 - */ - @Parameter(property = "anonymous", defaultValue = "true") - protected boolean anonymous; - /////////////////////////////////////////////////////////////////////////// /// Mojo internal attributes /////////////////////////////////////////////////////////////////////////// @@ -164,20 +151,6 @@ } /////////////////////////////////////////////////////////////////////////// - /// RedmineClientConfiguration - /////////////////////////////////////////////////////////////////////////// - - @Override - public boolean isAnonymous() { - return anonymous; - } - - @Override - public void setAnonymous(boolean anonymous) { - this.anonymous = anonymous; - } - - /////////////////////////////////////////////////////////////////////////// /// RedmineProjectAware /////////////////////////////////////////////////////////////////////////// @@ -301,7 +274,8 @@ throw new MojoFailureException( "The data type '" + dataType + "' can not be used, Accepted types are : " + - universeList); + universeList + ); } DataScope scope = namestoScope.get(dataType); @@ -314,7 +288,8 @@ throw new MojoFailureException( "The data type '" + dataType + "', depends on a project, but the 'projectId' was " + - "not filled."); + "not filled." + ); } } if (DataScope.VERSION.equals(scope)) { @@ -324,7 +299,8 @@ throw new MojoFailureException( "The data type '" + dataType + "', depends on a version, but the 'versionId' was " + - "not filled."); + "not filled." + ); } } results.add(dataType); Modified: trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/NextVersionMojo.java =================================================================== --- trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/NextVersionMojo.java 2014-04-16 12:52:49 UTC (rev 446) +++ trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/NextVersionMojo.java 2014-04-16 12:53:14 UTC (rev 447) @@ -52,17 +52,6 @@ /////////////////////////////////////////////////////////////////////////// /** - * Flag to know if anonymous connexion to redmine server is required. - * - * Note: If set to {@code false}, you should fill {@link #username} - * and {@link #password} properties. - * - * @since 1.1.3 - */ - @Parameter(property = "redmine.anonymous", defaultValue = "false") - protected boolean anonymous; - - /** * The news description to update to redmine server. * * Note : if not set, no update of the description will be made. @@ -155,20 +144,6 @@ } /////////////////////////////////////////////////////////////////////////// - /// RedmineClientConfiguration - /////////////////////////////////////////////////////////////////////////// - - @Override - public boolean isAnonymous() { - return anonymous; - } - - @Override - public void setAnonymous(boolean anonymous) { - this.anonymous = anonymous; - } - - /////////////////////////////////////////////////////////////////////////// /// DryRunAware /////////////////////////////////////////////////////////////////////////// Modified: trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/PublishAttachmentsMojo.java =================================================================== --- trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/PublishAttachmentsMojo.java 2014-04-16 12:52:49 UTC (rev 446) +++ trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/PublishAttachmentsMojo.java 2014-04-16 12:53:14 UTC (rev 447) @@ -53,17 +53,6 @@ /////////////////////////////////////////////////////////////////////////// /** - * Flag to know if anonymous connexion to redmine server is required. - * - * Note: If set to {@code false}, you should fill {@link #username} - * and {@link #password} properties. - * - * @since 1.1.3 - */ - @Parameter(property = "redmine.anonymous", defaultValue = "false") - protected boolean anonymous; - - /** * The path of a properties files where files to deploy are descriped. * * The key of a property is the sortor key @@ -134,20 +123,6 @@ } /////////////////////////////////////////////////////////////////////////// - /// RedmineClientConfiguration - /////////////////////////////////////////////////////////////////////////// - - @Override - public boolean isAnonymous() { - return anonymous; - } - - @Override - public void setAnonymous(boolean anonymous) { - this.anonymous = anonymous; - } - - /////////////////////////////////////////////////////////////////////////// /// DryRunAware /////////////////////////////////////////////////////////////////////////// Modified: trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/PublishNewsMojo.java =================================================================== --- trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/PublishNewsMojo.java 2014-04-16 12:52:49 UTC (rev 446) +++ trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/PublishNewsMojo.java 2014-04-16 12:53:14 UTC (rev 447) @@ -46,17 +46,6 @@ /////////////////////////////////////////////////////////////////////////// /** - * Flag to know if anonymous connexion to redmine server is required. - * - * Note: If set to {@code false}, you should fill {@link #username} - * and {@link #password} properties. - * - * @since 1.1.3 - */ - @Parameter(property = "redmine.anonymous", defaultValue = "false") - protected boolean anonymous; - - /** * The content file of the news. * * @since 1.0.0 @@ -123,20 +112,6 @@ } /////////////////////////////////////////////////////////////////////////// - /// RedmineClientConfiguration - /////////////////////////////////////////////////////////////////////////// - - @Override - public boolean isAnonymous() { - return anonymous; - } - - @Override - public void setAnonymous(boolean anonymous) { - this.anonymous = anonymous; - } - - /////////////////////////////////////////////////////////////////////////// /// DryRunAware /////////////////////////////////////////////////////////////////////////// Modified: trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/UpdateVersionMojo.java =================================================================== --- trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/UpdateVersionMojo.java 2014-04-16 12:52:49 UTC (rev 446) +++ trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/UpdateVersionMojo.java 2014-04-16 12:53:14 UTC (rev 447) @@ -52,17 +52,6 @@ /////////////////////////////////////////////////////////////////////////// /** - * Flag to know if anonymous connexion to redmine server is required. - * - * Note: If set to {@code false}, you should fill {@link #username} - * and {@link #password} properties. - * - * @since 1.1.3 - */ - @Parameter(property = "redmine.anonymous", defaultValue = "false") - protected boolean anonymous; - - /** * The version description to update to redmine server. * * Note : if not set, no update of the description will be made. @@ -160,20 +149,6 @@ } /////////////////////////////////////////////////////////////////////////// - /// RedmineClientConfiguration - /////////////////////////////////////////////////////////////////////////// - - @Override - public boolean isAnonymous() { - return anonymous; - } - - @Override - public void setAnonymous(boolean anonymous) { - this.anonymous = anonymous; - } - - /////////////////////////////////////////////////////////////////////////// /// DryRunAware /////////////////////////////////////////////////////////////////////////// @@ -258,7 +233,8 @@ } catch (ParseException e) { throw new MojoExecutionException( "could not parse effectivate date " + effectiveDate + - " for reason " + e.getMessage(), e); + " for reason " + e.getMessage(), e + ); } } else if (closeVersion) { date = new Date(); @@ -278,7 +254,8 @@ "could not parse status " + versionStatus + " for reason " + e.getMessage() + ", should be one of values : " + - Arrays.toString(VersionStatusEnum.values()), e); + Arrays.toString(VersionStatusEnum.values()), e + ); } if (closeVersion) { Modified: trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/announcement/AbstractAnnouncementMojo.java =================================================================== --- trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/announcement/AbstractAnnouncementMojo.java 2014-04-16 12:52:49 UTC (rev 446) +++ trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/announcement/AbstractAnnouncementMojo.java 2014-04-16 12:53:14 UTC (rev 447) @@ -52,19 +52,6 @@ /////////////////////////////////////////////////////////////////////////// /** - * Flag to know if anonymous connexion to redmine server is required. - * - * For this goal, the default value is {@code true} - * - * Note: If set to {@code false}, you should fill {@link #username} - * and {@link #password} properties. - * - * @since 1.1.3 - */ - @Parameter(property = "redmine.anonymous", defaultValue = "true") - protected boolean anonymous; - - /** * Directory where the template file will be generated. * * @since 1.0.0 @@ -378,20 +365,6 @@ } /////////////////////////////////////////////////////////////////////////// - /// AbstractRedmineMojo - /////////////////////////////////////////////////////////////////////////// - - @Override - public boolean isAnonymous() { - return anonymous; - } - - @Override - public void setAnonymous(boolean anonymous) { - this.anonymous = anonymous; - } - - /////////////////////////////////////////////////////////////////////////// /// AbstractPlugin /////////////////////////////////////////////////////////////////////////// Modified: trunk/jredmine-maven-plugin/src/test/java/org/nuiton/jredmine/RedmineFixtures.java =================================================================== --- trunk/jredmine-maven-plugin/src/test/java/org/nuiton/jredmine/RedmineFixtures.java 2014-04-16 12:52:49 UTC (rev 446) +++ trunk/jredmine-maven-plugin/src/test/java/org/nuiton/jredmine/RedmineFixtures.java 2014-04-16 12:53:14 UTC (rev 447) @@ -25,7 +25,6 @@ import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; -import org.nuiton.jredmine.service.DefaultRedmineAnonymousService; import org.nuiton.jredmine.service.DefaultRedmineService; import org.nuiton.jredmine.service.RedmineAnonymousService; import org.nuiton.jredmine.service.RedmineConfigurationUtil; @@ -80,7 +79,7 @@ public RedmineAnonymousService newRedmineAnonymousService(RedmineServiceConfiguration configuration) throws IOException, RedmineServiceException { - RedmineAnonymousService service = new DefaultRedmineAnonymousService(); + RedmineAnonymousService service = new DefaultRedmineService(); service.init(configuration); return service; } Modified: trunk/jredmine-maven-plugin/src/test/java/org/nuiton/jredmine/RedmineLogguedFixtureClassRule.java =================================================================== --- trunk/jredmine-maven-plugin/src/test/java/org/nuiton/jredmine/RedmineLogguedFixtureClassRule.java 2014-04-16 12:52:49 UTC (rev 446) +++ trunk/jredmine-maven-plugin/src/test/java/org/nuiton/jredmine/RedmineLogguedFixtureClassRule.java 2014-04-16 12:53:14 UTC (rev 447) @@ -29,6 +29,7 @@ import org.junit.rules.TestRule; import org.junit.runner.Description; import org.junit.runners.model.Statement; +import org.nuiton.jredmine.service.RedmineConfigurationUtil; import org.nuiton.jredmine.service.RedmineServiceConfiguration; import java.io.IOException; @@ -71,7 +72,7 @@ // configuration must NOT be anonymous - if (conf.isAnonymous()) { + if (!RedmineConfigurationUtil.isLoggued(conf)) { if (log.isWarnEnabled()) { log.warn("A authenticated configuration was required, will skip test " +

1 0

r446 - in trunk/jredmine-client/src: main/java/org/nuiton/jredmine/client main/java/org/nuiton/jredmine/service test/java/org/nuiton/jredmine test/java/org/nuiton/jredmine/client test/java/org/nuiton/jredmine/test
by tchemit＠users.nuiton.org April 16, 2014

April 16, 2014

Author: tchemit Date: 2014-04-16 14:52:49 +0200 (Wed, 16 Apr 2014) New Revision: 446 Url: http://forge.nuiton.org/projects/jredmine/repository/revisions/446 Log: fixes #3156 Removed: trunk/jredmine-client/src/main/java/org/nuiton/jredmine/service/DefaultRedmineAnonymousService.java Modified: trunk/jredmine-client/src/main/java/org/nuiton/jredmine/client/RedmineClient.java trunk/jredmine-client/src/main/java/org/nuiton/jredmine/client/RedmineClientAuthConfiguration.java trunk/jredmine-client/src/main/java/org/nuiton/jredmine/client/RedmineClientConfiguration.java trunk/jredmine-client/src/main/java/org/nuiton/jredmine/service/AbstractRedmineService.java trunk/jredmine-client/src/main/java/org/nuiton/jredmine/service/DefaultRedmineService.java trunk/jredmine-client/src/main/java/org/nuiton/jredmine/service/RedmineAnonymousService.java trunk/jredmine-client/src/main/java/org/nuiton/jredmine/service/RedmineConfigurationUtil.java trunk/jredmine-client/src/main/java/org/nuiton/jredmine/service/RedmineLogguedService.java trunk/jredmine-client/src/test/java/org/nuiton/jredmine/RedmineFixtures.java trunk/jredmine-client/src/test/java/org/nuiton/jredmine/client/RedmineClientTest.java trunk/jredmine-client/src/test/java/org/nuiton/jredmine/test/AbstractRedmineFixtures.java trunk/jredmine-client/src/test/java/org/nuiton/jredmine/test/RedmineFixtureClassRule.java Modified: trunk/jredmine-client/src/main/java/org/nuiton/jredmine/client/RedmineClient.java =================================================================== --- trunk/jredmine-client/src/main/java/org/nuiton/jredmine/client/RedmineClient.java 2014-04-16 12:52:01 UTC (rev 445) +++ trunk/jredmine-client/src/main/java/org/nuiton/jredmine/client/RedmineClient.java 2014-04-16 12:52:49 UTC (rev 446) @@ -124,7 +124,7 @@ CoreProtocolPNames.HTTP_CONTENT_CHARSET, configuration.getEncoding()); - if (!configuration.isAnonymous()) { +// if (!configuration.isAnonymous()) { RedmineClientAuthConfiguration authConfiguration = configuration.getAuthConfiguration(); if (authConfiguration.isUseApiKey()) { @@ -137,7 +137,7 @@ defaultHeaders.put("X-Redmine-API-Key", authConfiguration.getApiKey()); } - } +// } } public boolean isOpen() { @@ -167,28 +167,6 @@ throw e; } -// if (!configuration.isAnonymous()) { -// -// // log in -// RedmineClientAuthConfiguration conf = -// getConfiguration().getAuthConfiguration(); -// RedmineRequest<Void> loginRequest; -// -// if (conf.isUseApiKey()) { -// -// // use api key to login -// loginRequest = RedmineRequestHelper.postAction("login", void.class). -// parameter("apiKey", conf.getApiKey()); -// } else { -// // use normal login / password -// loginRequest = RedmineRequestHelper.postAction("login", void.class). -// parameter("username", conf.getUsername()). -// parameter("password", conf.getPassword()); -// } -// -// executeRequest(loginRequest); -// } - open = true; } } @@ -258,30 +236,9 @@ open = false; } - -// try { -// -// RedmineRequest<Void> request = -// RedmineRequestHelper.action("logout", void.class); -// executeRequest(request); -// } catch (Exception e) { -// throw new IOException( -// "could not close client for reason " + -// e.getMessage(), e); -// } finally { -// -// try { -// -// client.getConnectionManager().shutdown(); -// } finally { -// -// open = false; -// } -// } } } - protected <T> HttpRequestBase prepareRequest(RedmineRequest<T> request) throws IOException { if (showRequest) { Modified: trunk/jredmine-client/src/main/java/org/nuiton/jredmine/client/RedmineClientAuthConfiguration.java =================================================================== --- trunk/jredmine-client/src/main/java/org/nuiton/jredmine/client/RedmineClientAuthConfiguration.java 2014-04-16 12:52:01 UTC (rev 445) +++ trunk/jredmine-client/src/main/java/org/nuiton/jredmine/client/RedmineClientAuthConfiguration.java 2014-04-16 12:52:49 UTC (rev 446) @@ -78,4 +78,13 @@ public boolean isUseApiKey() { return StringUtils.isNotBlank(apiKey); } + + public boolean isUseUsername() { + return StringUtils.isNotBlank(username); + } + + public boolean isUseLoginPassword() { + return StringUtils.isNotBlank(username) && StringUtils.isNotBlank(password); + } + } Modified: trunk/jredmine-client/src/main/java/org/nuiton/jredmine/client/RedmineClientConfiguration.java =================================================================== --- trunk/jredmine-client/src/main/java/org/nuiton/jredmine/client/RedmineClientConfiguration.java 2014-04-16 12:52:01 UTC (rev 445) +++ trunk/jredmine-client/src/main/java/org/nuiton/jredmine/client/RedmineClientConfiguration.java 2014-04-16 12:52:49 UTC (rev 446) @@ -44,8 +44,8 @@ /** @return {@code true} to make verbose client (show request and parameters) */ boolean isVerbose(); - /** @return {@code true} if rest client does not need login */ - boolean isAnonymous(); +// /** @return {@code true} if rest client does not need login */ +// boolean isAnonymous(); /** @param url the url of server to set */ void setUrl(URL url); @@ -56,8 +56,8 @@ /** @param verbose the flag verbose to set */ void setVerbose(boolean verbose); - /** @param anonymous the flag anonymous to set */ - void setAnonymous(boolean anonymous); +// /** @param anonymous the flag anonymous to set */ +// void setAnonymous(boolean anonymous); RedmineClientAuthConfiguration getAuthConfiguration(); Modified: trunk/jredmine-client/src/main/java/org/nuiton/jredmine/service/AbstractRedmineService.java =================================================================== --- trunk/jredmine-client/src/main/java/org/nuiton/jredmine/service/AbstractRedmineService.java 2014-04-16 12:52:01 UTC (rev 445) +++ trunk/jredmine-client/src/main/java/org/nuiton/jredmine/service/AbstractRedmineService.java 2014-04-16 12:52:49 UTC (rev 446) @@ -134,7 +134,8 @@ checkInit(); checkSessionNotNull(); checkSessionConfigurationNotNull(); - if (client.getConfiguration().isAnonymous()) { + boolean anonymous = !RedmineConfigurationUtil.isLoggued(client.getConfiguration()); + if (anonymous) { throw new RedmineServiceLoginException( "can not access this service in anonymous mode"); } Deleted: trunk/jredmine-client/src/main/java/org/nuiton/jredmine/service/DefaultRedmineAnonymousService.java =================================================================== --- trunk/jredmine-client/src/main/java/org/nuiton/jredmine/service/DefaultRedmineAnonymousService.java 2014-04-16 12:52:01 UTC (rev 445) +++ trunk/jredmine-client/src/main/java/org/nuiton/jredmine/service/DefaultRedmineAnonymousService.java 2014-04-16 12:52:49 UTC (rev 446) @@ -1,264 +0,0 @@ -package org.nuiton.jredmine.service; - -/* - * #%L - * JRedmine :: Client - * $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.base.Preconditions; -import org.apache.commons.logging.Log; -import org.apache.commons.logging.LogFactory; -import org.nuiton.jredmine.client.RedmineRequest; -import org.nuiton.jredmine.client.RedmineRequestHelper; -import org.nuiton.jredmine.model.Attachment; -import org.nuiton.jredmine.model.Issue; -import org.nuiton.jredmine.model.IssueCategory; -import org.nuiton.jredmine.model.IssuePriority; -import org.nuiton.jredmine.model.IssueStatus; -import org.nuiton.jredmine.model.News; -import org.nuiton.jredmine.model.Project; -import org.nuiton.jredmine.model.TimeEntry; -import org.nuiton.jredmine.model.Tracker; -import org.nuiton.jredmine.model.User; -import org.nuiton.jredmine.model.Version; -import org.nuiton.jredmine.model.VersionStatusEnum; - -import java.util.ArrayList; -import java.util.Collections; -import java.util.Comparator; -import java.util.List; - - -/** - * Default implementation of {@link RedmineAnonymousService}. - * - * @author tchemit <chemit(a)codelutin.com> - * @plexus.component role="org.nuiton.jredmine.service.RedmineAnonymousService" role-hint="default" - * @since 1.5 - */ -public class DefaultRedmineAnonymousService extends AbstractRedmineService implements RedmineAnonymousService { - - /** Logger. */ - private static final Log log = - LogFactory.getLog(DefaultRedmineAnonymousService.class); - - @Override - public void init(RedmineServiceConfiguration configuration) throws RedmineServiceException { - Preconditions.checkNotNull(configuration, "the client configuration was not be null!"); - - // always clone the configuration to keep it in the service - RedmineServiceConfiguration serviceConfiguration = - RedmineConfigurationUtil.cloneConfiguration(configuration); - - // Force to not be loggued - serviceConfiguration.setAnonymous(true); - - if (log.isDebugEnabled()) { - log.debug("init service with configuration: " + serviceConfiguration); - } - if (!isInit()) { - super.init(serviceConfiguration); - } - } - - /////////////////////////////////////////////////////////////////////////// - /// RedmineAnonymousService - /////////////////////////////////////////////////////////////////////////// - - @Override - public IssueStatus[] getIssueStatuses() throws RedmineServiceException { - RedmineRequest<IssueStatus> request = - RedmineRequestHelper.action("get_issue_statuses.xml", - IssueStatus.class); - return executeRequestAndReturnDatas(request); - } - - @Override - public IssuePriority[] getIssuePriorities() throws RedmineServiceException { - RedmineRequest<IssuePriority> request = RedmineRequestHelper.action( - "get_issue_priorities.xml", IssuePriority.class); - return executeRequestAndReturnDatas(request); - } - - @Override - public Project[] getProjects() throws RedmineServiceException { - RedmineRequest<Project> request = RedmineRequestHelper.action( - "get_projects.xml", Project.class); - return executeRequestAndReturnDatas(request); - } - - @Override - public IssueCategory[] getIssueCategories(String projectName) throws RedmineServiceException { - RedmineRequest<IssueCategory> request = - RedmineRequestHelper.actionWithProject( - "get_issue_categories.xml", - IssueCategory.class, - projectName); - return executeRequestAndReturnDatas(request); - } - - @Override - public Project getProject(String projectName) throws RedmineServiceException { - RedmineRequest<Project> request = - RedmineRequestHelper.actionWithProject("get_project.xml", - Project.class, - projectName); - return executeRequestAndReturnData(request - ); - } - - @Override - public Tracker[] getTrackers(String projectName) throws RedmineServiceException { - RedmineRequest<Tracker> request = - RedmineRequestHelper.actionWithProject("get_project_trackers.xml", - Tracker.class, - projectName); - return executeRequestAndReturnDatas(request); - } - - @Override - public News[] getNews(String projectName) throws RedmineServiceException { - RedmineRequest<News> request = - RedmineRequestHelper.actionWithProject("get_project_news.xml", - News.class, - projectName); - return executeRequestAndReturnDatas(request); - } - - @Override - public User[] getProjectMembers(String projectName) throws RedmineServiceException { - RedmineRequest<User> request = - RedmineRequestHelper.actionWithProject("get_project_users.xml", - User.class, - projectName); - return executeRequestAndReturnDatas(request); - } - - @Override - public Version[] getVersions(String projectName) throws RedmineServiceException { - RedmineRequest<Version> request = - RedmineRequestHelper.actionWithProject("get_project_versions.xml", - Version.class, - projectName); - return executeRequestAndReturnDatas(request); - } - - @Override - public Version getVersion(String projectName, - String versionName) throws RedmineServiceException { - RedmineRequest<Version> request = - RedmineRequestHelper.actionWithProjectAndVersion("get_version.xml", - Version.class, - projectName, - versionName); - return executeRequestAndReturnData(request); - } - - @Override - public Version getLastestClosedVersion(String projectName) throws RedmineServiceException { - - // get all versions - Version[] versions = getVersions(projectName); - List<Version> closedVersions = new ArrayList<Version>(); - for (Version version : versions) { - if (version.getEffectiveDate() != null && - VersionStatusEnum.closed.name().equals(version.getStatus())) { - closedVersions.add(version); - } - } - - Version result = null; - if (!closedVersions.isEmpty()) { - Collections.sort(closedVersions, new Comparator<Version>() { - @Override - public int compare(Version o1, Version o2) { - return o2.getEffectiveDate().compareTo(o1.getEffectiveDate()); - } - }); - result = closedVersions.get(0); - } - return result; - } - - @Override - public Attachment[] getAttachments(String projectName, - String versionName) throws RedmineServiceException { - RedmineRequest<Attachment> request = - RedmineRequestHelper.actionWithProjectAndVersion("get_version_attachments.xml", - Attachment.class, - projectName, - versionName); - return executeRequestAndReturnDatas(request); - } - - @Override - public Issue[] getIssues(String projectName, - String versionName) throws RedmineServiceException { - RedmineRequest<Issue> request = - RedmineRequestHelper.actionWithProjectAndVersion("get_version_issues.xml", - Issue.class, - projectName, - versionName); - return executeRequestAndReturnDatas(request); - } - - @Override - public TimeEntry[] getIssueTimeEntries(String projectName, - String issueId) throws RedmineServiceException { - RedmineRequest<TimeEntry> request = - RedmineRequestHelper.actionWithProjectAndIssue("get_issue_times.xml", - TimeEntry.class, - projectName, - issueId); - return executeRequestAndReturnDatas(request); - } - - @Override - public Issue[] getIssues(String projectName) throws RedmineServiceException { - RedmineRequest<Issue> request = - RedmineRequestHelper.actionWithProject("get_project_issues.xml", - Issue.class, - projectName); - Issue[] result = executeRequestAndReturnDatas(request); - return result; - } - - @Override - public Issue[] getOpenedIssues(String projectName) throws RedmineServiceException { - RedmineRequest<Issue> request = - RedmineRequestHelper.actionWithProject("get_project_opened_issues.xml", - Issue.class, - projectName); - Issue[] result = executeRequestAndReturnDatas(request); - return result; - } - - @Override - public Issue[] getClosedIssues(String projectName) throws RedmineServiceException { - RedmineRequest<Issue> request = - RedmineRequestHelper.actionWithProject("get_project_closed_issues.xml", - Issue.class, - projectName); - Issue[] result = executeRequestAndReturnDatas(request); - return result; - } -} Modified: trunk/jredmine-client/src/main/java/org/nuiton/jredmine/service/DefaultRedmineService.java =================================================================== --- trunk/jredmine-client/src/main/java/org/nuiton/jredmine/service/DefaultRedmineService.java 2014-04-16 12:52:01 UTC (rev 445) +++ trunk/jredmine-client/src/main/java/org/nuiton/jredmine/service/DefaultRedmineService.java 2014-04-16 12:52:49 UTC (rev 446) @@ -44,7 +44,11 @@ import java.text.DateFormat; import java.text.SimpleDateFormat; +import java.util.ArrayList; +import java.util.Collections; +import java.util.Comparator; import java.util.Date; +import java.util.List; /** * Default implementation of the {@link RedmineService}. @@ -59,18 +63,6 @@ private static final Log log = LogFactory.getLog(DefaultRedmineService.class); - /** - * Delegate all anonymous call to the anonymous service. - * - * @see RedmineAnonymousService - * @since 1.5 - */ - protected final RedmineAnonymousService anonymousService; - - public DefaultRedmineService() { - this.anonymousService = new DefaultRedmineAnonymousService(); - } - @Override public void init(RedmineServiceConfiguration configuration) throws RedmineServiceException { if (log.isDebugEnabled()) { @@ -84,9 +76,6 @@ // init the service using the configuration super.init(serviceConfiguration); - - // init the anonymous service - anonymousService.init(configuration); } } @@ -96,107 +85,179 @@ @Override public IssueStatus[] getIssueStatuses() throws RedmineServiceException { - IssueStatus[] result = anonymousService.getIssueStatuses(); - return result; + RedmineRequest<IssueStatus> request = + RedmineRequestHelper.action("get_issue_statuses.xml", + IssueStatus.class); + return executeRequestAndReturnDatas(request); } @Override public IssuePriority[] getIssuePriorities() throws RedmineServiceException { - IssuePriority[] result = anonymousService.getIssuePriorities(); - return result; + RedmineRequest<IssuePriority> request = RedmineRequestHelper.action( + "get_issue_priorities.xml", IssuePriority.class); + return executeRequestAndReturnDatas(request); } @Override public Project[] getProjects() throws RedmineServiceException { - Project[] result = anonymousService.getProjects(); - return result; + RedmineRequest<Project> request = RedmineRequestHelper.action( + "get_projects.xml", Project.class); + return executeRequestAndReturnDatas(request); } @Override public IssueCategory[] getIssueCategories(String projectName) throws RedmineServiceException { - IssueCategory[] result = anonymousService.getIssueCategories(projectName); - return result; + RedmineRequest<IssueCategory> request = + RedmineRequestHelper.actionWithProject( + "get_issue_categories.xml", + IssueCategory.class, + projectName); + return executeRequestAndReturnDatas(request); } @Override public Project getProject(String projectName) throws RedmineServiceException { - Project result = anonymousService.getProject(projectName); - return result; + RedmineRequest<Project> request = + RedmineRequestHelper.actionWithProject("get_project.xml", + Project.class, + projectName); + return executeRequestAndReturnData(request + ); } @Override public Tracker[] getTrackers(String projectName) throws RedmineServiceException { - Tracker[] result = anonymousService.getTrackers(projectName); - return result; + RedmineRequest<Tracker> request = + RedmineRequestHelper.actionWithProject("get_project_trackers.xml", + Tracker.class, + projectName); + return executeRequestAndReturnDatas(request); } @Override public News[] getNews(String projectName) throws RedmineServiceException { - News[] result = anonymousService.getNews(projectName); - return result; + RedmineRequest<News> request = + RedmineRequestHelper.actionWithProject("get_project_news.xml", + News.class, + projectName); + return executeRequestAndReturnDatas(request); } @Override public User[] getProjectMembers(String projectName) throws RedmineServiceException { - User[] result = anonymousService.getProjectMembers(projectName); - return result; + RedmineRequest<User> request = + RedmineRequestHelper.actionWithProject("get_project_users.xml", + User.class, + projectName); + return executeRequestAndReturnDatas(request); } @Override public Version[] getVersions(String projectName) throws RedmineServiceException { - Version[] result = anonymousService.getVersions(projectName); - return result; + RedmineRequest<Version> request = + RedmineRequestHelper.actionWithProject("get_project_versions.xml", + Version.class, + projectName); + return executeRequestAndReturnDatas(request); } @Override public Version getVersion(String projectName, String versionName) throws RedmineServiceException { - Version result = anonymousService.getVersion(projectName, versionName); - return result; + RedmineRequest<Version> request = + RedmineRequestHelper.actionWithProjectAndVersion("get_version.xml", + Version.class, + projectName, + versionName); + return executeRequestAndReturnData(request); } @Override public Version getLastestClosedVersion(String projectName) throws RedmineServiceException { - Version result = anonymousService.getLastestClosedVersion(projectName); + + // get all versions + Version[] versions = getVersions(projectName); + List<Version> closedVersions = new ArrayList<Version>(); + for (Version version : versions) { + if (version.getEffectiveDate() != null && + VersionStatusEnum.closed.name().equals(version.getStatus())) { + closedVersions.add(version); + } + } + + Version result = null; + if (!closedVersions.isEmpty()) { + Collections.sort(closedVersions, new Comparator<Version>() { + @Override + public int compare(Version o1, Version o2) { + return o2.getEffectiveDate().compareTo(o1.getEffectiveDate()); + } + }); + result = closedVersions.get(0); + } return result; } @Override public Attachment[] getAttachments(String projectName, String versionName) throws RedmineServiceException { - Attachment[] result = anonymousService.getAttachments(projectName, versionName); - return result; + RedmineRequest<Attachment> request = + RedmineRequestHelper.actionWithProjectAndVersion("get_version_attachments.xml", + Attachment.class, + projectName, + versionName); + return executeRequestAndReturnDatas(request); } @Override public Issue[] getIssues(String projectName, String versionName) throws RedmineServiceException { - Issue[] result = anonymousService.getIssues(projectName, versionName); - return result; + RedmineRequest<Issue> request = + RedmineRequestHelper.actionWithProjectAndVersion("get_version_issues.xml", + Issue.class, + projectName, + versionName); + return executeRequestAndReturnDatas(request); } @Override public TimeEntry[] getIssueTimeEntries(String projectName, String issueId) throws RedmineServiceException { - TimeEntry[] result = anonymousService.getIssueTimeEntries(projectName, issueId); - return result; + RedmineRequest<TimeEntry> request = + RedmineRequestHelper.actionWithProjectAndIssue("get_issue_times.xml", + TimeEntry.class, + projectName, + issueId); + return executeRequestAndReturnDatas(request); } @Override public Issue[] getIssues(String projectName) throws RedmineServiceException { - Issue[] result = anonymousService.getIssues(projectName); + RedmineRequest<Issue> request = + RedmineRequestHelper.actionWithProject("get_project_issues.xml", + Issue.class, + projectName); + Issue[] result = executeRequestAndReturnDatas(request); return result; } @Override public Issue[] getOpenedIssues(String projectName) throws RedmineServiceException { - Issue[] result = anonymousService.getOpenedIssues(projectName); + RedmineRequest<Issue> request = + RedmineRequestHelper.actionWithProject("get_project_opened_issues.xml", + Issue.class, + projectName); + Issue[] result = executeRequestAndReturnDatas(request); return result; } @Override public Issue[] getClosedIssues(String projectName) throws RedmineServiceException { - Issue[] result = anonymousService.getClosedIssues(projectName); + RedmineRequest<Issue> request = + RedmineRequestHelper.actionWithProject("get_project_closed_issues.xml", + Issue.class, + projectName); + Issue[] result = executeRequestAndReturnDatas(request); return result; } @@ -209,10 +270,6 @@ checkLoggued(); RedmineRequest<User> request = RedmineRequestHelper.action("get_user.xml", User.class); - //FIXME-tchemit-2014-05-16 Should remove this! - if (getClient().getConfiguration().getAuthConfiguration().isUseApiKey()) { - request.getParams().put("key", getClient().getConfiguration().getAuthConfiguration().getApiKey()); - } return executeRequestAndReturnData(request); } Modified: trunk/jredmine-client/src/main/java/org/nuiton/jredmine/service/RedmineAnonymousService.java =================================================================== --- trunk/jredmine-client/src/main/java/org/nuiton/jredmine/service/RedmineAnonymousService.java 2014-04-16 12:52:01 UTC (rev 445) +++ trunk/jredmine-client/src/main/java/org/nuiton/jredmine/service/RedmineAnonymousService.java 2014-04-16 12:52:49 UTC (rev 446) @@ -37,14 +37,12 @@ import org.nuiton.jredmine.model.Version; /** - * Contract of all redmine operations which do not requires any login to server. + * Contract of all redmine operations which do not requires any login to server (if project is public). * - * A default implementation is {@link DefaultRedmineAnonymousService}. * * Created: 2 janv. 2010 * * @author tchemit <chemit(a)codelutin.com> - * @see DefaultRedmineAnonymousService * @since 1.0.3 */ public interface RedmineAnonymousService { Modified: trunk/jredmine-client/src/main/java/org/nuiton/jredmine/service/RedmineConfigurationUtil.java =================================================================== --- trunk/jredmine-client/src/main/java/org/nuiton/jredmine/service/RedmineConfigurationUtil.java 2014-04-16 12:52:01 UTC (rev 445) +++ trunk/jredmine-client/src/main/java/org/nuiton/jredmine/service/RedmineConfigurationUtil.java 2014-04-16 12:52:49 UTC (rev 446) @@ -33,6 +33,7 @@ import org.apache.commons.logging.LogFactory; import org.nuiton.jredmine.client.RedmineClient; import org.nuiton.jredmine.client.RedmineClientAuthConfiguration; +import org.nuiton.jredmine.client.RedmineClientConfiguration; import java.io.File; import java.io.IOException; @@ -163,6 +164,7 @@ overridePropertyFromProperties(result, "url", props); overridePropertyFromProperties(result, "encoding", props); overridePropertyFromProperties(result, "verbose", props); + // Anonymous means no credential filled overridePropertyFromProperties(authConfiguration, "username", props); overridePropertyFromProperties(authConfiguration, "password", props); overridePropertyFromProperties(authConfiguration, "apiKey", props); @@ -170,7 +172,6 @@ overridePropertyFromEnv(result, "url"); overridePropertyFromEnv(result, "encoding"); overridePropertyFromEnv(result, "verbose"); - result.setAnonymous(true); return result; } @@ -187,8 +188,6 @@ overridePropertyFromEnv(authConfiguration, "password"); overridePropertyFromEnv(authConfiguration, "apiKey"); - result.setAnonymous(false); - return result; } @@ -204,7 +203,6 @@ dst.setUrl(src.getUrl()); dst.setEncoding(src.getEncoding()); dst.setVerbose(src.isVerbose()); - dst.setAnonymous(src.isAnonymous()); RedmineClientAuthConfiguration authConfiguration = new RedmineClientAuthConfiguration(); authConfiguration.setApiKey(src.getAuthConfiguration().getApiKey()); @@ -223,7 +221,8 @@ } catch (Exception e) { throw new IOException( "Could not set property '" + prop + - "' with value '" + value + "' to configuration"); + "' with value '" + value + "' to configuration" + ); } } } @@ -237,11 +236,17 @@ } catch (Exception e) { throw new IOException( "Could not set property '" + prop + - "' with value '" + value + "' to configuration"); + "' with value '" + value + "' to configuration" + ); } } } + public static boolean isLoggued(RedmineClientConfiguration configuration) { + RedmineClientAuthConfiguration authConfiguration = configuration.getAuthConfiguration(); + return authConfiguration.isUseApiKey() || authConfiguration.isUseLoginPassword(); + } + /** * Configuration of a redmine service for test purposes. * @@ -254,8 +259,6 @@ boolean verbose; - boolean anonymous; - String encoding; RedmineClientAuthConfiguration authConfiguration; @@ -301,29 +304,12 @@ } @Override - public boolean isAnonymous() { - return anonymous; - } - - @Override - public void setAnonymous(boolean anonymous) { - this.anonymous = anonymous; - } - - @Override public String toString() { - ToStringBuilder b = new ToStringBuilder( - this, - ToStringStyle.MULTI_LINE_STYLE - ); + ToStringBuilder b = new ToStringBuilder(this, ToStringStyle.MULTI_LINE_STYLE); b.append("redmineUrl", url); - if (anonymous) { - b.append("anonymous", true); - } else { - b.append("apiKey", authConfiguration.getApiKey()); - b.append("username", authConfiguration.getUsername()); - b.append("password", "***"); - } + b.append("apiKey", authConfiguration.getApiKey()); + b.append("username", authConfiguration.getUsername()); + b.append("password", "***"); b.append("encoding", encoding); b.append("verbose", verbose); return b.toString(); Modified: trunk/jredmine-client/src/main/java/org/nuiton/jredmine/service/RedmineLogguedService.java =================================================================== --- trunk/jredmine-client/src/main/java/org/nuiton/jredmine/service/RedmineLogguedService.java 2014-04-16 12:52:01 UTC (rev 445) +++ trunk/jredmine-client/src/main/java/org/nuiton/jredmine/service/RedmineLogguedService.java 2014-04-16 12:52:49 UTC (rev 446) @@ -34,7 +34,7 @@ /** - * Contract of all redmine operations that needs a login to be performed. + * Contract of all redmine operations that needs a login to be performed (for both public or private projects). * * A default implementation is offered in {@link DefaultRedmineService}. * Modified: trunk/jredmine-client/src/test/java/org/nuiton/jredmine/RedmineFixtures.java =================================================================== --- trunk/jredmine-client/src/test/java/org/nuiton/jredmine/RedmineFixtures.java 2014-04-16 12:52:01 UTC (rev 445) +++ trunk/jredmine-client/src/test/java/org/nuiton/jredmine/RedmineFixtures.java 2014-04-16 12:52:49 UTC (rev 446) @@ -48,8 +48,6 @@ /** Logger. */ private static final Log log = LogFactory.getLog(RedmineFixtures.class); -// private ArrayListMultimap<Class<?>, Object> model; - public String projectName() { return "jredmine"; } Modified: trunk/jredmine-client/src/test/java/org/nuiton/jredmine/client/RedmineClientTest.java =================================================================== --- trunk/jredmine-client/src/test/java/org/nuiton/jredmine/client/RedmineClientTest.java 2014-04-16 12:52:01 UTC (rev 445) +++ trunk/jredmine-client/src/test/java/org/nuiton/jredmine/client/RedmineClientTest.java 2014-04-16 12:52:49 UTC (rev 446) @@ -44,6 +44,7 @@ import org.nuiton.jredmine.model.Tracker; import org.nuiton.jredmine.model.User; import org.nuiton.jredmine.model.Version; +import org.nuiton.jredmine.service.RedmineConfigurationUtil; import org.nuiton.jredmine.service.RedmineServiceConfiguration; import org.nuiton.jredmine.test.RedmineFixtureClassRule; @@ -253,25 +254,17 @@ } @Test - public void login() throws Exception { - - Assume.assumeTrue(!getConfiguration().isAnonymous()); - //Assume.assumeTrue(StringUtils.isBlank(getConfiguration().getApiKey())); - client.open(); + public void getUser() throws Exception { + assumeIsLoggued(); + RedmineRequest<User> request = RedmineRequestHelper.action( + "get_user.xml", User.class); + askData(request); } @Test - public void loginByApiKey() throws Exception { - - Assume.assumeTrue(!getConfiguration().isAnonymous()); - //Assume.assumeTrue(StringUtils.isNotBlank(getConfiguration().getAuthConfiguration().getApiKey())); - client.open(); - } - - @Test public void getUserProjects() throws Exception { - Assume.assumeTrue(!getConfiguration().isAnonymous()); + assumeIsLoggued(); RedmineRequest<Project> request = RedmineRequestHelper.action( "get_user_projects.xml", Project.class); askDatas(request); @@ -279,37 +272,42 @@ @Test public void addVersion() throws Exception { - Assume.assumeTrue(!getConfiguration().isAnonymous()); + assumeIsLoggued(); //TODO } @Test public void addAttachment() throws Exception { - Assume.assumeTrue(!getConfiguration().isAnonymous()); + assumeIsLoggued(); //TODO } @Test public void addNews() throws Exception { - Assume.assumeTrue(!getConfiguration().isAnonymous()); + assumeIsLoggued(); //TODO } + public void assumeIsLoggued() { + boolean loggued = RedmineConfigurationUtil.isLoggued(getConfiguration()); + Assume.assumeTrue(loggued); + } + @Test public void updateVersion() throws Exception { - Assume.assumeTrue(!getConfiguration().isAnonymous()); + assumeIsLoggued(); //TODO } @Test public void nextVersion() throws Exception { - Assume.assumeTrue(!getConfiguration().isAnonymous()); + assumeIsLoggued(); //TODO } @Test public void addIssueTime() throws Exception { - Assume.assumeTrue(!getConfiguration().isAnonymous()); + assumeIsLoggued(); //TODO } Modified: trunk/jredmine-client/src/test/java/org/nuiton/jredmine/test/AbstractRedmineFixtures.java =================================================================== --- trunk/jredmine-client/src/test/java/org/nuiton/jredmine/test/AbstractRedmineFixtures.java 2014-04-16 12:52:01 UTC (rev 445) +++ trunk/jredmine-client/src/test/java/org/nuiton/jredmine/test/AbstractRedmineFixtures.java 2014-04-16 12:52:49 UTC (rev 446) @@ -23,7 +23,6 @@ * #L% */ -import org.nuiton.jredmine.service.DefaultRedmineAnonymousService; import org.nuiton.jredmine.service.DefaultRedmineService; import org.nuiton.jredmine.service.RedmineAnonymousService; import org.nuiton.jredmine.service.RedmineConfigurationUtil; @@ -68,7 +67,7 @@ public RedmineAnonymousService newRedmineAnonymousService(RedmineServiceConfiguration configuration) throws IOException, RedmineServiceException { - RedmineAnonymousService service = new DefaultRedmineAnonymousService(); + RedmineAnonymousService service = new DefaultRedmineService(); service.init(configuration); return service; } Modified: trunk/jredmine-client/src/test/java/org/nuiton/jredmine/test/RedmineFixtureClassRule.java =================================================================== --- trunk/jredmine-client/src/test/java/org/nuiton/jredmine/test/RedmineFixtureClassRule.java 2014-04-16 12:52:01 UTC (rev 445) +++ trunk/jredmine-client/src/test/java/org/nuiton/jredmine/test/RedmineFixtureClassRule.java 2014-04-16 12:52:49 UTC (rev 446) @@ -80,18 +80,6 @@ testClass.getName()); } Assume.assumeTrue(false); - } else { - - // configuration must NOT be anonymous - - if (conf.isAnonymous()) { - - if (log.isWarnEnabled()) { - log.warn("A authenticated configuration was required, will skip test " + - testClass.getName()); - } - Assume.assumeTrue(false); - } } } catch (IOException e) {

1 0

r445 - trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin
by tchemit＠users.nuiton.org April 16, 2014

April 16, 2014

Author: tchemit Date: 2014-04-16 14:52:01 +0200 (Wed, 16 Apr 2014) New Revision: 445 Url: http://forge.nuiton.org/projects/jredmine/repository/revisions/445 Log: fixes #3157 Removed: trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/LoginMojo.java Deleted: trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/LoginMojo.java =================================================================== --- trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/LoginMojo.java 2014-04-16 12:51:47 UTC (rev 444) +++ trunk/jredmine-maven-plugin/src/main/java/org/nuiton/jredmine/plugin/LoginMojo.java 2014-04-16 12:52:01 UTC (rev 445) @@ -1,368 +0,0 @@ -/* - * #%L - * JRedmine :: Maven plugin - * - * $Id$ - * $HeadURL$ - * %% - * Copyright (C) 2009 - 2012 Tony Chemit, 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% - */ -package org.nuiton.jredmine.plugin; - -import org.apache.maven.model.IssueManagement; -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.apache.maven.settings.Settings; -import org.nuiton.helper.plugin.ShareServerSecretPlugin; -import org.nuiton.jredmine.client.RedmineClientAuthConfiguration; -import org.nuiton.jredmine.service.RedmineService; -import org.nuiton.jredmine.service.RedmineServiceConfiguration; -import org.nuiton.plugin.AbstractPlugin; -import org.sonatype.plexus.components.sec.dispatcher.SecDispatcher; - -import java.net.URL; -import java.util.Properties; - -/** - * Obtain login for the redmine server to use. - * - * @author Tony Chemit <chemit(a)codelutin.com> Copyright Code Lutin - * @since 1.2.1 - */ -@Mojo(name = "login", requiresOnline = true, requiresProject = true) -public class LoginMojo extends AbstractPlugin implements RedmineServiceConfiguration { - - /////////////////////////////////////////////////////////////////////////// - /// Mojo parameters - /////////////////////////////////////////////////////////////////////////// - - /** - * The real basedir redmine url. - * - * If no url is given, will use the issue management url. - * - * @since 1.2.1 - */ - @Parameter(property = "redmine.url") - protected URL url; - - /** - * Flag to activate verbose logging. - * - * @since 1.2.1 - */ - @Parameter(property = "redmine.verbose", defaultValue = "${maven.verbose}") - protected boolean verbose; - - /** - * Flag to check login is ok (will connect to server). - * - * @since 1.2.1 - */ - @Parameter(property = "redmine.checkLogin", defaultValue = "false") - protected boolean checkLogin; - - /** - * Flag to fail build if configuration is not ok. - * - * @since 1.2.1 - */ - @Parameter(property = "redmine.safe", defaultValue = "true") - protected boolean safe; - - /** - * Redmine server id to obtain login and password. - * - * The server must be defined in your settings.xml file in servers section. - * - * @since 1.2.1 - */ - @Parameter(property = "redmine.serverId", required = true) - protected String serverId; - - /** - * Un flag pour utiliser le clef API pluto que le login/password. - * - * Il faut alors que le serveur contienne en username le clef API (encrypté ou non). - * - * @since 2.0 - */ - @Parameter(property = "redmine.useApiKey", defaultValue = "false") - private boolean useApiKey; - - /////////////////////////////////////////////////////////////////////////// - /// Mojo components - /////////////////////////////////////////////////////////////////////////// - - /** - * Current project. - * - * @since 1.2.1 - */ - @Component - protected MavenProject project; - - /** - * Settings. - * - * @since 1.2.1 - */ - @Component - protected Settings settings; - - /** - * Password decypher. - * - * @since 1.2.1 - */ - @Component(hint = "helper-maven-plugin") - protected SecDispatcher sec; - - /** - * Redmine service. - * - * @component - * @since 1.2.1 - */ - @Component - protected RedmineService service; - - /////////////////////////////////////////////////////////////////////////// - /// Mojo internal attributes - /////////////////////////////////////////////////////////////////////////// - - /** - * Shared instance of delegate plugin to obtain login - * - * @since 1.2.1 - */ - protected static ShareServerSecretPlugin plugin; - - - private RedmineClientAuthConfiguration authConfiguration; - - private static final String REDMINE_USERNAME = "redmine.username"; - - private static final String REDMINE_PASSWORD = "redmine.password"; - - private static final String REDMINE_API_KEY = "redmine.apiKey"; - - /////////////////////////////////////////////////////////////////////////// - /// Plugin - /////////////////////////////////////////////////////////////////////////// - - @Override - public MavenProject getProject() { - return project; - } - - @Override - public void setProject(MavenProject project) { - this.project = project; - } - - @Override - public boolean isVerbose() { - return verbose; - } - - @Override - public void setVerbose(boolean verbose) { - this.verbose = verbose; - } - - /////////////////////////////////////////////////////////////////////////// - /// PluginWithEncoding - /////////////////////////////////////////////////////////////////////////// - - @Override - public String getEncoding() { - return null; - } - - @Override - public void setEncoding(String encoding) { - } - - /////////////////////////////////////////////////////////////////////////// - /// RedmineClientConfiguration - /////////////////////////////////////////////////////////////////////////// - - @Override - public boolean isAnonymous() { - return false; - } - - @Override - public void setAnonymous(boolean anonymous) { - } - - @Override - public URL getUrl() { - return url; - } - - @Override - public void setUrl(URL url) { - this.url = url; - } - - @Override - public RedmineClientAuthConfiguration getAuthConfiguration() { - return authConfiguration; - } - - @Override - public void setAuthConfiguration(RedmineClientAuthConfiguration authConfiguration) { - this.authConfiguration = authConfiguration; - } - - /////////////////////////////////////////////////////////////////////////// - /// AbstractPlugin - /////////////////////////////////////////////////////////////////////////// - - @Override - protected void init() throws Exception { - if (plugin == null) { - plugin = new ShareServerSecretPlugin(); - plugin.setServerId(serverId); - if (useApiKey) { - plugin.setUsernameOut(REDMINE_API_KEY); - } else { - plugin.setUsernameOut(REDMINE_USERNAME); - plugin.setPasswordOut(REDMINE_PASSWORD); - } - plugin.setRunOnce(true); - - plugin.setProject(project); - plugin.setLog(getLog()); - plugin.setVerbose(isVerbose()); - plugin.setSettings(settings); - plugin.setSec(sec); - - plugin.init(); - } - - if (!checkLogin) { - // no more thing to do - return; - } - - // check issue management - - IssueManagement issueManagement = project.getIssueManagement(); - - if (issueManagement == null) { - throw new MojoExecutionException("No Issue Management set."); - } else if ((issueManagement.getUrl() == null) || (issueManagement.getUrl().trim().equals(""))) { - throw new MojoExecutionException("No URL set in Issue Management."); - } else if ((issueManagement.getSystem() != null) && !(issueManagement.getSystem().equalsIgnoreCase(AbstractRedmineMojo.REDMINE_SYSTEM))) { - throw new MojoExecutionException("Redmine's Plugin only supports 'redmine' Issue Management system."); - } - - // prepare Redmine service configuration - - if (url == null || url.toString().isEmpty()) { - - // no redmine url specified, guess it from issueManagement - - url = new URL(issueManagement.getUrl()); - - if (verbose) { - getLog().info("use the url from issue management : " + url); - } - } - - // apply configuration - - setUrl(url); - } - - @Override - protected boolean checkSkip() { - - return plugin.checkSkip(); - } - - @Override - protected void doAction() throws Exception { - - getLog().info("Obtain redmine login for serverId '" + serverId + "'"); - // when plugin is executed, then will repass in method checkSkip and then - // will never execute the plugin :( Should instead open Plugin api to - // pass to public the doAction method - plugin.setRunOnce(false); - plugin.execute(); - plugin.setRunOnce(true); - - if (!checkLogin) { - // no more thing to do - return; - } - - Properties properties = project.getProperties(); - authConfiguration = RedmineClientAuthConfiguration.newConf( - properties.getProperty(REDMINE_API_KEY), - properties.getProperty(REDMINE_USERNAME), - properties.getProperty(REDMINE_PASSWORD) - ); - - // init Redmine service - - try { - - service.init(this); -// if (!((RedmineServiceImplementor) service).isInit()) { -// throw new MojoExecutionException("could not logged to redmine server"); -// } - - } catch (Exception e) { - if (safe) { - throw e; - } -// if (verbose) { - getLog().error("could not init Redmine service [" + getUrl() + "]", e); -// } -// return false; - } - - getLog().info("Connection to redmine server successfull."); - } - - - @Override - protected void afterExecute() { - if (service != null) { - - if (service.isInit()) { - try { - if (verbose) { - getLog().info("<<< Close redmine rest client..."); - } - service.destroy(); - } catch (Exception ex) { - getLog().error("could not close redmine client for reason " + ex.getMessage(), ex); - } - } - } - } - -}

1 0