Skip to content

Avoid (Missing Reference) on embedded material asset fields#758

Merged
adriengivry merged 1 commit intoOverload-Technologies:mainfrom
Gopmyc:756
Apr 22, 2026
Merged

Avoid (Missing Reference) on embedded material asset fields#758
adriengivry merged 1 commit intoOverload-Technologies:mainfrom
Gopmyc:756

Conversation

@Gopmyc
Copy link
Copy Markdown
Contributor

@Gopmyc Gopmyc commented Apr 22, 2026

Description

Asset fields were incorrectly showing "(Missing Reference)" for embedded material/texture references (virtual paths like model:embedded_material_*.ovmat), because existence checks used filesystem paths directly.

This PR updates the editor asset existence checker to:

  • detect embedded asset paths,
  • validate embedded material/texture references,
  • resolve existence via the source model file,
  • keep the previous behavior unchanged for non-embedded paths.

Related Issue(s)

Fixes #756

Review Guidance

  • Import/load a model with embedded materials.
  • In Inspector (Material Renderer) and material-related asset fields, verify embedded references no longer display "(Missing Reference)" when the source model exists.
  • Verify regular invalid asset paths still display "(Missing Reference)".

Screenshots/GIFs

N/A

AI Usage Disclosure

Debugging

Checklist

  • My code follows the project's code style guidelines
  • When applicable, I have commented my code, particularly in hard-to-understand areas
  • When applicable, I have updated the documentation accordingly
  • My changes don't generate new warnings or errors
  • I have reviewed and take responsibility for all code in this PR (including any AI-assisted contributions)

@adriengivry adriengivry changed the title Avoid `(Missing Reference)è on embedded material asset fields Avoid (Missing Reference) on embedded material asset fields Apr 22, 2026
@adriengivry adriengivry merged commit b949720 into Overload-Technologies:main Apr 22, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

(Missing Reference) in embedded material asset fields

2 participants