X-Git-Url: http://git.tdb.fi/?p=libs%2Fdatafile.git;a=blobdiff_plain;f=source%2Fcollection.h;h=be9ae88c491595f925db46e30aeff2daa67deb5f;hp=ec7cde31786adda2049631226231d75731757346;hb=698d4ae13da87b7c53aa96bca99c3deaff397eec;hpb=e93ac3d7eb94842161f0669cf78b4e2173dc0488

diff --git a/source/collection.h b/source/collection.h
index ec7cde3..be9ae88 100644
--- a/source/collection.h
+++ b/source/collection.h
@@ -109,16 +109,27 @@ public:
 	template<typename T>
 	T &get(const std::string &name) const
 	{
-		return *get_item(items, name).value<RefPtr<typename RemoveConst<T>::Type> >();
+		return extract<T>(get_item(items, name));
 	}
 
 	/** Gets a typed object from the collection.  If the name is not found,
 	automatic creation with the type's creator function (if defined) or from
 	sources (if present) is attempted. */
 	template<typename T>
-	T &get(const std::string &);
+	T &get(const std::string &name)
+	{
+		return extract<T>(get_var(name, get_type<T>()));
+	}
 
 private:
+	const Variant &get_var(const std::string &, const CollectionItemTypeBase *);
+
+	template<typename T>
+	T &extract(const Variant &var) const
+	{
+		return *var.value<RefPtr<typename RemoveConst<T>::Type> >();
+	}
+
 	template<typename T>
 	void collect_items(std::list<T *> *objects, std::list<std::string> *names, std::list<std::string> *future_names) const
 	{
@@ -450,33 +461,6 @@ public:
 };
 
 
-template<typename T>
-T &Collection::get(const std::string &name)
-{
-	typedef RefPtr<typename RemoveConst<T>::Type> RPNCT;
-
-	ItemMap::iterator i = items.find(name);
-	if(i!=items.end())
-		return *i->second.value<RPNCT>();
-
-	if(CollectionItemTypeBase *type = get_type<T>())
-	{
-		bool loaded = false;
-		if(type->can_create())
-		{
-			type->create_item(*this, name);
-			loaded = items.count(name);
-		}
-		for(SourceList::iterator j=sources.begin(); (!loaded && j!=sources.end()); ++j)
-		{
-			(*j)->load(*this, *type, name);
-			loaded = items.count(name);
-		}
-	}
-
-	return *get_item(items, name).value<RPNCT>();
-}
-
 template<typename T>
 CollectionItemType<T> &Collection::add_type()
 {