From 3c2cf1bb085ecbe906a978b92d29a3c14e334959 Mon Sep 17 00:00:00 2001
From: Thomas Krijnen <t.krijnen@gmail.com>
Date: Fri, 27 Dec 2013 09:33:57 +0000
Subject: [PATCH] Fix for Collada export of transparency and material libraries

---
 src/ifcconvert/ColladaSerializer.cpp | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/src/ifcconvert/ColladaSerializer.cpp b/src/ifcconvert/ColladaSerializer.cpp
index 9cd6d0ab2d..43fa842c96 100644
--- a/src/ifcconvert/ColladaSerializer.cpp
+++ b/src/ifcconvert/ColladaSerializer.cpp
@@ -173,7 +173,9 @@ void ColladaSerializer::ColladaExporter::ColladaMaterials::ColladaEffects::write
 	if (material.hasTransparency()) {
 		const double transparency = material.transparency();
 		if (transparency > 0) {
-			effect.setTransparency(transparency);
+			// The default opacity mode for Collada is A_ONE, which apparently indicates a
+			// transparency value of 1 to be fully opaque. Hence transparency is inverted.
+			effect.setTransparency(1.0 - transparency);
 		}
 	}
 	addEffectProfile(effect);
@@ -201,8 +203,9 @@ void ColladaSerializer::ColladaExporter::ColladaMaterials::write() {
 		const std::string& material_name = collada_id((*it).name());
 		openMaterial(material_name);
 		addInstanceEffect("#" + material_name + "-fx");
-		closeLibrary();
+		closeMaterial();
 	}
+	closeLibrary();
 }
 		
 void ColladaSerializer::ColladaExporter::startDocument() {