Merge pull request rubysherpas#243 from beefsack/gravatar-protocol-fix

Removed protocol from gravatar url to allow https.
IS3D · Jun 13, 2012 · fe18d8d · fe18d8d
2 parents 3dda6a2 + 9ed2d82
commit fe18d8d
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/app/helpers/forem/posts_helper.rb b/app/helpers/forem/posts_helper.rb
@@ -18,7 +18,7 @@ def avatar_url(email, options = {})
       options[:s] = options.delete(:size) || 60
       options[:d] = options.delete(:default) || default_gravatar
       options.delete(:d) unless options[:d]
-      "http://www.gravatar.com/avatar/#{md5}?#{options.to_param}"
+      "//www.gravatar.com/avatar/#{md5}?#{options.to_param}"
     end
 
     def default_gravatar

diff --git a/spec/helpers/posts_helper_spec.rb b/spec/helpers/posts_helper_spec.rb
@@ -10,7 +10,7 @@ module Forem
         let(:opts) { {} }
 
         it 'includes the default size' do
-          expected = "http://www.gravatar.com/avatar/#{email_hash}?s=60"
+          expected = "//www.gravatar.com/avatar/#{email_hash}?s=60"
           helper.avatar_url(email, opts).should eq(expected)
         end
       end
@@ -19,7 +19,7 @@ module Forem
         let(:opts) { {:size => 99} }
 
         it 'overwrites the default size' do
-          expected = "http://www.gravatar.com/avatar/#{email_hash}?s=99"
+          expected = "//www.gravatar.com/avatar/#{email_hash}?s=99"
           helper.avatar_url(email, opts).should eq(expected)
         end
       end
@@ -28,7 +28,7 @@ module Forem
         let(:opts) { {:default => 'mm'} }
 
         it 'includes a default parameter' do
-          expected = "http://www.gravatar.com/avatar/#{email_hash}?d=mm&s=60"
+          expected = "//www.gravatar.com/avatar/#{email_hash}?d=mm&s=60"
           helper.avatar_url(email, opts).should eq(expected)
         end
       end
@@ -37,7 +37,7 @@ module Forem
         let(:opts) { {:default => 'mm', :size => '99'} }
 
         it 'includes a default parameter and overwrites the default size' do
-          expected = "http://www.gravatar.com/avatar/#{email_hash}?d=mm&s=99"
+          expected = "//www.gravatar.com/avatar/#{email_hash}?d=mm&s=99"
           helper.avatar_url(email, opts).should eq(expected)
         end
       end