Mailing-List: contact geronimo-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: geronimo-dev@incubator.apache.org
Message-ID: <2847477.1071144655926.JavaMail.orion@hogshead.codehaus.org>
Date: Thu, 11 Dec 2003 06:10:55 -0600 (CST)
From: jira@codehaus.org
To: geronimo-dev@incubator.apache.org
Subject: [jira] Closed: (GERONIMO-122) Proposed refactorings for
 o.a.g.kernel.deployment.goal.*
Mime-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit

Message:

   The following issue has been closed.

   Resolver: Gianny DAMOUR
       Date: Thu, 11 Dec 2003 6:10 AM

Thanks for this patch Ed Letifov.
---------------------------------------------------------------------
View the issue:

  http://jira.codehaus.org/secure/ViewIssue.jspa?key=GERONIMO-122


Here is an overview of the issue:
---------------------------------------------------------------------
        Key: GERONIMO-122
    Summary: Proposed refactorings for o.a.g.kernel.deployment.goal.*
       Type: Improvement

     Status: Closed
   Priority: Trivial
 Resolution: FIXED

 Original Estimate: Unknown
 Time Spent: Unknown
  Remaining: Unknown

    Project: Apache Geronimo

   Assignee: Gianny DAMOUR
   Reporter: Ed Letifov

    Created: Tue, 2 Dec 2003 3:44 PM
    Updated: Thu, 11 Dec 2003 6:10 AM

Description:
I am under impression that some classes in o.a.g.kernel.deployment.goal.* can (should?) be refactored just a little to facilitate more reuse between them. IMHO an abstract class would be welcome in the hierarchy to reduce copy paste in the implementation (and also make tests simpler) 

I have some concerns about UndeployURL and RedeployURL that do not behave in the same way as other two: they do not normalize file-protocol based URLs and seem to accept null values for the arguments in the constructors. Is it in fact how it is supposed to be?


---------------------------------------------------------------------
JIRA INFORMATION:
This message is automatically generated by JIRA.

If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa

If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira