fixes avatar urls, now they can point to an absolute url or a local url.

CartoDB · Feb 11, 2016 · 4426c73 · 4426c73
1 parent 5a456a2
commit 4426c73
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 5 deletions.
diff --git a/app/models/user.rb b/app/models/user.rb
@@ -612,7 +612,7 @@ def reload_avatar
       # If the user doesn't have gravatar try to get a cartodb avatar
       if self.avatar_url.nil? || self.avatar_url == "//#{default_avatar}"
         # Only update the avatar if the user avatar is nil or the default image
-        self.avatar_url = "//#{cartodb_avatar}"
+        self.avatar_url = "#{cartodb_avatar}"
         self.this.update avatar_url: self.avatar_url
       end
     end
@@ -638,7 +638,7 @@ def avatar
   end
 
   def default_avatar
-    return "cartodb.s3.amazonaws.com/static/public_dashboard_default_avatar.png"
+    "/assets/unversioned/images/avatars/public_dashboard_default_avatar.png"
   end
 
   def gravatar(protocol = "http://", size = 128, default_image = default_avatar)

diff --git a/app/views/admin/shared/_dashboard_header.erb b/app/views/admin/shared/_dashboard_header.erb
@@ -39,7 +39,7 @@
 
         <li class="Header-settingsItem Header-settingsItem--avatar">
           <button class="UserAvatar js-settings-dropdown">
-            <%= image_tag current_user.avatar, :class => 'UserAvatar-img UserAvatar-img--medium' %>
+              <img src="<%= current_user.avatar %>" class='UserAvatar-img UserAvatar-img--medium' />
           </button>
         </li>
       </ul>

diff --git a/app/views/admin/shared/_private_header.erb b/app/views/admin/shared/_private_header.erb
@@ -43,10 +43,10 @@
         <% end %>
         <li class="Header-settingsItem Header-settingsItem--avatar">
           <button class="UserAvatar js-settings-dropdown">
-            <%= image_tag current_user.avatar, :class => 'UserAvatar-img UserAvatar-img--medium' %>
+            <img src="<%= current_user.avatar %>" class='UserAvatar-img UserAvatar-img--medium' />
           </button>
         </li>
       </ul>
     </div>
   </div>
-</div>
+</div>
diff --git a/lib/build/tasks/copy.js b/lib/build/tasks/copy.js
@@ -115,6 +115,14 @@
             dest: '<%= assets_dir %>/images/'
           },
 
+          // avatars should be placed in a unversioned folder
+          {
+            expand: true,
+            cwd: "app/assets/images/avatars/",
+            src: ['**/*'],
+            dest: '<%= root_assets_dir %>/unversioned/images/avatars/'
+          },
+
             // CartoDB.js images
 
           {