Your code loops around a 'product-announced' element, which doesn't exist in your file. Does that have something to do with it?
Since you say you're new to XML parsing, perhaps you'd like to see an example of an alternate solution. The following is XML::LibXML code, and XML::Xpath code would be virtually identical. use XML::LibXML; $doc = XML::LibXML->new->parse_file('20040817.xml'); foreach my $product ($doc->findnodes('//product-updated')) { my $name = $product->findvalue('product-name'); my $upc = $product->findvalue('upc'); my $msrp = $product->findvalue('msrp'); #etc } -- Mark Thomas [EMAIL PROTECTED] Internet Systems Architect DigitalNet, Inc. $_=q;KvtuyboopuifeyQQfeemyibdlfee;; y.e.s. ;y+B-x+A-w+s; ;y;y; ;;print;; -- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] <http://learn.perl.org/> <http://learn.perl.org/first-response>