pauldix-sax-machine 0.0.7 → 0.0.8

data/README.textile

@@ -48,6 +48,15 @@ feed.entries.first.title # => title of the first entry
 feed.entries.first.author # => the author of the first entry
 feed.entries.first.url # => the permalink on the blog for this entry
 # etc ...
+
+# you can also use the elements method without specifying a class like so
+class SomeServiceResponse
+  elements :message, :as => :messages
+end
+
+response = SomeServiceResponse.parse("<response><message>hi</message><message>world</message></response>")
+response.messages.first # => "hi"
+response.messages.last  # => "world"
 </pre>
 
 h2. LICENSE

data/lib/sax-machine/sax_document.rb

@@ -35,7 +35,16 @@ module SAXMachine
     
     def elements(name, options = {})
       options[:as] ||= name
-      sax_config.add_collection_element(name, options)
+      if options[:class]
+        sax_config.add_collection_element(name, options)
+      else
+        class_eval <<-SRC
+          def add_#{options[:as]}(value)
+            #{options[:as]} << value
+          end
+        SRC
+        sax_config.add_top_level_element(name, options.merge(:collection => true))
+      end
       
       class_eval <<-SRC
         def #{options[:as]}

data/lib/sax-machine/sax_element_config.rb

@@ -2,7 +2,7 @@ module SAXMachine
   class SAXConfig
     
     class ElementConfig
-      attr_reader :name
+      attr_reader :name, :setter
       
       def initialize(name, options)
         @name = name.to_s
@@ -21,16 +21,19 @@ module SAXMachine
         end
         
         @as = options[:as]
+        @collection = options[:collection]
+        
+        if @collection
+          @setter = "add_#{options[:as]}"
+        else
+          @setter = "#{@as}="
+        end
       end
 
       def value_from_attrs(attrs)
         attrs[attrs.index(@value) + 1]
       end
       
-      def setter
-        "#{@as}="
-      end
-      
       def attrs_match?(attrs)
         if @with
           @with == (@with & attrs)
@@ -42,6 +45,10 @@ module SAXMachine
       def has_value_and_attrs_match?(attrs)
         !@value.nil? && attrs_match?(attrs)
       end
+      
+      def collection?
+        @collection
+      end
     end
     
   end

data/lib/sax-machine/sax_handler.rb

@@ -74,7 +74,7 @@ module SAXMachine
     end
     
     def mark_as_parsed
-      @parsed_configs[@element_config] = true
+      @parsed_configs[@element_config] = true unless @element_config.collection?
     end
     
     def parsed_config?

data/spec/sax-machine/sax_document_spec.rb

@@ -214,32 +214,30 @@ describe "SAXMachine" do
   end
   
   describe "elements" do
-    # I took this stuff out because I'm not sure yet if this is something I want to bother supporting.
-    
-    # describe "when parsing multiple elements" do
-    #   before :each do
-    #     @klass = Class.new do
-    #       include SAXMachine
-    #       elements :entry, :as => :entries
-    #     end
-    #   end
-    #   
-    #   it "should provide a collection accessor" do
-    #     document = @klass.new
-    #     document.entries << :foo
-    #     document.entries.should == [:foo]
-    #   end
-    #   
-    #   it "should parse a single element" do
-    #     document = @klass.parse("<entry>hello</entry>")
-    #     document.entries.should == ["hello"]
-    #   end
-    #   
-    #   it "should parse multiple elements" do
-    #     document = @klass.parse("<xml><entry>hello</entry><entry>world</entry></xml>")
-    #     document.entries.should == ["hello", "world"]
-    #   end
-    # end
+    describe "when parsing multiple elements" do
+      before :each do
+        @klass = Class.new do
+          include SAXMachine
+          elements :entry, :as => :entries
+        end
+      end
+      
+      it "should provide a collection accessor" do
+        document = @klass.new
+        document.entries << :foo
+        document.entries.should == [:foo]
+      end
+      
+      it "should parse a single element" do
+        document = @klass.parse("<entry>hello</entry>")
+        document.entries.should == ["hello"]
+      end
+      
+      it "should parse multiple elements" do
+        document = @klass.parse("<xml><entry>hello</entry><entry>world</entry></xml>")
+        document.entries.should == ["hello", "world"]
+      end
+    end
     
     describe "when using the class option" do
       before :each do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: pauldix-sax-machine
 version: !ruby/object:Gem::Version 
-  version: 0.0.7
+  version: 0.0.8
 platform: ruby
 authors: 
 - Paul Dix