Bug #15306
Incorrect ScreenMessage shown for failed EVA out of noAutoEVA part using CrewHatchDialog
0%
![Arrow u r green](/plugin_assets/vote_on_issues/images/arrow-u-r-green.png?1560451728)
![Arrow d r red](/plugin_assets/vote_on_issues/images/arrow-d-r-red.png?1560451728)
Description
In general, clicking on an airlock and using the button in CrewHatchDialog to attempt EVA before it has been unlocked via Astronaut Complex shows the following ScreenMessage
"Cannot disembark while off of Kerbin's surface.\nAstronaut Complex upgrade required."
However, doing the same with parts that are noAutoEVA (i.e. MK1CrewCabin) will instead show
"Automatic EVA disabled. Activate a hatch."
Which is the wrong error message, since the user has indeed attempted EVA by manually using the crew hatch.
[This error message is intended for the tooltip on the EVA button available in KerbalPortrait. That does work correctly.]
The root cause of this is most likelyGameVariables.GetEVALockedReason(Vessel v, ProtoCrewMember crew)
The method does not take any parameter that provides context for whether the request for EVA originated from KerbalPortrait or CrewHatchDialog.
Steps to reproduce:
- start new career game, make stock aircraft available
- cheat in science and funds as needed
- unlock tech/parts needed for "Velociteze" stock aircraft
- do not upgrade Astronaut Complex
- launch "Velociteze" fully crewed, take off from runway
- click on the crew hatch of Mark1Cockpit to bring up CrewHatchDialog
- click EVA button, note correct ScreenMessage
- click on the forward crew hatch of the MK1CrewCabin (leads into hollow structural fuselage) to bring up CrewHatchDialog
- click EVA button, note incorrect ScreenMessage
History
#1
Updated by cake-pie almost 7 years ago
- Platform Linux, OSX added
- Expansion Core Game added
- Language deleted (
English (US))
Issue still present in 1.4.3