godotengine / godotengine/godot

FBXDocument register/unregister document extensions are non functional

Open
#89,087 0 comments 0 reactions 0 assignees View on GitHub
bug enhancement topic:3d topic:import
Dominant language
C++
Stars
117k
Forks
26.8k
PR merge metrics
PR metrics pending

Description

### Tested versions

- Reproducable in https://github.com/godotengine/godot/commit/f2045ba822bff7d34964901393581a3117c394a9
- not Reproducable in dev3

The FBXDocument is a new feature

### System information

Windows 11, Nvidia 3090, Vulkan forward+, Vulkan mobile

### Issue description

During the merge of FBXDocument, the GLTFExtension system was not implemented, commented stubs were left in c++ where the extensions should go.

### Steps to reproduce

I have not executed this sample code code.

```gdscript
var fbx = FBXDocument.new()
fbx.register_gltf_document_extension # ...
fbx.append_from_file # ...
fbx.generate_scene # ...
```

### Minimal reproduction project (MRP)

The bug is visible in a brand new project.

Contributor guide

Open the contributing guide

Research direction

Start by inspecting the FBXDocument implementation and the commented C++ extension stubs described in the issue. Reproduce the failure with the shown GDScript flow using register_gltf_document_extension, append_from_file, and generate_scene; done means document extension registration and unregistration function through that flow.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
game-dev
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.