Basic prototype of concerto#909 - button placement on settings page needs CSS help...badly

Author: augustf

app/assets/images/elements/green_button.svg

@@ -10,6 +10,18 @@ def initiate_restart
     restart_webserver()
     redirect_to :action => :show
   end
+  
+  def config_check
+    @imagemagick_installed = command?("convert")
+    @rmagick_installed = Gem::Specification::find_all_by_name('rmagick').any?
+    @not_using_sqlite = ActiveRecord::Base.configurations[Rails.env]['adapter'] != "sqlite3"
+    @not_world_writable = !(File.stat(Rails.root).world_writable?)
+    #Using Ruby methods to stat a directory and convert the mod bit to the familiar 3-digit octal
+    #The logic here and in the view assumes a *nix system - no idea what other posix systems will return
+    @rails_root_perms = File.stat(Rails.root).mode.to_s(8)[-3,3] == "700" #should be 700 on a shared box
+    @rails_log_perms = File.stat(Rails.root.join('log')).mode.to_s(8)[-3,3] == "600" #should be 600 on a shared box
+    @webserver_ownership = File.stat(Rails.root).owned?
+  end
 
   # get a hash of concerto_config keys and values and update them using the ConcertoConfig setter
   # PUT /settings

app/assets/images/elements/red_button.svg

@@ -25,4 +25,46 @@ def generate_tabs configs
     headers << '</ul>'
     headers.join.html_safe
   end
+  
+  def test_symbol(test_boolean)
+    image_tag("elements/#{test_boolean ? "green" : "red"}_button.svg", :width => 15, :height => 15)
+  end
+    
+  def imagemagick_text
+    "ImageMagick is #{@imagemagick_installed ? "installed" : "not installed"} at #{which("convert")}"
+  end
+  
+  def rmagick_text
+    "RMagick is #{@rmagick_installed ? "installed" : "not installed"}"
+  end
+  
+  def world_writable_text
+    @not_world_writable ? "The Concerto directory is not world writable" : "The Concerto directory is world writable"
+  end
+  
+  def root_perms_text
+    "#{Rails.root} has permission #{File.stat(Rails.root).mode.to_s(8)[-3,3]}" + "#{@rails_root_perms == 700 ? "" : " instead of 700"}"
+  end
+  
+  def rails_log_text
+    "#{Rails.root.join('log')} has permission #{File.stat(Rails.root.join('log')).mode.to_s(8)[-3,3]}" + "#{@rails_log_perms == 600 ? "" : " instead of 600"}"
+  end
+  
+  def webserver_ownership_text
+    Rails.root.to_s + (@webserver_ownership ? " is owned by the webserver" : " is not owned by the webserver")
+  end
+  
+  def db_text
+    adapter = ActiveRecord::Base.configurations[Rails.env]['adapter']
+    if @not_using_sqlite == false && (system_has_mysql? && system_has_postgres?)
+      "#{adapter} is your current database adapter. It's not recommended for production deployments and both MySQL (#{mysql_location}) and Postgresql (#{postgres_location}) are available on your system."
+    elsif @not_using_sqlite == false && system_has_mysql?
+      "#{adapter} is your current database adapter. It's not recommended for production deployments and MySQL is installed at #{mysql_location}"
+    elsif @not_using_sqlite == false && system_has_postgres?
+      "#{adapter} is your current database adapter. It's not recommended for production deployments and Postgresql is installed at #{postgres_location}"
+    else
+      "#{adapter} is properly installed"
+    end
+  end
+  
 end

app/controllers/concerto_config_controller.rb

@@ -0,0 +1,24 @@
+<section class="viewblock">
+  <header class="viewblock-header">
+    <h1 class="default-padding">
+      <%= t('.configuration_check') %>
+    </h1>
+    <div class="viewblock-header_right">
+
+    </div>
+
+  </header>
+  <div class="viewblock-cont">
+
+    <div class="subnav clearfix">
+      <p><%= test_symbol(@imagemagick_installed) %><%=imagemagick_text%></p>
+      <p><%= test_symbol(@rmagick_installed) %><%= rmagick_text %></p>
+      <p><%= test_symbol(@not_using_sqlite) %><%= db_text %></p>
+      <p><%= test_symbol(@not_world_writable) %><%= world_writable_text %></p>
+      <p><%= test_symbol(@rails_root_perms) %><%= root_perms_text %></p>
+      <p><%= test_symbol(@rails_log_perms) %><%= rails_log_text %></p>
+      <p><%= test_symbol(@webserver_ownership) %><%= webserver_ownership_text %></p>      
+    </div>
+
+  </div>
+</section>

app/helpers/concerto_config_helper.rb

@@ -3,7 +3,7 @@
 <section class="viewblock">
   <header class="viewblock-header">
     <h1 class="default-padding">
-      <%= t('.edit_settings') %>
+      <%= t('.edit_settings') %><%= button_to t('.check_configuration'), {:action => :config_check}, {:method => :get } %>
     </h1>
     <div class="viewblock-header_right">
       <div class="button-padding">

app/views/concerto_config/config_check.html.erb

@@ -36,4 +36,7 @@ en:
       edit_settings: "Edit Global Settings"
       restart_confirm: "Restarting the webserver may take a 10 to 20 seconds. Are you sure you'd like to restart?"
       restart_webserver: "Restart Webserver"
+      check_configuration: "Check Configuration"
       save_settings: "Save Settings"
+    config_check:
+      configuration_check: "Configuration Check"

app/views/concerto_config/show.html.erb

@@ -111,8 +111,9 @@ def matches?(request)
   resources :html_texts, :controller => :contents, :except => [:index, :show], :path => "content"
   resources :client_times, :controller => :contents, :except => [:index, :show], :path => "content"
 
-  resource :concerto_config, :controller => :concerto_config, :only => [:show, :update], :path => "settings" do
+  resource :concerto_config, :controller => :concerto_config, :only => [:show, :update,], :path => "settings" do
     post :initiate_restart
+    get :config_check
   end
 
   # This is the catch-all path we use for people who type /content when they

config/locales/views/concerto_config/en.yml

@@ -29,3 +29,11 @@ def system_has_mysql?
 def system_has_postgres?
   Gem::Specification::find_all_by_name('pg').any? || command?('pg_config')
 end
+
+def mysql_location
+  which('mysql_config').split('/bin').first
+end
+
+def postgres_location
+  which('pg_config').split('/bin').first
+end