You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.
Describe the bug eosio::chain::abi_serializer puts built-in types first, so even if user defines new type with built-in type name, it would be ignored. However, eosjs instantiates built-in types by createInitialTypes(), but it would be overwritten by types read from abi. It would be better to keep consistency between two serializers.
To Reproduce
Steps to reproduce the behavior:
Define new type with built-in type name in contract ABI like:
Version of EOSJS
v21.0.3
Describe the bug
eosio::chain::abi_serializer
puts built-in types first, so even if user defines new type with built-in type name, it would be ignored. However, eosjs instantiates built-in types bycreateInitialTypes()
, but it would be overwritten by types read from abi. It would be better to keep consistency between two serializers.To Reproduce
Steps to reproduce the behavior:
cleos
: cleos useseosio::chain::abi_serializer
, sobytes
still follows the serialization rule of built-in typebytes
.eosjs
: eosjs overwrites built-in type serializer with user-defined one, so it would be serialized likechecksum256
here.The text was updated successfully, but these errors were encountered: