From 0d670478b51c55e44f57995fe3ca8a4585723a6c Mon Sep 17 00:00:00 2001 From: Adrian Kummerlaender Date: Tue, 17 Jun 2014 21:28:04 +0200 Subject: Added context awareness to XObjectValue casting logic * added support for defining boost::filesystem::path as a external function parameter ** boost::filesystem::path parameters are resolved against the appropriate FilesystemContext and IncludeEntityResolver instances * xalan::XSLTInputSource parameter source paths are also resolved * removed need for passing a reference FilesystemContext to "constructDocument" methods ** they now only accept the parameters of the external function implemented by them ** all path resolution logic is wrapped by the newly created XObjectValue class * converted XObjectValue namespace into class ** the "get" template method is now a template member method ** this was needed to enable value casting logic to access the appropriate FilesystemContext and IncludeEntityResolver instances * this commit marks the next step towards the goals defined in 741a70f --- src/support/type/xobject_value.cc | 55 ++++++++++++++++++++++++++++++++------- src/support/type/xobject_value.h | 21 ++++++++++++--- 2 files changed, 62 insertions(+), 14 deletions(-) (limited to 'src/support/type') diff --git a/src/support/type/xobject_value.cc b/src/support/type/xobject_value.cc index 812be52..0236597 100644 --- a/src/support/type/xobject_value.cc +++ b/src/support/type/xobject_value.cc @@ -4,6 +4,7 @@ #include #include +#include "boost/filesystem.hpp" #include @@ -11,34 +12,68 @@ namespace InputXSLT { -namespace XObjectValue { +XObjectValue::XObjectValue( + const boost::filesystem::path& path, + const IncludeEntityResolver* resolver +): + filesystem_context_(path), + include_resolver_(resolver) { } template <> -std::string get(const xalan::XObjectPtr& ptr) { +std::string XObjectValue::get( + const xalan::XObjectPtr& ptr) const { return boost::trim_copy(toString(ptr->str())); } template <> -xalan::XObjectPtr get(const xalan::XObjectPtr& ptr) { +boost::filesystem::path XObjectValue::get( + const xalan::XObjectPtr& ptr) const { + const boost::filesystem::path rawPath( + toString(ptr->str()) + ); + + const boost::filesystem::path filePath( + this->filesystem_context_.resolve(rawPath) + ); + + if ( !(boost::filesystem::exists(filePath) && + boost::filesystem::is_regular_file(filePath)) ) { + if ( auto resolvedPath = this->include_resolver_->resolve(rawPath) ) { + return *resolvedPath; + } else { + return filePath; + } + } else { + return filePath; + } +} + +template <> +xalan::XObjectPtr XObjectValue::get( + const xalan::XObjectPtr& ptr) const { return ptr; } template <> -xalan::XSLTInputSource get( - const xalan::XObjectPtr& ptr) { +xalan::XSLTInputSource XObjectValue::get( + const xalan::XObjectPtr& ptr) const { switch ( ptr->getType() ) { case xalan::XObject::eObjectType::eTypeNodeSet: { - return xalan::XSLTInputSource(ptr->nodeset().item(0)); + return xalan::XSLTInputSource( + ptr->nodeset().item(0) + ); } case xalan::XObject::eObjectType::eTypeResultTreeFrag: { - return xalan::XSLTInputSource(ptr->rtree().getFirstChild()); + return xalan::XSLTInputSource( + ptr->rtree().getFirstChild() + ); } default: { - return xalan::XSLTInputSource(ptr->str()); + return xalan::XSLTInputSource( + this->get(ptr).string().data() + ); } } } } - -} diff --git a/src/support/type/xobject_value.h b/src/support/type/xobject_value.h index bb602a4..cfc259d 100644 --- a/src/support/type/xobject_value.h +++ b/src/support/type/xobject_value.h @@ -4,13 +4,26 @@ #include #include "common.h" +#include "support/filesystem_context.h" +#include "support/include_entity_resolver.h" namespace InputXSLT { -namespace XObjectValue { - template - Type get(const xalan::XObjectPtr&); -} +class XObjectValue { + public: + XObjectValue( + const boost::filesystem::path&, + const IncludeEntityResolver* + ); + + template + Type get(const xalan::XObjectPtr&) const; + + private: + const FilesystemContext filesystem_context_; + const IncludeEntityResolver* const include_resolver_; + +}; } -- cgit v1.2.3