mirror of
https://github.com/redmine/redmine.git
synced 2025-11-15 17:56:03 +01:00
Thumbnails in issue history (#13429).
git-svn-id: http://svn.redmine.org/redmine/trunk@14731 e93f8b46-1217-0410-a6f0-8f06a7374b81
This commit is contained in:
@@ -18,6 +18,13 @@
|
|||||||
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
|
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
|
||||||
|
|
||||||
module JournalsHelper
|
module JournalsHelper
|
||||||
|
|
||||||
|
# Returns the attachments of a journal that are displayed as thumbnails
|
||||||
|
def journal_thumbnail_attachments(journal)
|
||||||
|
ids = journal.details.select {|d| d.property == 'attachment' && d.value.present?}.map(&:prop_key)
|
||||||
|
ids.any? ? Attachment.where(:id => ids).select(&:thumbnailable?) : []
|
||||||
|
end
|
||||||
|
|
||||||
def render_notes(issue, journal, options={})
|
def render_notes(issue, journal, options={})
|
||||||
content = ''
|
content = ''
|
||||||
editable = User.current.logged? && (User.current.allowed_to?(:edit_issue_notes, issue.project) || (journal.user == User.current && User.current.allowed_to?(:edit_own_issue_notes, issue.project)))
|
editable = User.current.logged? && (User.current.allowed_to?(:edit_issue_notes, issue.project) || (journal.user == User.current && User.current.allowed_to?(:edit_own_issue_notes, issue.project)))
|
||||||
|
|||||||
@@ -13,6 +13,13 @@
|
|||||||
<li><%= string %></li>
|
<li><%= string %></li>
|
||||||
<% end %>
|
<% end %>
|
||||||
</ul>
|
</ul>
|
||||||
|
<% if Setting.thumbnails_enabled? && (thumbnail_attachments = journal_thumbnail_attachments(journal)).any? %>
|
||||||
|
<div class="thumbnails">
|
||||||
|
<% thumbnail_attachments.each do |attachment| %>
|
||||||
|
<div><%= thumbnail_tag(attachment) %></div>
|
||||||
|
<% end %>
|
||||||
|
</div>
|
||||||
|
<% end %>
|
||||||
<% end %>
|
<% end %>
|
||||||
<%= render_notes(issue, journal, :reply_links => reply_links) unless journal.notes.blank? %>
|
<%= render_notes(issue, journal, :reply_links => reply_links) unless journal.notes.blank? %>
|
||||||
</div>
|
</div>
|
||||||
|
|||||||
@@ -599,7 +599,8 @@ div.attachments span.author { font-size: 0.9em; color: #888; }
|
|||||||
|
|
||||||
div.thumbnails {margin-top:0.6em;}
|
div.thumbnails {margin-top:0.6em;}
|
||||||
div.thumbnails div {background:#fff;border:2px solid #ddd;display:inline-block;margin-right:2px;}
|
div.thumbnails div {background:#fff;border:2px solid #ddd;display:inline-block;margin-right:2px;}
|
||||||
div.thumbnails img {margin: 3px;}
|
div.thumbnails img {margin: 3px; vertical-align: middle;}
|
||||||
|
#history div.thumbnails {margin-left: 2em;}
|
||||||
|
|
||||||
p.other-formats { text-align: right; font-size:0.9em; color: #666; }
|
p.other-formats { text-align: right; font-size:0.9em; color: #666; }
|
||||||
.other-formats span + span:before { content: "| "; }
|
.other-formats span + span:before { content: "| "; }
|
||||||
|
|||||||
48
test/unit/helpers/journals_helper_test.rb
Normal file
48
test/unit/helpers/journals_helper_test.rb
Normal file
@@ -0,0 +1,48 @@
|
|||||||
|
# Redmine - project management software
|
||||||
|
# Copyright (C) 2006-2015 Jean-Philippe Lang
|
||||||
|
#
|
||||||
|
# This program is free software; you can redistribute it and/or
|
||||||
|
# modify it under the terms of the GNU General Public License
|
||||||
|
# as published by the Free Software Foundation; either version 2
|
||||||
|
# of the License, or (at your option) any later version.
|
||||||
|
#
|
||||||
|
# This program is distributed in the hope that it will be useful,
|
||||||
|
# but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||||
|
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
||||||
|
# GNU General Public License for more details.
|
||||||
|
#
|
||||||
|
# You should have received a copy of the GNU General Public License
|
||||||
|
# along with this program; if not, write to the Free Software
|
||||||
|
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
|
||||||
|
|
||||||
|
require File.expand_path('../../../test_helper', __FILE__)
|
||||||
|
|
||||||
|
class JournalsHelperTest < ActionView::TestCase
|
||||||
|
include JournalsHelper
|
||||||
|
|
||||||
|
fixtures :projects, :trackers, :issue_statuses, :issues,
|
||||||
|
:enumerations, :issue_categories,
|
||||||
|
:projects_trackers,
|
||||||
|
:users, :roles, :member_roles, :members,
|
||||||
|
:enabled_modules,
|
||||||
|
:custom_fields,
|
||||||
|
:attachments,
|
||||||
|
:versions
|
||||||
|
|
||||||
|
def test_journal_thumbnail_attachments_should_return_thumbnailable_attachments
|
||||||
|
issue = Issue.generate!
|
||||||
|
|
||||||
|
journal = new_record(Journal) do
|
||||||
|
issue.init_journal(User.find(1))
|
||||||
|
issue.attachments << Attachment.new(:file => mock_file_with_options(:original_filename => 'image.png'), :author => User.find(1))
|
||||||
|
issue.attachments << Attachment.new(:file => mock_file_with_options(:original_filename => 'foo'), :author => User.find(1))
|
||||||
|
issue.save
|
||||||
|
end
|
||||||
|
assert_equal 2, journal.details.count
|
||||||
|
|
||||||
|
thumbnails = journal_thumbnail_attachments(journal)
|
||||||
|
assert_equal 1, thumbnails.count
|
||||||
|
assert_kind_of Attachment, thumbnails.first
|
||||||
|
assert_equal 'image.png', thumbnails.first.filename
|
||||||
|
end
|
||||||
|
end
|
||||||
Reference in New Issue
Block a user