cleans up implied properties and fixes spec

lib/microformats2/implied_property/foundation.rb

@@ -1,7 +1,6 @@
 module Microformats2
   module ImpliedProperty
     class Foundation
-      attr_reader :selector
 
       def initialize(element)
         @element = element
@@ -30,33 +29,28 @@ module Microformats2
       protected
 
       def element_value
-        ev = nil
         name_map.each_pair do |elname, attr|
           if elname == @element.name && @element.attribute(attr)
-            ev ||= @element.attribute(attr).to_s
+            return @element.attribute(attr).to_s
-            @selector ||= elname
           end
         end
-        ev
+        nil
+      end
+
+      def name_map
+        {}
       end
 
       def selector_value
-        sv = nil
         selector_map.each_pair do |sel, attr|
-          selected_elements = @element.css(sel)
+          if selected_element = @element.css(sel).first
-          if selected_elements.first
+            return selected_element.attribute(attr).to_s
-            sv ||= selected_elements.first.attribute(attr).to_s
-            @selector ||= sel
           end
         end
-        sv
+        nil
       end
 
-      def attribute
+      def selector_map
-        attr_map[@element.name]
-      end
-
-      def attr_map
         {}
       end
     end

lib/microformats2/implied_property/name.rb

@@ -21,13 +21,12 @@ module Microformats2
         { ">img[alt]:only-of-type" => "alt",
           ">abbr[title]:only-of-type" => "title",
           ">:only-of-type>img[alt]:only-of-type" => "alt",
-          ">:only-of-type>abbr[title]:only-of-type" => "title" } # only-of-type bit not working
+          ">:only-of-type>abbr[title]:only-of-type" => "title" }
       end
 
       private
 
       def text_value
-        @selector = "inner_text"
         @element.inner_text.gsub(/\n+/, " ").gsub(/\s+/, " ").strip
       end
     end

spec/lib/microformats2/collection_spec.rb

@@ -93,7 +93,8 @@ describe Microformats2::Collection do
                   :url => ["http://twitter.com/jlsuttles"],
                   :name => ["Jessica Lynn Suttles"]
                 }
-              }]
+              }],
+              :name => ["Jessica Lynn Suttles"]
             }
           }]
         }