partigi-partigirb 0.2.4 → 0.2.5

data/README.rdoc

@@ -49,7 +49,7 @@ For POST requests just change <tt>?</tt> by <tt>!</tt>:
 === Parameter handling
 
 - All parameters are URL encoded as necessary.
-- If you use a File object as a parameter it will be POSTed to Twitter in a multipart request.
+- If you use a File object as a parameter it will be POSTed to Partigi in a multipart request.
 - If you use a Time object as a parameter, .httpdate will be called on it and that value will be used
 
 === Return values
@@ -62,7 +62,7 @@ When using Atom format Partigi returns some XML elements using namespaces. In th
 
 === Error handling
 
-In case Partigi returns an error response, this is turned into a PartigiError object having the error string returned in the XML response as message attribute.
+In case Partigi returns an error response, this is turned into a PartigiError object with message attribute set to the error string returned in the XML response.
 
 == Requirements
 

data/VERSION

@@ -1 +1 @@
-0.2.4
+0.2.5

data/lib/partigirb/handlers/xml_handler.rb

@@ -11,7 +11,7 @@ module Partigirb
         def load_recursive(node)
           if array_node?(node)
             node.elements.map {|e| load_recursive(e)}
-          elsif node.elements.size > 0 || node.attributes.size > 0
+          elsif (node.elements.size > 0 || node.attributes.size > 0) && !ignore_attributes?(node)
             build_struct(node)          
           else
             value = node.text
@@ -37,8 +37,7 @@ module Partigirb
           node.elements.each do |e|
             property = ""
             
-            if !e.namespace.blank?
-              ns = e.namespaces.invert[e.namespace]
+            if ns = node_namespace(e)
               property << "#{ns}_" unless ns == 'xmlns'
             end
             
@@ -64,6 +63,20 @@ module Partigirb
         def fixnum?(value)
           value =~ /^\d+$/
         end
+        
+        def node_namespace(node)
+          node.namespace.blank? ? nil : node.namespaces.invert[node.namespace]
+        end
+        
+        def ignore_attributes?(node)
+          element_name = node.name
+          ns = node_namespace(node)
+          element_name.insert(0, "#{ns}:") if ns
+          
+          IGNORE_ATTRIBUTES_FOR.include?(element_name)
+        end
+        
+        IGNORE_ATTRIBUTES_FOR = ['ptItem:synopsis', 'ptItem:title']
     end
   end
 end

data/partigirb.gemspec

@@ -2,11 +2,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{partigirb}
-  s.version = "0.2.4"
+  s.version = "0.2.5"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Alvaro Bautista", "Fernando Blat"]
-  s.date = %q{2009-07-30}
+  s.date = %q{2009-08-03}
   s.email = ["alvarobp@gmail.com", "ferblape@gmail.com"]
   s.extra_rdoc_files = [
     "LICENSE",

data/test/xml_handler_test.rb

@@ -101,4 +101,20 @@ class XMLHandlerTest < Test::Unit::TestCase
     assert_equal 'alternate', res.links.first.rel
     assert_equal 'self', res.links[1].rel
   end
+  
+  should "extract values and ignore attributes for elements in IGNORE_ATTRIBUTES_FOR" do
+    xmls = build_xml_string do |xml|
+      xml.instruct!
+      xml.entry({"xmlns:ptItem" => 'http://schemas.overtheworld.com/v200/ptItem'}) do
+        xml.id 123
+        xml.ptItem :synopsis, {:lang => 'en', :type => 'text'}, 'In my opinion...'
+        xml.ptItem :title, {:lang => 'en'}, 'Wadus Movie'
+      end
+    end
+    
+    res = @handler.decode_response(xmls)
+    assert res.is_a?(Partigirb::PartigiStruct)
+    assert_equal 'In my opinion...', res.ptItem_synopsis
+    assert_equal 'Wadus Movie', res.ptItem_title
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: partigi-partigirb
 version: !ruby/object:Gem::Version 
-  version: 0.2.4
+  version: 0.2.5
 platform: ruby
 authors: 
 - Alvaro Bautista
@@ -10,7 +10,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-07-30 00:00:00 -07:00
+date: 2009-08-03 00:00:00 -07:00
 default_executable: 
 dependencies: []