it's now possible to change the XPath engine for each transformer

added a test that uses XML::XPathEngine and HTML::TreeBuilder::XPath

2 files changed, 47 insertions, 10 deletions

diff --git a/lib/Tree/Transform/XSLTish.pm b/lib/Tree/Transform/XSLTish.pm
index 390616f..8be326d 100644
--- a/lib/Tree/Transform/XSLTish.pm
+++ b/lib/Tree/Transform/XSLTish.pm
@@ -58,9 +58,9 @@ sub tree_rule {
 sub _transformer_class { 'Tree::Transform::XSLTish::Transformer' };
 
 sub new_transformer {
-    my ($rules_package)=@_;
+    my $rules_package=shift;
 
-    return _transformer_class->new(rules_package=>$rules_package);
+    return _transformer_class->new(rules_package=>$rules_package,@_);
 }
 
 1;
@@ -70,6 +70,34 @@ __END__
 
 Tree::Transform::XSLTish - transform tree data, like XSLT but in Perl
 
+=head1 SYNOPSIS
+
+  package MyTransform;
+  use Tree::Transform::XSLTish;
+
+  default_rules;
+
+  tree_rule match => 'node[@id=5]', action => sub {
+    return $_[0]->it->data();
+  };
+
+  package main;
+  use My::Tree;
+
+  my $tree= My::Tree->new();
+  # build something inside the tree
+
+  my ($node5_data)=MyTransform->new->transform($tree);
+
+=head1 DESCRIPTION
+
+This module allows you to transform tree with Perl subroutines, just
+like XSLT does for XML documents.
+
+It tries to model as closely as reasonable the semantic of XSLT.
+
+=head1 REQUIREMENTS
+
 =head1 AUTHOR
 
 Gianni Ceccarelli <dakkar@thenautilus.net>
diff --git a/lib/Tree/Transform/XSLTish/Transformer.pm b/lib/Tree/Transform/XSLTish/Transformer.pm
index 2f8fc52..8b06422 100644
--- a/lib/Tree/Transform/XSLTish/Transformer.pm
+++ b/lib/Tree/Transform/XSLTish/Transformer.pm
@@ -1,12 +1,23 @@
 package Tree::Transform::XSLTish::Transformer;
 use Moose;
 use MooseX::AttributeHelpers;
+use Moose::Util::TypeConstraints;
 use Params::Validate ':all';
 use Tree::Transform::XSLTish::Utils;
 use Tree::Transform::XSLTish::Context;
 use Tree::XPathEngine;
 use Carp::Clan qw(^Tree::Transform::XSLTish);
 
+subtype 'Tree::Transform::XSLTish::Engine'
+    => as 'Object'
+    => where {
+        my $object=$_;
+        for my $meth (qw(findnodes matches exists)) {
+            return unless $object->can($meth);
+        }
+        return 1;
+    };
+
 has 'rules_package' => (is => 'ro', isa => 'ClassName');
 
 has 'context_stack' => (
@@ -24,7 +35,7 @@ has 'context_stack' => (
 
 has 'engine' => (
     is => 'ro',
-    isa => 'Tree::XPathEngine',
+    isa => 'Tree::Transform::XSLTish::Engine',
     default => sub { Tree::XPathEngine->new() },
 );
 
@@ -33,7 +44,7 @@ sub it { $_[0]->context->current_node }
 sub transform {
     my ($self,$tree)=@_;
 
-    return $self->apply_rules($tree->xpath_get_root_node);
+    return $self->apply_rules($self->engine->findnodes('/',$tree));
 }
 
 sub apply_rules {
@@ -44,7 +55,7 @@ sub apply_rules {
             carp 'apply_rules called without nodes nor context!';
             return;
         }
-        @nodes=$self->it->xpath_get_child_nodes();
+        @nodes=$self->engine->findnodes('*',$self->it);
     };
 
     my $guard=Tree::Transform::XSLTish::ContextGuard->new
@@ -153,17 +164,15 @@ sub rule_matches {
     # XXX check the semantic
 
     my $base_node=$node;
-    while (1) {
+    while ($base_node) {
 
         #warn "# Testing <$path> against @{[ $node ]} based on @{[ $base_node ]}";
 
         if ($self->engine->matches($node,$path,$base_node)) {
             return 1;
         }
-        if ($base_node->xpath_is_document_node) {
-            return;
-        }
-        $base_node=$base_node->xpath_get_parent_node;
+
+        ($base_node)=$self->engine->findnodes('..',$base_node);
     }
     return;
 }