Remove redundant _string2cdata() method, use xml_encode() instead.
authorMike Taylor <mike@indexdata.com>
Thu, 30 Nov 2006 12:01:23 +0000 (12:01 +0000)
committerMike Taylor <mike@indexdata.com>
Thu, 30 Nov 2006 12:01:23 +0000 (12:01 +0000)
lib/ZOOM/IRSpy/Record.pm

index 9141a51..a0738c8 100644 (file)
@@ -1,4 +1,4 @@
-# $Id: Record.pm,v 1.20 2006-11-29 18:17:16 mike Exp $
+# $Id: Record.pm,v 1.21 2006-11-30 12:01:23 mike Exp $
 
 package ZOOM::IRSpy::Record;
 
@@ -98,7 +98,7 @@ sub store_result {
     my $xml = "<irspy:$type";
 
     foreach my $key (keys %info) {
-        $xml .= " $key=\"" . $this->_string2cdata($info{$key}) . "\"";
+        $xml .= " $key=\"" . xml_encode($info{$key}) . "\"";
     }
 
     $xml .= ">" . isodate(time()) . "</irspy:$type>\n";
@@ -158,21 +158,6 @@ 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/&/&amp;/gs;
-    $buffer =~ s/</&lt;/gs;
-    $buffer =~ s/>/&gt;/gs;
-    $buffer =~ s/"/&quot;/gs;
-    $buffer =~ s/'/&apos;/gs;
-
-    return $buffer;
-}
-
-
 =head1 SEE ALSO
 
 ZOOM::IRSpy