/*-
 * Copyright (c) 2013 Red Hat, Inc.
 *
 * Licensed 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.
 */
package org.fedoraproject.maven.config;

import java.util.Properties;

import org.codehaus.plexus.PlexusContainer;
import org.codehaus.plexus.component.annotations.Component;
import org.codehaus.plexus.component.annotations.Requirement;
import org.codehaus.plexus.component.repository.exception.ComponentLookupException;
import org.codehaus.plexus.util.xml.Xpp3Dom;
import org.fedoraproject.maven.repository.Repository;

/**
 * @author Mikolaj Izdebski
 */
@Component( role = RepositoryConfigurator.class )
public class DefaultRepositoryConfigurator
    implements RepositoryConfigurator
{
    @Requirement
    private Configurator configurator;

    @Requirement
    private PlexusContainer container;

    @Override
    public Repository configureRepository( String repoId )
    {
        Properties properties = new Properties();
        Xpp3Dom configurationXml = new Xpp3Dom( "configuration" );

        for ( org.fedoraproject.maven.config.Repository repository : configurator.getConfiguration().getRepositories() )
        {
            if ( repository.getId() != null && repository.getId().equals( repoId ) )
            {
                properties = repository.getProperties();
                configurationXml = (Xpp3Dom) repository.getConfiguration();
                break;
            }
        }

        try
        {
            Repository repository = container.lookup( Repository.class, repoId );
            repository.configure( properties, configurationXml );
            return repository;
        }
        catch ( ComponentLookupException e )
        {
            throw new RuntimeException( e );
        }
    }
}

@@ -1415,23 +1415,11 @@
  = n
-ew Properties()
+ull
 ;%0A  
@@ -1456,37 +1456,39 @@
  = n
-ew Xpp3Dom( %22configuration%22 )
+ull;%0A        String type = null
 ;%0A%0A 
@@ -1845,24 +1845,69 @@
 guration();%0A
+                type = repository.getType();%0A
             
@@ -1942,16 +1942,252 @@
      %7D%0A%0A
+        if ( properties == null )%0A            properties = new Properties();%0A        if ( configurationXml == null )%0A            configurationXml = new Xpp3Dom( %22configuration%22 );%0A        if ( type == null )%0A            type = repoId;%0A%0A
         
@@ -2272,22 +2272,20 @@
 .class, 
-repoId
+type
  );%0A    

Fix bugs detected by CustomRepositoryTest