clarify some TODO elements

git-svn-id: https://svn.apache.org/repos/asf/maven/archiva/trunk@441803 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/archiva-core/src/main/java/org/apache/maven/archiva/scheduler/DefaultRepositoryTaskScheduler.java b/archiva-core/src/main/java/org/apache/maven/archiva/scheduler/DefaultRepositoryTaskScheduler.java
index 3db17e8edf3977bbd3c4bc9e62ca2ff20653dc5b..bee12f0784a93ab1ec367c7135bbdd4335e0c68d 100644 (file)
--- a/archiva-core/src/main/java/org/apache/maven/archiva/scheduler/DefaultRepositoryTaskScheduler.java
+++ b/archiva-core/src/main/java/org/apache/maven/archiva/scheduler/DefaultRepositoryTaskScheduler.java
@@ -97,7 +97,7 @@ public class DefaultRepositoryTaskScheduler
     private void scheduleJobs( Configuration configuration )
         throws ParseException, SchedulerException
     {
-        // TODO! would be nice to queue jobs that are triggered so we could avoid two running at the same time (so have a queue for discovery based jobs so they didn't thrash the repo)
+        // TODO: would be nice to queue jobs that are triggered so we could avoid two running at the same time (so have a queue for discovery based jobs so they didn't thrash the repo)
         if ( configuration.getIndexPath() != null )
         {
             JobDetail jobDetail = createJobDetail( INDEXER_JOB, indexerTask );
@@ -107,7 +107,7 @@ public class DefaultRepositoryTaskScheduler
                 new CronTrigger( INDEXER_JOB + "Trigger", DISCOVERER_GROUP, configuration.getIndexerCronExpression() );
             scheduler.scheduleJob( jobDetail, trigger );
 
-            // TODO: run as a job so it doesn't block startup/configuration saving!
+            // TODO: run as a job so it doesn't block startup/configuration saving
             try
             {
                 indexerTask.executeNowIfNeeded();

diff --git a/archiva-reports-standard/src/main/java/org/apache/maven/archiva/reporting/BadMetadataReportProcessor.java b/archiva-reports-standard/src/main/java/org/apache/maven/archiva/reporting/BadMetadataReportProcessor.java
index a307e16f22fe6e38107c638426a5113b4d9b2a58..907bed9b0bfc50362a38d8ed3d6701c00e643707 100644 (file)
--- a/archiva-reports-standard/src/main/java/org/apache/maven/archiva/reporting/BadMetadataReportProcessor.java
+++ b/archiva-reports-standard/src/main/java/org/apache/maven/archiva/reporting/BadMetadataReportProcessor.java
@@ -264,7 +264,7 @@ public class BadMetadataReportProcessor
         File versionsDir =
             new File( repository.getBasedir(), repository.pathOfRemoteRepositoryMetadata( metadata ) ).getParentFile();
 
-        // TODO! I don't know how this condition can happen, but it was seen on the main repository.
+        // TODO: I don't know how this condition can happen, but it was seen on the main repository.
         // Avoid hard failure
         if ( versionsDir.exists() )
         {

diff --git a/archiva-webapp/src/main/java/org/apache/maven/archiva/web/action/ProxyAction.java b/archiva-webapp/src/main/java/org/apache/maven/archiva/web/action/ProxyAction.java
index c7c57f16e93955653eaed905caea54a1c80c6c87..219dbfcaeff3ada1e96921c43714eff9854c3ee3 100644 (file)
--- a/archiva-webapp/src/main/java/org/apache/maven/archiva/web/action/ProxyAction.java
+++ b/archiva-webapp/src/main/java/org/apache/maven/archiva/web/action/ProxyAction.java
@@ -58,7 +58,7 @@ public class ProxyAction
 
             artifactStream = new FileInputStream( file );
 
-            // TODO: could be better!
+            // TODO: set the correct content type and other headers!
             contentType = "application/octet-stream";
 
             filename = file.getName();

diff --git a/archiva-webapp/src/main/java/org/apache/maven/archiva/web/action/admin/DeleteSyncedRepositoryAction.java b/archiva-webapp/src/main/java/org/apache/maven/archiva/web/action/admin/DeleteSyncedRepositoryAction.java
index dc286c65600bfaa94b041e1f52bc3fb9dcc8ec0b..f562f33f56a7cfb3992ceaf2d050ceb3d20bbeb9 100644 (file)
--- a/archiva-webapp/src/main/java/org/apache/maven/archiva/web/action/admin/DeleteSyncedRepositoryAction.java
+++ b/archiva-webapp/src/main/java/org/apache/maven/archiva/web/action/admin/DeleteSyncedRepositoryAction.java
@@ -43,6 +43,6 @@ public class DeleteSyncedRepositoryAction
     protected void removeContents( AbstractRepositoryConfiguration existingRepository )
         throws IOException
     {
-        // TODO!
+        // TODO! remove the contents
     }
 }

diff --git a/archiva-webapp/src/main/webapp/WEB-INF/jsp/showArtifact.jsp b/archiva-webapp/src/main/webapp/WEB-INF/jsp/showArtifact.jsp
index ce9d5eabf0fbe05087df21016d091096f8d00b5c..d05ce8586b93c62129f842aaba43d648f36524e6 100644 (file)
--- a/archiva-webapp/src/main/webapp/WEB-INF/jsp/showArtifact.jsp
+++ b/archiva-webapp/src/main/webapp/WEB-INF/jsp/showArtifact.jsp
@@ -74,7 +74,7 @@
       <ww:url action="proxy">
         <%-- TODO! create a tag for this, include classifier for javadoc, sources below --%>
         <%-- TODO: what about other repositories? --%>
-        <%-- TODO! extension probably doesn't match type --%>
+        <%-- TODO! extension probably doesn't match type. Use artifact handler instead. --%>
         <ww:param name="path"
                   value="%{'${model.groupId}/${model.artifactId}/${model.version}/${model.artifactId}-${model.version}.${model.packaging}'}"/>
       </ww:url>