fe 2.1.9 → 2.1.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f80bfad3559c2f47bb6297a80dae9645ffe078100986ff735da80c8af903985d
-  data.tar.gz: 31df747d0f5d44fb5db832eafe67176256de53f32d7bea7bf2dcca90d64219a8
+  metadata.gz: 1b53a2e7051a1d412fad535434c300275a075e512e693d7ac4007ab00a6fedfc
+  data.tar.gz: 28a168ce6df5ed1eedda2e9523d2784a9d0e01a2526d1e8c3708b77b3cfce481
 SHA512:
-  metadata.gz: 67dc5b54f91a0c83c0c5bb2145b5d0669b8b90823341d2c67247bf83915f1bb9b06deeaa7f0ce5207982fd42dcb2b233efc91c51a53818d790db9149e3f9f143
-  data.tar.gz: b709b379290e78641f48907cea8ea7d630131c0c7c501a265e4ddb09405ae32d4d7f9b841d8f994984aac0f86a2850a7c344cfb57f1a6158bdb04237fd4a1b12
+  metadata.gz: b902c50a56960f86d77911ffb67c09c4fd7df56a2995c971b395457015ea7848038d535116d5c1c4322aa6ba13c126592b3481c05f646dd9324758cb8240f2d5
+  data.tar.gz: a1890338895c3f944a2fd777050ce161e709a2f8a4d1084965ca6e18c2b6dbfefcb4a8fa52e90a4486ebb5c09c9c8254cff0ffe3e95732396ace1812d886fb62

data/Rakefile

@@ -2,7 +2,7 @@ require 'bundler/setup'
 require 'thor'
 
 APP_RAKEFILE = File.expand_path("../spec/dummy/Rakefile", __FILE__)
-load 'rails/tasks/engine.rake' if File.exists? 'spec/dummy/Rakefile'
+load 'rails/tasks/engine.rake' if File.exist? 'spec/dummy/Rakefile'
 
 Bundler::GemHelper.install_tasks
 

data/app/controllers/fe/admin/elements_controller.rb

@@ -57,7 +57,7 @@ class Fe::Admin::ElementsController < ApplicationController
         @page_element = Fe::PageElement.create(element: @element, page: @page)
         format.js
       else
-        format.js { render action: 'error.js.erb' }
+        format.js { render 'error' }
       end
     end
   end
@@ -70,7 +70,7 @@ class Fe::Admin::ElementsController < ApplicationController
       if @element.update(element_params)
         format.js
       else
-        format.js { render action: 'error.js.erb' }
+        format.js { render 'error' }
       end
     end
   end

data/app/models/fe/question_sheet.rb

@@ -85,10 +85,10 @@ module Fe
       # set page conditional_id values to new ids based on old_id
       question_sheet.all_elements.each do |el|
         if el.conditional_type != "Fe::Page" && el.conditional_id.present?
-          # noop
+          el.update(conditional_id: question_sheet.element_id_mappings[el.conditional_id])
         end
 
-        # note that conditional elements are already translated to new ids in the element import so no need to do it here
+        # handle page conditionals separately since they use page.old_id instead of element_id_mappings
         if el.conditional_type == "Fe::Page" && el.conditional_id
           el.update(conditional_id: question_sheet.pages.detect{ |el2| el2.old_id == el.conditional_id }&.id)
         end

data/app/views/fe/questions/fe/_attachment_field.html.erb

@@ -29,7 +29,7 @@
         <% session_key_name = Rails.application.config.session_options[:key] %>
         $('#<%= dom_id(attachment_field) %>_file_uploader').uploadify({
             uploader        : '<%= asset_path "fe/uploadify/uploadify.swf" %>',
-            script          : '<%= save_file_fe_answer_sheet_page_path(@presenter.active_answer_sheet, @page, question_id: element_id, answer_sheet_type: @answer_sheet.class.name) %>',
+            script          : '<%= save_file_fe_answer_sheet_page_path(@presenter.active_answer_sheet, @page, question_id: element_id, answer_sheet_type: @answer_sheet.class.name).html_safe %>',
             fileDesc        : 'Pick a file',
             fileExt         : '<%= attachment_field.content || "*.*" %>',
             auto            : true,
@@ -64,7 +64,7 @@
           $('<%= base_element %> .remove_current').click(function() {
             $.ajax({
               type: 'DELETE',
-              url: '<%= delete_file_fe_answer_sheet_page_path(@presenter.active_answer_sheet, @page, question_id: element_id, answer_sheet_type: @answer_sheet.class.name) %>',
+              url: '<%= delete_file_fe_answer_sheet_page_path(@presenter.active_answer_sheet, @page, question_id: element_id, answer_sheet_type: @answer_sheet.class.name).html_safe %>',
               success: function(result) {
                 $.get("/fe/answer_sheets/<%= @answer_sheet.id %>/page/<%= @page.id %>/edit?answer_sheet_type=<%= @answer_sheet.class.name %>", function(data) {
                   $('<%= base_element %>').replaceWith($(data).find('<%= base_element %>'))
@@ -98,7 +98,7 @@
 
               let $input = $("<%= base_element %> .upload_field").html5_upload({
                   url: function(number) {
-                    return "<%= save_file_fe_answer_sheet_page_path(@presenter.active_answer_sheet, @page, question_id: element_id, answer_sheet_type: @answer_sheet.class.name) %>";
+                    return "<%= save_file_fe_answer_sheet_page_path(@presenter.active_answer_sheet, @page, question_id: element_id, answer_sheet_type: @answer_sheet.class.name).html_safe %>";
                   },
                   sendBoundary: window.FormData || $.browser.mozilla,
                   onStart: function(event, total) {

data/lib/fe/version.rb

@@ -1,3 +1,3 @@
 module Fe
-  VERSION = "2.1.9"
+  VERSION = "2.1.12"
 end

data/spec/controllers/fe/admin/elements_controller_spec.rb

@@ -87,7 +87,7 @@ describe Fe::Admin::ElementsController, type: :controller do
       }.to change{Fe::Element.count}.by(0)
 
       expect(assigns(:page_element)).to be_nil
-      expect(response).to render_template('error.js.erb')
+      expect(response).to render_template('error')
     end
   end
   context '#update' do
@@ -114,7 +114,7 @@ describe Fe::Admin::ElementsController, type: :controller do
 
       put :update, params: {element: { slug: "Illegal Chars: #@$!" }, question_sheet_id: question_sheet.id, page_id: page.id, id: element.id}, xhr: true
       expect(assigns(:element)).to eq(element)
-      expect(response).to render_template('error.js.erb')
+      expect(response).to render_template('error')
     end
   end
   context '#destroy' do

data/spec/dummy/config/application.rb

@@ -25,7 +25,8 @@ module Dummy
     # The default locale is :en and all translations from config/locales/*.rb,yml are auto loaded.
     # config.i18n.load_path += Dir[Rails.root.join('my', 'locales', '*.{rb,yml}').to_s]
     config.i18n.default_locale = :en
-    config.action_view.raise_on_missing_translations = false
+    # In Rails 7+, this setting has moved to I18n
+    config.i18n.raise_on_missing_translations = false
   end
 end
 

data/spec/dummy/config/database.yml

@@ -1,5 +1,5 @@
 mysql: &mysql
-  adapter: mysql<%= "2" unless ( ENV["TRAVIS_RUBY_VERSION"].include? "j" if ENV["TRAVIS_RUBY_VERSION"] ) %>
+  adapter: mysql<%= "2" unless (ENV["TRAVIS_RUBY_VERSION"].include? "j" if ENV["TRAVIS_RUBY_VERSION"]) %>
   username: root
   password:
 
@@ -9,20 +9,35 @@ postgresql: &postgresql
   password:
   min_messages: ERROR
 
+sqlite3: &sqlite3
+  adapter: sqlite3
+
 defaults: &defaults
   pool: 5
   timeout: 5000
   host: localhost
-  <<: *<%= ENV['DB'] || "mysql" %>
+  <<: *<%= ENV['DB'] || "sqlite3" %>
 
 development:
+  <% if (ENV['DB'] || "sqlite3") == "sqlite3" %>
+  database: db/fe_dev.sqlite3
+  <% else %>
   database: fe_dev
+  <% end %>
   <<: *defaults
 
 test:
+  <% if (ENV['DB'] || "sqlite3") == "sqlite3" %>
+  database: db/fe_test.sqlite3
+  <% else %>
   database: fe_test
+  <% end %>
   <<: *defaults
 
 production:
+  <% if (ENV['DB'] || "sqlite3") == "sqlite3" %>
+  database: db/fe_prod.sqlite3
+  <% else %>
   database: fe_prod
-  <<: *defaults
+  <% end %>
+  <<: *defaults

data/spec/dummy/db/schema.rb

@@ -2,17 +2,16 @@
 # of editing this file, please use the migrations feature of Active Record to
 # incrementally modify your database, and then regenerate this schema definition.
 #
-# Note that this schema.rb definition is the authoritative source for your
-# database schema. If you need to create the application database on another
-# system, you should be using db:schema:load, not running all the migrations
-# from scratch. The latter is a flawed and unsustainable approach (the more migrations
-# you'll amass, the slower it'll run and the greater likelihood for issues).
+# This file is the source Rails uses to define your schema when running `bin/rails
+# db:schema:load`. When creating a new database, `bin/rails db:schema:load` tends to
+# be faster and is potentially less error prone than running all of your
+# migrations from scratch. Old migrations may fail to apply correctly if those
+# migrations use external dependencies or application code.
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20181108201746) do
-
-  create_table "fe_addresses", force: :cascade, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8" do |t|
+ActiveRecord::Schema[7.2].define(version: 2018_11_08_201746) do
+  create_table "fe_addresses", force: :cascade do |t|
     t.datetime "startdate"
     t.datetime "enddate"
     t.string   "address1"
@@ -29,38 +28,38 @@ ActiveRecord::Schema.define(version: 20181108201746) do
     t.datetime "updated_at"
   end
 
-  create_table "fe_answer_sheet_question_sheets", force: :cascade, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8" do |t|
+  create_table "fe_answer_sheet_question_sheets", force: :cascade do |t|
     t.integer  "answer_sheet_id"
     t.integer  "question_sheet_id"
     t.datetime "created_at"
     t.datetime "updated_at"
-    t.index ["answer_sheet_id", "question_sheet_id"], name: "answer_sheet_question_sheet", using: :btree
+    t.index ["answer_sheet_id", "question_sheet_id"], name: "answer_sheet_question_sheet"
   end
 
-  create_table "fe_answers", force: :cascade, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8" do |t|
+  create_table "fe_answers", force: :cascade do |t|
     t.integer  "answer_sheet_id",                       null: false
     t.integer  "question_id",                           null: false
-    t.text     "value",                   limit: 65535
+    t.text     "value"
     t.integer  "attachment_file_size"
     t.string   "attachment_content_type"
     t.string   "attachment_file_name"
     t.datetime "attachment_updated_at"
     t.datetime "created_at"
     t.datetime "updated_at"
-    t.index ["answer_sheet_id", "question_id"], name: "answer_sheet_question", using: :btree
+    t.index ["answer_sheet_id", "question_id"], name: "answer_sheet_question"
   end
 
-  create_table "fe_applications", force: :cascade, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8" do |t|
+  create_table "fe_applications", force: :cascade do |t|
     t.integer  "applicant_id"
     t.string   "status"
     t.datetime "submitted_at"
     t.datetime "created_at"
     t.datetime "updated_at"
     t.string   "locale",       default: "en"
-    t.index ["applicant_id"], name: "question_sheet_id", using: :btree
+    t.index ["applicant_id"], name: "question_sheet_id"
   end
 
-  create_table "fe_conditions", force: :cascade, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8" do |t|
+  create_table "fe_conditions", force: :cascade do |t|
     t.integer  "question_sheet_id", null: false
     t.integer  "trigger_id",        null: false
     t.string   "expression",        null: false
@@ -68,18 +67,18 @@ ActiveRecord::Schema.define(version: 20181108201746) do
     t.integer  "toggle_id"
     t.datetime "created_at"
     t.datetime "updated_at"
-    t.index ["question_sheet_id"], name: "index_fe_conditions_on_question_sheet_id", using: :btree
-    t.index ["toggle_id"], name: "index_fe_conditions_on_toggle_id", using: :btree
-    t.index ["toggle_page_id"], name: "index_fe_conditions_on_toggle_page_id", using: :btree
-    t.index ["trigger_id"], name: "index_fe_conditions_on_trigger_id", using: :btree
+    t.index ["question_sheet_id"], name: "index_fe_conditions_on_question_sheet_id"
+    t.index ["toggle_id"], name: "index_fe_conditions_on_toggle_id"
+    t.index ["toggle_page_id"], name: "index_fe_conditions_on_toggle_page_id"
+    t.index ["trigger_id"], name: "index_fe_conditions_on_trigger_id"
   end
 
-  create_table "fe_elements", force: :cascade, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8" do |t|
+  create_table "fe_elements", force: :cascade do |t|
     t.integer  "question_grid_id"
     t.string   "kind",                      limit: 40,                    null: false
     t.string   "style",                     limit: 40
     t.string   "label"
-    t.text     "content",                   limit: 65535
+    t.text     "content"
     t.boolean  "required"
     t.string   "slug",                      limit: 36
     t.integer  "position"
@@ -97,41 +96,41 @@ ActiveRecord::Schema.define(version: 20181108201746) do
     t.datetime "updated_at"
     t.integer  "related_question_sheet_id"
     t.integer  "conditional_id"
-    t.text     "tooltip",                   limit: 65535
+    t.text     "tooltip"
     t.boolean  "hide_label",                              default: false
     t.boolean  "hide_option_labels",                      default: false
     t.integer  "max_length"
     t.string   "conditional_type"
-    t.text     "conditional_answer",        limit: 65535
+    t.text     "conditional_answer"
     t.integer  "choice_field_id"
     t.boolean  "share",                                   default: false
-    t.text     "label_translations",        limit: 65535
-    t.text     "tip_translations",          limit: 65535
-    t.text     "content_translations",      limit: 65535
-    t.index ["conditional_id"], name: "index_fe_elements_on_conditional_id", using: :btree
-    t.index ["question_grid_id"], name: "index_fe_elements_on_question_grid_id", using: :btree
-    t.index ["slug"], name: "index_fe_elements_on_slug", using: :btree
+    t.text     "label_translations"
+    t.text     "tip_translations"
+    t.text     "content_translations"
+    t.index ["conditional_id"], name: "index_fe_elements_on_conditional_id"
+    t.index ["question_grid_id"], name: "index_fe_elements_on_question_grid_id"
+    t.index ["slug"], name: "index_fe_elements_on_slug"
   end
 
-  create_table "fe_email_templates", force: :cascade, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8" do |t|
+  create_table "fe_email_templates", force: :cascade do |t|
     t.string   "name",       limit: 1000,  null: false
-    t.text     "content",    limit: 65535
+    t.text     "content"
     t.boolean  "enabled"
     t.string   "subject"
     t.datetime "created_at"
     t.datetime "updated_at"
   end
 
-  create_table "fe_page_elements", force: :cascade, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8" do |t|
+  create_table "fe_page_elements", force: :cascade do |t|
     t.integer  "page_id"
     t.integer  "element_id"
     t.integer  "position"
     t.datetime "created_at"
     t.datetime "updated_at"
-    t.index ["page_id", "element_id"], name: "page_element", using: :btree
+    t.index ["page_id", "element_id"], name: "page_element"
   end
 
-  create_table "fe_pages", force: :cascade, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8" do |t|
+  create_table "fe_pages", force: :cascade do |t|
     t.integer  "question_sheet_id",                                null: false
     t.string   "label",              limit: 60,                    null: false
     t.integer  "number"
@@ -139,11 +138,11 @@ ActiveRecord::Schema.define(version: 20181108201746) do
     t.boolean  "hidden",                           default: false
     t.datetime "created_at"
     t.datetime "updated_at"
-    t.text     "all_element_ids",    limit: 65535
-    t.text     "label_translations", limit: 65535
+    t.text     "all_element_ids"
+    t.text     "label_translations"
   end
 
-  create_table "fe_people", force: :cascade, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8" do |t|
+  create_table "fe_people", force: :cascade do |t|
     t.string   "first_name", limit: 50
     t.string   "last_name",  limit: 50
     t.integer  "user_id"
@@ -152,15 +151,15 @@ ActiveRecord::Schema.define(version: 20181108201746) do
     t.datetime "updated_at"
   end
 
-  create_table "fe_question_sheets", force: :cascade, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8" do |t|
+  create_table "fe_question_sheets", force: :cascade do |t|
     t.string   "label",      limit: 100,                   null: false
     t.boolean  "archived",                 default: false
     t.datetime "created_at"
     t.datetime "updated_at"
-    t.text     "languages",  limit: 65535
+    t.text     "languages"
   end
 
-  create_table "fe_references", force: :cascade, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8" do |t|
+  create_table "fe_references", force: :cascade do |t|
     t.integer  "question_id"
     t.integer  "applicant_answer_sheet_id"
     t.datetime "email_sent_at"
@@ -180,11 +179,11 @@ ActiveRecord::Schema.define(version: 20181108201746) do
     t.integer  "question_sheet_id"
     t.boolean  "visible"
     t.string   "visibility_cache_key"
-    t.index ["applicant_answer_sheet_id"], name: "index_fe_references_on_applicant_answer_sheet_id", using: :btree
-    t.index ["question_id"], name: "index_fe_references_on_question_id", using: :btree
+    t.index ["applicant_answer_sheet_id"], name: "index_fe_references_on_applicant_answer_sheet_id"
+    t.index ["question_id"], name: "index_fe_references_on_question_id"
   end
 
-  create_table "fe_users", force: :cascade, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8" do |t|
+  create_table "fe_users", force: :cascade do |t|
     t.integer  "user_id"
     t.datetime "last_login"
     t.string   "type"
@@ -193,7 +192,7 @@ ActiveRecord::Schema.define(version: 20181108201746) do
     t.datetime "updated_at"
   end
 
-  create_table "phone_numbers", force: :cascade, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8" do |t|
+  create_table "phone_numbers", force: :cascade do |t|
     t.string   "number"
     t.string   "extensions"
     t.integer  "person_id"
@@ -206,20 +205,19 @@ ActiveRecord::Schema.define(version: 20181108201746) do
     t.datetime "updated_at"
   end
 
-  create_table "users", force: :cascade, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8" do |t|
+  create_table "users", force: :cascade do |t|
     t.string  "username"
     t.string  "email"
     t.integer "person_id"
   end
 
-  create_table "versions", force: :cascade, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8mb4" do |t|
+  create_table "versions", force: :cascade do |t|
     t.string   "item_type",  limit: 191,        null: false
     t.integer  "item_id",                       null: false
     t.string   "event",                         null: false
     t.string   "whodunnit"
-    t.text     "object",     limit: 4294967295
+    t.text     "object"
     t.datetime "created_at"
-    t.index ["item_type", "item_id"], name: "index_versions_on_item_type_and_item_id", using: :btree
+    t.index ["item_type", "item_id"], name: "index_versions_on_item_type_and_item_id"
   end
-
-end
+end

data/spec/models/fe/question_sheet_conditional_import_export_spec.rb

@@ -0,0 +1,121 @@
+require 'rails_helper'
+
+describe Fe::QuestionSheet, 'conditional import/export', type: :model do
+  let!(:temp_yaml_file) { Rails.root.join('tmp', 'test_export.yml') }
+
+  after do
+    File.delete(temp_yaml_file) if File.exist?(temp_yaml_file)
+  end
+
+  context 'conditional element and page mapping during export/import' do
+    it 'should preserve conditional relationships after export and import' do
+      # Create question sheet with two pages
+      question_sheet = create(:question_sheet, label: "Conditional Test Sheet")
+
+      # Page 1 - contains element-to-element conditional
+      page1 = create(:page, question_sheet: question_sheet, label: "Personal Info", number: 1)
+
+      # Page 2 - target for page conditional
+      page2 = create(:page, question_sheet: question_sheet, label: "Additional Details", number: 2)
+
+      # Page 1: Simple text field
+      name_field = create(:text_field_element, label: "Your Name")
+      create(:page_element, page: page1, element: name_field)
+
+      # Page 1: Yes/No question that controls visibility of next element on same page
+      has_spouse_field = create(:choice_field_element,
+        label: "Are you married?",
+        style: "yes-no",
+        content: "Yes\r\nNo",
+        conditional_type: "Fe::Element"
+      )
+      create(:page_element, page: page1, element: has_spouse_field)
+
+      # Page 1: Element conditionally shown based on previous yes/no
+      spouse_name_field = create(:text_field_element, label: "Spouse Name")
+      create(:page_element, page: page1, element: spouse_name_field)
+
+      # Page 1: Yes/No question that controls visibility of entire next page
+      has_children_field = create(:choice_field_element,
+        label: "Do you have children?",
+        style: "yes-no",
+        content: "Yes\r\nNo",
+        conditional_type: "Fe::Page"
+      )
+      create(:page_element, page: page1, element: has_children_field)
+
+      # Page 2: Simple text field (entire page controlled by has_children_field)
+      children_details_field = create(:text_field_element, label: "Tell us about your children")
+      create(:page_element, page: page2, element: children_details_field)
+
+      # Set up the conditionals properly
+      page1.reload
+      page2.reload
+
+      # Update all_element_ids for proper ordering
+      page1.update_column(:all_element_ids, "#{name_field.id},#{has_spouse_field.id},#{spouse_name_field.id},#{has_children_field.id}")
+      page2.update_column(:all_element_ids, "#{children_details_field.id}")
+
+      # Set up conditional relationships
+      has_spouse_field.reload
+      has_spouse_field.update!(conditional_id: spouse_name_field.id)
+
+      has_children_field.reload
+      has_children_field.update!(conditional_id: page2.id)
+
+      # Verify setup
+      expect(has_spouse_field.conditional_type).to eq("Fe::Element")
+      expect(has_spouse_field.conditional_id).to eq(spouse_name_field.id)
+      expect(has_children_field.conditional_type).to eq("Fe::Page")
+      expect(has_children_field.conditional_id).to eq(page2.id)
+
+      # Export to YAML
+      File.write(temp_yaml_file, question_sheet.export_to_yaml)
+      expect(File.exist?(temp_yaml_file)).to be true
+
+      # Store original IDs for comparison
+      original_spouse_element_id = spouse_name_field.id
+      original_page2_id = page2.id
+      original_has_spouse_conditional_id = has_spouse_field.conditional_id
+      original_has_children_conditional_id = has_children_field.conditional_id
+
+      # Import from YAML (creates new sheet with new IDs)
+      imported_sheet = Fe::QuestionSheet.create_from_yaml(temp_yaml_file)
+
+      # Verify basic structure
+      expect(imported_sheet.pages.count).to eq(2)
+      expect(imported_sheet.all_elements.count).to eq(5)
+
+      # Get imported elements by their labels
+      imported_page1 = imported_sheet.pages.find_by(label: "Personal Info")
+      imported_page2 = imported_sheet.pages.find_by(label: "Additional Details")
+
+      imported_name_field = imported_sheet.all_elements.find_by(label: "Your Name")
+      imported_has_spouse_field = imported_sheet.all_elements.find_by(label: "Are you married?")
+      imported_spouse_name_field = imported_sheet.all_elements.find_by(label: "Spouse Name")
+      imported_has_children_field = imported_sheet.all_elements.find_by(label: "Do you have children?")
+      imported_children_details_field = imported_sheet.all_elements.find_by(label: "Tell us about your children")
+
+      # Verify all elements were imported
+      expect(imported_name_field).to be_present
+      expect(imported_has_spouse_field).to be_present
+      expect(imported_spouse_name_field).to be_present
+      expect(imported_has_children_field).to be_present
+      expect(imported_children_details_field).to be_present
+
+      # Verify IDs are different (new database records)
+      expect(imported_spouse_name_field.id).not_to eq(original_spouse_element_id)
+      expect(imported_page2.id).not_to eq(original_page2_id)
+
+      # Verify element-to-element conditional was properly mapped
+      expect(imported_has_spouse_field.conditional_type).to eq("Fe::Element")
+      expect(imported_has_spouse_field.conditional_id).to eq(imported_spouse_name_field.id)
+      expect(imported_has_spouse_field.conditional_id).not_to eq(original_has_spouse_conditional_id)
+
+      # Verify element-to-page conditional was properly mapped
+      expect(imported_has_children_field.conditional_type).to eq("Fe::Page")
+      expect(imported_has_children_field.conditional_id).to eq(imported_page2.id)
+      expect(imported_has_children_field.conditional_id).not_to eq(original_has_children_conditional_id)
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fe
 version: !ruby/object:Gem::Version
-  version: 2.1.9
+  version: 2.1.12
 platform: ruby
 authors:
 - CruGlobal
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-09-30 00:00:00.000000000 Z
+date: 2025-11-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -17,6 +17,9 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 5.0.7
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '8'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -24,6 +27,9 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 5.0.7
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '8'
 - !ruby/object:Gem::Dependency
   name: acts_as_list
   requirement: !ruby/object:Gem::Requirement
@@ -576,6 +582,7 @@ files:
 - spec/dummy/config/locales/en.yml
 - spec/dummy/config/routes.rb
 - spec/dummy/config/secrets.yml
+- spec/dummy/db/fe_test.sqlite3
 - spec/dummy/db/migrate/20141203214017_core.fe_engine.rb
 - spec/dummy/db/migrate/20141203214018_create_reference_sheets.fe_engine.rb
 - spec/dummy/db/migrate/20141203214019_add_element_and_answer_fields.fe_engine.rb
@@ -649,6 +656,7 @@ files:
 - spec/models/fe/question_grid_spec.rb
 - spec/models/fe/question_grid_with_total_spec.rb
 - spec/models/fe/question_set_spec.rb
+- spec/models/fe/question_sheet_conditional_import_export_spec.rb
 - spec/models/fe/question_sheet_spec.rb
 - spec/models/fe/question_spec.rb
 - spec/models/fe/reference_question_spec.rb
@@ -728,6 +736,7 @@ test_files:
 - spec/dummy/config/routes.rb
 - spec/dummy/config/secrets.yml
 - spec/dummy/config.ru
+- spec/dummy/db/fe_test.sqlite3
 - spec/dummy/db/migrate/20141203214017_core.fe_engine.rb
 - spec/dummy/db/migrate/20141203214018_create_reference_sheets.fe_engine.rb
 - spec/dummy/db/migrate/20141203214019_add_element_and_answer_fields.fe_engine.rb
@@ -801,6 +810,7 @@ test_files:
 - spec/models/fe/question_grid_spec.rb
 - spec/models/fe/question_grid_with_total_spec.rb
 - spec/models/fe/question_set_spec.rb
+- spec/models/fe/question_sheet_conditional_import_export_spec.rb
 - spec/models/fe/question_sheet_spec.rb
 - spec/models/fe/question_spec.rb
 - spec/models/fe/reference_question_spec.rb