[ldtk] Set the def field in LdtkFieldInstance

[gaboose]

I noticed that all LdtkFieldInstances return null identifiers. Turns out that's because the def class field is never set. This PR fixes that. Let me know if the fix looks good or needs some tweaking :)

Tracing the field instances of entities from the ceramic-samples/ldtk-level.

Before the change:

-- Field instances of entity "Item" --
field: LdtkFieldInstance(identifier=null value=Gold tile=null), field.def: null
field: LdtkFieldInstance(identifier=null value=0 tile=null), field.def: null
field: LdtkFieldInstance(identifier=null value=100 tile=null), field.def: null
-- Field instances of entity "Item" --
field: LdtkFieldInstance(identifier=null value=Healing_potion tile=null), field.def: null
field: LdtkFieldInstance(identifier=null value=0 tile=null), field.def: null
field: LdtkFieldInstance(identifier=null value=1 tile=null), field.def: null
-- Field instances of entity "Player" --
field: LdtkFieldInstance(identifier=null value=[Ammo,Bow] tile=null), field.def: null
field: LdtkFieldInstance(identifier=null value=10 tile=null), field.def: null
-- Field instances of entity "Ladder" --
-- Field instances of entity "Item" --
field: LdtkFieldInstance(identifier=null value=Ammo tile=null), field.def: null
field: LdtkFieldInstance(identifier=null value=0 tile=null), field.def: null
field: LdtkFieldInstance(identifier=null value=5 tile=null), field.def: null

After the change:

-- Field instances of entity "Item" --
field: LdtkFieldInstance(identifier=type value=Gold tile=null), field.def: LdtkFieldDefinition(type=LocalEnum.ItemType identifier=type uid=76 isArray=false)
field: LdtkFieldInstance(identifier=price value=0 tile=null), field.def: LdtkFieldDefinition(type=Int identifier=price uid=93 isArray=false)
field: LdtkFieldInstance(identifier=count value=100 tile=null), field.def: LdtkFieldDefinition(type=Int identifier=count uid=139 isArray=false)
-- Field instances of entity "Item" --
field: LdtkFieldInstance(identifier=type value=Healing_potion tile=null), field.def: LdtkFieldDefinition(type=LocalEnum.ItemType identifier=type uid=76 isArray=false)
field: LdtkFieldInstance(identifier=price value=0 tile=null), field.def: LdtkFieldDefinition(type=Int identifier=price uid=93 isArray=false)
field: LdtkFieldInstance(identifier=count value=1 tile=null), field.def: LdtkFieldDefinition(type=Int identifier=count uid=139 isArray=false)
-- Field instances of entity "Player" --
field: LdtkFieldInstance(identifier=inventory value=[Ammo,Bow] tile=null), field.def: LdtkFieldDefinition(type=Array<LocalEnum.ItemType> identifier=inventory uid=91 isArray=true)
field: LdtkFieldInstance(identifier=HP value=10 tile=null), field.def: LdtkFieldDefinition(type=Int identifier=HP uid=92 isArray=false)
-- Field instances of entity "Ladder" --
-- Field instances of entity "Item" --
field: LdtkFieldInstance(identifier=type value=Ammo tile=null), field.def: LdtkFieldDefinition(type=LocalEnum.ItemType identifier=type uid=76 isArray=false)

[jeremyfa]

The changes look good so far so I'll merge it and check that it doesn't break anything. likely to be included in next Ceramic release thanks!

[gaboose]

Thanks! :)