Author: evenisse
Date: Mon Sep 25 00:30:34 2006
New Revision: 449590

URL: http://svn.apache.org/viewvc?view=rev&rev=449590
Log:
[SCM-199] Fix startDate  parameter for ClearcaseLT

Modified:
    
maven/scm/trunk/maven-scm-providers/maven-scm-provider-clearcase/src/main/java/org/apache/maven/scm/provider/clearcase/command/changelog/ClearCaseChangeLogCommand.java
    
maven/scm/trunk/maven-scm-providers/maven-scm-provider-clearcase/src/test/java/org/apache/maven/scm/provider/clearcase/command/changelog/ClearCaseChangeLogCommandTest.java

Modified: 
maven/scm/trunk/maven-scm-providers/maven-scm-provider-clearcase/src/main/java/org/apache/maven/scm/provider/clearcase/command/changelog/ClearCaseChangeLogCommand.java
URL: 
http://svn.apache.org/viewvc/maven/scm/trunk/maven-scm-providers/maven-scm-provider-clearcase/src/main/java/org/apache/maven/scm/provider/clearcase/command/changelog/ClearCaseChangeLogCommand.java?view=diff&rev=449590&r1=449589&r2=449590
==============================================================================
--- 
maven/scm/trunk/maven-scm-providers/maven-scm-provider-clearcase/src/main/java/org/apache/maven/scm/provider/clearcase/command/changelog/ClearCaseChangeLogCommand.java
 (original)
+++ 
maven/scm/trunk/maven-scm-providers/maven-scm-provider-clearcase/src/main/java/org/apache/maven/scm/provider/clearcase/command/changelog/ClearCaseChangeLogCommand.java
 Mon Sep 25 00:30:34 2006
@@ -38,6 +38,7 @@
  * @author <a href="mailto:[EMAIL PROTECTED]">Emmanuel Venisse</a>
  * @author <a href="mailto:[EMAIL PROTECTED]">Trygve Laugst&oslash;l</a>
  * @author <a href="mailto:[EMAIL PROTECTED]">Frederic Mura</a>
+ * @author <a href="mailto:[EMAIL PROTECTED]">Mark Holster</a>
  * @version $Id$
  */
 public class ClearCaseChangeLogCommand
@@ -109,15 +110,12 @@
         format.append( "COMM:%-12.12o - %o - %c - Activity: %[activity]p\\n" );
         format.append( "USER:%-8.8u\\n" );
 
-        if ( !isClearCaseLT() )
-        {
-            command.createArgument().setValue( "-fmt" );
-            command.createArgument().setValue( format.toString() );
-        }
+        command.createArgument().setValue( "-fmt" );
+        command.createArgument().setValue( format.toString() );
         command.createArgument().setValue( "-recurse" );
         command.createArgument().setValue( "-nco" );
 
-        if ( startDate != null && !isClearCaseLT() )
+        if ( startDate != null)
         {
             SimpleDateFormat sdf = new SimpleDateFormat( "dd-MMM-yyyy", 
Locale.ENGLISH );
 
@@ -138,25 +136,5 @@
         }
 
         return command;
-    }
-
-    /**
-     * @return the value of the setting property 'clearcaseLT'
-     */
-    protected static boolean isClearCaseLT()
-    {
-        return settings.isClearcaseLT();
-    }
-
-    /**
-     * Frederic Mura
-     * Only use for test case
-     *
-     * @param isClearCaseLT
-     * @deprecated
-     */
-    protected static void setIsClearCaseLT( boolean isClearCaseLT )
-    {
-        settings.setClearcaseLT( isClearCaseLT );
     }
 }

Modified: 
maven/scm/trunk/maven-scm-providers/maven-scm-provider-clearcase/src/test/java/org/apache/maven/scm/provider/clearcase/command/changelog/ClearCaseChangeLogCommandTest.java
URL: 
http://svn.apache.org/viewvc/maven/scm/trunk/maven-scm-providers/maven-scm-provider-clearcase/src/test/java/org/apache/maven/scm/provider/clearcase/command/changelog/ClearCaseChangeLogCommandTest.java?view=diff&rev=449590&r1=449589&r2=449590
==============================================================================
--- 
maven/scm/trunk/maven-scm-providers/maven-scm-provider-clearcase/src/test/java/org/apache/maven/scm/provider/clearcase/command/changelog/ClearCaseChangeLogCommandTest.java
 (original)
+++ 
maven/scm/trunk/maven-scm-providers/maven-scm-provider-clearcase/src/test/java/org/apache/maven/scm/provider/clearcase/command/changelog/ClearCaseChangeLogCommandTest.java
 Mon Sep 25 00:30:34 2006
@@ -25,6 +25,7 @@
 /**
  * @author <a href="mailto:[EMAIL PROTECTED]">Emmanuel Venisse</a>
  * @author <a href="mailto:[EMAIL PROTECTED]">Trygve Laugst&oslash;l</a>
+ * @author <a href="mailto:[EMAIL PROTECTED]">Mark Holster</a>
  * @version $Id$
  */
 public class ClearCaseChangeLogCommandTest
@@ -41,8 +42,6 @@
 
         testCommandLine( tag, startDate, endDate,
                          "cleartool lshistory -fmt 
\"NAME:%En\\nDATE:%Nd\\nCOMM:%-12.12o - %o - %c - Activity: 
%[activity]p\\nUSER:%-8.8u\\n\" -recurse -nco" );
-
-        testClearCaseLTCommandLine( tag, startDate, endDate, "cleartool 
lshistory -recurse -nco" );
     }
 
     public void testGetCommandLineWithTag()
@@ -56,8 +55,6 @@
 
         testCommandLine( tag, startDate, endDate,
                          "cleartool lshistory -fmt 
\"NAME:%En\\nDATE:%Nd\\nCOMM:%-12.12o - %o - %c - Activity: 
%[activity]p\\nUSER:%-8.8u\\n\" -recurse -nco -branch myBranch" );
-
-        testClearCaseLTCommandLine( tag, startDate, endDate, "cleartool 
lshistory -recurse -nco -branch myBranch" );
     }
 
     public void testGetCommandLineWithStartDate()
@@ -71,8 +68,6 @@
 
         testCommandLine( tag, startDate, endDate,
                          "cleartool lshistory -fmt 
\"NAME:%En\\nDATE:%Nd\\nCOMM:%-12.12o - %o - %c - Activity: 
%[activity]p\\nUSER:%-8.8u\\n\" -recurse -nco -since 10-Sep-2003" );
-
-        testClearCaseLTCommandLine( tag, startDate, endDate, "cleartool 
lshistory -recurse -nco" );
     }
 
     public void testGetCommandLineWithTagAndStartDate()
@@ -86,8 +81,6 @@
 
         testCommandLine( tag, startDate, endDate,
                          "cleartool lshistory -fmt 
\"NAME:%En\\nDATE:%Nd\\nCOMM:%-12.12o - %o - %c - Activity: 
%[activity]p\\nUSER:%-8.8u\\n\" -recurse -nco -since 10-Sep-2003 -branch 
myBranch" );
-
-        testClearCaseLTCommandLine( tag, startDate, endDate, "cleartool 
lshistory -recurse -nco -branch myBranch" );
     }
 
     // ----------------------------------------------------------------------
@@ -98,21 +91,6 @@
         throws Exception
     {
         File workingDirectory = getTestFile( 
"target/clearcare-changelog-command-test" );
-
-        ClearCaseChangeLogCommand.setIsClearCaseLT( false );
-
-        Commandline cl = ClearCaseChangeLogCommand.createCommandLine( 
workingDirectory, tag, startDate );
-
-        assertEquals( commandLine, cl.toString() );
-    }
-
-    private void testClearCaseLTCommandLine( String tag, Date startDate, Date 
endDate, String commandLine )
-        throws Exception
-    {
-
-        File workingDirectory = getTestFile( 
"target/clearcare-changelog-command-test" );
-
-        ClearCaseChangeLogCommand.setIsClearCaseLT( true );
 
         Commandline cl = ClearCaseChangeLogCommand.createCommandLine( 
workingDirectory, tag, startDate );
 


Reply via email to