search

HaywireInteractive / OnAllFronts-Public

UE5 MilSim FPS / RTS Game (formerly ProjectM)
MIT License
126 stars 24 forks source link

Rename all C++ classes/structs to match our new convention of having PM prefix instead of ProjectM #995

Open Kalchev89 opened 3 months ago

Kalchev89 commented 3 months ago

Add the prefix to types that are missing it. Use Core Redirects so we don't have to update Blueprint references. Then as a separate PR we can update Blueprint references and remove the Core Redirects. Talk to Leroy for details.

Ctrl+Shift+H to find/replace in files.

Example PR.

See here for more information about Core Redirects. To remove core redirects later see here.

Kalchev89 commented 2 months ago

I have renamed all the files (Subsystems, evaluators and tasks). The build in VS is successful but in UE5 when try to compile ST_Soldier I get the following error: "MoveToVehicle Task 'Mass Nav Mesh Move Task': Failed to find binding source property 'AgentRadius' for target Task 'Mass Nav Mesh Move Task' DestinationRadius." And i think i got this compiling error before I changed the name of "PMMassNavMeshMoveTask"

Leroy231 commented 2 months ago

I'm reopening this because there are still other classes you need to rename, but you should wait because I wanted to prevent conflicts with some big open PRs.

Leroy231 commented 2 months ago

@Kalchev89 this should be fine to continue on now in the other ProjectM subfolders that you didn't do the renaming yet.