Merge pull request rails#53982 from kivikakk/push-znknxlolwmxv

byroot · web-flow · commit 60cccb72f190 · 2024-12-18T10:49:22.000+01:00
Use `limit` in `#sole`, not `first`.
diff --git a/activerecord/lib/active_record/relation/finder_methods.rb b/activerecord/lib/active_record/relation/finder_methods.rb
@@ -141,7 +141,7 @@ def take!
     #
     #   Product.where(["price = %?", price]).sole
     def sole
-      found, undesired = first(2)
+      found, undesired = limit(2)
 
       if found.nil?
         raise_record_not_found_exception!

Original file line number	Diff line number	Diff line change
`@@ -141,7 +141,7 @@ def take!`
`141`	`141`	`#`
`142`	`142`	`# Product.where(["price = %?", price]).sole`
`143`	`143`	`def sole`
`144`		`- found, undesired = first(2)`
	`144`	`+ found, undesired = limit(2)`
`145`	`145`
`146`	`146`	`if found.nil?`
`147`	`147`	`raise_record_not_found_exception!`