js-code-wrapper 0.1.2 → 0.1.3

data/README.md

@@ -14,4 +14,5 @@ to your `Gemfile`.
 
 ``` ruby
   wrap_js_code("<script>alert('wrap me!');</script>") # => "<code><script>alert('wrap me!');</script></code>"
+  wrap_js_code("<script>alert('wrap me!');</script>", :escape_js => true) # => "&lt;script&gt;alert('wrap me!');&lt;/script&gt;"
 ```

data/VERSION

@@ -1 +1 @@
-0.1.2
+0.1.3

data/js-code-wrapper.gemspec

@@ -4,14 +4,14 @@
 # -*- encoding: utf-8 -*-
 
 Gem::Specification.new do |s|
-  s.name = "js-code-wrapper"
-  s.version = "0.1.2"
+  s.name = %q{js-code-wrapper}
+  s.version = "0.1.3"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
-  s.authors = ["dom1nga"]
-  s.date = "2011-11-29"
-  s.description = "Usage: wrap_js_code('<script>alert('wrap me!');</script>') # => '<code><script>alert('wrap me!');</script></code>'"
-  s.email = "beloved.dom1nga@gmail.com"
+  s.authors = [%q{dom1nga}]
+  s.date = %q{2011-12-06}
+  s.description = %q{Usage: wrap_js_code('<script>alert('wrap me!');</script>') # => '<code><script>alert('wrap me!');</script></code>'}
+  s.email = %q{beloved.dom1nga@gmail.com}
   s.extra_rdoc_files = [
     "LICENSE.txt",
     "README.md"
@@ -30,11 +30,11 @@ Gem::Specification.new do |s|
     "spec/lib/js_code_wrapper_spec.rb",
     "spec/spec_helper.rb"
   ]
-  s.homepage = "http://github.com/dom1nga/js-code-wrapper"
-  s.licenses = ["MIT"]
-  s.require_paths = ["lib"]
-  s.rubygems_version = "1.8.10"
-  s.summary = "Provides helper method 'wrap_js_code' that wraps javascript code into '<code></code>' tags"
+  s.homepage = %q{http://github.com/dom1nga/js-code-wrapper}
+  s.licenses = [%q{MIT}]
+  s.require_paths = [%q{lib}]
+  s.rubygems_version = %q{1.8.7}
+  s.summary = %q{Provides helper method 'wrap_js_code' that wraps javascript code into '<code></code>' tags}
 
   if s.respond_to? :specification_version then
     s.specification_version = 3

data/lib/js-code-wrapper.rb

@@ -1,12 +1,18 @@
 require 'nokogiri'
+require 'cgi'
 
 module JsCodeWrapper
-  def wrap_js_code(text)
+  def wrap_js_code(text, options = { :escape_js => false })
     doc = ::Nokogiri::HTML("<text>#{text}</text>")
-    doc.xpath("//script[not(ancestor::code)] | //*/@*[starts-with(name(.), 'on')]/..").wrap('<code>')
+    if options[:escape_js]
+      doc.xpath("//script[not(ancestor::code)] | //*[@*[starts-with(name(), 'on')] and not(ancestor::code)]").each {|elem| elem.replace(::CGI.escapeHTML(elem.to_html)) }
+    else
+      doc.xpath("//script[not(ancestor::code)] | //*[@*[starts-with(name(), 'on')] and not(ancestor::code)]").wrap('<code>')
+    end
     doc.xpath("//body/*").to_s.scan(/<text>(.*)<\/text>/im)[0][0]
   end
+
+  module_function :wrap_js_code
 end
 
-Object.send(:include, JsCodeWrapper)
 ActionView::Base.send(:include, JsCodeWrapper) if defined?(ActionView::Base)

data/spec/lib/js_code_wrapper_spec.rb

@@ -4,44 +4,54 @@ describe JsCodeWrapper do
 
   describe "#wrap_js_code" do
     it "should return the text without changes, if it doesn't contains script tags" do
-      wrap_js_code("text <p>text</p><h1>text</h1>").should ==
+      JsCodeWrapper.wrap_js_code("text <p>text</p><h1>text</h1>").should ==
                    "text <p>text</p>\n<h1>text</h1>"
     end
 
     it "should return the text with script body (type1) wrapped into code tags" do
-      wrap_js_code("<script>alert('wrap me!')</script>").should ==
+      JsCodeWrapper.wrap_js_code("<script>alert('wrap me!')</script>").should ==
                    "<code><script>alert('wrap me!')</script></code>"
     end
 
     it "should return the text with script body (type2) wrapped into code tags" do
-      wrap_js_code('<script type="text/javascript" src="example.com/example.js"></script>').should ==
+      JsCodeWrapper.wrap_js_code('<script type="text/javascript" src="example.com/example.js"></script>').should ==
                    '<code><script type="text/javascript" src="example.com/example.js"></script></code>'
     end
 
     it "should return the text with script body wrapped into code tags when script body duplicated in text" do
-      wrap_js_code('<code><script>alert("wrap me!")</script></code><script>alert("wrap me!")</script>').should ==
+      JsCodeWrapper.wrap_js_code('<code><script>alert("wrap me!")</script></code><script>alert("wrap me!")</script>').should ==
                    '<code><script>alert("wrap me!")</script></code><code><script>alert("wrap me!")</script></code>'
     end
 
     it "should return the text with script body wrapped into code tags" do
-      wrap_js_code("<code><script>alert('wrap me1!')</script><script type='text/javascript' src='example.com/example1.js'></script></code><p>text1</p><script>alert('wrap me2!')</script><script type='text/javascript' src='example.com/example2.js'></script><code><script>alert('wrap me3!')</script></code>").should ==
+      JsCodeWrapper.wrap_js_code("<code><script>alert('wrap me1!')</script><script type='text/javascript' src='example.com/example1.js'></script></code><p>text1</p><script>alert('wrap me2!')</script><script type='text/javascript' src='example.com/example2.js'></script><code><script>alert('wrap me3!')</script></code>").should ==
                    "<code><script>alert('wrap me1!')</script><script type=\"text/javascript\" src=\"example.com/example1.js\"></script></code><p>text1</p>\n<code><script>alert('wrap me2!')</script></code><code><script type=\"text/javascript\" src=\"example.com/example2.js\"></script></code><code><script>alert('wrap me3!')</script></code>"
     end
 
     it "should return the text without changes, if script body into code tags" do
-      wrap_js_code("<code><script>alert('wrap me!')</script></code>").should ==
+      JsCodeWrapper.wrap_js_code("<code><script>alert('wrap me!')</script></code>").should ==
                    "<code><script>alert('wrap me!')</script></code>"
     end
 
     it "should return the text with link wrapped into code tags if link contains event attribute" do
-      wrap_js_code('<a href="#" onclick="alert(\'wrap me!\'); return 0;">link</a>').should ==
+      JsCodeWrapper.wrap_js_code('<a href="#" onclick="alert(\'wrap me!\'); return 0;">link</a>').should ==
                    "<code><a href=\"#\" onclick=\"alert('wrap me!'); return 0;\">link</a></code>"
     end
 
     it "should return the text with tags without changes if tags not contains event attributes" do
-      wrap_js_code('<a href="#">link</a><div class="div1"></div><p>text1</p>').should ==
+      JsCodeWrapper.wrap_js_code('<a href="#">link</a><div class="div1"></div><p>text1</p>').should ==
                    "<a href=\"#\">link</a><div class=\"div1\"></div>\n<p>text1</p>"
     end
 
+    it "should return the text with escaped java script" do
+      JsCodeWrapper.wrap_js_code("<script>alert('wrap me!')</script>", :escape_js => true).should ==
+                   "&lt;script&gt;alert('wrap me!')&lt;/script&gt;"
+    end
+
+    it "should return the text with script body wrapped into code tags when tag contains script body duplicated in text" do
+      JsCodeWrapper.wrap_js_code('<code><a href="#" onclick="alert(\'wrap me!\'); return 0;">link</a></code><a href="#" onclick="alert(\'wrap me!\'); return 0;">link</a>').should ==
+                                 '<code><a href="#" onclick="alert(\'wrap me!\'); return 0;">link</a></code><code><a href="#" onclick="alert(\'wrap me!\'); return 0;">link</a></code>'
+    end
+
   end
 end

metadata

@@ -1,88 +1,69 @@
---- !ruby/object:Gem::Specification 
+--- !ruby/object:Gem::Specification
 name: js-code-wrapper
-version: !ruby/object:Gem::Version 
-  hash: 31
+version: !ruby/object:Gem::Version
+  version: 0.1.3
   prerelease: 
-  segments: 
-  - 0
-  - 1
-  - 2
-  version: 0.1.2
 platform: ruby
-authors: 
+authors:
 - dom1nga
 autorequire: 
 bindir: bin
 cert_chain: []
-
-date: 2011-11-29 00:00:00 Z
-dependencies: 
-- !ruby/object:Gem::Dependency 
-  requirement: &id001 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
-  version_requirements: *id001
+date: 2011-12-06 00:00:00.000000000Z
+dependencies:
+- !ruby/object:Gem::Dependency
   name: rspec
-  prerelease: false
-  type: :development
-- !ruby/object:Gem::Dependency 
-  requirement: &id002 !ruby/object:Gem::Requirement 
+  requirement: &449670 !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
-  version_requirements: *id002
-  name: rake
-  prerelease: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-- !ruby/object:Gem::Dependency 
-  requirement: &id003 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
-  version_requirements: *id003
-  name: jeweler
   prerelease: false
+  version_requirements: *449670
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: &449370 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-- !ruby/object:Gem::Dependency 
-  requirement: &id004 !ruby/object:Gem::Requirement 
+  prerelease: false
+  version_requirements: *449370
+- !ruby/object:Gem::Dependency
+  name: jeweler
+  requirement: &449010 !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
-  version_requirements: *id004
-  name: nokogiri
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
   prerelease: false
+  version_requirements: *449010
+- !ruby/object:Gem::Dependency
+  name: nokogiri
+  requirement: &448640 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-description: "Usage: wrap_js_code('<script>alert('wrap me!');</script>') # => '<code><script>alert('wrap me!');</script></code>'"
+  prerelease: false
+  version_requirements: *448640
+description: ! 'Usage: wrap_js_code(''<script>alert(''wrap me!'');</script>'') # =>
+  ''<code><script>alert(''wrap me!'');</script></code>'''
 email: beloved.dom1nga@gmail.com
 executables: []
-
 extensions: []
-
-extra_rdoc_files: 
+extra_rdoc_files:
 - LICENSE.txt
 - README.md
-files: 
+files:
 - .document
 - .rspec
 - Gemfile
@@ -96,37 +77,32 @@ files:
 - spec/lib/js_code_wrapper_spec.rb
 - spec/spec_helper.rb
 homepage: http://github.com/dom1nga/js-code-wrapper
-licenses: 
+licenses:
 - MIT
 post_install_message: 
 rdoc_options: []
-
-require_paths: 
+require_paths:
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement 
+required_ruby_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+      segments:
       - 0
-      version: "0"
-required_rubygems_version: !ruby/object:Gem::Requirement 
+      hash: -385486207
+required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
 requirements: []
-
 rubyforge_project: 
-rubygems_version: 1.8.10
+rubygems_version: 1.8.7
 signing_key: 
 specification_version: 3
-summary: Provides helper method 'wrap_js_code' that wraps javascript code into '<code></code>' tags
+summary: Provides helper method 'wrap_js_code' that wraps javascript code into '<code></code>'
+  tags
 test_files: []
-