CRUD for Questionnaire

app/controllers/item_types_controller.rb

@@ -0,0 +1,7 @@
+class ItemTypesController < ApplicationController
+  # GET /item_types
+  def index
+    item_types = ItemType.all
+    render json: item_types
+  end
+end

app/controllers/items_controller.rb

@@ -0,0 +1,16 @@
+# app/controllers/items_controller.rb
+class ItemsController < ApplicationController
+  def index
+    questionnaire = Questionnaire.find(params[:questionnaire_id])
+
+    Rails.logger.info "Items for Questionnaire #{questionnaire.id}:"
+    questionnaire.items.each do |item|
+      Rails.logger.info item.attributes.inspect
+    end
+
+    items_json = questionnaire.items.as_json
+    Rails.logger.info "JSON being rendered: #{items_json.inspect}"
+
+    render json: items_json
+  end
+end

app/controllers/questionnaire_types_controller.rb

@@ -0,0 +1,7 @@
+class QuestionnaireTypesController < ApplicationController
+  # GET /questionnaire_types
+  def index
+    questionnaire_types = QuestionnaireType.all
+    render json: questionnaire_types
+  end
+end

app/controllers/questionnaires_controller.rb

@@ -21,39 +21,46 @@ def show
   # Create method creates a questionnaire and returns the JSON object of the created questionnaire
   # POST on /questionnaires
   # Instructor Id statically defined since implementation of Instructor model is out of scope of E2345.
-  def create
-    begin
-      @questionnaire = Questionnaire.new(questionnaire_params)
-      @questionnaire.display_type = sanitize_display_type(@questionnaire.questionnaire_type)
-      @questionnaire.save!
-      render json: @questionnaire, status: :created and return
-    rescue ActiveRecord::RecordInvalid
-      render json: $ERROR_INFO.to_s, status: :unprocessable_entity
-    end
+ def create
+  begin
+    @questionnaire = Questionnaire.new(questionnaire_params)
+    @questionnaire.display_type = sanitize_display_type(@questionnaire.questionnaire_type)
+    @questionnaire.save!
+    render json: @questionnaire, status: :created and return
+  rescue ActiveRecord::RecordInvalid => e
+    render json: { errors: e.record.errors.full_messages }, status: :unprocessable_entity
+
+  rescue => e
+    render json: { error: e.message, backtrace: e.backtrace.take(5) }, status: :internal_server_error
   end
+end
 
   # Destroy method deletes the questionnaire object with id- {:id}
   # DELETE on /questionnaires/:id
   def destroy
-    begin
-      @questionnaire = Questionnaire.find(params[:id])
-      @questionnaire.delete
-    rescue ActiveRecord::RecordNotFound
+  begin
+    @questionnaire = Questionnaire.find(params[:id])
+    @questionnaire.destroy! # ensures dependent items are removed
+  rescue ActiveRecord::RecordNotFound
       render json: $ERROR_INFO.to_s, status: :not_found and return
     end
-  end
+end
 
   # Update method updates the questionnaire object with id - {:id} and returns the updated questionnaire JSON object
   # PUT on /questionnaires/:id
 
-  def update
-    @questionnaire = Questionnaire.find(params[:id])
-    if @questionnaire.update(questionnaire_params)
-      render json: @questionnaire, status: :ok
-    else
-      render json: @questionnaire.errors.full_messages, status: :unprocessable_entity
-    end
+def update
+  @questionnaire = Questionnaire.find(params[:id])
+
+  if @questionnaire.update(questionnaire_params)
+    render json: @questionnaire, status: :ok
+  else
+    render json: { errors: @questionnaire.errors.full_messages }, status: :unprocessable_entity
   end
+rescue ActiveRecord::RecordNotFound
+  render json: { error: 'Questionnaire not found' }, status: :not_found
+end
+
   # Copy method creates a copy of questionnaire with id - {:id} and return its JSON object
   # POST on /questionnaires/copy/:id
   def copy
@@ -87,9 +94,16 @@ def toggle_access
   private
 
   def questionnaire_params
-    params.require(:questionnaire).permit(:name, :questionnaire_type, :private, :min_question_score, :max_question_score, :instructor_id)
+    params.require(:questionnaire).permit(:name, :questionnaire_type, :private,
+    :min_question_score, :max_question_score, :instructor_id,
+    items_attributes: [
+      :id, :txt, :question_type, :seq, :weight,
+      :size, :alternatives, :min_label, :max_label, :textarea_width, :textarea_height, :textbox_width, :col_names, :row_names,
+      :break_before, :_destroy, :max_value
+    ])
   end
 
+  # To match the expected format, replace a space by a for questionnaire types with 2 or more words
   def sanitize_display_type(type)
     display_type = type.split('Questionnaire')[0]
     if %w[AuthorFeedback CourseSurvey TeammateReview GlobalSurvey AssignmentSurvey BookmarkRating].include?(display_type)
@@ -98,4 +112,4 @@ def sanitize_display_type(type)
     display_type
   end
 
-end
+end

app/models/Item.rb

@@ -2,7 +2,7 @@
 
 class Item < ApplicationRecord
   before_create :set_seq
-  belongs_to :questionnaire # each item belongs to a specific questionnaire
+  belongs_to :questionnaire, optional: true
   has_many :answers, dependent: :destroy, foreign_key: 'item_id'
   attr_accessor :choice_strategy
 
@@ -18,14 +18,18 @@ def scorable?
   def scored?
     question_type.in?(%w[ScaleItem CriterionItem])
   end
-    
+
   def set_seq
     self.seq = questionnaire.items.size + 1
   end
 
   def as_json(options = {})
       super(options.merge({
-                            only: %i[txt weight seq question_type size alternatives break_before min_label max_label created_at updated_at],
+                            only: %i[
+      id txt weight seq question_type size alternatives break_before 
+      min_label max_label created_at updated_at textarea_width 
+      textarea_height textbox_width col_names row_names
+    ],
                             include: {
                               questionnaire: { only: %i[name id] }
                             }
@@ -73,4 +77,4 @@ def self.for(record)
     # Cast the existing record to the desired subclass
     klass.new(record.attributes)
   end
-end
+end

app/models/item_type.rb

@@ -0,0 +1,4 @@
+class ItemType < ApplicationRecord
+  # Validations
+  validates :name, presence: true, uniqueness: true
+end

app/models/questionnaire.rb

@@ -2,13 +2,14 @@
 
 class Questionnaire < ApplicationRecord
   belongs_to :instructor
-  has_many :items, class_name: "Item", foreign_key: "questionnaire_id", dependent: :destroy # the collection of questions associated with this Questionnaire
+  has_many :items, -> { order(:seq) }, class_name: "Item", foreign_key: "questionnaire_id", dependent: :destroy # the collection of questions associated with this Questionnaire
+  accepts_nested_attributes_for :items, allow_destroy: true 
   before_destroy :check_for_question_associations
 
   validate :validate_questionnaire
   validates :name, presence: true
   validates :max_question_score, :min_question_score, numericality: true 
-  
+
 
   # after_initialize :post_initialization
     # @print_name = 'Review Rubric'
@@ -28,7 +29,7 @@ def symbol
     def get_assessments_for(participant)
       participant.reviews
     end
-    
+
   # validate the entries for this questionnaire
   def validate_questionnaire
     errors.add(:max_question_score, 'The maximum item score must be a positive integer.') if max_question_score < 1
@@ -64,9 +65,9 @@ def self.copy_questionnaire_details(params)
     questionnaire
   end
 
-  # Check_for_question_associations checks if questionnaire has associated questions or not
+  # Check_for_question_associations checks if questionnaire has associated items or not
   def check_for_question_associations
-    if questions.any?
+    if items.any?
       raise ActiveRecord::DeleteRestrictionError.new( "Cannot delete record because dependent questions exist")
     end
   end
@@ -82,4 +83,4 @@ def as_json(options = {})
         hash['instructor'] ||= { id: nil, name: nil }
       end
   end
-end
+end

app/models/questionnaire_type.rb

@@ -0,0 +1,7 @@
+class QuestionnaireType < ApplicationRecord
+  # Validations
+  validates :name, presence: true, uniqueness: true
+
+  # Associations (if any later)
+  # has_many :questionnaires, foreign_key: :questionnaire_type, primary_key: :name
+end

config/routes.rb

@@ -69,6 +69,13 @@
         end
       end
 
+      resources :questionnaire_types, only: [:index] do
+      end
+
+      resources :item_types, only: [:index] do
+      end
+
+
       resources :questions do
         collection do
           get :types
@@ -77,6 +84,12 @@
         end
       end
 
+      # config/routes.rb
+      resources :questionnaires do
+        resources :items, only: [:index]
+      end
+
+
       resources :signed_up_teams do
         collection do
           post '/sign_up', to: 'signed_up_teams#sign_up'
@@ -149,4 +162,4 @@
           get '/:participant_id/instructor_review', to: 'grades#instructor_review'
         end
       end
-end
+end

db/migrate/20251013123456_add_textarea_width_to_items.rb

@@ -0,0 +1,7 @@
+class AddTextareaWidthToItems < ActiveRecord::Migration[8.0]
+  def change
+    add_column :items, :textarea_width, :integer
+    add_column :items, :textarea_height, :integer
+    add_column :items, :textbox_width, :integer
+  end
+end

db/migrate/20251013123457_add_cols_rows_to_items.rb

@@ -0,0 +1,6 @@
+class AddColsRowsToItems < ActiveRecord::Migration[8.0]
+  def change
+    add_column :items, :col_names, :string
+    add_column :items, :row_names, :string
+  end
+end

db/migrate/20251230123456_rename_question_types_to_item_types.rb.rb

@@ -0,0 +1,5 @@
+class RenameQuestionTypesToItemTypes < ActiveRecord::Migration[8.0]
+  def change
+    rename_table :question_types, :item_types
+  end
+end