Skip to content

Commit 8246923

Browse files
authored
Merge pull request #590 from moberegger/moberegger/optimize_is_collection
Optimize `_is_collection?` method
2 parents d4c8827 + 7e1f2ef commit 8246923

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

lib/jbuilder.rb

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -350,16 +350,12 @@ def _scope
350350
end
351351

352352
def _is_collection?(object)
353-
_object_respond_to?(object, :map, :count) && !(::Struct === object)
353+
object.respond_to?(:map) && object.respond_to?(:count) && !(::Struct === object)
354354
end
355355

356356
def _blank?(value=@attributes)
357357
BLANK == value
358358
end
359-
360-
def _object_respond_to?(object, *methods)
361-
methods.all?{ |m| object.respond_to?(m) }
362-
end
363359
end
364360

365361
require 'jbuilder/railtie' if defined?(Rails)

0 commit comments

Comments
 (0)