Opened 6 years ago

Closed 6 years ago

#9982 closed bug (fixed)

[Installer] doesn't quit correctly on alt+w at EULA window

Reported by: diver Owned by: stpere
Priority: normal Milestone: R1
Component: Applications/Installer Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: yes Platform: All

Description

Closing Installer using alt+w while it shows EULA window makes it hang in the Deskbar. Used to work fine in A3.

Attachments (3)

0001-installer-eula-fix.patch (17.1 KB ) - added by Freeman 6 years ago.
0001-installer-eula-fix-goes-to-next-window.2.patch (17.4 KB ) - added by Freeman 6 years ago.
0001-installer-eula-fix-goes-to-next-window.patch (17.4 KB ) - added by Freeman 6 years ago.

Download all attachments as: .zip

Change History (15)

comment:1 by stpere, 6 years ago

Resolution: fixed
Status: newclosed

should be fixed in hrev46245. Thanks!

comment:2 by diver, 6 years ago

Resolution: fixed
Status: closedreopened

The fix was reverted in hrev46248.

comment:3 by diver, 6 years ago

Owner: changed from korli to stpere
Status: reopenedassigned

comment:4 by axeld, 6 years ago

Should it really quit when closing the window? Wouldn't it make more sense if it would just continue to the application? I mean, there is still Alt-Q to close the application if that's your intent.

comment:5 by diver, 6 years ago

As long as it doesn't just hang in Deskbar without any window it should be fine ;-)

comment:6 by siarzhuk, 6 years ago

Are there any objections to publish this puzzle as GCI 2013 task? ;-)

comment:7 by pulkomandy, 6 years ago

I vote for having it continue with the installation as well. And, yes, let's see what GCI students can come up with.

comment:8 by Freeman, 6 years ago

Has a Patch: set

by Freeman, 6 years ago

comment:9 by Freeman, 6 years ago

Two patches are included. One to fix the original exit bug, the other to open the installation window after the EULA window has been closed. Time to pick...

comment:10 by korli, 6 years ago

A few remarks:

  • The line "BString infoText;" in your patches is misaligned.
  • The copyright header seems wrong formatted (missing comma and period).
  • In a .cpp there are two blank lines after the copyright header.
  • Line 7, 9 and 13 of the EULAWindow.h should be also two blank lines.
  • EULAWindow doesn't need member attributes (local variables are OK).

Thanks!

comment:11 by Freeman, 6 years ago

Is anyone getting crashes at InstallerApp::MessageReceived()? The current mentor is getting them but I can't replicate the bug.

comment:12 by siarzhuk, 6 years ago

Resolution: fixed
Status: assignedclosed

Applied in hrev46670. Thanks.

Note: See TracTickets for help on using tickets.