[Bf-blender-cvs] [14bb6a964a1] collada: fix: Collada: Error handler does not report Parsing errors and continues processing instead of stopping the import/export

Gaia Clary noreply at git.blender.org
Wed Oct 24 22:06:46 CEST 2018


Commit: 14bb6a964a1632943102c5153b36f559872a0435
Author: Gaia Clary
Date:   Wed Oct 24 22:06:40 2018 +0200
Branches: collada
https://developer.blender.org/rB14bb6a964a1632943102c5153b36f559872a0435

fix: Collada: Error handler does not report Parsing errors and continues processing instead of stopping the import/export

===================================================================

M	source/blender/collada/ErrorHandler.cpp
M	source/blender/editors/io/io_collada.c

===================================================================

diff --git a/source/blender/collada/ErrorHandler.cpp b/source/blender/collada/ErrorHandler.cpp
index caa73900632..22ec432d237 100644
--- a/source/blender/collada/ErrorHandler.cpp
+++ b/source/blender/collada/ErrorHandler.cpp
@@ -49,47 +49,64 @@ ErrorHandler::~ErrorHandler()
 //--------------------------------------------------------------------
 bool ErrorHandler::handleError(const COLLADASaxFWL::IError *error)
 {
-	/* This method must return true when Collada should continue.
+	/* This method must return false when Collada should continue.
 	 * See https://github.com/KhronosGroup/OpenCOLLADA/issues/442
 	 */
-	bool isWarning = false;
+	bool isError = true;
+	std::string error_context;
+	std::string error_message;
 
 	if (error->getErrorClass() == COLLADASaxFWL::IError::ERROR_SAXPARSER) {
+		error_context = "Schema validation";
+
 		COLLADASaxFWL::SaxParserError *saxParserError = (COLLADASaxFWL::SaxParserError *) error;
 		const GeneratedSaxParser::ParserError& parserError = saxParserError->getError();
+		error_message = parserError.getErrorMessage();
 
-		// Workaround to avoid wrong error
 		if (parserError.getErrorType() == GeneratedSaxParser::ParserError::ERROR_VALIDATION_MIN_OCCURS_UNMATCHED) {
 			if (STREQ(parserError.getElement(), "effect")) {
-				isWarning = true;
+				isError = false;
 			}
 		}
+
 		if (parserError.getErrorType() == GeneratedSaxParser::ParserError::ERROR_VALIDATION_SEQUENCE_PREVIOUS_SIBLING_NOT_PRESENT) {
 			if (!(STREQ(parserError.getElement(), "extra") &&
 			      STREQ(parserError.getAdditionalText().c_str(), "sibling: fx_profile_abstract")))
 			{
-				isWarning = true;
+				isError = false;
 			}
 		}
 
 		if (parserError.getErrorType() == GeneratedSaxParser::ParserError::ERROR_COULD_NOT_OPEN_FILE) {
-			std::cout << "Couldn't open file" << std::endl;
+			isError = true;
+			error_context = "File access";
 		}
-
-		std::cout << "Schema validation error: " << parserError.getErrorMessage() << std::endl;
 	}
 	else if (error->getErrorClass() == COLLADASaxFWL::IError::ERROR_SAXFWL) {
+		error_context = "Sax FWL";
 		COLLADASaxFWL::SaxFWLError *saxFWLError = (COLLADASaxFWL::SaxFWLError *) error;
+		error_message = saxFWLError->getErrorMessage();
+
 		/*
 		 * Accept non critical errors as warnings (i.e. texture not found)
 		 * This makes the importer more graceful, so it now imports what makes sense.
 		 */
-		isWarning = (saxFWLError->getSeverity() == COLLADASaxFWL::IError::SEVERITY_ERROR_NONCRITICAL);
-		std::cout << "Sax FWL Error: " << saxFWLError->getErrorMessage() << std::endl;
+
+		isError = (saxFWLError->getSeverity() != COLLADASaxFWL::IError::SEVERITY_ERROR_NONCRITICAL);
+		
 	}
 	else {
-		std::cout << "opencollada error: " << error->getFullErrorMessage() << std::endl;
+		error_context = "OpenCollada";
+		error_message = error->getFullErrorMessage();
+		isError = true;
 	}
 
-	return isWarning;
+	std::string severity = (isError) ? "Error" : "Warning";
+	std::cout << error_context << " (" << severity << "): " << error_message << std::endl;
+	if (isError) {
+		std::cout << "The Collada import has been forced to stop." << std::endl;
+		std::cout << "Please fix the reported error and then try again.";
+		mError = true;
+	}
+	return isError;
 }
diff --git a/source/blender/editors/io/io_collada.c b/source/blender/editors/io/io_collada.c
index dfcc3a8d8c3..288f3b24fcd 100644
--- a/source/blender/editors/io/io_collada.c
+++ b/source/blender/editors/io/io_collada.c
@@ -606,7 +606,7 @@ static int wm_collada_import_exec(bContext *C, wmOperator *op)
 		return OPERATOR_FINISHED;
 	}
 	else {
-		BKE_report(op->reports, RPT_ERROR, "Errors found during parsing COLLADA document (see console for details)");
+		BKE_report(op->reports, RPT_ERROR, "Parsing errors in Document (see Blender Console)");
 		return OPERATOR_CANCELLED;
 	}
 }



More information about the Bf-blender-cvs mailing list