X-Git-Url: http://git.indexdata.com/?p=irspy-moved-to-github.git;a=blobdiff_plain;f=lib%2FZOOM%2FIRSpy%2FRecord.pm;h=9141a51575c4154e831e7559884130378e556e1d;hp=081e19d064f4db4971263ad3817806109be76101;hb=bdf3cf0e4ebca2f323ac5b709aced7de95286f21;hpb=d279b72e4e1af82605815866f0f52a40328531bf diff --git a/lib/ZOOM/IRSpy/Record.pm b/lib/ZOOM/IRSpy/Record.pm index 081e19d..9141a51 100644 --- a/lib/ZOOM/IRSpy/Record.pm +++ b/lib/ZOOM/IRSpy/Record.pm @@ -1,4 +1,4 @@ -# $Id: Record.pm,v 1.11 2006-09-25 12:39:34 mike Exp $ +# $Id: Record.pm,v 1.20 2006-11-29 18:17:16 mike Exp $ package ZOOM::IRSpy::Record; @@ -8,7 +8,7 @@ use warnings; use XML::LibXML; use XML::LibXML::XPathContext; - +use ZOOM::IRSpy::Utils qw(xml_encode isodate); =head1 NAME @@ -26,7 +26,7 @@ I<## To follow> sub new { my $class = shift(); - my($target, $zeerex) = @_; + my($irspy, $target, $zeerex) = @_; if (!defined $zeerex) { $zeerex = _empty_zeerex_record($target); @@ -34,6 +34,7 @@ sub new { my $parser = new XML::LibXML(); return bless { + irspy => $irspy, target => $target, parser => $parser, zeerex => $parser->parse_string($zeerex)->documentElement(), @@ -62,22 +63,6 @@ __EOT__ } -# I can't -- just can't, can't, can't -- believe that this function -# isn't provided by one of the core XML modules. But the evidence all -# says that it's not: among other things, XML::Generator and -# Template::Plugin both roll their own. So I will do likewise. D'oh! -# -sub xml_encode { - my ($text) = @_; - $text =~ s/&/&/g; - $text =~ s//>/g; - $text =~ s/['']/'/g; - $text =~ s/[""]/"/g; - return $text; -} - - sub append_entry { my $this = shift(); my($xpath, $frag) = @_; @@ -86,7 +71,7 @@ sub append_entry { my $root = $this->{zeerex}; # XML::LibXML::Element ISA XML::LibXML::Node my $xc = XML::LibXML::XPathContext->new($root); $xc->registerNs(zeerex => "http://explain.z3950.org/dtd/2.0/"); - $xc->registerNs(irspy => "http://indexdata.com/irspy/1.0"); + $xc->registerNs(irspy => $ZOOM::IRSpy::Utils::IRSPY_NS); my @nodes = $xc->findnodes($xpath); if (@nodes == 0) { @@ -101,13 +86,26 @@ sub append_entry { if @nodes == 0; } - ZOOM::Log::log("irspy", - scalar(@nodes), " matches for '$xpath': using first") + $this->{irspy}->log("warn", + scalar(@nodes), " matches for '$xpath': using first") if @nodes > 1; $this->_half_decent_appendWellBalancedChunk($nodes[0], $frag); } +sub store_result { + my ($this, $type, %info) = @_; + my $xml = "_string2cdata($info{$key}) . "\""; + } + + $xml .= ">" . isodate(time()) . "\n"; + + $this->append_entry('irspy:status', $xml); +} + # *sigh* # @@ -125,7 +123,7 @@ sub append_entry { # namespace mapping for that node -- but that only affects pre-parsed # trees, and is no use for parsing. Hence the following pair of lines # DOES NOT WORK: -# $node->setNamespace("http://indexdata.com/irspy/1.0", "irspy", 0); +# $node->setNamespace($ZOOM::IRSpy::Utils::IRSPY_NS, "irspy", 0); # $node->appendWellBalancedChunk($frag); # # Instead I have to go the long way round, hence this method. I have @@ -139,7 +137,7 @@ sub _half_decent_appendWellBalancedChunk { my($node, $frag) = @_; if (1) { - $frag =~ s,>, xmlns:irspy="http://indexdata.com/irspy/1.0">,; + $frag =~ s,>, xmlns:irspy="$ZOOM::IRSpy::Utils::IRSPY_NS">,; $node->appendWellBalancedChunk($frag); return; } @@ -160,6 +158,21 @@ sub _half_decent_appendWellBalancedChunk { } +# Yes, I know that this is already implemented in IRSpy.pm. I suggest that we +# introduce a toolkit package with such subroutines... +# +sub _string2cdata { + my ($this, $buffer) = @_; + $buffer =~ s/&/&/gs; + $buffer =~ s//>/gs; + $buffer =~ s/"/"/gs; + $buffer =~ s/'/'/gs; + + return $buffer; +} + + =head1 SEE ALSO ZOOM::IRSpy