Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Intervention: losing MCV loaded in transport doesn't fail mission #8294

Closed
MER-C opened this issue May 30, 2015 · 4 comments · Fixed by #21398
Closed

Intervention: losing MCV loaded in transport doesn't fail mission #8294

MER-C opened this issue May 30, 2015 · 4 comments · Fixed by #21398

Comments

@MER-C
Copy link

MER-C commented May 30, 2015

Version: Playtest 20150524
Mission: Intervention (Red Alert)

Steps to reproduce:

  1. Proceed normally by building a power plant, ore refinery, naval yard and transport.
  2. Load the MCV into the transport.
  3. Get the transport sunk while carrying the MCV by heading towards the turrets in the top left.

This should fail the mission but doesn't.

@pchote
Copy link
Member

pchote commented May 30, 2015

#8226 puts the framework in place to be able to fix this. A subsequent PR will be able to add an actor disposed trigger, which will be called from all situations that cause an actor to be permanently removed from the world (dying, dying in a transport, selling, etc).

@ghost
Copy link

ghost commented Aug 13, 2018

This should fail the mission but doesn't.

Present in playtest-20180729.

@abcdefg30 abcdefg30 self-assigned this Aug 14, 2018
@Smittytron
Copy link
Member

A subsequent PR will be able to add an actor disposed trigger, which will be called from all situations that cause an actor to be permanently removed from the world (dying, dying in a transport, selling, etc).

I believe the above is done now. Problem is an actor reference won't transfer on transformation, in this case MCV to Conyard and back.

@pchote
Copy link
Member

pchote commented Feb 4, 2021

the mcv disposed trigger could check the map for the presence of a construction yard, and attach a new trigger to that (and vice versa)? If a replacement isn't found, fail.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants