Merge pull request #3402 from codealchemy/patch/field-names/open-fix

Ensure `open` can be used as a field name

Author: mshibuya

spec/dummy_app/app/mongoid/field_test.rb

@@ -31,6 +31,7 @@ class FieldTest
   field :time_field, type: Time
 
   field :format, type: String
+  field :open, type: Boolean
   field :restricted_field, type: String
   field :protected_field, type: String
   has_mongoid_attached_file :paperclip_asset, styles: {thumb: '100x100>'}

spec/dummy_app/db/migrate/20211011235734_add_bool_field_open.rb

@@ -0,0 +1,5 @@
+class AddBoolFieldOpen < ActiveRecord::Migration[6.0]
+  def change
+    add_column :field_tests, :open, :boolean
+  end
+end

spec/integration/actions/edit_spec.rb

@@ -828,4 +828,20 @@ class HelpTest < Tableless
       expect(@record.format).to eq('test for format')
     end
   end
+
+  context "with a field with 'open' as a name" do
+    it 'is updatable without any error' do
+      RailsAdmin.config FieldTest do
+        edit do
+          field :open
+        end
+      end
+      record = FieldTest.create
+      visit edit_path(model_name: 'field_test', id: record.id)
+      expect do
+        check 'field_test[open]'
+        click_button 'Save'
+      end.to change { record.reload.open }.from(nil).to(true)
+    end
+  end
 end