Author: brett
Date: Thu Apr 3 06:57:40 2008
New Revision: 644317
URL: http://svn.apache.org/viewvc?rev=644317&view=rev
Log:
[MRM-746] also need to make sure the metadata is not scanned in getVersions
Added:
archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/repositories/metadata-repository/org/apache/archiva/metadata/tests/include_xml/
archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/repositories/metadata-repository/org/apache/archiva/metadata/tests/include_xml/1.0/
archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/repositories/metadata-repository/org/apache/archiva/metadata/tests/include_xml/1.0/include_xml-1.0.pom
(with props)
archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/repositories/metadata-repository/org/apache/archiva/metadata/tests/include_xml/1.0/include_xml-1.0.xml
(with props)
archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/repositories/metadata-repository/org/apache/archiva/metadata/tests/include_xml/1.0/maven-metadata.xml
(with props)
Modified:
archiva/branches/archiva-1.0.x/archiva-base/archiva-configuration/src/main/java/org/apache/maven/archiva/configuration/FileTypes.java
archiva/branches/archiva-1.0.x/archiva-base/archiva-consumers/archiva-consumer-api/src/main/java/org/apache/maven/archiva/consumers/AbstractMonitoredConsumer.java
archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/main/java/org/apache/maven/archiva/repository/content/ManagedDefaultRepositoryContent.java
archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/java/org/apache/maven/archiva/repository/content/ManagedDefaultRepositoryContentTest.java
Modified:
archiva/branches/archiva-1.0.x/archiva-base/archiva-configuration/src/main/java/org/apache/maven/archiva/configuration/FileTypes.java
URL:
http://svn.apache.org/viewvc/archiva/branches/archiva-1.0.x/archiva-base/archiva-configuration/src/main/java/org/apache/maven/archiva/configuration/FileTypes.java?rev=644317&r1=644316&r2=644317&view=diff
==============================================================================
---
archiva/branches/archiva-1.0.x/archiva-base/archiva-configuration/src/main/java/org/apache/maven/archiva/configuration/FileTypes.java
(original)
+++
archiva/branches/archiva-1.0.x/archiva-base/archiva-configuration/src/main/java/org/apache/maven/archiva/configuration/FileTypes.java
Thu Apr 3 06:57:40 2008
@@ -35,6 +35,7 @@
import java.lang.reflect.Field;
import java.util.ArrayList;
+import java.util.Arrays;
import java.util.Collections;
import java.util.HashMap;
import java.util.List;
@@ -72,6 +73,15 @@
private List<String> artifactPatterns;
+ /**
+ * Default exclusions from artifact consumers that are using the file
types. Note that this is simplistic in the
+ * case of the support files (based on extension) as it is elsewhere - it
may be better to match these to actual
+ * artifacts and exclude later during scanning.
+ */
+ public static final List<String> DEFAULT_EXCLUSIONS = Arrays.asList(
"**/maven-metadata.xml",
+
"**/maven-metadata-*.xml", "**/*.sha1",
+
"**/*.asc", "**/*.md5", "**/*.pgp" );
+
public void setArchivaConfiguration( ArchivaConfiguration
archivaConfiguration )
{
this.archivaConfiguration = archivaConfiguration;
@@ -125,7 +135,7 @@
{
artifactPatterns = getFileTypePatterns( ARTIFACTS );
}
-
+
for ( String pattern : artifactPatterns )
{
if ( SelectorUtils.matchPath( pattern, relativePath, false ) )
@@ -139,13 +149,31 @@
return false;
}
+ public boolean matchesDefaultExclusions( String relativePath )
+ {
+ // Correct the slash pattern.
+ relativePath = relativePath.replace( '\\', '/' );
+
+ for ( String pattern : DEFAULT_EXCLUSIONS )
+ {
+ if ( SelectorUtils.matchPath( pattern, relativePath, false ) )
+ {
+ // Found match
+ return true;
+ }
+ }
+
+ // No match.
+ return false;
+ }
+
public void initialize()
throws InitializationException
{
// TODO: why is this done by hand?
String errMsg = "Unable to load default archiva configuration for
FileTypes: ";
-
+
try
{
CommonsConfigurationRegistry commonsRegistry = new
CommonsConfigurationRegistry();
@@ -156,7 +184,7 @@
fld.set( commonsRegistry, new CombinedConfiguration() );
commonsRegistry.enableLogging( getLogger() );
commonsRegistry.addConfigurationFromResource(
"org/apache/maven/archiva/configuration/default-archiva.xml" );
-
+
// Read configuration as it was intended.
ConfigurationRegistryReader configReader = new
ConfigurationRegistryReader();
Configuration defaultConfig = configReader.read( commonsRegistry );
Modified:
archiva/branches/archiva-1.0.x/archiva-base/archiva-consumers/archiva-consumer-api/src/main/java/org/apache/maven/archiva/consumers/AbstractMonitoredConsumer.java
URL:
http://svn.apache.org/viewvc/archiva/branches/archiva-1.0.x/archiva-base/archiva-consumers/archiva-consumer-api/src/main/java/org/apache/maven/archiva/consumers/AbstractMonitoredConsumer.java?rev=644317&r1=644316&r2=644317&view=diff
==============================================================================
---
archiva/branches/archiva-1.0.x/archiva-base/archiva-consumers/archiva-consumer-api/src/main/java/org/apache/maven/archiva/consumers/AbstractMonitoredConsumer.java
(original)
+++
archiva/branches/archiva-1.0.x/archiva-base/archiva-consumers/archiva-consumer-api/src/main/java/org/apache/maven/archiva/consumers/AbstractMonitoredConsumer.java
Thu Apr 3 06:57:40 2008
@@ -19,9 +19,9 @@
* under the License.
*/
+import org.apache.maven.archiva.configuration.FileTypes;
import org.codehaus.plexus.logging.AbstractLogEnabled;
-import java.util.Arrays;
import java.util.HashSet;
import java.util.Iterator;
import java.util.List;
@@ -39,15 +39,6 @@
{
private Set<ConsumerMonitor> monitors = new HashSet<ConsumerMonitor>();
- /**
- * Default exclusions from artifact consumers that are using the file
types. Note that this is simplistic in the
- * case of the support files (based on extension) as it is elsewhere - it
may be better to match these to actual
- * artifacts and exclude later during scanning.
- */
- private static final List<String> DEFAULT_EXCLUSIONS = Arrays.asList(
"**/maven-metadata.xml",
-
"**/maven-metadata-*.xml", "**/*.sha1",
-
"**/*.asc", "**/*.md5", "**/*.pgp" );
-
public void addConsumerMonitor( ConsumerMonitor monitor )
{
monitors.add( monitor );
@@ -113,6 +104,6 @@
protected List<String> getDefaultArtifactExclusions()
{
- return DEFAULT_EXCLUSIONS;
+ return FileTypes.DEFAULT_EXCLUSIONS;
}
}
Modified:
archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/main/java/org/apache/maven/archiva/repository/content/ManagedDefaultRepositoryContent.java
URL:
http://svn.apache.org/viewvc/archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/main/java/org/apache/maven/archiva/repository/content/ManagedDefaultRepositoryContent.java?rev=644317&r1=644316&r2=644317&view=diff
==============================================================================
---
archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/main/java/org/apache/maven/archiva/repository/content/ManagedDefaultRepositoryContent.java
(original)
+++
archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/main/java/org/apache/maven/archiva/repository/content/ManagedDefaultRepositoryContent.java
Thu Apr 3 06:57:40 2008
@@ -242,6 +242,12 @@
String relativePath = PathUtil.getRelative(
repository.getLocation(), repoFiles[i] );
+ if ( filetypes.matchesDefaultExclusions( relativePath ) )
+ {
+ // Skip it, it's metadata or similar
+ continue;
+ }
+
if ( filetypes.matchesArtifactPattern( relativePath ) )
{
ArtifactReference artifact = toArtifactReference( relativePath
);
Modified:
archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/java/org/apache/maven/archiva/repository/content/ManagedDefaultRepositoryContentTest.java
URL:
http://svn.apache.org/viewvc/archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/java/org/apache/maven/archiva/repository/content/ManagedDefaultRepositoryContentTest.java?rev=644317&r1=644316&r2=644317&view=diff
==============================================================================
---
archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/java/org/apache/maven/archiva/repository/content/ManagedDefaultRepositoryContentTest.java
(original)
+++
archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/java/org/apache/maven/archiva/repository/content/ManagedDefaultRepositoryContentTest.java
Thu Apr 3 06:57:40 2008
@@ -20,6 +20,9 @@
*/
import org.apache.maven.archiva.common.utils.VersionComparator;
+import org.apache.maven.archiva.configuration.ArchivaConfiguration;
+import org.apache.maven.archiva.configuration.FileType;
+import org.apache.maven.archiva.configuration.FileTypes;
import org.apache.maven.archiva.configuration.ManagedRepositoryConfiguration;
import org.apache.maven.archiva.model.ArtifactReference;
import org.apache.maven.archiva.model.ProjectReference;
@@ -122,6 +125,12 @@
}
}
+ public void testExcludeMetadataFile()
+ throws Exception
+ {
+ assertVersions( "include_xml", "1.0", new String[] { "1.0" } );
+ }
+
private void assertGetVersions( String artifactId, List<String>
expectedVersions )
throws Exception
{
@@ -186,6 +195,14 @@
File repoDir = getTestFile( "src/test/repositories/default-repository"
);
ManagedRepositoryConfiguration repository = createRepository(
"testRepo", "Unit Test Repo", repoDir );
+
+ ArchivaConfiguration archivaConfiguration = (ArchivaConfiguration)
lookup( ArchivaConfiguration.ROLE );
+ FileType fileType = (FileType)
archivaConfiguration.getConfiguration().getRepositoryScanning().getFileTypes().get(
0 );
+ fileType.addPattern( "**/*.xml" );
+ assertEquals( FileTypes.ARTIFACTS, fileType.getId() );
+
+ FileTypes fileTypes = (FileTypes) lookup( FileTypes.class );
+ fileTypes.afterConfigurationChange( null, "fileType", null );
repoContent = (ManagedRepositoryContent) lookup(
ManagedRepositoryContent.class, "default" );
repoContent.setRepository( repository );
Added:
archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/repositories/metadata-repository/org/apache/archiva/metadata/tests/include_xml/1.0/include_xml-1.0.pom
URL:
http://svn.apache.org/viewvc/archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/repositories/metadata-repository/org/apache/archiva/metadata/tests/include_xml/1.0/include_xml-1.0.pom?rev=644317&view=auto
==============================================================================
---
archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/repositories/metadata-repository/org/apache/archiva/metadata/tests/include_xml/1.0/include_xml-1.0.pom
(added)
+++
archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/repositories/metadata-repository/org/apache/archiva/metadata/tests/include_xml/1.0/include_xml-1.0.pom
Thu Apr 3 06:57:40 2008
@@ -0,0 +1,25 @@
+<!--
+ ~ Licensed to the Apache Software Foundation (ASF) under one
+ ~ or more contributor license agreements. See the NOTICE file
+ ~ distributed with this work for additional information
+ ~ regarding copyright ownership. The ASF licenses this file
+ ~ to you under the Apache License, Version 2.0 (the
+ ~ "License"); you may not use this file except in compliance
+ ~ with the License. You may obtain a copy of the License at
+ ~
+ ~ http://www.apache.org/licenses/LICENSE-2.0
+ ~
+ ~ Unless required by applicable law or agreed to in writing,
+ ~ software distributed under the License is distributed on an
+ ~ "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ ~ KIND, either express or implied. See the License for the
+ ~ specific language governing permissions and limitations
+ ~ under the License.
+ -->
+<project>
+ <modelVersion>4.0.0</modelVersion>
+ <groupId>org.apache.archiva.metadata.tests</groupId>
+ <artifactId>include_xml</artifactId>
+ <version>1.0</version>
+ <packaging>xml</packaging>
+</project>
Propchange:
archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/repositories/metadata-repository/org/apache/archiva/metadata/tests/include_xml/1.0/include_xml-1.0.pom
------------------------------------------------------------------------------
svn:eol-style = native
Added:
archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/repositories/metadata-repository/org/apache/archiva/metadata/tests/include_xml/1.0/include_xml-1.0.xml
URL:
http://svn.apache.org/viewvc/archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/repositories/metadata-repository/org/apache/archiva/metadata/tests/include_xml/1.0/include_xml-1.0.xml?rev=644317&view=auto
==============================================================================
---
archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/repositories/metadata-repository/org/apache/archiva/metadata/tests/include_xml/1.0/include_xml-1.0.xml
(added)
+++
archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/repositories/metadata-repository/org/apache/archiva/metadata/tests/include_xml/1.0/include_xml-1.0.xml
Thu Apr 3 06:57:40 2008
@@ -0,0 +1,2 @@
+<xml />
+
\ No newline at end of file
Propchange:
archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/repositories/metadata-repository/org/apache/archiva/metadata/tests/include_xml/1.0/include_xml-1.0.xml
------------------------------------------------------------------------------
svn:eol-style = native
Added:
archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/repositories/metadata-repository/org/apache/archiva/metadata/tests/include_xml/1.0/maven-metadata.xml
URL:
http://svn.apache.org/viewvc/archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/repositories/metadata-repository/org/apache/archiva/metadata/tests/include_xml/1.0/maven-metadata.xml?rev=644317&view=auto
==============================================================================
---
archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/repositories/metadata-repository/org/apache/archiva/metadata/tests/include_xml/1.0/maven-metadata.xml
(added)
+++
archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/repositories/metadata-repository/org/apache/archiva/metadata/tests/include_xml/1.0/maven-metadata.xml
Thu Apr 3 06:57:40 2008
@@ -0,0 +1,3 @@
+<metadata>
+
+</metadata>
\ No newline at end of file
Propchange:
archiva/branches/archiva-1.0.x/archiva-base/archiva-repository-layer/src/test/repositories/metadata-repository/org/apache/archiva/metadata/tests/include_xml/1.0/maven-metadata.xml
------------------------------------------------------------------------------
svn:eol-style = native
