Udpate conditional logic, update YARD comment

Author: jgarber623

lib/indieweb/endpoints/parsers/base_parser.rb

@@ -37,15 +37,15 @@ def parsed_response_headers
         end
 
         def results_from_body
-          return unless response.mime_type == 'text/html'
+          return if response.mime_type != 'text/html'
 
           Services::ResponseParserService.parse_body(parsed_response_body, self.class.identifier)
         end
 
         def results_from_headers
           return if parsed_response_headers.none?
 
-          Services::ResponseParserService.parse_headers(parsed_response_headers, self.class.identifier)
+          Services::ResponseParserService.parse_headers(parsed_response_headers.group_by_relation_type, self.class.identifier)
         end
 
         def results_from_http_request

lib/indieweb/endpoints/parsers/redirect_uri_parser.rb

@@ -8,7 +8,7 @@ class RedirectUriParser < BaseParser
 
         # @return [Array<String>, nil]
         def results
-          return unless mapped_results.any?
+          return if mapped_results.none?
 
           mapped_results
         end

lib/indieweb/endpoints/services/response_parser_service.rb

@@ -10,16 +10,14 @@ def self.parse_body(body, identifier, node = 'link')
           body.css(%(#{node}[rel~="#{identifier}"][href]:not([href*="#"]))).map { |element| element['href'] }
         end
 
-        # @param headers [LinkHeaderParser::LinkHeadersCollection]
+        # @param headers [Hash{Symbol => Array<LinkHeaderParser::LinkHeader}]
         # @param identifier [Symbol]
         # @return [Array<String>, nil]
         def self.parse_headers(headers, identifier)
-          headers_for_identifier = headers.group_by_relation_type[identifier]
-
-          return unless headers_for_identifier
+          return unless headers.key?(identifier)
 
           # Reject endpoints that contain a fragment identifier
-          headers_for_identifier.reject { |header| Addressable::URI.parse(header.target_uri).fragment }.map(&:target_uri)
+          headers[identifier].reject { |header| Addressable::URI.parse(header.target_uri).fragment }.map(&:target_uri)
         end
       end
     end