Actions
Feature #15311
closed
Add an indication to admin/info whether or not ImageMagick convert is available
Start date:
Due date:
% Done:
0%
Estimated time:
Resolution:
Fixed
Description
ImageMagick convert is now used for generating thumbnails1 which makes it (specifically ImageMagick) an optional dependency.
Inspired by #15297#note-1 and some others I think it would be good if its availability is indicated on admin/info.
1 for both issue attachments thumbnails as well as macro thumbnails
Files
Updated by Mischa The Evil over 9 years ago
- File rm15311-add_convert_availability_indication-trunk_r12240.diff rm15311-add_convert_availability_indication-trunk_r12240.diff added
- File rm15311-add_convert_availability_indication.png rm15311-add_convert_availability_indication.png added
Patch, including English and Dutch translations (EN stubs for others) attached. A test for this tiny change is omitted.
The essence of the patch:
Index: app/controllers/admin_controller.rb
===================================================================
--- app/controllers/admin_controller.rb (revision 12240)
+++ app/controllers/admin_controller.rb (working copy)
@@ -77,7 +77,8 @@
[:text_default_administrator_account_changed, User.default_admin_account_changed?],
[:text_file_repository_writable, File.writable?(Attachment.storage_path)],
[:text_plugin_assets_writable, File.writable?(Redmine::Plugin.public_directory)],
- [:text_rmagick_available, Object.const_defined?(:Magick)]
+ [:text_rmagick_available, Object.const_defined?(:Magick)],
+ [:text_convert_available, Redmine::Thumbnail.convert_available?]
]
end
end
This changes admin/info like:
Updated by Jean-Philippe Lang over 9 years ago
- Category changed from UI to Administration
- Status changed from New to Closed
- Target version set to 2.4.0
- Resolution set to Fixed
Thanks for patch, it's committed in r12278.
Actions