Restart campaign doesn't clear assets at locations

[Rushyo]

Clicking 'restart campaign' doesn't seem to clear the OPFOR assets at locations, so restarted campaigns are always harder than the first one and get harder the more you do it.

Workaround is to delete the campaign files and restart the application.

Issue is present in 1.5 RC1

[JohanAberg]

I can reproduce this.
The issue appears to be the ConflictTheater class implementations.

dcs_liberation/theater/persiangulf.py

Line 24 in 63da350

al_minhad = ControlPoint.from_airport(persiangulf.Al_Minhad_AB, LAND, SIZE_REGULAR, 1.1)

The control points are store as class attributes (apposed to instance attributes), so they will be remembered even when a new instance of say PersianGulfTheater is made after File > Restart Campain.
I was able to get the right/expected behaviour by making the ControlPoints instance attributes.

[Khopa]

Reproduced and fixed in my dev branch.

https://github.com/Khopa/dcs_liberation/blob/fbd01fbfdbe204dc36a64829471101a15b30c981/theater/start_generator.py#L27-L32

Another related issue is that "strike targets' (namely 'ground objects') were not cleared either.
Also fixed in my dev branch.

https://github.com/Khopa/dcs_liberation/blob/fbd01fbfdbe204dc36a64829471101a15b30c981/theater/start_generator.py#L86-L87

In the meantime, you can also fall back to the old way of restarting the game, that consist of deleting the save file : C:/Users//Saved Games/dcs/liberation_save. And then restart DCS liberation to get a clean new start.

[shdwp]

@Khopa are you able to merge into develop and master? I think I gave you maintainer access, meaning that you should be able to make releases as well.

[Khopa]

Yes i think i should be able to do this it when needed.

But the version on my branch is not stable yet, moreover I and @wrycu have some pending changes on our branches as well, so personally, i am not planning to make a release immediately.