konquets existsing game aborts without warning if new selected from game menu

Bug #1022414 reported by michael
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
kdegames (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

konquest exectutes the following on selecting game->new.
MainWindow::startNewGame()
{
    m_gameView->deleteLater();
    m_game->deleteLater();
    setupGameView();
    m_gameView->startNewGame();
}

By calling setupGameView(); we are aborting the current game regardlless of whether or not the new game dialog is cancled. This should only only be done if GameView::shutdownGame() returns true. Further, since startNewGame is called on the new GameView it will never display confirmation before terminating the existing game.
Edit: Just realized I patched my 4.8.3 copy too.

michael (reeves-87)
description: updated
description: updated
michael (reeves-87)
description: updated
Revision history for this message
michael (reeves-87) wrote :

Sorry I had a little trouble with the interface. Upstream has fixed this in commit 0f4655733289d6703db20d89d9d49b7732d303fd. Please see https://git.reviewboard.kde.org/r/108460/. I am requesting that this patch be back-ported to kde 10 as well. Next kde release will have it if nothing else.

michael (reeves-87)
tags: added: patch-accepted-upstream
Changed in kdegames (Ubuntu):
status: New → Confirmed
status: Confirmed → New
Revision history for this message
michael (reeves-87) wrote :
michael (reeves-87)
Changed in kdegames (Ubuntu):
status: New → Fix Committed
Revision history for this message
michael (reeves-87) wrote :

kubuntu-ppa just got kde 4.10.2 which fixes the issue. Raring should be getting this too https://wiki.kubuntu.org/RaringRingtail/Beta2/Kubuntu.

Changed in kdegames (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.