Skip to content

Fix OrderManager being deleted in map editor#21573

Merged
Mailaender merged 1 commit into
OpenRA:bleedfrom
PunkPun:fix-freeze
Sep 12, 2024
Merged

Fix OrderManager being deleted in map editor#21573
Mailaender merged 1 commit into
OpenRA:bleedfrom
PunkPun:fix-freeze

Conversation

@PunkPun
Copy link
Copy Markdown
Member

@PunkPun PunkPun commented Sep 5, 2024

Fixes 1 cause for #21290

The mission selector deletes OrderManager on exit, which doesn't make much sense. We can just remove it

@Mailaender
Copy link
Copy Markdown
Member

It seems you are just moving it.

@PunkPun
Copy link
Copy Markdown
Member Author

PunkPun commented Sep 5, 2024

By moving it I'm removing it from IngameMenuLogic

@Mailaender
Copy link
Copy Markdown
Member

Fix OrderManager being deleted in map editor

This is about MissionBrowserLogic so the commit message seems off.

@PunkPun
Copy link
Copy Markdown
Member Author

PunkPun commented Sep 9, 2024

But it is specifically fixing mission browser in the map editor

@Mailaender
Copy link
Copy Markdown
Member

The mission browser can't be accessed through the map editor. I can't follow. Try to explain in more detail with a step by step on how to reproduce the original issue.

@JovialFeline
Copy link
Copy Markdown
Contributor

JovialFeline commented Sep 10, 2024

The mission browser can't be accessed through the map editor. I can't follow. Try to explain in more detail with a step by step on how to reproduce the original issue.

This is at least one of the issues, I take it.

2024-09-10_17-22-10.mp4
  • Load a mission in the editor.
  • Click Play Map.
  • Click Back.
  • Try to exit the editor.

@Mailaender
Copy link
Copy Markdown
Member

Thanks. Now I get it. I didn't know that we launch the mission browser when pressing play map because for skirmish maps it is the lobby. It still freezes for skirmish maps. I assume the fix is the same problem, so would be nice to have that fixed as well.
grafik

@PunkPun
Copy link
Copy Markdown
Member Author

PunkPun commented Sep 12, 2024

It is the same problem, but both the reason and the fix to it is much much more complex. To the point where I want to just hack exiting behaviour

@Mailaender Mailaender merged commit ab6dc5d into OpenRA:bleed Sep 12, 2024
@Mailaender
Copy link
Copy Markdown
Member

Changelog

@PunkPun PunkPun deleted the fix-freeze branch September 13, 2024 15:01
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.

3 participants