Skip to content

Commit

Permalink
Fixes filename and extension for cloudinary
Browse files Browse the repository at this point in the history
  • Loading branch information
mathieugagne committed Mar 17, 2014
1 parent c51a58f commit e95f6b3
Show file tree
Hide file tree
Showing 4 changed files with 18 additions and 4 deletions.
5 changes: 5 additions & 0 deletions app/models/ckeditor/asset.rb
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,12 @@ def to_jq_upload
}
end

def short_name
@short_name ||= self.data.file.filename.dup.gsub(/(.*)\.#{extension}/, '\1') if data.file and data.file.filename
end

def extension
self.data_content_type ||= self.data.file.content_type
@extension ||= data_content_type.gsub(/.*\/(.*)/, '\1')
end

Expand Down
9 changes: 9 additions & 0 deletions app/models/ckeditor/picture.rb
Original file line number Diff line number Diff line change
Expand Up @@ -12,4 +12,13 @@ def pages
@pages ||= contents.map(&:contentable).map(&:page).compact.uniq
end

def filename
@filename ||=
if defined?(Cloudinary)
data_file_name.dup.gsub(/v\d{10}\/(.*)/, '\1') || data_file_name
else
data_file_name
end
end

end
2 changes: 1 addition & 1 deletion app/views/georgia/ckeditor/pictures/_picture.html.erb
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
</td>
<td class='thumbnail-cell'>
<%= link_to picture.url_content, class: 'media-link bg-transparent', rel: 'shadowbox[gallery]' do %>
<%= image_tag picture.url(:tiny), title: picture.data_file_name, class: 'media-image' %>
<%= image_tag picture.url(:tiny), title: picture.filename, class: 'media-image' %>
<% end %>
</td>
<td>
Expand Down
6 changes: 3 additions & 3 deletions app/views/georgia/media/edit.html.erb
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
<div class="col-md-9">

<h2>
<%= link_to @asset.data_file_name, @asset.url %>
<%= link_to @asset.filename, @asset.url %>
<% unless @asset.is_a?(Ckeditor::Picture) %>
<%= link_to icon_tag('link'), @asset.url, title: @asset.url, target: '_blank', class: 'btn btn-default' %>
<% end -%>
Expand All @@ -25,7 +25,7 @@
</span>
<% end -%>
</div>
<%= image_tag @asset.url(:content), title: @asset.data_file_name, class: 'media-image--inline' %>
<%= image_tag @asset.url(:content), title: @asset.filename, class: 'media-image--inline' %>
</div>
<% end -%>
</div>
Expand All @@ -36,7 +36,7 @@
</div>
<div class="panel-body">
<p>
<b>File Name</b>: <%= @asset.data_file_name %><br />
<b>File Name</b>: <%= @asset.filename %><br />
<b>Content Type</b>: <%= @asset.data_content_type %><br />
<% if @asset.is_a?(Ckeditor::Picture) %>
<b>File Size</b>: <%= number_to_human_size(@asset.data_file_size) %><br />
Expand Down

0 comments on commit e95f6b3

Please sign in to comment.