Return-Path: Delivered-To: apmail-continuum-commits-archive@www.apache.org Received: (qmail 58535 invoked from network); 26 Jun 2009 04:48:50 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.3) by minotaur.apache.org with SMTP; 26 Jun 2009 04:48:50 -0000 Received: (qmail 42555 invoked by uid 500); 26 Jun 2009 04:49:01 -0000 Delivered-To: apmail-continuum-commits-archive@continuum.apache.org Received: (qmail 42501 invoked by uid 500); 26 Jun 2009 04:49:01 -0000 Mailing-List: contact commits-help@continuum.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@continuum.apache.org Delivered-To: mailing list commits@continuum.apache.org Received: (qmail 42492 invoked by uid 99); 26 Jun 2009 04:49:00 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 26 Jun 2009 04:49:00 +0000 X-ASF-Spam-Status: No, hits=-2000.0 required=10.0 tests=ALL_TRUSTED X-Spam-Check-By: apache.org Received: from [140.211.11.4] (HELO eris.apache.org) (140.211.11.4) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 26 Jun 2009 04:48:58 +0000 Received: by eris.apache.org (Postfix, from userid 65534) id D89EC23888D6; Fri, 26 Jun 2009 04:48:38 +0000 (UTC) Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: svn commit: r788573 - in /continuum/branches/continuum-1.3.x/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action: AddProjectAction.java BuildProjectAction.java Date: Fri, 26 Jun 2009 04:48:38 -0000 To: commits@continuum.apache.org From: evenisse@apache.org X-Mailer: svnmailer-1.0.8 Message-Id: <20090626044838.D89EC23888D6@eris.apache.org> X-Virus-Checked: Checked by ClamAV on apache.org Author: evenisse Date: Fri Jun 26 04:48:38 2009 New Revision: 788573 URL: http://svn.apache.org/viewvc?rev=788573&view=rev Log: Add some TODO Modified: continuum/branches/continuum-1.3.x/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/AddProjectAction.java continuum/branches/continuum-1.3.x/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/BuildProjectAction.java Modified: continuum/branches/continuum-1.3.x/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/AddProjectAction.java URL: http://svn.apache.org/viewvc/continuum/branches/continuum-1.3.x/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/AddProjectAction.java?rev=788573&r1=788572&r2=788573&view=diff ============================================================================== --- continuum/branches/continuum-1.3.x/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/AddProjectAction.java (original) +++ continuum/branches/continuum-1.3.x/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/AddProjectAction.java Fri Jun 26 04:48:38 2009 @@ -23,6 +23,8 @@ import java.util.Collection; import java.util.List; +import org.apache.continuum.web.util.AuditLog; +import org.apache.continuum.web.util.AuditLogConstants; import org.apache.maven.continuum.ContinuumException; import org.apache.maven.continuum.builddefinition.BuildDefinitionServiceException; import org.apache.maven.continuum.model.project.BuildDefinitionTemplate; @@ -32,8 +34,6 @@ import org.apache.maven.continuum.profile.ProfileException; import org.apache.maven.continuum.profile.ProfileService; import org.apache.maven.continuum.web.exception.AuthorizationRequiredException; -import org.apache.continuum.web.util.AuditLog; -import org.apache.continuum.web.util.AuditLogConstants; import org.codehaus.plexus.util.StringUtils; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -148,6 +148,7 @@ String projectNameTrim = projectName.trim(); String versionTrim = projectVersion.trim(); String scmTrim = projectScmUrl.trim(); + //TODO: Instead of get all projects then test them, it would be better to check it directly in the DB for ( Project project : getContinuum().getProjects() ) { // CONTINUUM-1445 Modified: continuum/branches/continuum-1.3.x/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/BuildProjectAction.java URL: http://svn.apache.org/viewvc/continuum/branches/continuum-1.3.x/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/BuildProjectAction.java?rev=788573&r1=788572&r2=788573&view=diff ============================================================================== --- continuum/branches/continuum-1.3.x/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/BuildProjectAction.java (original) +++ continuum/branches/continuum-1.3.x/continuum-webapp/src/main/java/org/apache/maven/continuum/web/action/BuildProjectAction.java Fri Jun 26 04:48:38 2009 @@ -76,6 +76,7 @@ } else { + //TODO: Check if it is called somewhere getContinuum().buildProjects(); } }