inline_forms 7.5.2 → 7.9.1

data/lib/installer_templates/example_app_tests/test/integration/example_app_apartment_top_level_pagination_test.rb

@@ -0,0 +1,40 @@
+# frozen_string_literal: true
+
+require_relative "../example_app/example_integration_test_case"
+
+# Step 4: top-level index pagination swaps `<turbo-frame id="apartments_list">`.
+class ExampleAppApartmentTopLevelPaginationTest < ExampleAppIntegrationTestCase
+  setup do
+    @frame = "apartments_list"
+    @frame_headers = { "Turbo-Frame" => @frame, "Accept" => "text/html" }
+    @original_per_page = Apartment.per_page
+    Apartment.per_page = 3
+    6.times do |i|
+      Apartment.find_or_create_by!(name: "Paginate Apt #{i}") do |a|
+        a.title = "Pagination seed #{i}"
+      end
+    end
+  end
+
+  teardown do
+    Apartment.per_page = @original_per_page
+  end
+
+  test "top-level pagination links target apartments_list frame id" do
+    get apartments_path
+    assert_response :success
+    assert_match %r{class="pagination"}, @response.body
+    assert_match(
+      /update=apartments_list/,
+      @response.body,
+      "pagination must pass update=apartments_list for Turbo frame swap"
+    )
+  end
+
+  test "top-level page 2 returns matching turbo-frame with Turbo-Frame header" do
+    get apartments_path(page: 2, update: @frame, ul_needed: true), headers: @frame_headers
+    assert_response :success
+    assert_includes @response.body, %(<turbo-frame id="#{@frame}")
+    refute_match(/id="outer_container"/, @response.body)
+  end
+end

data/lib/installer_templates/example_app_tests/test/integration/example_app_apartment_versions_turbo_test.rb

@@ -24,4 +24,97 @@ class ExampleAppApartmentVersionsTurboTest < ExampleAppIntegrationTestCase
     assert_includes @response.body, %(<turbo-frame id="#{@versions_frame}">)
     refute_includes @response.body, "Changeset"
   end
+
+  test "expanded row versions open link uses Turbo not UJS remote" do
+    row_frame = "apartment_#{@apartment.id}"
+    get apartment_path(@apartment, update: row_frame),
+        headers: { "Turbo-Frame" => row_frame, "Accept" => "text/html" }
+    assert_response :success
+    assert_select "turbo-frame##{@versions_frame} a[data-turbo='true'][data-turbo-frame='#{@versions_frame}']", minimum: 1
+    refute_select "turbo-frame##{@versions_frame} a[data-remote='true']"
+  end
+
+  test "restore link in versions list requests turbo-stream (nested versions frame)" do
+    apt = Apartment.create!(name: "Stream Link", title: "T")
+    apt.update!(title: "T2")
+    vf = "apartment_#{apt.id}_versions"
+    get list_versions_apartment_path(apt, update: vf),
+        headers: { "Turbo-Frame" => vf, "Accept" => "text/html" }
+    assert_response :success
+    assert_match(/data-turbo-stream="true"/, @response.body,
+      "restore from inside …_versions must use turbo-stream to avoid Turbo-Frame mismatch")
+  end
+
+  test "revert from versions list closes row via turbo-stream when Turbo-Frame is versions" do
+    apt = Apartment.create!(name: "Versions Stream Revert", title: "Before")
+    apt.update!(title: "After")
+    row_frame = "apartment_#{apt.id}"
+    versions_frame = "#{row_frame}_versions"
+    version = apt.versions.where(event: "update").order(:id).last
+    assert version, "expected an update version to revert"
+
+    post revert_apartment_path(version.id, update: row_frame),
+         headers: {
+           "Turbo-Frame" => versions_frame,
+           "Accept" => "text/vnd.turbo-stream.html"
+         }
+    assert_response :success
+    assert_includes @response.body, %(action="replace")
+    assert_includes @response.body, %(target="#{row_frame}")
+    assert_includes @response.body, %(target="#{versions_frame}")
+    assert_equal "Before", apt.reload.title
+  end
+
+  test "revert from versions list closes row via Turbo POST on row frame" do
+    apt = Apartment.create!(name: "Versions Revert", title: "Before")
+    apt.update!(title: "After")
+    row_frame = "apartment_#{apt.id}"
+    versions_frame = "#{row_frame}_versions"
+    # 7.9.0 dropped the `format.html` fallback in `revert`; the restore link
+    # always requests a turbo-stream now, even when the click happened on
+    # the row frame, so the test mirrors that contract.
+    row_headers = {
+      "Turbo-Frame" => row_frame,
+      "Accept" => "text/vnd.turbo-stream.html"
+    }
+
+    version = apt.versions.where(event: "update").order(:id).last
+    assert version, "expected an update version to revert"
+
+    post revert_apartment_path(version.id, update: row_frame), headers: row_headers
+    assert_response :success
+    assert_includes @response.body, %(action="replace")
+    assert_includes @response.body, %(target="#{row_frame}")
+    assert_includes @response.body, %(target="#{versions_frame}")
+    assert_equal "Before", apt.reload.title
+  end
+
+  test "revert from versions list restores rich_text body via turbo-stream" do
+    apt = Apartment.create!(name: "RichText Revert", title: "T")
+    apt.update!(description: "<p>old body</p>")
+    apt.update!(description: "<p>new body</p>")
+
+    rich_text = ActionText::RichText.find_by!(
+      record_type: Apartment.name, record_id: apt.id, name: "description"
+    )
+    rich_text_version = rich_text.versions.where(event: "update").order(:id).last
+    assert rich_text_version,
+      "expected a PaperTrail update version on ActionText::RichText for the description edit"
+
+    row_frame = "apartment_#{apt.id}"
+    versions_frame = "#{row_frame}_versions"
+    post revert_apartment_path(rich_text_version.id, update: row_frame),
+         headers: {
+           "Turbo-Frame" => versions_frame,
+           "Accept" => "text/vnd.turbo-stream.html"
+         }
+    assert_response :success
+    assert_includes @response.body, %(action="replace")
+    assert_includes @response.body, %(target="#{row_frame}")
+    assert_includes @response.body, %(target="#{versions_frame}")
+
+    apt.reload
+    assert_includes apt.description.body.to_html, "old body",
+      "rich_text revert should restore the previous body content"
+  end
 end

data/lib/installer_templates/example_app_tests/test/integration/example_app_photo_revert_test.rb

@@ -0,0 +1,94 @@
+# frozen_string_literal: true
+
+require_relative "../example_app/example_integration_test_case"
+
+# 7.9.0 regression: revert from the versions panel for a Photo whose
+# `image` column changed must put back the previous CarrierWave file
+# bytes, not just the previous filename string. Relies on the
+# `ImageUploader` knobs the installer ships:
+#
+#   * `CarrierWave.configure { config.remove_previously_stored_files_after_update = false }`
+#     (config/initializers/carrierwave.rb)
+#   * `remove!` no-op
+#   * unique per-upload filename prefix
+#
+# See https://stackoverflow.com/questions/9423279/papertrail-and-carrierwave
+class ExampleAppPhotoRevertTest < ExampleAppIntegrationTestCase
+  # Mirror the setup of ExampleAppApartmentPhotosPaginationTest: re-seed
+  # Konferensha photos from db/seed_images/ so the test asserts against
+  # real CarrierWave file mounts without depending on dev DB state.
+  setup do
+    @apartment = Apartment.find_or_create_by!(name: "Konferensha") do |a|
+      a.title = "Konferensha sobre Papiamentu"
+    end
+    seed_dir = Rails.root.join("db", "seed_images")
+    if seed_dir.directory?
+      Dir.glob(seed_dir.join("*.{jpg,jpeg,png,gif}"), File::FNM_CASEFOLD).sort.each do |abs|
+        base = File.basename(abs)
+        next if Photo.exists?(name: base, apartment_id: @apartment.id)
+        File.open(abs, "rb") do |io|
+          Photo.create!(
+            name: base,
+            caption: "Konferensha foto #{base}",
+            apartment: @apartment,
+            image: io
+          )
+        end
+      end
+    end
+  end
+
+  test "revert restores previous CarrierWave file bytes on disk" do
+    photo = @apartment.photos.first!
+    original_identifier = photo.image.identifier
+    original_path = photo.image.path
+    assert File.exist?(original_path), "expected seeded photo file on disk: #{original_path}"
+    original_size = File.size(original_path)
+
+    seed_dir = Rails.root.join("db", "seed_images")
+    jpgs = Dir.glob(seed_dir.join("*.{jpg,jpeg}"), File::FNM_CASEFOLD).sort
+    replacement = jpgs.find { |abs| File.basename(abs) != photo.name } || jpgs.last
+    assert replacement, "need at least one seed jpg different from the photo's current mount"
+    refute_equal File.size(replacement), original_size,
+      "test needs a replacement file with a different byte length so the assertion is meaningful"
+
+    frame_id = "apartment_#{@apartment.id}_photo_#{photo.id}_image"
+    turbo_headers = { "Turbo-Frame" => frame_id, "Accept" => "text/html" }
+    uploaded = Rack::Test::UploadedFile.new(replacement, "image/jpeg")
+    put photo_path(
+      photo,
+      attribute: "image",
+      form_element: "image_field",
+      update: frame_id
+    ),
+        params: { image: uploaded },
+        headers: turbo_headers
+    assert_response :success
+    photo.reload
+    refute_equal original_identifier, photo.image.identifier,
+      "update should have changed the image identifier"
+
+    version = photo.versions.where(event: "update").order(:id).last
+    assert version, "expected a Photo update version after the image PUT"
+
+    row_frame = "apartment_#{@apartment.id}_photo_#{photo.id}"
+    versions_frame = "apartment_#{@apartment.id}_photo_#{photo.id}_versions"
+    post revert_photo_path(version.id, update: row_frame),
+         headers: {
+           "Turbo-Frame" => versions_frame,
+           "Accept" => "text/vnd.turbo-stream.html"
+         }
+    assert_response :success
+    assert_includes @response.body, %(action="replace")
+    assert_includes @response.body, %(target="#{row_frame}")
+    assert_includes @response.body, %(target="#{versions_frame}")
+
+    photo.reload
+    assert_equal original_identifier, photo.image.identifier,
+      "revert should restore the previous CarrierWave identifier"
+    assert File.exist?(photo.image.path),
+      "revert should leave the previous file bytes on disk (carrierwave config)"
+    assert_equal original_size, File.size(photo.image.path),
+      "previous file bytes must be byte-identical to the original"
+  end
+end

data/lib/installer_templates/example_app_tests/test/integration/example_app_turbo_layout_test.rb

@@ -2,14 +2,11 @@
 
 require_relative "../example_app/example_integration_test_case"
 
-# Smoke test for rollout step 1 of stuff/ujs-to-turbo.md (gem-side):
-# the layout loads Turbo as an ES module and disables Drive so existing
-# UJS-driven links/forms keep working unchanged. If this asserts ever
-# fails, frame/stream conversions in later slices will silently fall
-# back to full-page navigation (Turbo not loaded) instead of using the
-# `<turbo-frame>` fast path.
+# Smoke test: layouts load Turbo as an ES module (no Sprockets ESM parse error).
+# Step 5 (7.8.0) leaves Turbo Drive at its default (enabled); inline flows use
+# `<turbo-frame>` + HTML, not jquery-ujs.
 class ExampleAppTurboLayoutTest < ExampleAppIntegrationTestCase
-  test "inline_forms layout loads turbo.min.js as an ES module with drive disabled" do
+  test "inline_forms layout loads turbo.min.js as an ES module" do
     get apartments_path
     assert_response :success
 
@@ -19,10 +16,10 @@ class ExampleAppTurboLayoutTest < ExampleAppIntegrationTestCase
       "expected the inline_forms layout to import turbo.min.js as a module"
     )
 
-    assert_match(
+    refute_match(
       /Turbo\.session\.drive\s*=\s*false/,
       @response.body,
-      "expected Turbo.session.drive = false so existing UJS keeps working"
+      "Step 5 enables Turbo Drive by default; disabling it would regress full-page Turbo"
     )
   end
 end

data/lib/installer_templates/example_app_tests/test/models/example_app_apartment_name_validation_test.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+require "test_helper"
+
+class ExampleAppApartmentNameValidationTest < ActiveSupport::TestCase
+  test "apartment requires name" do
+    apartment = Apartment.new(title: "No name")
+    assert_not apartment.valid?
+    assert_includes apartment.errors[:name], "can't be blank"
+  end
+
+  test "apartment is valid with name" do
+    apartment = Apartment.new(name: "North", title: "Tower")
+    assert apartment.valid?
+  end
+end

data/test/archived_form_elements_test.rb

@@ -0,0 +1,41 @@
+# frozen_string_literal: true
+
+require_relative "test_helper"
+require "inline_forms/archived_form_elements"
+
+class ArchivedFormElementsTest < Minitest::Test
+  class ModelWithArchivedElement
+    def inline_forms_attribute_list
+      [[:address, "Address", :geo_code_curacao]]
+    end
+  end
+
+  class ModelWithArchivedTree
+    def inline_forms_attribute_list
+      [[:children, "Children", :tree]]
+    end
+  end
+
+  def test_raises_when_model_declares_archived_geo_code_curacao
+    err = assert_raises(InlineForms::ArchivedFormElementError) do
+      InlineForms.validate_no_archived_form_elements_for!(ModelWithArchivedElement)
+    end
+    assert_includes err.message, "geo_code_curacao"
+    assert_includes err.message, "7.6.0"
+    assert_includes err.message, "archived/form_elements/geo_code_curacao"
+  end
+
+  def test_raises_when_model_declares_archived_tree
+    err = assert_raises(InlineForms::ArchivedFormElementError) do
+      InlineForms.validate_no_archived_form_elements_for!(ModelWithArchivedTree)
+    end
+    assert_includes err.message, "tree"
+    assert_includes err.message, "archived/form_elements/tree"
+  end
+
+  def test_archived_registry_documents_absence_list_without_path
+    meta = InlineForms::ARCHIVED_FORM_ELEMENTS[:absence_list]
+    assert_equal "6.3.0", meta[:removed_in_version]
+    assert_nil meta[:archive_path]
+  end
+end

data/test/form_element_from_callee_test.rb

@@ -45,9 +45,9 @@ class FormElementFromCalleeTest < Minitest::Test
   end
 
   def test_multi_word_element_name_underscores_preserved
-    got = InlineForms.form_element_string_from_callee(:chicas_dropdown_with_family_members_show)
+    got = InlineForms.form_element_string_from_callee(:text_area_without_ckeditor_show)
     assert_equal(
-      "chicas_dropdown_with_family_members",
+      "text_area_without_ckeditor",
       got,
       "Form element names with underscores must survive unchanged except for _show."
     )

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: inline_forms
 version: !ruby/object:Gem::Version
-  version: 7.5.2
+  version: 7.9.1
 platform: ruby
 authors:
 - Ace Suares
@@ -471,15 +471,11 @@ files:
 - app/assets/stylesheets/jquery_ui/themes/_jquery.ui.vader.css.scss
 - app/controllers/concerns/cancan_concern.rb
 - app/controllers/concerns/versions_concern.rb
-- app/controllers/geo_code_curacao_controller.rb
 - app/controllers/inline_forms_application_controller.rb
 - app/controllers/inline_forms_controller.rb
 - app/helpers/form_elements/audio_field.rb
 - app/helpers/form_elements/check_box.rb
 - app/helpers/form_elements/check_list.rb
-- app/helpers/form_elements/chicas_dropdown_with_family_members.rb
-- app/helpers/form_elements/chicas_family_photo_list.rb
-- app/helpers/form_elements/chicas_photo_list.rb
 - app/helpers/form_elements/ckeditor.rb
 - app/helpers/form_elements/date.rb
 - app/helpers/form_elements/decimal_field.rb
@@ -491,17 +487,14 @@ files:
 - app/helpers/form_elements/dropdown_with_values.rb
 - app/helpers/form_elements/dropdown_with_values_with_stars.rb
 - app/helpers/form_elements/file_field.rb
-- app/helpers/form_elements/geo_code_curacao.rb
 - app/helpers/form_elements/header.rb
 - app/helpers/form_elements/image_field.rb
 - app/helpers/form_elements/info.rb
 - app/helpers/form_elements/info_list.rb
 - app/helpers/form_elements/integer_field.rb
-- app/helpers/form_elements/kansen_slider.rb
 - app/helpers/form_elements/money_field.rb
 - app/helpers/form_elements/month_select.rb
 - app/helpers/form_elements/month_year_picker.rb
-- app/helpers/form_elements/move.rb
 - app/helpers/form_elements/multi_image_field.rb
 - app/helpers/form_elements/pdf_link.rb
 - app/helpers/form_elements/plain_text.rb
@@ -519,7 +512,6 @@ files:
 - app/helpers/form_elements/time.rb
 - app/helpers/inline_forms_helper.rb
 - app/models/concerns/inline_forms/soft_deletable.rb
-- app/models/geo_code_curacao.rb
 - app/validators/curacao_id_number_validator.rb
 - app/validators/is_curacao_phone_validator.rb
 - app/validators/is_email_address_validator.rb
@@ -537,8 +529,6 @@ files:
 - app/views/devise/sessions/new.html.erb
 - app/views/devise/shared/_header_and_errors.html.erb
 - app/views/devise/shared/_links.erb
-- app/views/geo_code_curacao/list_streets.html.erb
-- app/views/geo_code_curacao/list_streets.js.erb
 - app/views/inline_forms/_close.html.erb
 - app/views/inline_forms/_edit.html.erb
 - app/views/inline_forms/_flash.html.erb
@@ -547,30 +537,39 @@ files:
 - app/views/inline_forms/_new.html.erb
 - app/views/inline_forms/_new_nested.html.erb
 - app/views/inline_forms/_show.html.erb
-- app/views/inline_forms/_tree.html.erb
 - app/views/inline_forms/_versions.html.erb
 - app/views/inline_forms/_versions_list.html.erb
-- app/views/inline_forms/close.js.erb
 - app/views/inline_forms/create_list_frame.html.erb
 - app/views/inline_forms/extract_translations.erb
 - app/views/inline_forms/field_edit.html.erb
 - app/views/inline_forms/field_show.html.erb
-- app/views/inline_forms/list.js.erb
-- app/views/inline_forms/new.js.erb
 - app/views/inline_forms/new_record.html.erb
-- app/views/inline_forms/record_destroyed.js.erb
 - app/views/inline_forms/row_close.html.erb
 - app/views/inline_forms/row_destroyed.html.erb
 - app/views/inline_forms/row_show.html.erb
-- app/views/inline_forms/show.js.erb
-- app/views/inline_forms/show_undo.js.erb
-- app/views/inline_forms/versions.js.erb
-- app/views/inline_forms/versions_list.js.erb
 - app/views/inline_forms/versions_list_panel.html.erb
 - app/views/inline_forms/versions_panel.html.erb
 - app/views/layouts/application.html.erb
 - app/views/layouts/devise.html.erb
 - app/views/layouts/inline_forms.html.erb
+- archived/README.md
+- archived/form_elements/README.md
+- archived/form_elements/chicas/README.md
+- archived/form_elements/chicas/app/helpers/form_elements/chicas_dropdown_with_family_members.rb
+- archived/form_elements/chicas/app/helpers/form_elements/chicas_family_photo_list.rb
+- archived/form_elements/chicas/app/helpers/form_elements/chicas_photo_list.rb
+- archived/form_elements/geo_code_curacao/README.md
+- archived/form_elements/geo_code_curacao/app/controllers/geo_code_curacao_controller.rb
+- archived/form_elements/geo_code_curacao/app/helpers/form_elements/geo_code_curacao.rb
+- archived/form_elements/geo_code_curacao/app/models/geo_code_curacao.rb
+- archived/form_elements/geo_code_curacao/app/views/geo_code_curacao/list_streets.html.erb
+- archived/form_elements/geo_code_curacao/app/views/geo_code_curacao/list_streets.js.erb
+- archived/form_elements/kansen_slider/README.md
+- archived/form_elements/kansen_slider/app/helpers/form_elements/kansen_slider.rb
+- archived/form_elements/tree/README.md
+- archived/form_elements/tree/app/helpers/form_elements/move.rb
+- archived/form_elements/tree/app/views/inline_forms/_show_tree.html.erb
+- archived/form_elements/tree/app/views/inline_forms/_tree.html.erb
 - bin/inline_forms
 - bin/inline_forms_app_template.rb
 - bin/inline_forms_installer_core.rb
@@ -593,6 +592,7 @@ files:
 - lib/generators/templates/test.erb
 - lib/generators/templates/unicorn/production.rb
 - lib/inline_forms.rb
+- lib/inline_forms/archived_form_elements.rb
 - lib/inline_forms/form_element_from_callee.rb
 - lib/inline_forms/version.rb
 - lib/installer_templates/dartsass/devise_main.scss
@@ -601,14 +601,18 @@ files:
 - lib/installer_templates/example_app_tests/test/example_app/example_integration_test_case.rb
 - lib/installer_templates/example_app_tests/test/integration/example_app_apartment_field_turbo_test.rb
 - lib/installer_templates/example_app_tests/test/integration/example_app_apartment_name_list_test.rb
+- lib/installer_templates/example_app_tests/test/integration/example_app_apartment_name_required_test.rb
 - lib/installer_templates/example_app_tests/test/integration/example_app_apartment_photos_pagination_test.rb
 - lib/installer_templates/example_app_tests/test/integration/example_app_apartment_row_turbo_test.rb
 - lib/installer_templates/example_app_tests/test/integration/example_app_apartment_top_level_new_test.rb
+- lib/installer_templates/example_app_tests/test/integration/example_app_apartment_top_level_pagination_test.rb
 - lib/installer_templates/example_app_tests/test/integration/example_app_apartment_versions_turbo_test.rb
 - lib/installer_templates/example_app_tests/test/integration/example_app_guest_access_test.rb
+- lib/installer_templates/example_app_tests/test/integration/example_app_photo_revert_test.rb
 - lib/installer_templates/example_app_tests/test/integration/example_app_photos_test.rb
 - lib/installer_templates/example_app_tests/test/integration/example_app_routing_test.rb
 - lib/installer_templates/example_app_tests/test/integration/example_app_turbo_layout_test.rb
+- lib/installer_templates/example_app_tests/test/models/example_app_apartment_name_validation_test.rb
 - lib/installer_templates/example_app_tests/test/models/example_app_apartment_photo_test.rb
 - lib/installer_templates/example_app_tests/test/models/example_app_paper_trail_changeset_test.rb
 - lib/installer_templates/example_app_tests/test/models/example_app_plain_text_rich_text_edge_cases_test.rb
@@ -622,6 +626,7 @@ files:
 - lib/otherstuff/mkrole
 - lib/otherstuff/roles_users.sql
 - lib/vagrant/vagrantbox-inline_forms.zip
+- test/archived_form_elements_test.rb
 - test/form_element_from_callee_test.rb
 - test/inline_edit_polymorphic_path_test.rb
 - test/inline_forms_generator_test.rb

data/app/views/geo_code_curacao/list_streets.html.erb

@@ -1 +0,0 @@
-<%= @streets.each { |s| puts s} %>

data/app/views/geo_code_curacao/list_streets.js.erb

@@ -1 +0,0 @@
-<%= @streets %>

data/app/views/inline_forms/close.js.erb

@@ -1,4 +0,0 @@
-$('#<%= @update_span %>').fadeOut("slow", function() {
-$(this).html('<%= escape_javascript(render(:partial => 'inline_forms/close' ))%>');
-$(this).fadeIn("slow");
-});

data/app/views/inline_forms/list.js.erb

@@ -1 +0,0 @@
-$('#<%= @update_span %>').html('<%= escape_javascript(render(:partial => 'inline_forms/list' ))%>')

data/app/views/inline_forms/new.js.erb

@@ -1 +0,0 @@
-$('#<%= @update_span %>').html('<%= escape_javascript(render(:partial => 'inline_forms/new' ))%>')

data/app/views/inline_forms/record_destroyed.js.erb

@@ -1 +0,0 @@
-$('#<%= @update_span %>').fadeOut("slow")

data/app/views/inline_forms/show.js.erb

@@ -1 +0,0 @@
-$('#<%= @update_span %>').html('<%= escape_javascript(render(:partial => 'inline_forms/show' ))%>')

data/app/views/inline_forms/show_undo.js.erb

@@ -1 +0,0 @@
-$('#<%= @update_span %>').html('<%= escape_javascript(link_to t('inline_forms.view.undo'), send('revert_' + @object.class.to_s.underscore + "_path", @object.versions.scope.last, :update => @update_span), :remote => true, :method => :post)%>')

data/app/views/inline_forms/versions.js.erb

@@ -1,4 +0,0 @@
-$('#<%= @update_span %>').fadeOut("slow", function() {
-$(this).html('<%= escape_javascript(render(:partial => 'inline_forms/versions' ))%>');
-$(this).fadeIn("slow");
-});

data/app/views/inline_forms/versions_list.js.erb

@@ -1 +0,0 @@
-$('#<%= @update_span %>').html('<%= escape_javascript(render(:partial => 'inline_forms/versions_list' ))%>')