[browsermedia#623] Make path automatically detected

If a file_picker needs to render a path field, it will.

Author: peakpg

app/inputs/file_picker_input.rb

@@ -4,7 +4,6 @@
 # @param [Hash] options
 # @option options [String] :label (method) If no label is specified, the human readable name for method will be used.
 # @option options [String] :hint (blank) Helpful tips for the person entering the field, appears blank if nothing is specified.
-# @option options [Boolean] :edit_path (false) If true, render a text field to allow users to edit path for this file.
 # @option options [Boolean] :edit_section (false) If true, render a select box which allows users to choose which section this attachment should be placed in.
 class FilePickerInput < SimpleForm::Inputs::Base
 
@@ -13,7 +12,7 @@ def input
     object.ensure_attachment_exists if object.respond_to?(:ensure_attachment_exists)
 
     html = ""
-    if options[:edit_section]
+    if render_section_picker?
       sections = sections_with_full_paths
       sections.each do |s|
         html << template.tag(:span, :class => "section_id_map", style: 'display: hidden', :data => {:id => s.id, :path => s.prependable_path})
@@ -22,10 +21,10 @@ def input
     @builder.simple_fields_for :attachments do |a|
       html << a.hidden_field("attachment_name", value: attribute_name.to_s)
       html << a.file_field(:data, input_html_options.merge('data-purpose' => "cms_file_field"))
-      if options[:edit_section]
+      if render_section_picker?
         html << a.input(:section_id, collection: sections, label_method: :full_path, include_blank: false, label: "Section", input_html: {'data-purpose' => "section_selector"})
       end
-      if options[:edit_path]
+      if render_path_input?
         klass = object.new_record? ? "suggest_file_path" : "keep_existing_path"
         html << a.input(:data_file_path, label: "Path", input_html: {class: klass})
       end
@@ -34,6 +33,15 @@ def input
 
   end
 
+  def render_path_input?
+    object.respond_to?(:set_attachment_path)
+  end
+
+  def render_section_picker?
+    #object.respond_to? :set_attachment_section
+    options[:edit_section]
+  end
+
 
   def sections_with_full_paths
     root = Cms::Section.root.first

app/views/cms/file_blocks/_form.html.erb

@@ -1,3 +1,3 @@
 <%= f.input :name %>
-<%= f.input :file, hint: "Select a file to upload.", as: :file_picker, edit_path: true, edit_section: true %>
+<%= f.input :file, hint: "Select a file to upload.", as: :file_picker, edit_section: true %>
 <%= f.input :tag_list, as: :tag_list, label: "Tags", input_html: {autocomplete: 'off'}, hint: "A space separated list of tags." %>

app/views/cms/image_blocks/_form.html.erb

@@ -1,3 +1,3 @@
 <%= f.input :name %>
-<%= f.input :file, :label => "Image", :hint=>"Select a file to upload.", as: :file_picker, edit_path: true, edit_section: true %>
+<%= f.input :file, :label => "Image", :hint=>"Select a file to upload.", as: :file_picker,  edit_section: true %>
 <%= f.input :tag_list, as: :tag_list, label: "Tags", input_html: {autocomplete: 'off'}, hint: "A space separated list of tags." %>

lib/cms/behaviors/attaching.rb

@@ -140,7 +140,8 @@ def validates_attachment_content_type(name, options = {})
           end
         end
 
-        # Define at :set_attachment_path if you would like to override the way file_path is set
+        # Define the #set_attachment_path method if you would like to override the way file_path is set.
+        # A path input will be rendered for content types having #set_attachment_path.
         def handle_setting_attachment_path
           if self.respond_to? :set_attachment_path
             set_attachment_path