Author: dennisl
Date: Tue Jan  4 20:49:03 2011
New Revision: 1055177

URL: http://svn.apache.org/viewvc?rev=1055177&view=rev
Log:
o Use the Locale object when formating dates, instead of using manual 
formatting patterns stored in resource bundles.

Modified:
    
maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/issues/IssuesReportGenerator.java
    
maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/JiraMojo.java
    
maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/trac/TracMojo.java
    
maven/plugins/trunk/maven-changes-plugin/src/main/resources/trac-report.properties
    
maven/plugins/trunk/maven-changes-plugin/src/main/resources/trac-report_de.properties
    
maven/plugins/trunk/maven-changes-plugin/src/main/resources/trac-report_fr.properties
    
maven/plugins/trunk/maven-changes-plugin/src/main/resources/trac-report_sv.properties

Modified: 
maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/issues/IssuesReportGenerator.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/issues/IssuesReportGenerator.java?rev=1055177&r1=1055176&r2=1055177&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/issues/IssuesReportGenerator.java
 (original)
+++ 
maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/issues/IssuesReportGenerator.java
 Tue Jan  4 20:49:03 2011
@@ -22,9 +22,10 @@ package org.apache.maven.plugin.issues;
 import org.apache.maven.doxia.sink.Sink;
 import org.apache.maven.reporting.MavenReportException;
 
+import java.text.DateFormat;
 import java.text.SimpleDateFormat;
 import java.util.List;
-import java.util.MissingResourceException;
+import java.util.Locale;
 import java.util.ResourceBundle;
 
 /**
@@ -64,13 +65,13 @@ public class IssuesReportGenerator
         sinkEndReport( sink );
     }
 
-    public void doGenerateReport( ResourceBundle bundle, Sink sink, List 
issueList )
+    public void doGenerateReport( ResourceBundle bundle, Sink sink, List 
issueList, Locale locale )
     {
         sinkBeginReport( sink, bundle );
 
         constructHeaderRow( sink, issueList, bundle );
 
-        constructDetailRows( sink, issueList, bundle );
+        constructDetailRows( sink, issueList, bundle, locale );
 
         sinkEndReport( sink );
     }
@@ -155,7 +156,7 @@ public class IssuesReportGenerator
         sink.tableRow_();
     }
 
-    private void constructDetailRows( Sink sink, List issueList, 
ResourceBundle bundle )
+    private void constructDetailRows( Sink sink, List issueList, 
ResourceBundle bundle, Locale locale )
     {
         if ( issueList == null )
         {
@@ -164,15 +165,8 @@ public class IssuesReportGenerator
 
         for ( int idx = 0; idx < issueList.size(); idx++ )
         {
-            SimpleDateFormat sdf;
-            try
-            {
-                sdf = new SimpleDateFormat( bundle.getString( 
"report.issues.dateformat" ) );
-            }
-            catch ( MissingResourceException mre )
-            {
-                sdf = new SimpleDateFormat();
-            }
+            // Use a DateFormat based on the the Locale
+            DateFormat df = DateFormat.getDateInstance(DateFormat.SHORT, 
locale);
 
             Issue issue = (Issue) issueList.get( idx );
 
@@ -191,7 +185,7 @@ public class IssuesReportGenerator
                         break;
 
                     case IssuesReportHelper.COLUMN_CREATED:
-                        sinkCell( sink, sdf.format( issue.getCreated() ) );
+                        sinkCell( sink, df.format( issue.getCreated() ) );
                         break;
 
                     case IssuesReportHelper.COLUMN_FIX_VERSION:
@@ -239,7 +233,7 @@ public class IssuesReportGenerator
                         break;
 
                     case IssuesReportHelper.COLUMN_UPDATED:
-                        sinkCell( sink, sdf.format( issue.getUpdated() ) );
+                        sinkCell( sink, df.format( issue.getUpdated() ) );
                         break;
 
                     case IssuesReportHelper.COLUMN_VERSION:

Modified: 
maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/JiraMojo.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/JiraMojo.java?rev=1055177&r1=1055176&r2=1055177&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/JiraMojo.java
 (original)
+++ 
maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/JiraMojo.java
 Tue Jan  4 20:49:03 2011
@@ -302,7 +302,7 @@ public class JiraMojo
                     getLog().info( "The JIRA Report will contain issues only 
for the current version." );
                 }
 
-                report.doGenerateReport( getBundle( locale ), getSink(), 
issueList );
+                report.doGenerateReport( getBundle( locale ), getSink(), 
issueList, locale );
             }
             else
             {

Modified: 
maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/trac/TracMojo.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/trac/TracMojo.java?rev=1055177&r1=1055176&r2=1055177&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/trac/TracMojo.java
 (original)
+++ 
maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/trac/TracMojo.java
 Tue Jan  4 20:49:03 2011
@@ -144,7 +144,7 @@ public class TracMojo
             {
                 try
                 {
-                    report.doGenerateReport( getBundle( locale ), getSink(), 
issueList );
+                    report.doGenerateReport( getBundle( locale ), getSink(), 
issueList, locale );
                 }
                 catch ( Exception e )
                 {

Modified: 
maven/plugins/trunk/maven-changes-plugin/src/main/resources/trac-report.properties
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changes-plugin/src/main/resources/trac-report.properties?rev=1055177&r1=1055176&r2=1055177&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-changes-plugin/src/main/resources/trac-report.properties
 (original)
+++ 
maven/plugins/trunk/maven-changes-plugin/src/main/resources/trac-report.properties
 Tue Jan  4 20:49:03 2011
@@ -20,7 +20,6 @@ report.issues.description=Report on tick
 report.issues.error=An error occured that made it impossible to generate this 
report. \
   Please check the console for information on what might be the cause of this.
 report.issues.header=Trac Tickets
-report.issues.dateformat=dd MMM yy
 report.issues.label.assignee=Owner
 report.issues.label.component=Component
 report.issues.label.created=Created

Modified: 
maven/plugins/trunk/maven-changes-plugin/src/main/resources/trac-report_de.properties
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changes-plugin/src/main/resources/trac-report_de.properties?rev=1055177&r1=1055176&r2=1055177&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-changes-plugin/src/main/resources/trac-report_de.properties
 (original)
+++ 
maven/plugins/trunk/maven-changes-plugin/src/main/resources/trac-report_de.properties
 Tue Jan  4 20:49:03 2011
@@ -20,7 +20,6 @@ report.issues.description=Bericht \u00FC
 report.issues.error=Es ist ein Fehler aufgetreten, der die Generierung dieses 
Berichts verhinderte. \
   Bitte kontrollieren Sie die Konsolenausgabe im Hinblick auf Informationen 
\u00FCber die Ursache.
 report.issues.header=Trac-Tickets
-report.issues.dateformat=dd MMM yy
 report.issues.label.assignee=Besitzer
 report.issues.label.component=Komponente
 report.issues.label.created=Erstellt

Modified: 
maven/plugins/trunk/maven-changes-plugin/src/main/resources/trac-report_fr.properties
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changes-plugin/src/main/resources/trac-report_fr.properties?rev=1055177&r1=1055176&r2=1055177&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-changes-plugin/src/main/resources/trac-report_fr.properties
 (original)
+++ 
maven/plugins/trunk/maven-changes-plugin/src/main/resources/trac-report_fr.properties
 Tue Jan  4 20:49:03 2011
@@ -20,7 +20,6 @@ report.issues.description=Rapport des ti
 report.issues.error=Une erreur est survenue, ce qui ne permet pas de 
g\u00E9n\u00E9rer le rapport. \
   Merci de v\u00E9rifier les traces dans la console pour conna\u00EEtre la 
cause de l'erreur.
 report.issues.header=Tickets Trac
-report.issues.dateformat=dd MMM yy
 report.issues.label.assignee=Propri\u00E9taire
 report.issues.label.component=Composant
 report.issues.label.created=Cr\u00E9e

Modified: 
maven/plugins/trunk/maven-changes-plugin/src/main/resources/trac-report_sv.properties
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changes-plugin/src/main/resources/trac-report_sv.properties?rev=1055177&r1=1055176&r2=1055177&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-changes-plugin/src/main/resources/trac-report_sv.properties
 (original)
+++ 
maven/plugins/trunk/maven-changes-plugin/src/main/resources/trac-report_sv.properties
 Tue Jan  4 20:49:03 2011
@@ -20,7 +20,6 @@ report.issues.description=Rapport \u00f6
 report.issues.error=Ett fel intr\u00e4ffade som gjorde det om\u00f6jligt att 
skapa denna rapport. \
   V\u00e4nligen kontrollera kommandoraden f\u00f6r information om vad som kan 
ha orsakat detta.
 report.issues.header=Trac-rapport
-report.issues.dateformat=yyyy-MM-dd
 report.issues.label.assignee=\u00c4gare
 report.issues.label.component=Komponent
 report.issues.label.created=Skapad


Reply via email to