first shot at using formtastic

Author: Mark Yoon

Rakefile

@@ -11,6 +11,7 @@ begin
     gem.authors = ["Brian Chamberlain", "Mark Yoon"]
     gem.add_dependency 'haml'
     gem.add_dependency 'fastercsv'
+    gem.add_dependency 'formtastic'
     gem.add_development_dependency "yard", ">= 0"
     # gem is a Gem::Specification... see http://www.rubygems.org/read/chapter/20 for additional settings
   end

app/helpers/surveyor_helper.rb

@@ -1,5 +1,4 @@
 module SurveyorHelper
-
   # Layout: stylsheets and javascripts
   def surveyor_includes
     surveyor_stylsheets + surveyor_javascripts    
@@ -34,7 +33,28 @@ def next_section
     # refactored to use copy in memory instead of making extra db calls
     @sections.last == @section ? submit_tag(t('surveyor.click_here_to_finish'), :name => "finish") : submit_tag(t('surveyor.next_section'), :name => "section[#{@sections[@sections.index(@section)+1].id}]")
   end
-  
+
+  # new methods
+  def q_text(obj)
+    @n ||= 0
+    return image_tag(obj.text) if obj.is_a?(Question) and obj.display_type == "image"
+    return obj.text if obj.is_a?(Question) and (obj.dependent? or obj.display_type == "label" or obj.part_of_group?)
+    "#{@n += 1}) #{obj.text}"
+  end
+  def response_for(response_set, question, answer = nil)
+    return nil unless response_set && question && question.id
+    if answer.nil?
+      result = response_set.responses.detect{|r| r.question_id == question.id}
+      result.blank? ? response_set.responses.build(:question => question) : result
+    elsif
+      result = response_set.responses.detect{|r| r.question_id == question.id && r.answer_id == answer.id}
+      result.blank? ? response_set.responses.build(:question => question, :answer => answer) : result
+    end
+  end
+  def response_idx(increment = true)
+    @rc ||= 0
+    (increment ? @rc += 1 : @rc).to_s
+  end
   # Questions
   def next_number
     @n ||= 0

app/views/partials/_answer.html.haml

@@ -1,58 +1,19 @@
-- renderer = answer.renderer(question)
-- response_group ||= nil
-- hide_label ||= false
-- disabled ||= false
-- fields_for_response(response_obj, response_group) do |response_form|
-  - case renderer
-  - when :any_answer
-    = response_form.survey_check_box(:selected, :class => (answer.is_exclusive? ? "exclusive" : ""))
-    = response_form.label("selected", answer.text) unless hide_label # This is a temp fix til I figure out why the id of the checkbox is all jacked up
-  - when :any_other_and_string
-    = response_form.survey_check_box(:selected, :class => (answer.is_exclusive? ? "exclusive" : ""))
-    %label Other
-    = response_form.text_field(:other_value)
-    = response_form.text_field(:string_value)
-  - when :any_string
-    = response_form.survey_check_box(:selected, :class => (answer.is_exclusive? ? "exclusive" : ""))
-    = response_form.label(:answer_id, answer.text)
-    = response_form.text_field(:string_value, :size => 25, :maxlength => 250)
-  - when :one_answer, :one_string, :one_integer
-    - fields_for_radio(response_obj) do |radio_form|
-      = radio_form.radio_button(:answer_id, answer.id, :checked => response_obj.selected?, :title => answer.text)
-      = radio_form.label("answer_id_#{answer.id}", split_text(answer.text)[:prefix]) unless hide_label
-      - if renderer == :one_string
-        = response_form.text_field(:string_value, :size => 25, :maxlength => 250, :autocomplete => "off")
-      - if renderer == :one_integer
-        = response_form.text_field(:integer_value, :size => 5, :maxlength => 10, :autocomplete => "off")    
-      = radio_form.label("answer_id_#{answer.id}", split_text(answer.text)[:postfix]) unless hide_label
-  - when :none_date
-    = response_form.label(:datetime_value, split_text(answer.text)[:prefix], :class => "#{(disabled)? 'disabled' : 'enabled'}") unless hide_label
-    = response_form.text_field(:datetime_value)
-    = response_form.label(:datetime_value, split_text(answer.text)[:postfix], :class => "#{(disabled)? 'disabled' : 'enabled'}") unless hide_label
-  - when :none_datetime
-    = response_form.label(:datetime_value, split_text(answer.text)[:prefix], :class => "#{(disabled)? 'disabled' : 'enabled'}") unless hide_label
-    = response_form.datetime_select(:datetime_value, {:include_blank => true})
-    = response_form.label(:datetime_value, split_text(answer.text)[:postfix], :class => "#{(disabled)? 'disabled' : 'enabled'}") unless hide_label
-  - when :none_float
-    = response_form.label(:float_value, split_text(answer.text)[:prefix]) unless hide_label
-    = response_form.text_field(:float_value, :size => 5, :maxlength => 10, :autocomplete => "off")
-    = response_form.label(:float_value, split_text(answer.text)[:postfix]) unless hide_label
-  - when :none_integer
-    = response_form.label(:integer_value, split_text(answer.text)[:prefix], :class => "#{(disabled)? 'disabled' : 'enabled'}") unless hide_label
-    = response_form.text_field(:integer_value, :size => 5, :maxlength => 10, :autocomplete => "off", :disabled => disabled)
-    = response_form.label(:integer_value, split_text(answer.text)[:postfix], :class => "#{(disabled)? 'disabled' : 'enabled'}") unless hide_label
-  - when :none_string
-    = response_form.label(:string_value, split_text(answer.text)[:prefix], :class => "#{(disabled)? 'disabled' : 'enabled'}") unless hide_label
-    = response_form.text_field(:string_value, :size => 25, :maxlength => 250, :autocomplete => "off", :disabled => disabled)
-    = response_form.label(:string_value, split_text(answer.text)[:postfix], :class => "#{(disabled)? 'disabled' : 'enabled'}") unless hide_label
-  - when :none_text
-    = response_form.label(:text_value, split_text(answer.text)[:prefix]) unless hide_label
-    = response_form.text_area(:text_value, :cols => 80, :rows => 10)
-    = response_form.label(:text_value, split_text(answer.text)[:postfix]) unless hide_label
-  - when :none_time
-    = response_form.label(:datetime_value, split_text(answer.text)[:prefix], :class => "#{(disabled)? 'disabled' : 'enabled'}") unless hide_label
-    = response_form.time_select(:datetime_value,{ :include_blank => true })
-    = response_form.label(:datetime_value, split_text(answer.text)[:postfix], :class => "#{(disabled)? 'disabled' : 'enabled'}") unless hide_label
-  - else
-    = answer.text
-  
+-# TODO: class for a.exclusive, disabled, hide_label, prefix/post labels
+- r = response_for(@response_set, q, a)
+- i = response_idx(q.pick != "one") # argument will be false (don't increment i) if we're on radio buttons
+- f.semantic_fields_for i, r do |g|
+  = g.input :question_id, :as => :hidden unless q.pick == "one" # don't repeat question_id if we're on radio buttons
+  - case q.pick
+  - when "one"
+    = g.input :answer_id, :as => :radio, :collection => [[a.text, a.id]], :label => false
+    = g.input :string if a.response_class == 'string'
+    = g.input :integer if a.response_class == 'integer'
+  - when "any"
+    = g.input :answer_id, :as => :check_boxes, :collection => [[a.text, a.id]], :label => false
+    = g.input :other_value, :label => false if a.response_class == "other_and_string"
+    = g.input :string_value, :label => false if %w(string other_and_string).include? a.response_class
+  - when "none"
+    - if %w(date datetime time float integer string text time).include? a.response_class
+      = g.input a.response_class.to_sym, :label => a.text
+    - else
+      = a.text

app/views/partials/_dependents.html.haml

@@ -0,0 +1,5 @@
+#dependents
+  .title Follow-up questions from your answers on the previous page
+  -# @dependents.each_with_index do |question, index|
+    = dependency_explanation_helper(question, @response_set)
+    = render question.custom_renderer || "/partials/question", :question => question, :response_set => @response_set, :number => "D#{index+1}"

app/views/partials/_question.html.haml

@@ -1,72 +1,20 @@
-- renderer = question.renderer(question_group ||= nil)
-- case renderer
-- when :label
-  - div_for question, :class => "label #{question.css_class(response_set)}" do
-    %span.text= question.text
-    %span.help= question.help_text
-- when :image
-  .image= image_tag(question.text)
-- when :dropdown, :slider
-  - if renderer == :slider
-    %script{:type => "text/javascript"}= "$(function(){$('#slider_#{question.id}').accessibleUISlider({width: 400, labels:#{question.answers.size} }).hide();});"
-  - div_for question, :class => question.css_class(response_set) do
-    .number= next_number unless question.dependent?
-    %fieldset
-      %legend{:style => "display:inline;"}
-        %span.text= question.text
-        %span.help= question.help_text
-        = hidden_field_tag("responses[#{question.id}][question_id]", question.id)
-      %ol.answers
-        %li.answer
-          - options = question.answers.collect{|a| "<option #{ (response_set.response_for(question.id, a.id).selected?)? "selected='selected'" : nil } value ='#{a.id}'>#{a.text}</option>" }
-          = renderer == :slider ? select_tag("responses[#{question.id}][answer_id]", options, {:id => "slider_#{question.id}"}) : select_tag("responses[#{question.id}][answer_id]", ["<option value=''>#{t('surveyor.Select_one')}</option>"].concat(options))
-- when :grid_default
-  - reset_cycle("col")
-  - content_tag_for :tr, question, :class => question.css_class(response_set) do
-    %th.question_prefix
-      %span.text= split_text(question.text)[:prefix]
-      %span.help= question.help_text
-      = hidden_field_tag("responses[#{question.id}][question_id]", question.id)
-    - question.answers.each do |answer|
-      %td{:class => "#{cycle("column_highlight", "", :name => "col")}"}
-        .answer{:class => answer.custom_class}= render(:partial => answer.custom_renderer || "/partials/answer", :locals => {:answer => answer, :question => question, :response_obj => @response_set.response_for(question.id, answer.id), :hide_label => true})
-    %th.question_postfix
-      %span.text= split_text(question.text)[:postfix]
-- when :repeater_default, :repeater_dropdown
-  - disabled ||= false
-  - div_for question, :class => "#{disabled ? 'dis' : 'en'}abled #{question.css_class(response_set)}" do
-    %span.text= question.text
-    %span.help= question.help_text
-    = hidden_field_tag("response_groups[#{question.id}][#{response_group}][response_group]", response_group)
-    = hidden_field_tag("response_groups[#{question.id}][#{response_group}][question_id]", question.id)
-    %br
-    %ol.answers
-      - if renderer == :repeater_default  
-        - question.answers.each do |answer|
-          - content_tag_for :li, answer, :class => answer.custom_class do
-            = render(:partial => answer.custom_renderer || "/partials/answer", :locals => {:answer => answer, :question => question, :response_obj => @response_set.response_for(question.id, answer.id, response_group), :response_group => response_group, :hide_label => true, :disabled => disabled})
-      - else
-        %li.answer
-          - options = question.answers.collect{|a| "<option #{ (response_set.response_for(question.id, a.id, response_group).selected?)? "selected='selected'" : nil } value ='#{a.id}'>#{a.text}</option>" }
-          = select_tag("response_groups[#{question.id}][#{response_group}][answer_id]", ["<option value=''>Select one...</option>"].concat(options), :disabled => disabled)
-- when :inline_default, :inline_dropdown
-  - div_for question, :class => "inline #{question.css_class(response_set)}" do
-    = hidden_field_tag("responses[#{question.id}][question_id]", question.id)
-    - if renderer == :inline_default
-      - question.answers.each do |answer|
-        .answer{:class => answer.custom_class}= render(:partial => answer.custom_renderer || "/partials/answer", :locals => {:answer => answer, :question => question, :response_obj => @response_set.response_for(question.id, answer.id), :hide_label => answer.hide_label})
-    - else
-      - options = question.answers.collect{|a| "<option #{ (response_set.response_for(question.id, a.id).selected?)? "selected='selected'" : nil } value ='#{a.id}'>#{a.text}</option>" }
-      = select_tag("responses[#{question.id}][answer_id]", ["<option value=''>Select one...</option>"].concat(options))
-- else # :default, :inline
-  - div_for question, :class => question.css_class(response_set) do
-    .number= next_number unless question.dependent?
-    %fieldset
-      %legend
-        %span.text= question.text
-        %span.help= question.help_text
-        = hidden_field_tag("responses[#{question.id}][question_id]", question.id)
-      %ol.answers{:class => renderer == :inline ? "inline" : nil}
-        - question.answers.each do |answer|
-          - content_tag_for :li, answer, :class => answer.custom_class do
-            = render(:partial => answer.custom_renderer || "/partials/answer", :locals => {:answer => answer, :question => question, :response_obj => @response_set.response_for(question.id, answer.id), :hide_label => answer.hide_label})
+-# TODO: js for slider pre/post text for question in a grid
+- renderer = q.renderer(g ||= nil)
+- f.inputs q_text(q), :id => "q_#{q.id}", :class => "q_#{renderer} #{q.css_class(@response_set)}" do
+  %span.help= q.help_text
+  - case renderer
+  - when :image, :label
+  - when :dropdown, :inline_dropdown, :slider
+    - r = response_for(@response_set, q)
+    - i = response_idx
+    - f.semantic_fields_for i, r do |g|
+      = g.input :question_id, :as => :hidden
+      = g.input :answer_id, :collection => q.answers.map{|a| [a.text, a.id]}, :label => false
+  - else # :default, :inline, :inline_default, :inline_dropdown
+    - if q.pick == "one"
+      - r = response_for(@response_set, q)
+      - i = response_idx
+      - f.semantic_fields_for i, r do |g|
+        = g.input :question_id, :as => :hidden
+    - q.answers.each do |a|
+      = render a.custom_renderer || '/partials/answer', :q => q, :a => a, :f => f

app/views/partials/_question_group.html.haml

@@ -1,39 +1,30 @@
-- renderer = question_group.renderer
-- locals = {:question_group => question_group, :response_set => response_set, :hide_label => true}
-- div_for question_group, :class => question_group.css_class(response_set) do
-  .head
-    .number= next_number
-    %span.text= question_group.text
-    %span.help= question_group.help_text
-
-  - case question_group.renderer
-  - when :grid
-    %table.grid
-      %thead
-        %tr
-          %th= " "
-          - reset_cycle("column_highlight")
-          - group_questions.first.answers.each do |answer|
-            %th{:class => "#{cycle("column_highlight", "", :name => "column_highlight")}"}= answer.text
-      %tbody
-        - group_questions.each_with_index do |question, i|
-          = render(:partial => question.custom_renderer || "/partials/question", :locals => locals.merge({:question => question}))
-          - if i%10 == 9
-            %th= " "
-            - reset_cycle("column_highlight")
-            - group_questions.first.answers.each do |answer|
-              %th{:class => "#{cycle("column_highlight", "", :name => "column_highlight")}"}= answer.text
-  - when :repeater
-    %ul.repeater
-      - (response_set.count_group_responses(group_questions) + 1).times do |group_number|
-        %li.repeater
-          - group_questions.each do |question|
-            = render(:partial => question.custom_renderer || "/partials/question", :locals => locals.merge({:question => question, :response_group => group_number, :disabled => false}))
-      %li.repeater
-        - group_questions.each do |question|
-          = render(:partial => question.custom_renderer || "/partials/question", :locals => locals.merge({:question => question, :response_group => 9999, :disabled => true}))
-        = submit_tag("<= add row", :name => "section[#{group_questions.first.survey_section_id}]question_group_#{question_group.id}", :class => "add_row")
-  - else # :inline
-    %ul
-      - group_questions.each do |question|
-        %li{:class => question_group.renderer == :inline ? "inline" : nil}= render(:partial => question.custom_renderer || "/partials/question", :locals => locals.merge({:question => question}))
+-# TODO: inline, disabled
+- renderer = g.renderer
+- f.inputs q_text(g), :id => "g_#{g.id}", :class => "g_#{renderer} #{g.css_class(@response_set)}" do
+  %li.help= g.help_text
+  - case renderer
+    - when :grid
+      %li
+        %table
+          %tbody
+            - qs.each_slice(10) do |ten_questions|
+              -# header row every 10
+              %tr
+                %th  
+                - ten_questions.first.answers.each do |a|
+                  %th= a.text
+              - ten_questions.each_with_index do |q, i|
+                %tr{:id => "q_#{q.id}", :class => "q_#{renderer} #{q.css_class(@response_set)}"}
+                  %th= q.text
+                  - q.answers.each do |a|
+                    %td= render a.custom_renderer || '/partials/answer', :g => g, :q => q, :a => a, :f => f
+    - when :repeater
+      - (@response_set.count_group_responses(qs) + 1).times do |rg|
+        - qs.each do |q|
+          = render q.custom_renderer || "/partials/question", :g => g, :rg => rg, :q => q, :f => f
+      - qs.each do |q|
+        = render q.custom_renderer || "/partials/question", :g => g, :q => q, :f => f, :disabled => true
+      = submit_tag("<= add row", :name => "s[#{@section.id}]g_#{g.id}", :class => "add_row")
+    - else # :inline
+      - qs.each do |q|
+        = render q.custom_renderer || "/partials/question", :g => g, :q => q, :f => f

app/views/partials/_section.html.haml

@@ -1,13 +1,13 @@
 - div_for @section, :class => @section.custom_class do
   %span.title= @section.title
-  .questions
-    - group_questions ||= []
-    - @questions.each_with_index do |question, index|
-      - unless question.part_of_group?
-        = render(:partial => question.custom_renderer || "/partials/question", :locals => {:question => question, :response_set => @response_set})
-      - else # gather up the group questions
-        - group_questions << question
-        - if (index + 1 >= @questions.size) or (question.question_group_id != @questions[index + 1].question_group_id)
-          - # this is the last question of the section, or the group
-          = render(:partial => question.question_group.custom_renderer || "/partials/question_group", :locals => {:question_group => question.question_group, :response_set => @response_set, :group_questions => group_questions})
-          - group_questions = []
+  = hidden_field_tag :current_section_id, @section.id
+  - qs ||= []
+  - (questions = @section.questions).each_with_index do |q, i|
+    - if q.part_of_group?
+      - qs << q # gather up the group questions
+      - if (i+1 >= questions.size) or (q.question_group_id != questions[i+1].question_group_id)
+        - # this is the last question of the section, or the group
+        = render q.question_group.custom_renderer || "/partials/question_group", :g => q.question_group, :qs => qs, :f => f
+        - qs = []
+    - else # gather up the group questions
+      = render q.custom_renderer || "/partials/question", :q => q, :f => f

app/views/partials/_section_menu.html.haml

@@ -0,0 +1,5 @@
+#menu
+  = t('surveyor.sections')
+  %ul
+    - @sections.each do |s|
+      %li{:class => ("active" if s == @section)}= menu_button_for(s)