Mission screen opportunity not fired [Solved]
Moderators: winston, another_commander, Getafix
- Okti
- ---- E L I T E ----
- Posts: 700
- Joined: Sun Sep 26, 2010 1:51 pm
- Location: A GH shop, near witchpoint to Oresrati in Galaxy 8
Mission screen opportunity not fired [Solved]
Hi,
One of the test pilots testing a WIP found an unexpected behaviour in the OXP... when I checked the save file I came to the conclusion that when the arrival report is displayed and if the player presses F2 instead of space, the mission screens are not displayed. That means missionScreenOpportunity is not fired. If the player launches and docks again the mission screen for the related stage will be displayed. Thanks to Wyvern for finding this bug.
One of the test pilots testing a WIP found an unexpected behaviour in the OXP... when I checked the save file I came to the conclusion that when the arrival report is displayed and if the player presses F2 instead of space, the mission screens are not displayed. That means missionScreenOpportunity is not fired. If the player launches and docks again the mission screen for the related stage will be displayed. Thanks to Wyvern for finding this bug.
Last edited by Okti on Sat Jul 30, 2011 5:53 pm, edited 1 time in total.
- Fatleaf
- Intergalactic Spam Assassin
- Posts: 1988
- Joined: Tue Jun 08, 2010 5:11 am
- Location: In analysis mode on Phaelon
- Contact:
Re: Mission screen opportunity not fired from arrival report
Just tested and can confirm this.
Find out about the early influences of Fatleaf here. Also his OXP's!
Holds the Ooniversal record for "Thread Necromancy"
Holds the Ooniversal record for "Thread Necromancy"
- Eric Walch
- Slightly Grand Rear Admiral
- Posts: 5536
- Joined: Sat Jun 16, 2007 3:48 pm
- Location: Netherlands
Re: Mission screen opportunity not fired from arrival report
In playerEntitiesControls.m are the missionscreens handled special so they only accept certain keys. Probably the same should happen for report screens. Best place to fix is probably line 3110:
into
Untested though and only changing here won't be enough. And maybe it needs an even more complex fix with the risk of breaking other stuff.
Code: Select all
if (gui_screen == GUI_SCREEN_MISSION)
Code: Select all
if (gui_screen == GUI_SCREEN_MISSION || gui_screen == GUI_SCREEN_REPORT)
UPS-Courier & DeepSpacePirates & others at the box and some older versions
Re: Mission screen opportunity not fired from arrival report
Meh, I've just had some fun with semi-random breakages related to new event functionality, so I couldn't agree more!Eric Walch wrote:And maybe it needs an even more complex fix with the risk of breaking other stuff.
Hey, free OXPs: farsun v1.05 & tty v0.5! :0)
- Eric Walch
- Slightly Grand Rear Admiral
- Posts: 5536
- Joined: Sat Jun 16, 2007 3:48 pm
- Location: Netherlands
Re: Mission screen opportunity not fired from arrival report
Therefor I want to do some longer testing before committing.Kaks wrote:Meh, I've just had some fun with semi-random breakages related to new event functionality, so I couldn't agree more!
First tests I just did, show that it works as wanted. The report page now only accepts a spacebar and the F1 key like the missionscreen. And missionscreens are triggered correct after leaving the report page.
UPS-Courier & DeepSpacePirates & others at the box and some older versions
- Eric Walch
- Slightly Grand Rear Admiral
- Posts: 5536
- Joined: Sat Jun 16, 2007 3:48 pm
- Location: Netherlands
Re: Mission screen opportunity not fired from arrival report
Fixed in r4586.
The report screen now only accepts the spacebar as input (And the F-1 key to launch).
It was the addition of a single check after all that is not likely to interfere with other stuff.
I could not find any incompatibilities, not even when testing legacy mission screens.
The report screen now only accepts the spacebar as input (And the F-1 key to launch).
It was the addition of a single check after all that is not likely to interfere with other stuff.
I could not find any incompatibilities, not even when testing legacy mission screens.
UPS-Courier & DeepSpacePirates & others at the box and some older versions
-
- Quite Grand Sub-Admiral
- Posts: 6683
- Joined: Wed Feb 28, 2007 7:54 am
Re: Mission screen opportunity not fired from arrival report
Eric, I think it's fixed in r4588.
- Okti
- ---- E L I T E ----
- Posts: 700
- Joined: Sun Sep 26, 2010 1:51 pm
- Location: A GH shop, near witchpoint to Oresrati in Galaxy 8
Re: Mission screen opportunity not fired from arrival report
Thanks for the efforts
- Eric Walch
- Slightly Grand Rear Admiral
- Posts: 5536
- Joined: Sat Jun 16, 2007 3:48 pm
- Location: Netherlands
Re: Mission screen opportunity not fired from arrival report
Yep, that was my commit number. Somehow there is a lot of update action the past days so that we are further than I thought.another_commander wrote:Eric, I think it's fixed in r4588.
And I have still one other in the pipeline with a correct size check on launching ships. It's fully ready but I want to find some time to do some in-game tests with a few models that just fit.
UPS-Courier & DeepSpacePirates & others at the box and some older versions