[Kde-games-devel] Review Request 114674: Fix bug 310411: Crash when selecting 'Game>Restart puzzle'

[Albert Astals Cid]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/114674/#review46195
-----------------------------------------------------------

Ship it!


Makes sense :-)

Any reason you only want to commit to master and not KDE/4.11 and then merge to 4.12 and master?

- Albert Astals Cid


On Dec. 26, 2013, 9:49 p.m., Ian Wadham wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/114674/
> -----------------------------------------------------------
> 
> (Updated Dec. 26, 2013, 9:49 p.m.)
> 
> 
> Review request for KDE Games.
> 
> 
> Bugs: 310411
>     http://bugs.kde.org/show_bug.cgi?id=310411
> 
> 
> Repository: palapeli
> 
> 
> Description
> -------
> 
> The problem occurs because no puzzle has been selected and loaded when Palapeli starts, therefore there is no puzzle to "restart".
> 
> The solution is to enable 'Game>Restart puzzle' only when a puzzle has been selected and loaded and is being solved (the action makes no sense in Palapeli's Collection mode), also to return immediately from Palapeli::Scene:restartPuzzle() if, for any other reason, no puzzle has been successfully loaded. 
> 
> 
> Diffs
> -----
> 
>   src/engine/scene.cpp 8be0925 
>   src/window/mainwindow.cpp e3305fb 
> 
> Diff: https://git.reviewboard.kde.org/r/114674/diff/
> 
> 
> Testing
> -------
> 
> Tried 'Game>Restart puzzle' from Collection mode (no action) and from Puzzle mode (puzzle pieces are dismantled and re-shuffled, as expected).
> 
> 
> Thanks,
> 
> Ian Wadham
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20131227/364e87f7/attachment.html>