Message:

   The following issue has been resolved as FIXED.

   Resolver: Michal Maczka
       Date: Tue, 5 Aug 2003 2:29 PM

We don't need it any more.
${artifact.parent.parent} does a trick
---------------------------------------------------------------------
View the issue:

  http://jira.codehaus.org/secure/ViewIssue.jspa?key=MAVEN-195


Here is an overview of the issue:
---------------------------------------------------------------------
        Key: MAVEN-195
    Summary: Artifact enhancement - add method getArtifactSubDirectory()
       Type: Improvement

     Status: Resolved
   Priority: Minor
 Resolution: FIXED

 Time Spent: Unknown
  Remaining: Unknown

    Project: maven
   Fix Fors:
             1.0-rc1
   Versions:
             1.0-beta-8

   Assignee: Jason van Zyl
   Reporter: Michal Maczka

    Created: Sun, 12 Jan 2003 11:28 AM
    Updated: Tue, 5 Aug 2003 2:29 PM

Description:
I think that it will be nice to have single place in GenericArtifact class 
which hided strategy how the directory where artifact file is kept 
is named. 

Currently the strategy to name this directory is:


getDependency().getType() + "s"



My proposition is to have:

/**
 * Get the directory to place the artifact in 
 *
 * @return The artifact sub directory.
 */             
 public String getArtifactSubDirectory()
 {
   // simple version. Later this can be made more
   // complicated by providing more adavanced ArtifactFactoryClasse(s)
   // but the staregy will be hidden grace to this method
   return getType()+"s";
 }


This will result in possiblity of rewriting methods like getUrlPath in following way:


  /** @see Artifact#getUrlPath */
    public String getUrlPath()
    {
        return "/" + getDependency().getArtifactDirectory()
             + "/" + getArtifactSubDirectory()
             + "/" + getDependency().getArtifact();
    }








    


---------------------------------------------------------------------
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


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to