Skip to content
This repository was archived by the owner on Oct 19, 2018. It is now read-only.

Fix Fixnum Bignum deprecation warning #220

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 10 additions & 4 deletions lib/reactive-ruby/serializers.rb
Original file line number Diff line number Diff line change
@@ -1,7 +1,13 @@
[Bignum, FalseClass, Fixnum, Float, Integer, NilClass, String, Symbol, Time, TrueClass].each do |klass|
klass.send(:define_method, :react_serializer) do
as_json
end
[TrueClass, FalseClass, NilClass, Float, String, Symbol, Time].each do |klass|
klass.send(:define_method, :react_serializer) { as_json }
end
# Ruby 2.4 unifies Fixnum and Bignum into Integer
# and prints a warning if the old constants are accessed.
if 0.class == Integer
Integer.send(:define_method, :react_serializer) { as_json }
else
Fixnum.send(:define_method, :react_serializer) { as_json }
Bignum.send(:define_method, :react_serializer) { as_json }
end

BigDecimal.send(:define_method, :react_serializer) { as_json } rescue nil
Expand Down