microformats 4.0.7 → 4.0.9

data/Rakefile

@@ -1,9 +1,8 @@
-require "bundler/gem_tasks"
-require "rspec/core/rake_task"
-require "nokogiri"
-require "open-uri"
-require "pp"
+require 'bundler/gem_tasks'
+require 'rspec/core/rake_task'
+require 'rubocop/rake_task'
 
-RSpec::Core::RakeTask.new(:spec)
-task :default => [:spec]
+RSpec::Core::RakeTask.new
+RuboCop::RakeTask.new
 
+task default: :spec

data/bin/microformats

@@ -4,25 +4,26 @@ require 'microformats'
 require 'json'
 
 def print_usage
-  puts "Usage: microformats (URL, filepath, or HTML)"
+  puts 'Usage: microformats (URL, filepath, or HTML)'
 end
 
 def process_html(html)
-    collection = Microformats.parse(html)
-    puts JSON.pretty_generate(JSON[collection.to_json.to_s])
+  collection = Microformats.parse(html)
+  puts JSON.pretty_generate(JSON[collection.to_json.to_s])
 end
 
 if STDIN.tty?
-  unless ARGV[0].nil?
-    process_html(ARGV[0])
-  else
+  if ARGV[0].nil?
     print_usage
+  else
+    process_html(ARGV[0])
   end
 else
   html = STDIN.read
-  unless html.nil?
-    process_html(html)
-  else
+
+  if html.nil?
     print_usage
+  else
+    process_html(html)
   end
 end

data/lib/microformats.rb

@@ -1,17 +1,18 @@
-require "nokogiri"
-require "open-uri"
-require "json"
+require 'nokogiri'
+require 'open-uri'
+require 'json'
+require 'set'
 
-require "microformats/version"
-require "microformats/absolute_uri"
-require "microformats/parser_core"
-require "microformats/parser"
-require "microformats/format_parser"
-require "microformats/property_parser"
-require "microformats/time_property_parser"
-require "microformats/results/parser_result"
-require "microformats/results/collection"
-require "microformats/results/property_set"
+require 'microformats/version'
+require 'microformats/absolute_uri'
+require 'microformats/parser_core'
+require 'microformats/parser'
+require 'microformats/format_parser'
+require 'microformats/property_parser'
+require 'microformats/time_property_parser'
+require 'microformats/results/parser_result'
+require 'microformats/results/collection'
+require 'microformats/results/property_set'
 
 module Microformats
   class << self
@@ -22,7 +23,7 @@ module Microformats
     def read_html(html)
       Parser.new.read_html(html)
     end
-  end # class << self
+  end
 
   class InvalidPropertyPrefix < StandardError; end
 end

data/lib/microformats/absolute_uri.rb

@@ -11,20 +11,16 @@ module Microformats
 
     def absolutize
       return relative if base.nil?
-      return base if relative.nil? or relative == ""
-      return relative if relative =~ /^https?:\/\//
+      return base if relative.nil? || relative == ''
+      return relative if relative =~ %r{^https?://}
       return base + relative if relative =~ /^#/
 
       uri = URI.parse(relative)
-
-      if base && !uri.absolute?
-        uri = URI.join(base.to_s, relative.to_s)
-      end
+      uri = URI.join(base.to_s, relative.to_s) if base && !uri.absolute?
 
       uri.normalize!
       uri.to_s
-
-    rescue URI::BadURIError, URI::InvalidURIError => e
+    rescue URI::BadURIError, URI::InvalidURIError
       relative.to_s
     end
   end

data/lib/microformats/format_parser.rb

@@ -1,343 +1,363 @@
 module Microformats
   class FormatParser < ParserCore
-
-    def parse(element, base:nil, element_type:nil, format_class_array:[], backcompat:false)
+    def parse(element, base: nil, element_type: nil, format_class_array: [], backcompat: false)
       @base = base
 
       @mode_backcompat = backcompat
 
       @properties = {}
       @children = []
+      @found_prefixes = Set.new
 
       @format_property_type = element_type
       @value = nil
 
       @mode_backcompat = backcompat
 
-      @fmt_classes =  format_class_array
+      @fmt_classes = format_class_array
 
       parse_node(element.children)
 
-      ##### Implied Properties######
-      #NOTE: much of this code may be simplified by using element.css, not sure yet, but coding to have passing tests first
-      # can optimize this later
-      unless @mode_backcompat
-        if @properties['name'].nil?
-
-          if element.name == 'img' and not element.attribute('alt').nil?
-            @properties['name'] = [element.attribute('alt').value.strip]
-          elsif element.name == 'area' and not element.attribute('alt').nil?
-            @properties['name'] = [element.attribute('alt').value.strip]
-          elsif element.name == 'abbr' and not element.attribute('title').nil?
-            @properties['name'] = [element.attribute('title').value.strip]
-
-          else
-            child_nodes = element.children.select{|n| not n.is_a?(Nokogiri::XML::Text)}
+      # check properties for any missing h-* so we know not to imply anything
+      check_for_h_properties
 
-            if child_nodes.count == 1 and child_nodes.first.is_a?(Nokogiri::XML::Element) and format_classes(child_nodes.first).empty?
-              node = child_nodes.first
+      imply_properties(element)
 
-              #else if .h-x>img:only-child[alt]:not([alt=""]):not[.h-*] then use that img’s alt for name
-              if node.name == 'img' and not node.attribute('alt').nil? and not node.attribute('alt').value.empty?
-                @properties['name'] = [node.attribute('alt').value.strip]
+      if @value.nil? || @value.empty?
+        if element_type == 'p' && !@properties['name'].nil? && !@properties['name'].empty?
+          @value = @properties['name'].first
+        elsif element_type == 'u' && !@properties['url'].nil? && !@properties['url'].empty?
+          @value = @properties['url'].first
+        elsif !element_type.nil?
+          @value = PropertyParser.new.parse(element, base: @base, element_type: element_type, backcompat: @mode_backcompat)
+        end
+      end
 
-              #else if .h-x>area:only-child[alt]:not([alt=""]):not[.h-*] then use that area’s alt for name
-              elsif node.name == 'area' and not node.attribute('alt').nil? and not node.attribute('alt').value.empty?
-                @properties['name'] = [node.attribute('alt').value.strip]
+      h_object = {}
 
-              #else if .h-x>abbr:only-child[title]:not([title=""]):not[.h-*] then use that abbr title for name
-              elsif node.name == 'abbr' and not node.attribute('title').nil? and not node.attribute('title').value.empty?
-                @properties['name'] = [node.attribute('title').value.strip]
+      h_object['value'] = @value unless @value.nil?
+      h_object['type'] = format_class_array
+      h_object['properties'] = @properties
 
-              else
-                child_nodes = node.children.select{|n| not n.is_a?(Nokogiri::XML::Text)}
-                if child_nodes.count == 1 and child_nodes.first.is_a?(Nokogiri::XML::Element) and format_classes(child_nodes.first).empty?
-                  node = child_nodes.first
+      h_object['children'] = @children unless @children.empty?
 
-                  #else if .h-x>:only-child:not[.h-*]>img:only-child[alt]:not([alt=""]):not[.h-*] then use that img’s alt for name
-                  if node.name == 'img' and not node.attribute('alt').nil? and not node.attribute('alt').value.empty?
-                    @properties['name'] = [node.attribute('alt').value.strip]
+      if @format_property_type == 'e'
+        h_object['value'] = render_text(element)
+        h_object['html'] = element.inner_html.strip
+      end
 
-                  #else if .h-x>:only-child:not[.h-*]>area:only-child[alt]:not([alt=""]):not[.h-*] then use that area’s alt for name
-                  elsif node.name == 'area' and not node.attribute('alt').nil? and not node.attribute('alt').value.empty?
-                    @properties['name'] = [node.attribute('alt').value.strip]
+      # TODO: fall back to p- dt- u- parsing if value still not set?
+      # not sure that is correct by the spec actually
+      h_object
+    end
 
-                  #else if .h-x>:only-child:not[.h-*]>abbr:only-child[title]:not([title=""]):not[.h-*] use that abbr’s title for name
-                  elsif node.name == 'abbr' and not node.attribute('title').nil? and not node.attribute('title').value.empty?
-                    @properties['name'] = [node.attribute('title').value.strip]
+    def parse_element(element)
+      prop_classes = property_classes(element)
+      prop_classes = backcompat_property_classes(element) if @mode_backcompat
 
-                  else
-                    @properties['name'] = [element.text.strip]
+      bc_classes_found = false
+      fmt_classes = format_classes(element)
 
-                  end
-                else
-                  @properties['name'] = [element.text.strip]
-                end
-              end
-            else
-              @properties['name'] = [element.text.strip]
-            end
-          end
-        end # end implied name
+      prop_classes.each do |p_class|
+        element_type = prefix_from_class(p_class)
+        @found_prefixes.add(element_type.to_sym)
+      end
+      fmt_classes.each do |p_class|
+        element_type = prefix_from_class(p_class)
+        @found_prefixes.add(element_type.to_sym)
+      end
 
+      if fmt_classes.empty?
+        fmt_classes = backcompat_format_classes(element)
+        bc_classes_found = true unless fmt_classes.empty?
+      end
 
-        if @properties['photo'].nil?
-          if element.name == 'img' and not element.attribute('src').nil?
-            @properties['photo'] = [element.attribute('src').value]
-          elsif element.name == 'object' and not element.attribute('data').nil?
-            @properties['photo'] = [element.attribute('data').value]
-          else
+      if prop_classes.length >= 1
+        if fmt_classes.length >= 1
+          prop_classes.each do |element_class|
+            element_type = prefix_from_class(element_class)
+            property_name = property_from_class(element_class)
 
-            #else if .h-x>img[src]:only-of-type:not[.h-*] then use that img src for photo
+            parsed_format = FormatParser.new.parse(element, base: @base, element_type: element_type, format_class_array: fmt_classes, backcompat: bc_classes_found)
 
-            child_img_tags_with_src = element.children.select do |child|
-              child.is_a?(Nokogiri::XML::Element) and child.name == 'img' and not child.attribute('src').nil?
-            end
-            if child_img_tags_with_src.count == 1
-              node = child_img_tags_with_src.first
-              if format_classes(node).empty?
-                @properties['photo'] = [node.attribute('src').value.strip]
+            if @value.nil?
+              if @format_property_type == 'p' && property_name == 'name'
+                @value = parsed_format['value']
+              # elsif @format_property_type == 'dt' and property_name == '???'
+              #   @value = parsed_format['value']
+              elsif @format_property_type == 'u' && property_name == 'url'
+                @value = parsed_format['value']
               end
             end
 
-            if @properties['photo'].nil?
+            @properties[property_name] = [] if @properties[property_name].nil?
+            @properties[property_name] << parsed_format
+          end
+        else
+          prop_classes.each do |element_class|
+            element_type = prefix_from_class(element_class)
+            property_name = property_from_class(element_class)
 
-              #else if .h-x>object[data]:only-of-type:not[.h-*] then use that object’s data for photo
+            parsed_property = PropertyParser.new.parse(element, base: @base, element_type: element_type, backcompat: @mode_backcompat)
 
-              child_object_tags_with_data = element.children.select do |child|
-                child.is_a?(Nokogiri::XML::Element) and child.name == 'object' and not child.attribute('data').nil?
-              end
-              if child_object_tags_with_data.count == 1
-                node = child_object_tags_with_data.first
-                if format_classes(node).empty?
-                  @properties['photo'] = [node.attribute('data').value.strip]
-                end
-              end
+            unless parsed_property.nil?
+              @properties[property_name] = [] if @properties[property_name].nil?
+              @properties[property_name] << parsed_property
             end
+          end
 
-            child_elements = element.children.select do |child| not child.is_a?(Nokogiri::XML::Text) end
-
-            if @properties['photo'].nil? and child_elements.count == 1 and format_classes(child_elements.first).empty?
-
-              #else if .h-x>:only-child:not[.h-*]>img[src]:only-of-type:not[.h-*], then use that img’s src for photo
+          parse_nodeset(element.children)
+        end
+      elsif fmt_classes.length >= 1
+        @children << FormatParser.new.parse(element, base: @base, format_class_array: fmt_classes, backcompat: bc_classes_found)
+      else
+        parse_nodeset(element.children)
+      end
+    end
 
-              child_img_tags_with_src = child_elements.first.children.select do |child|
-                child.is_a?(Nokogiri::XML::Element) and child.name == 'img' and not child.attribute('src').nil?
-              end
-              if child_img_tags_with_src.count == 1
-                node = child_img_tags_with_src.first
-                if format_classes(node).empty?
-                  @properties['photo'] = [node.attribute('src').value.strip]
-                end
-              end
+    def check_for_h_properties
+      @properties.each do |_, prop|
+        prop.each do |prop_entry|
+          next unless prop_entry.respond_to?(:key) && prop_entry.key?('type')
+          next if prop_entry['type'].nil?
 
-              if @properties['photo'].nil?
-
-                #else if .h-x>:only-child:not[.h-*]>object[data]:only-of-type:not[.h-*], then use that object’s data for photo
-                #
-                child_object_tags_with_data = child_elements.first.children.select do |child|
-                  child.is_a?(Nokogiri::XML::Element) and child.name == 'object' and not child.attribute('data').nil?
-                end
-                if child_object_tags_with_data.count == 1
-                  node = child_object_tags_with_data.first
-                  if format_classes(node).empty?
-                    @properties['photo'] = [node.attribute('data').value.strip]
-                  end
-                end
-              end
-            end
-
-          end
-          unless @properties['photo'].nil?
-            @properties['photo'] = [ Microformats::AbsoluteUri.new(@properties['photo'].first, base: @base).absolutize ]
+          prop_entry['type'].each do |type|
+            @found_prefixes.add(prefix_from_class(type).to_sym)
           end
         end
+      end
+    end
 
-        if @properties['url'].nil?
-          if element.name == 'a' and not element.attribute('href').nil?
-            @properties['url'] = [element.attribute('href').value]
-          elsif element.name == 'area' and not element.attribute('href').nil?
-            @properties['url'] = [element.attribute('href').value]
+    def imply_name(element)
+      return unless @properties['name'].nil? && !@found_prefixes.include?(:e) && !@found_prefixes.include?(:p) && !@found_prefixes.include?(:h) && @children.empty?
+
+      if element.name == 'img' && !element.attribute('alt').nil?
+        @properties['name'] = [element.attribute('alt').value.strip]
+      elsif element.name == 'area' && !element.attribute('alt').nil?
+        @properties['name'] = [element.attribute('alt').value.strip]
+      elsif element.name == 'abbr' && !element.attribute('title').nil?
+        @properties['name'] = [element.attribute('title').value.strip]
+      else
+        child_nodes = element.children.reject { |n| n.is_a?(Nokogiri::XML::Text) }
+
+        if child_nodes.count == 1 && child_nodes.first.is_a?(Nokogiri::XML::Element) && format_classes(child_nodes.first).empty?
+          node = child_nodes.first
+
+          # else if .h-x>img:only-child[alt]:not([alt=""]):not[.h-*] then use that img's alt for name
+          if node.name == 'img' && !node.attribute('alt').nil? && !node.attribute('alt').value.empty?
+            @properties['name'] = [node.attribute('alt').value.strip]
+          # else if .h-x>area:only-child[alt]:not([alt=""]):not[.h-*] then use that area's alt for name
+          elsif node.name == 'area' && !node.attribute('alt').nil? && !node.attribute('alt').value.empty?
+            @properties['name'] = [node.attribute('alt').value.strip]
+          # else if .h-x>abbr:only-child[title]:not([title=""]):not[.h-*] then use that abbr title for name
+          elsif node.name == 'abbr' && !node.attribute('title').nil? && !node.attribute('title').value.empty?
+            @properties['name'] = [node.attribute('title').value.strip]
           else
-            #else if .h-x>a[href]:only-of-type:not[.h-*], then use that [href] for url
-            child_a_tags_with_href = element.children.select do |child|
-              child.is_a?(Nokogiri::XML::Element) and child.name == 'a' and not child.attribute('href').nil?
-            end
-            if child_a_tags_with_href.count == 1
-              node = child_a_tags_with_href.first
-              if format_classes(node).empty?
-                @properties['url'] = [node.attribute('href').value.strip]
-              end
-            end
+            child_nodes = node.children.reject { |n| n.is_a?(Nokogiri::XML::Text) }
 
-            if @properties['url'].nil?
+            if child_nodes.count == 1 && child_nodes.first.is_a?(Nokogiri::XML::Element) && format_classes(child_nodes.first).empty?
+              node = child_nodes.first
 
-              #else if .h-x>area[href]:only-of-type:not[.h-*], then use that [href] for url
-              child_area_tags_with_href = element.children.select do |child|
-                child.is_a?(Nokogiri::XML::Element) and child.name == 'area' and not child.attribute('href').nil?
-              end
-              if child_area_tags_with_href.count == 1
-                node = child_area_tags_with_href.first
-                if format_classes(node).empty?
-                  @properties['url'] = [node.attribute('href').value.strip]
-                end
+              # else if .h-x>:only-child:not[.h-*]>img:only-child[alt]:not([alt=""]):not[.h-*] then use that img's alt for name
+              if node.name == 'img' && !node.attribute('alt').nil? && !node.attribute('alt').value.empty?
+                @properties['name'] = [node.attribute('alt').value.strip]
+              # else if .h-x>:only-child:not[.h-*]>area:only-child[alt]:not([alt=""]):not[.h-*] then use that area's alt for name
+              elsif node.name == 'area' && !node.attribute('alt').nil? && !node.attribute('alt').value.empty?
+                @properties['name'] = [node.attribute('alt').value.strip]
+              # else if .h-x>:only-child:not[.h-*]>abbr:only-child[title]:not([title=""]):not[.h-*] use that abbr's title for name
+              elsif node.name == 'abbr' && !node.attribute('title').nil? && !node.attribute('title').value.empty?
+                @properties['name'] = [node.attribute('title').value.strip]
+              else
+                @properties['name'] = [render_text(element)]
               end
+            else
+              @properties['name'] = [render_text(element)]
             end
+          end
+        else
+          @properties['name'] = [render_text(element)]
+        end
+      end
+    end
 
-            child_elements = element.children.select do |child| not child.is_a?(Nokogiri::XML::Text) end
+    def imply_photo(element)
+      return unless @properties['photo'].nil?
 
-            if @properties['url'].nil? and child_elements.count == 1 and format_classes(child_elements.first).empty?
-              child_element = child_elements.first
+      if element.name == 'img' && !element.attribute('src').nil?
+        @properties['photo'] = [element.attribute('src').value]
+      elsif element.name == 'object' && !element.attribute('data').nil?
+        @properties['photo'] = [element.attribute('data').value]
+      else
+        # else if .h-x>img[src]:only-of-type:not[.h-*] then use that img src for photo
+        child_img_tags_with_src = element.children.select do |child|
+          child.is_a?(Nokogiri::XML::Element) && child.name == 'img' && !child.attribute('src').nil?
+        end
 
-              #else if .h-x>:only-child:not[.h-*]>a[href]:only-of-type:not[.h-*], then use that [href] for url
-              child_a_tags_with_href = child_element.children.select do |child|
-                child.is_a?(Nokogiri::XML::Element) and child.name == 'a' and not child.attribute('href').nil?
-              end
-              if child_a_tags_with_href.count == 1
-                node = child_a_tags_with_href.first
-                if format_classes(node).empty?
-                  @properties['url'] = [node.attribute('href').value.strip]
-                end
-              end
+        if child_img_tags_with_src.count == 1
+          node = child_img_tags_with_src.first
 
-              if @properties['url'].nil?
-
-                #else if .h-x>:only-child:not[.h-*]>area[href]:only-of-type:not[.h-*], then use that [href] for url
-                child_area_tags_with_href = child_element.children.select do |child|
-                  child.is_a?(Nokogiri::XML::Element) and child.name == 'area' and not child.attribute('href').nil?
-                end
-                if child_area_tags_with_href.count == 1
-                  node = child_area_tags_with_href.first
-                  if format_classes(node).empty?
-                    @properties['url'] = [node.attribute('href').value.strip]
-                  end
-                end
-              end
-            end
-          end
+          @properties['photo'] = [node.attribute('src').value.strip] if format_classes(node).empty?
+        end
 
-          unless @properties['url'].nil?
-            @properties['url'] = [ Microformats::AbsoluteUri.new(@properties['url'].first, base: @base).absolutize ]
+        if @properties['photo'].nil?
+          # else if .h-x>object[data]:only-of-type:not[.h-*] then use that object's data for photo
+          child_object_tags_with_data = element.children.select do |child|
+            child.is_a?(Nokogiri::XML::Element) && child.name == 'object' && !child.attribute('data').nil?
           end
-        end
 
-      end
-      ##### END Implied Properties when not in backcompat mode######
+          if child_object_tags_with_data.count == 1
+            node = child_object_tags_with_data.first
 
-      ### imply date for dt-end if dt-start is defined with a date ###
-      if not @properties['end'].nil? and not @properties['start'].nil?
-        start_date = nil
-        @properties['start'].each do |start_val|
-          if start_val =~ /^(\d{4}-[01]\d-[0-3]\d)/
-            start_date = $1 if start_date.nil?
-          elsif start_val =~ /^(\d{4}-[0-3]\d\d)/
-            start_date = $1 if start_date.nil?
-          end
-        end
-        unless start_date.nil?
-          @properties['end'].map! do |end_val|
-            if end_val=~ /^\d{4}-[01]\d-[0-3]\d/
-              end_val
-            elsif end_val=~ /^\d{4}-[0-3]\d\d/
-              end_val
-            else
-              start_date + ' ' + end_val
+            if format_classes(node).empty?
+              @properties['photo'] = [node.attribute('data').value.strip]
             end
           end
         end
-      end
 
-      if @value.nil? or @value.empty?
-        if element_type == 'p' and not @properties['name'].nil? and not @properties['name'].empty?
-          @value = @properties['name'].first
-        elsif element_type == 'u' and not @properties['url'].nil? and not @properties['url'].empty?
-          @value = @properties['url'].first
-        elsif not element_type.nil?
-          @value = PropertyParser.new.parse(element, base: @base, element_type: element_type,  backcompat: @mode_backcompat)
-        end
-      end
+        child_elements = element.children.reject { |child| child.is_a?(Nokogiri::XML::Text) }
 
-      h_object = {}
+        if @properties['photo'].nil? && child_elements.count == 1 && format_classes(child_elements.first).empty?
+          # else if .h-x>:only-child:not[.h-*]>img[src]:only-of-type:not[.h-*], then use that img's src for photo
+          child_img_tags_with_src = child_elements.first.children.select do |child|
+            child.is_a?(Nokogiri::XML::Element) && child.name == 'img' && !child.attribute('src').nil?
+          end
 
-      h_object['value'] = @value unless @value.nil?
-      h_object['type'] = format_class_array
-      h_object['properties'] = @properties
+          if child_img_tags_with_src.count == 1
+            node = child_img_tags_with_src.first
 
-      h_object['children'] = @children unless @children.empty?
+            if format_classes(node).empty?
+              @properties['photo'] = [node.attribute('src').value.strip]
+            end
+          end
 
-      if @format_property_type == 'e'
-        h_object['value'] = element.text.strip
-        h_object['html'] = element.inner_html
-      end
+          if @properties['photo'].nil?
+            # else if .h-x>:only-child:not[.h-*]>object[data]:only-of-type:not[.h-*], then use that object's data for photo
+            child_object_tags_with_data = child_elements.first.children.select do |child|
+              child.is_a?(Nokogiri::XML::Element) && child.name == 'object' && !child.attribute('data').nil?
+            end
 
-      ##todo fall back to p- dt- u- parsing if value still not set?
-      #  not sure that is correct by the spec actually
+            if child_object_tags_with_data.count == 1
+              node = child_object_tags_with_data.first
 
-      h_object
+              if format_classes(node).empty?
+                @properties['photo'] = [node.attribute('data').value.strip]
+              end
+            end
+          end
+        end
+      end
 
+      @properties['photo'] = [Microformats::AbsoluteUri.new(@properties['photo'].first, base: @base).absolutize] unless @properties['photo'].nil?
     end
 
+    def imply_url(element)
+      return unless @properties['url'].nil? && !@found_prefixes.include?(:u) && !@found_prefixes.include?(:h) && @children.empty?
 
-    def parse_element(element)
+      if element.name == 'a' && !element.attribute('href').nil?
+        @properties['url'] = [element.attribute('href').value]
+      elsif element.name == 'area' && !element.attribute('href').nil?
+        @properties['url'] = [element.attribute('href').value]
+      else
+        # else if .h-x>a[href]:only-of-type:not[.h-*], then use that [href] for url
+        child_a_tags_with_href = element.children.select do |child|
+          child.is_a?(Nokogiri::XML::Element) && child.name == 'a' && !child.attribute('href').nil?
+        end
 
-      prop_classes = property_classes(element)
-      prop_classes = backcompat_property_classes(element) if @mode_backcompat
+        if child_a_tags_with_href.count == 1
+          node = child_a_tags_with_href.first
+          @properties['url'] = [node.attribute('href').value.strip] if format_classes(node).empty?
+        end
 
-      bc_classes_found = false
-      fmt_classes = format_classes(element)
+        if @properties['url'].nil?
+          # else if .h-x>area[href]:only-of-type:not[.h-*], then use that [href] for url
+          child_area_tags_with_href = element.children.select do |child|
+            child.is_a?(Nokogiri::XML::Element) && child.name == 'area' && !child.attribute('href').nil?
+          end
 
-      if fmt_classes.empty?
-          fmt_classes = backcompat_format_classes(element)
-          bc_classes_found = true unless fmt_classes.empty?
-      end
+          if child_area_tags_with_href.count == 1
+            node = child_area_tags_with_href.first
+            @properties['url'] = [node.attribute('href').value.strip] if format_classes(node).empty?
+          end
+        end
 
-      if prop_classes.length >= 1
+        child_elements = element.children.reject { |child| child.is_a?(Nokogiri::XML::Text) }
 
-        if fmt_classes.length >= 1
+        if @properties['url'].nil? && child_elements.count == 1 && format_classes(child_elements.first).empty?
+          child_element = child_elements.first
+          # else if .h-x>:only-child:not[.h-*]>a[href]:only-of-type:not[.h-*], then use that [href] for url
+          child_a_tags_with_href = child_element.children.select do |child|
+            child.is_a?(Nokogiri::XML::Element) && child.name == 'a' && !child.attribute('href').nil?
+          end
 
-          prop_classes.each do |element_class|
-            element_type = element_class.downcase.split('-')[0]
-            property_name = element_class.downcase.split('-')[1..-1].join('-')
+          if child_a_tags_with_href.count == 1
+            node = child_a_tags_with_href.first
+            @properties['url'] = [node.attribute('href').value.strip] if format_classes(node).empty?
+          end
 
-            parsed_format = FormatParser.new.parse(element, base:@base, element_type: element_type, format_class_array: fmt_classes, backcompat: bc_classes_found )
+          if @properties['url'].nil?
+            # else if .h-x>:only-child:not[.h-*]>area[href]:only-of-type:not[.h-*], then use that [href] for url
+            child_area_tags_with_href = child_element.children.select do |child|
+              child.is_a?(Nokogiri::XML::Element) && child.name == 'area' && !child.attribute('href').nil?
+            end
 
-            if @value.nil?
-              if @format_property_type == 'p' and property_name == 'name'
-                @value = parsed_format['value']
-              #elsif @format_property_type == 'dt' and property_name == '???'
-                #@value = parsed_format['value']
-              elsif @format_property_type == 'u' and property_name == 'url'
-                @value = parsed_format['value']
-              end
+            if child_area_tags_with_href.count == 1
+              node = child_area_tags_with_href.first
+              @properties['url'] = [node.attribute('href').value.strip] if format_classes(node).empty?
             end
+          end
+        end
+      end
 
-            @properties[property_name] = []  if @properties[property_name].nil?
-            @properties[property_name] << parsed_format
+      @properties['url'] = [Microformats::AbsoluteUri.new(@properties['url'].first, base: @base).absolutize] unless @properties['url'].nil?
+    end
 
-          end
+    def prefix_from_class(class_name)
+      class_name.downcase.split('-')[0]
+    end
 
-        else
+    def property_from_class(class_name)
+      class_name.downcase.split('-')[1..-1].join('-')
+    end
 
-          prop_classes.each do |element_class|
-            element_type = element_class.downcase.split('-')[0]
-            property_name = element_class.downcase.split('-')[1..-1].join('-')
+    # imply date for dt-end if dt-start is defined with a date ###
+    def imply_dates
+      return unless !@properties['end'].nil? && !@properties['start'].nil?
 
-            parsed_property = PropertyParser.new.parse(element, base: @base, element_type: element_type, backcompat:  @mode_backcompat)
+      start_date = nil
 
-            if not parsed_property.nil? 
-              @properties[property_name] = []  if @properties[property_name].nil?
-              @properties[property_name] << parsed_property
-            end
-          end
-          parse_nodeset(element.children)
+      @properties['start'].each do |start_val|
+        if start_val =~ /^(\d{4}-[01]\d-[0-3]\d)/
+          start_date = Regexp.last_match(1) if start_date.nil?
+        elsif start_val =~ /^(\d{4}-[0-3]\d\d)/
+          start_date = Regexp.last_match(1) if start_date.nil?
         end
+      end
 
-      elsif fmt_classes.length >= 1
-        @children << FormatParser.new.parse(element, base: @base, format_class_array: fmt_classes, backcompat: bc_classes_found )
-      else
-        parse_nodeset(element.children)
+      unless start_date.nil?
+        @properties['end'].map! do |end_val|
+          if end_val.match?(/^\d{4}-[01]\d-[0-3]\d/)
+            end_val
+          elsif end_val.match?(/^\d{4}-[0-3]\d\d/)
+            end_val
+          else
+            start_date + ' ' + end_val
+          end
+        end
       end
     end
 
+    def imply_properties(element)
+      ##### Implied Properties ######
+      # NOTE: much of this code may be simplified by using element.css, not sure yet, but coding to have passing tests first
+      # can optimize this later
+      unless @mode_backcompat
+        imply_name(element)
+        imply_photo(element)
+        imply_url(element)
+      end
+      ##### END Implied Properties when not in backcompat mode######
+
+      imply_dates
+    end
   end
 end