We don't need all crudify stuff for form generated settings controller so write edit and update actions ourselves.

Author: ugisozols

Diff for: core/lib/generators/refinery/form/templates/app/controllers/refinery/namespace/admin/settings_controller.rb.erb

@@ -3,46 +3,40 @@ module Refinery
     module Admin
       class SettingsController < Refinery::AdminController
 
-        crudify :'refinery/setting',
-                :title_attribute => 'name',
-                :order => 'name ASC',
-                :redirect_to_url => 'refinery.<%= namespacing.underscore %>_admin_<%= plural_name %>_path'
+        before_filter :check_setting, :only => [:edit, :update]
+        after_filter :save_subject_for_confirmation, :save_message_for_confirmation, :only => :update
 
-        before_filter :redirect_back_to_<%= plural_name %>?, :only => [:index]
-        before_filter :set_url_override?, :only => [:edit, :update]
-        after_filter :save_subject_for_confirmation?, :only => [:create, :update]
-        around_filter :rewrite_flash?, :only => [:create, :update]
+        def edit
+          @setting = Refinery::Setting.find(params[:id])
+        end
 
-      protected
-        def rewrite_flash?
-          yield
+        def update
+          @setting = Refinery::Setting.find(params[:id])
 
-          flash[:notice] = flash[:notice].to_s.gsub(/(\'.*\')/) {|m| m.titleize}.gsub('<%= class_name %> ', '')
-        end
+          if @setting.update_attributes(params[:setting])
+            flash[:notice] = t('refinery.crudify.updated', :what => @setting.name.gsub("<%= singular_name %>_", "").titleize)
 
-        def save_subject_for_confirmation?
-          Refinery::<%= namespacing %>::Setting.confirmation_subject = params[:subject] if params.keys.include?('subject')
+            unless request.xhr? or from_dialog?
+              redirect_back_or_default(refinery.<%= namespacing.underscore %>_admin_<%= plural_name %>_path)
+            else
+              render :text => "<script type='text/javascript'>parent.window.location = '#{refinery.<%= namespacing.underscore %>_admin_<%= plural_name %>_path}';</script>"
+            end
+          end
         end
 
-        def redirect_back_to_<%= plural_name %>?
-          redirect_to refinery.<%= namespacing.underscore %>_admin_<%= plural_name %>_path
-        end
+      protected
+        def check_setting
+          setting = params[:id].gsub("<%= singular_name %>_", "")
 
-        def set_url_override?
-          @url_override = refinery.<%= namespacing.underscore %>_admin_setting_url(@setting, :dialog => from_dialog?)
+          Refinery::<%= namespacing %>::Setting.send(setting) if Refinery::<%= namespacing %>::Setting.respond_to?(setting)
         end
 
-        def find_setting
-          # ensure that we're dealing with the name of the setting, not the id.
-          if params[:id].to_s.to_i.to_s == params[:id]
-            params[:id] = Refinery::Setting.find(params[:id]).name.to_s
-          end
+        def save_subject_for_confirmation
+          Refinery::<%= namespacing %>::Setting.confirmation_subject = params[:subject] if params.keys.include?('subject')
+        end
 
-          # prime the setting first, if it's valid.
-          if Refinery::<%= namespacing %>::Setting.methods.map(&:to_sym).include?(params[:id].to_s.gsub('<%= singular_name %>_', '').to_sym)
-            Refinery::<%= namespacing %>::Setting.send(params[:id].to_s.gsub('<%= singular_name %>_', '').to_sym)
-          end
-          @setting = Refinery::Setting.find_by_name(params[:id])
+        def save_message_for_confirmation
+          Refinery::<%= namespacing %>::Setting.confirmation_message = params[:message] if params.keys.include?('message')
         end
 
       end

Diff for: core/lib/generators/refinery/form/templates/app/views/refinery/namespace/admin/settings/_confirmation_email_form.html.erb

@@ -1,4 +1,4 @@
-<%%= form_for([refinery, :admin, @setting], :url => (@url_override || @url)) do |f| %>
+<%%= form_for @setting, :url => refinery.<%= namespacing.underscore %>_admin_setting_path do |f| %>
 
   <p>
     <%%= t('.explanation') %>

Diff for: core/lib/generators/refinery/form/templates/app/views/refinery/namespace/admin/settings/_notification_recipients_form.html.erb

@@ -1,4 +1,4 @@
-<%%= form_for([refinery, :admin, @setting], :url => (@url_override || @url)) do |f| %>
+<%%= form_for @setting, :url => refinery.<%= namespacing.underscore %>_admin_setting_path do |f| %>
 
   <div class='field'>
     <span class='label_with_help'>

Diff for: core/lib/generators/refinery/form/templates/config/routes.rb.erb

@@ -1,4 +1,4 @@
-Refinery::Core::Engine.routes.append do
+Refinery::Core::Engine.routes.draw do
   # Frontend routes
   namespace :<%= namespacing.underscore %> do
     resources :<%= plural_name %><%= ", :path => ''" if namespacing.underscore == plural_name %>, :only => [:new, :create] do
@@ -19,7 +19,7 @@ Refinery::Core::Engine.routes.append do
           get :toggle_spam
         end
       end<% end %>
-      resources :settings
+      resources :settings, :only => [:edit, :update]
     end
   end
 end