Project

General

Profile

Actions

Feature #15311

closed

Add an indication to admin/info whether or not ImageMagick convert is available

Added by Mischa The Evil over 10 years ago. Updated over 10 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Administration
Target version:
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

Actions #1

Updated by Mischa The Evil over 10 years ago

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:

rm15311-add_convert_availability_indication.png

Actions #2

Updated by Jean-Philippe Lang over 10 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

Also available in: Atom PDF