/*
 * Copyright (c) 2009, Rickard Öberg. All Rights Reserved.
 *
 * 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.qi4j.spi.value;

import java.lang.reflect.Type;
import java.util.Collection;
import java.util.List;
import java.util.Set;
import org.qi4j.api.util.Classes;

/**
 * TODO
 */
public class CollectionType
    implements ValueType
{
    public static boolean isCollection( Type type)
    {
        Class cl = Classes.getRawClass( type );

        if (cl.equals( Collection.class) || cl.equals( List.class) || cl.equals( Set.class))
            return true;

        return false;
    }

    private String collectionType;
    private ValueType type;

    public CollectionType( String collectionType, ValueType type )
    {
        this.collectionType = collectionType;
        this.type = type;
    }

    public String collectionType()
    {
        return collectionType;
    }

    public ValueType type()
    {
        return type;
    }

    @Override public String toString()
    {
        return collectionType+"<"+type+">";
    }
}

@@ -897,32 +897,129 @@
 ype type)%0A    %7B%0A
+        // TODO maybe: return Collection.class.isAssignableFrom( Classes.getRawClass( type ) ); %0A
         Class cl
@@ -1050,25 +1050,24 @@
 type );%0A
-%0A
         
 if (cl.e
@@ -1062,12 +1062,15 @@
     
-if (
+return 
 cl.e
@@ -1088,24 +1088,25 @@
 ection.class
+ 
 ) %7C%7C cl.equa
@@ -1119,16 +1119,17 @@
 st.class
+ 
 ) %7C%7C cl.
@@ -1149,57 +1149,10 @@
 lass
+ 
 )
-)%0A            return true;%0A%0A        return false
 ;%0A  

Simplify. Shouldn't any type of collection be supported?