Merge pull request #13412 from arthurnn/quote_id_not_id

quoting: Check if id is a valid method before using it

activerecord/lib/active_record/connection_adapters/abstract/quoting.rb

@@ -43,7 +43,9 @@ def quote(value, column = nil)
       # SQLite does not understand dates, so this method will convert a Date
       # to a String.
       def type_cast(value, column)
-        return value.id if value.respond_to?(:quoted_id)
+        if value.respond_to?(:quoted_id) && value.respond_to?(:id)
+          return value.id
+        end
 
         case value
         when String, ActiveSupport::Multibyte::Chars

activerecord/test/cases/adapters/sqlite3/quoting_test.rb

@@ -95,6 +95,13 @@ def id
             end
           }.new
           assert_equal 10, @conn.type_cast(quoted_id_obj, nil)
+
+          quoted_id_obj = Class.new {
+            def quoted_id
+              "'zomg'"
+            end
+          }
+          assert_raise(TypeError) { @conn.type_cast(quoted_id_obj, nil) }
         end
       end
     end