Fixes #1520

wpscanteam · Jul 18, 2020 · cba6e74 · cba6e74
1 parent 981bcf5
commit cba6e74
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 1 deletion.
diff --git a/app/models/theme.rb b/app/models/theme.rb
@@ -101,7 +101,7 @@ def parse_style
       #
       # @return [ String ]
       def parse_style_tag(body, tag)
-        value = body[/#{Regexp.escape(tag)}:[\t ]*([^\r\n*]+)/i, 1]
+        value = body[/\b#{Regexp.escape(tag)}:[\t ]*([^\r\n*]+)/, 1]
 
         value && !value.strip.empty? ? value.strip : nil
       end

diff --git a/spec/app/models/theme_spec.rb b/spec/app/models/theme_spec.rb
@@ -41,6 +41,12 @@
       its(:style_uri)   { should eql 'http://www.elegantthemes.com/gallery/divi/' }
       its(:license_uri) { should eql 'http://www.gnu.org/licenses/gpl-2.0.html' }
     end
+
+    context 'when no tags' do
+      let(:fixture) { fixtures.join('no_tags.css') }
+
+      its(:author) { should eql nil }
+    end
   end
 
   describe '#version' do

diff --git a/spec/fixtures/models/theme/no_tags.css b/spec/fixtures/models/theme/no_tags.css
@@ -0,0 +1,3 @@
+comment-author:before,.comment-author:after,.widget-area:before,.widget-area:after,.widget:before,.widget:after {
+  /* Some style */
+}