sax-machine 1.1.1 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7e1a19d1fe1c96e358244af5eddf73e658c43041
-  data.tar.gz: ab4ab21c950788a7ca9a05e9daa678f929bd42cd
+  metadata.gz: d70f53b72ae5d965a51b002434716f68b7d6dd80
+  data.tar.gz: a96da94d49a82ace8f74778aae5fe7b1e7ff2067
 SHA512:
-  metadata.gz: edbbbf1cd1dd4c9cafe1f4172601b63f4fca893175027c0fc09f8c24f92c3857af0b9a924fed787b8e3410b5192ae8d20818704ee1a42f83d093be7f73f03404
-  data.tar.gz: 91c2272b5be0c789d633767f0619b7ef2201ec1d8c2af8ae7e0eb36e199d472d30853b9578ff364af5722bee96dadad72aa720b1f5b7c89527cc6b0bd46d1857
+  metadata.gz: 618645fdf96799bf57df46b97c1e0244a933137085f1162c72735822b79a38c7b8c4fe7e3e73c7aba95611f36bc84bb91199af3e294faa924c18edc5b7fe0228
+  data.tar.gz: 7c0700b3c34bcd6812294cc67dc3571a35f4b4957a9aa2cef485aa15ab56292041ec1df056ebe861a5410cdcdb666041d85167aa00da43a45e92e0eb25bfbe6e

data/HISTORY.md

@@ -1,5 +1,9 @@
 # HEAD
 
+# 1.2.0
+
+* Add support for blocks as value modifiers [[#61](https://github.com/pauldix/sax-machine/pull/61)]
+
 # 1.1.1
 
 * Fix Nokogiri autoloading [[#60](https://github.com/pauldix/sax-machine/pull/60)]

data/README.md

@@ -80,7 +80,10 @@ end
 
 class Atom
   include SAXMachine
-  element :title
+  # Use block to modify the returned value
+  element :title do |title|
+    title.strip
+  end
   # The :with argument means that you only match a link tag
   # that has an attribute of type: "text/html"
   element :link, value: :href, as: :url, with: {

data/lib/sax-machine/sax_document.rb

@@ -39,16 +39,16 @@ module SAXMachine
       new.parse(*args)
     end
 
-    def element(name, options = {})
+    def element(name, options = {}, &block)
       real_name = (options[:as] ||= name).to_s
       sax_config.add_top_level_element(name, options)
-      create_attr(real_name)
+      create_attr(real_name, &block)
     end
 
-    def attribute(name, options = {})
+    def attribute(name, options = {}, &block)
       real_name = (options[:as] ||= name).to_s
       sax_config.add_top_level_attribute(self.class.to_s, options.merge(name: name))
-      create_attr(real_name)
+      create_attr(real_name, &block)
     end
 
     def value(name, options = {})
@@ -115,9 +115,18 @@ module SAXMachine
     # we only want to insert the getter and setter if they haven't defined it from elsewhere.
     # this is how we allow custom parsing behavior. So you could define the setter
     # and have it parse the string into a date or whatever.
-    def create_attr(real_name)
+    def create_attr(real_name, &block)
       attr_reader(real_name) unless method_defined?(real_name)
-      attr_writer(real_name) unless method_defined?("#{real_name}=")
+
+      if !method_defined?("#{real_name}=")
+        if block_given?
+          define_method("#{real_name}=") do |value|
+             instance_variable_set("@#{real_name}", (yield value))
+          end
+        else
+          attr_writer(real_name)
+        end
+      end
     end
   end
 end

data/lib/sax-machine/version.rb

@@ -1,3 +1,3 @@
 module SAXMachine
-  VERSION = "1.1.1"
+  VERSION = "1.2.0"
 end

data/spec/sax-machine/sax_document_spec.rb

@@ -218,6 +218,41 @@ describe "SAXMachine" do
           expect(@klass.required?(:date)).to be_truthy
         end
       end
+
+      describe "the block" do
+        before do
+          @klass = Class.new do
+            include SAXMachine
+
+            element :title do |title|
+              "#{title}!!!"
+            end
+
+            attribute :id do |id|
+              id.to_i
+            end
+
+            element :link, value: :foo do |link|
+              link.downcase
+            end
+          end
+        end
+
+        it "uses block for element" do
+          document = @klass.parse("<title>SAX</title>")
+          expect(document.title).to eq("SAX!!!")
+        end
+
+        it 'uses block for attribute' do
+          document = @klass.parse("<title id='345'>SAX</title>")
+          expect(document.id).to eq(345)
+        end
+
+        it "uses block for attribute value" do
+          document = @klass.parse("<link foo='tEst'>hello</link>")
+          expect(document.link).to eq("test")
+        end
+      end
     end
 
     describe "when parsing multiple elements" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sax-machine
 version: !ruby/object:Gem::Version
-  version: 1.1.1
+  version: 1.2.0
 platform: ruby
 authors:
 - Paul Dix
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-11-20 00:00:00.000000000 Z
+date: 2014-12-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec