1 files changed, 20 insertions, 20 deletions

diff --git a/src/function/read_xml_file.cc b/src/function/read_xml_file.cc
index ac42288..d429f34 100644
--- a/src/function/read_xml_file.cc
+++ b/src/function/read_xml_file.cc
@@ -13,28 +13,28 @@ FunctionReadXmlFile::FunctionReadXmlFile(const FunctionReadXmlFile& src):
 	parser_() { }
 
 xalan::XObjectPtr FunctionReadXmlFile::execute(
-	xalan::XPathExecutionContext&                executionContext,
-	xalan::XalanNode*                            context,
-	const xalan::Function::XObjectArgVectorType& arguments,
-	const xalan::Locator*                        locator
+	xalan::XPathExecutionContext& executionContext,
+	xalan::XalanNode*,
+	const xalan::XObjectPtr argument,
+	const xalan::Locator*
 ) const {
-	if ( arguments.size() != 1 ) {
-		xalan::XPathExecutionContext::GetAndReleaseCachedString guard(
-			executionContext
-		);
-
-		this->generalError(executionContext, context, locator);
-	}
-
-	boost::filesystem::ifstream file(
-		this->fs_context_.resolve(arguments[0]->str())
+	const boost::filesystem::path filePath(
+		this->fs_context_.resolve(argument->str())
 	);
 
-	return executionContext.getXObjectFactory().createNodeSet(
-		this->parser_.parseXMLStream(
-			xalan::XSLTInputSource(file)
-		)
-	);
+	if ( boost::filesystem::is_regular_file(filePath) ) {
+		boost::filesystem::ifstream file(filePath);
+
+		return executionContext.getXObjectFactory().createNodeSet(
+			this->parser_.parseXMLStream(
+				xalan::XSLTInputSource(file)
+			)
+		);
+	} else {
+		return executionContext.getXObjectFactory().createString(
+			xalan::XalanDOMString("io error")
+		);
+	}
 }
 
 FunctionReadXmlFile* FunctionReadXmlFile::clone(
@@ -44,7 +44,7 @@ FunctionReadXmlFile* FunctionReadXmlFile::clone(
 
 const xalan::XalanDOMString& FunctionReadXmlFile::getError(
 	xalan::XalanDOMString& result) const {
-	result.assign("The read-xml-file() function expects one argument.");
+	result.assign("The read-xml-file() function expects one argument of type string.");
 
 	return result;
 }