Skip to content

Update/crash fix for Blender 4.4#260

Merged
paroj merged 1 commit intoOGRECave:masterfrom
rpgplayerrobin:patch-1
Apr 11, 2025
Merged

Update/crash fix for Blender 4.4#260
paroj merged 1 commit intoOGRECave:masterfrom
rpgplayerrobin:patch-1

Conversation

@rpgplayerrobin
Copy link
Copy Markdown
Contributor

@rpgplayerrobin rpgplayerrobin commented Apr 10, 2025

When trying to export on Blender 4.4, it crashes in this file (shown in the system console) and then has no UI in the export dialogue.

This commit fixes that and with this fix it works to export in Blender 4.4 successfully.

See more about why this is needed here:
https://developer.blender.org/docs/release_notes/4.4/python_api/

When trying to export on Blender 4.4, it crashes in this file (shown in the system console) and then has no UI in the export dialogue.

This fixes that and with this fix it works to export in Blender 4.4 successfully.

See more about why this is needed here:
https://developer.blender.org/docs/release_notes/4.4/python_api/
@rpgplayerrobin rpgplayerrobin changed the title Update export.py for Blender 4.4 Update/crash fix for Blender 4.4 Apr 10, 2025
@paroj
Copy link
Copy Markdown
Member

paroj commented Apr 10, 2025

did you check with master? it sounds like #259 already attempted to fix this and we only need one of them.

I am in favor of reverting #259 as this is more clear.

@rpgplayerrobin
Copy link
Copy Markdown
Contributor Author

rpgplayerrobin commented Apr 10, 2025

You are correct, that version also works.
I just downloaded the version for 4.3 and tried that on 4.4, which of course failed.
But I should of course have downloaded the latest code from here instead.

I am unsure which version of these two are the best.
Mine keeps the same system as it was before, since before 4.4 the __init__ function was automatically called for all inherited classes, but now with 4.4, you must specifically call them yourself instead (which this commit does).

@paroj paroj merged commit 57cdf56 into OGRECave:master Apr 11, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants