Cannot set reference field value about pandora

[VojtaStruhar]

The issue is, that the PandoraReference class isn't considered of type "reference" and fails the is_valid_type check.

func(variant): return variant is PandoraEntity,

Adding PandoraReference to the type check should be fine, I think. I tried it, seemed to work just fine.

func(variant): return variant is PandoraEntity or variant is PandoraReference,

But I don't understand the codebase well enough (yet!) to be fully confident.

[bitbrain]

@VojtaStruhar the idea is that the reference gets created dynamically inside Pandora. From an API perspective, it should always be PandoraEntity.

Let me see today what the issue is here!

[VojtaStruhar]

I dug a little more and found, that after selecting entity from a dropdown, you wrap it in PandoraReference and then call set_default_value with it:

# reference_property.gd, _ready
entity_picker.entity_selected.connect(
  func(entity:PandoraEntity):
    var reference = PandoraReference.new(entity.get_entity_id(), 1 if entity is PandoraCategory else 0)
    _property.set_default_value(reference)
    property_value_changed.emit())

But in set_default_value you have a very similar piece of code that does the same thing conditionally:

if value is PandoraEntity:
  value = PandoraReference.new(value.get_entity_id(), PandoraReference.Type.CATEGORY if value is PandoraCategory else PandoraReference.Type.ENTITY)

I think that in the entity_picker, we are wrapping the entity in a PandoraReference too early! I tried to pass in just the entity, seems to work just fine. I had a look in the stored data too - it seems to add the override correctly:

If this sounds good to you, I'll make a pull request, it's a one-liner :)