fix for pretty_generate throwing wrong number of arguments error

Cody Horton · Cody Horton · commit 3d741e8a9faa · 2025-05-16T09:58:27.000-05:00
diff --git a/CHANGES.md b/CHANGES.md
@@ -2,6 +2,8 @@
 
 ### Unreleased
 
+* Fix for JSON.pretty_generate to use passed state object's generate instead of state class as the required parameters aren't available.
+
 ### 2025-05-12 (2.12.0)
 
 * Improve floating point generation to not use scientific notation as much.
diff --git a/lib/json/common.rb b/lib/json/common.rb
@@ -490,7 +490,7 @@ def fast_generate(obj, opts = nil)
   #   }
   #
   def pretty_generate(obj, opts = nil)
-    return state.generate(obj) if State === opts
+    return opts.generate(obj) if State === opts
 
     options = PRETTY_GENERATE_OPTIONS
 

Original file line number	Diff line number	Diff line change
`@@ -490,7 +490,7 @@ def fast_generate(obj, opts = nil)`
`490`	`490`	`# }`
`491`	`491`	`#`
`492`	`492`	`def pretty_generate(obj, opts = nil)`
`493`		`- return state.generate(obj) if State === opts`
	`493`	`+ return opts.generate(obj) if State === opts`
`494`	`494`
`495`	`495`	`options = PRETTY_GENERATE_OPTIONS`
`496`	`496`