netshade-oembed_links 0.1.7 → 0.1.8

data/README.txt

@@ -62,7 +62,7 @@ OEmbed.transform("Here is a link to amazon http://www.amazon.com/Complete-Aubrey
 end
 
 
-To get started quickly in Rails (< 2.2.  oembed_links doesn't work in 2.2 yet, when rendering w/ Rails templates):
+To get started quickly in Rails:
 
 Copy the included oembed_links_example.yml file to RAILS_ROOT/config/oembed_links.yml,
 add a dependency to the gem in your environment.rb ( config.gem "oembed_links" )

data/lib/oembed_links/response.rb

@@ -80,6 +80,10 @@ class OEmbed
       end
     end
 
+    def has_rendered?
+      !@rendered.nil?
+    end    
+
     private
 
     # Needlessly stupid priority for rendering.
@@ -95,9 +99,6 @@ class OEmbed
       end
     end
 
-    def has_rendered?
-      !@rendered.nil?
-    end
     
     def render_content(*args, &block)
       options = (args.last.is_a?(Hash)) ? args.last : { }

data/lib/oembed_links.rb

@@ -352,7 +352,7 @@ class OEmbed
         txt.gsub!(u, response.to_s)
       else
         yield(response, u)
-        txt.gsub!(u, response.rendered_content)
+        (response.has_rendered?) ? txt.gsub!(u, response.rendered_content) : txt
       end
     else
       if block.nil?
@@ -360,7 +360,7 @@ class OEmbed
       else
         response = OEmbed::Response.new("", u, {})
         yield(response, u)
-        txt.gsub!(u, response.rendered_content)
+        (response.has_rendered?) ? txt.gsub!(u, response.rendered_content) : txt
       end      
     end
   end

data/oembed_links.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name = %q{oembed_links}
-  s.version = "0.1.7"
+  s.version = "0.1.8"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Indianapolis Star MD&D"]

data/spec/oembed_links_spec.rb

@@ -166,7 +166,13 @@ describe OEmbed, "transforming functions" do
       r.hedgehog? { |v| "hedgey"}
       r.from?(:test2) { |t| "test2" }
       r.matches?(/baz/) { |m| "regex" }
-    end.should == "foo" 
+    end.should == "http://test1.net/foo" 
+  end
+
+  it "should transform only urls which have registered providers" do
+    OEmbed.transform("http://test1.net/foo and http://not.a.valid.url.host/fake are urls") do |r, url|
+      r.video? { |v| "video" }
+    end.should == "video and http://not.a.valid.url.host/fake are urls"    
   end
 
   it "should pass control to the .none? block if no scheme matched" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: netshade-oembed_links
 version: !ruby/object:Gem::Version 
-  version: 0.1.7
+  version: 0.1.8
 platform: ruby
 authors: 
 - Indianapolis Star MD&D