html-proofer 3.7.0 → 3.7.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 6849cc3fcff2f9484b194cbeb1e6a74c5ce569df
-  data.tar.gz: a926c5b784dc04b8437b042b11fd96b9f2e0e822
+  metadata.gz: 2e2e987f27cea394b1d6fd60937e45e32addbd15
+  data.tar.gz: 894dba51116323f95f00720d9317c75e00be75fc
 SHA512:
-  metadata.gz: bc5fb91918a157a54496abc4340f4731507a393aca4da5ce21f2afd42c83d49b7636d48777acabb1542e4854944aa0242c798605b3966fb5e55ee805648736d3
-  data.tar.gz: 1d7b2107525185bcdc87647c958f71c00d0fa575665a852306de8f945fb18652ad467b76473d5255d1b11de5c101e164363984340385de47aee7cc5482a713ea
+  metadata.gz: 27cb87adf3a63d61e020d55418d2bb33983e1b50a709d3251caf5e66850ea694f812e51eaaff2ec08a3cdff081c6a2ad18fa6db402b90d36fa4507815c552274
+  data.tar.gz: 1a126cc3aabbd152d8afac93b3b1a4133498b95d6218aeb1885142623f324628dfd367f3b4919deeefb7ca093ab2d605f09273ed7089b5aac58a51321dcdb40b

data/lib/html-proofer/check/scripts.rb

@@ -29,11 +29,11 @@ class ScriptCheck < ::HTMLProofer::Check
   end
 
   def check_sri(line, content)
-    if !defined? @script.integrity && !defined? @script.crossorigin
+    if !defined?(@script.integrity) && !defined?(@script.crossorigin)
       add_issue("SRI and CORS not provided in: #{@script.src}", line: line, content: content)
-    elsif !defined? @script.integrity
+    elsif !defined?(@script.integrity)
       add_issue("Integrity is missing in: #{@script.src}", line: line, content: content)
-    elsif !defined? @script.crossorigin
+    elsif !defined?(@script.crossorigin)
       add_issue("CORS not provided for external resource in: #{@script.src}", line: line, content: content)
     end
   end

data/lib/html-proofer/element.rb

@@ -16,7 +16,9 @@ module HTMLProofer
         instance_variable_set("@#{name}", value.value)
       end
 
-      @aria_hidden = @aria_hidden == 'true' ? true : false
+      @aria_hidden = (defined?(@aria_hidden) && @aria_hidden == 'true') ? true : false
+
+      @data_proofer_ignore = defined?(@data_proofer_ignore)
 
       @text = obj.content
       @check = check
@@ -31,9 +33,23 @@ module HTMLProofer
       @parent_ignorable = parent_attributes.any? { |a| !a['data-proofer-ignore'].nil? }
 
       # fix up missing protocols
-      @href.insert 0, 'http:' if @href =~ %r{^//}
-      @src.insert 0, 'http:' if @src =~ %r{^//}
-      @srcset.insert 0, 'http:' if @srcset =~ %r{^//}
+      if defined?(@href)
+        @href.insert(0, 'http:') if @href =~ %r{^//}
+      else
+        @href = nil
+      end
+
+      if defined?(@src)
+        @src.insert(0, 'http:') if @src =~ %r{^//}
+      else
+        @src = nil
+      end
+
+      if defined?(@srcset)
+        @srcset.insert(0, 'http:') if @srcset =~ %r{^//}
+      else
+        @srcset = nil
+      end
     end
 
     def url
@@ -50,7 +66,7 @@ module HTMLProofer
 
     def parts
       @parts ||= Addressable::URI.parse url
-    rescue URI::Error, Addressable::URI::InvalidURIError => e
+    rescue URI::Error, Addressable::URI::InvalidURIError
       @parts = nil
     end
 

data/lib/html-proofer/runner.rb

@@ -142,7 +142,7 @@ module HTMLProofer
     end
 
     def checks
-      return @checks unless @checks.nil?
+      return @checks if defined?(@checks) && !@checks.nil?
       @checks = HTMLProofer::Check.subchecks.map(&:name)
       @checks.delete('FaviconCheck') unless @options[:check_favicon]
       @checks.delete('HtmlCheck') unless @options[:check_html]

data/lib/html-proofer/version.rb

@@ -1,3 +1,3 @@
 module HTMLProofer
-  VERSION = '3.7.0'.freeze
+  VERSION = '3.7.1'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: html-proofer
 version: !ruby/object:Gem::Version
-  version: 3.7.0
+  version: 3.7.1
 platform: ruby
 authors:
 - Garen Torikian
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-05-20 00:00:00.000000000 Z
+date: 2017-05-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mercenary