Mission screen opportunity not fired [Solved]

For test results, bug reports, announcements of new builds etc.

Moderators: winston, another_commander, Getafix

Post Reply
User avatar
Okti
---- E L I T E ----
---- 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]

Post by Okti »

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. :D
Last edited by Okti on Sat Jul 30, 2011 5:53 pm, edited 1 time in total.
My OXP's
And Latest Mission Coyote's Run
User avatar
Fatleaf
Intergalactic Spam Assassin
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

Post by Fatleaf »

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"
User avatar
Eric Walch
Slightly Grand Rear Admiral
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

Post by Eric Walch »

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:

Code: Select all

if (gui_screen == GUI_SCREEN_MISSION)
into

Code: Select all

if (gui_screen == GUI_SCREEN_MISSION || gui_screen == GUI_SCREEN_REPORT)
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.
User avatar
Kaks
Quite Grand Sub-Admiral
Quite Grand Sub-Admiral
Posts: 3009
Joined: Mon Jan 21, 2008 11:41 pm
Location: The Big Smoke

Re: Mission screen opportunity not fired from arrival report

Post by Kaks »

Eric Walch wrote:
And maybe it needs an even more complex fix with the risk of breaking other stuff.
Meh, I've just had some fun with semi-random breakages related to new event functionality, so I couldn't agree more! :)
Hey, free OXPs: farsun v1.05 & tty v0.5! :0)
User avatar
Eric Walch
Slightly Grand Rear Admiral
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

Post by Eric Walch »

Kaks wrote:
Meh, I've just had some fun with semi-random breakages related to new event functionality, so I couldn't agree more! :)
Therefor I want to do some longer testing before committing.

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.
User avatar
Eric Walch
Slightly Grand Rear Admiral
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

Post by Eric Walch »

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.
another_commander
Quite Grand Sub-Admiral
Quite Grand Sub-Admiral
Posts: 6683
Joined: Wed Feb 28, 2007 7:54 am

Re: Mission screen opportunity not fired from arrival report

Post by another_commander »

Eric, I think it's fixed in r4588.
User avatar
Okti
---- E L I T E ----
---- 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

Post by Okti »

Thanks for the efforts
My OXP's
And Latest Mission Coyote's Run
User avatar
Eric Walch
Slightly Grand Rear Admiral
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

Post by Eric Walch »

another_commander wrote:
Eric, I think it's fixed in r4588.
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. :D
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.
Post Reply