Skip to content

IMapManager to hospice care#6579

Open
TemporalOroboros wants to merge 13 commits into
space-wizards:masterfrom
TemporalOroboros:refactor/ecs-grids-2
Open

IMapManager to hospice care#6579
TemporalOroboros wants to merge 13 commits into
space-wizards:masterfrom
TemporalOroboros:refactor/ecs-grids-2

Conversation

@TemporalOroboros
Copy link
Copy Markdown
Contributor

@TemporalOroboros TemporalOroboros commented May 19, 2026

Includes #6578

Moves (nearly (WIP)) all functionality from MapManager to SharedMapSystem
The remaining functionality was the pause commands which have been converted to LocalizedEntityCommands

TODO:

  • Move the grid history culling from INetworkedMapManager over
  • Test for whether the map nuke on shutdown/restart is really needed. (All call locations are immediately preceeded by EntityManager.FlushEntities` which should delete them anyway)
  • Doc comments for everything

Moves all variants of FindGridsIntersecting/TryFindGridAt to SharedMapSystem
Hollows out the MapManager methods and converts them into relays to SharedMapSystem
Moves the functionality for CreateGrid and its variants to SharedMapSystem
Hollows out the MapManager methods and converts them into relays for the SharedMapSystem methods
Obsoletes them too
Also moves over the GetAllMapGrids and GetAllGrids methods
Also moves the SuppressOnTileChanged flag member to SharedMapSystem
Hollows out and obsoletes the MapManager versions
Moves the GridCreationOptions struct to the same namespace as SharedMapSystem and converts it into a record struct
Move the GridCallback delegates to the same namespace as SharedMapSystem
Well, that was less painful than I thought it would be
@TemporalOroboros TemporalOroboros marked this pull request as ready for review May 20, 2026 06:25
@TemporalOroboros TemporalOroboros added T: Cleanup Type: Code cleanup D2: Medium A good amount of codebase knowledge required. A: ECS Core functionality for the ECS S: Needs Review This PR needs to be reviewed before it can be merged. labels May 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A: ECS Core functionality for the ECS D2: Medium A good amount of codebase knowledge required. S: Needs Review This PR needs to be reviewed before it can be merged. T: Cleanup Type: Code cleanup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant