[Kde-games-devel] Review Request: Konquest: Implement standing orders

Jeffrey Kelling overlordapophis at gmail.com
Tue Sep 13 18:58:24 UTC 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/6805/
-----------------------------------------------------------

Review request for KDE Games and Pierre Ducroquet.


Summary
-------

This patch implements standing orders, i.e. orders that are repeated each turn if possible. Standing orders are issued the same way as normal order, additionally the added check box has to be checked. They are also removed the same way as normal orders (in the fleet overview). The orders are executed at the end of turn if enough ships are available, if not a message is issued. If the Planet is no longer under the players control the order is deleted. 


This addresses bug 87882.
    https://bugs.kde.org/show_bug.cgi?id=87882


Diffs
-----

  trunk/KDE/kdegames/konquest/fleet.h 1253268 
  trunk/KDE/kdegames/konquest/fleet.cc 1253268 
  trunk/KDE/kdegames/konquest/fleetdlg.cc 1253268 
  trunk/KDE/kdegames/konquest/gamelogic.cc 1253268 
  trunk/KDE/kdegames/konquest/gameview.h 1253268 
  trunk/KDE/kdegames/konquest/gameview.cc 1253268 
  trunk/KDE/kdegames/konquest/player.h 1253268 
  trunk/KDE/kdegames/konquest/player.cc 1253268 

Diff: http://svn.reviewboard.kde.org/r/6805/diff


Testing
-------

Played the game, issued standing orders, ...


Screenshots
-----------

Spawn Fleet
  http://svn.reviewboard.kde.org/r/6805/s/621/
Fleet Overview
  http://svn.reviewboard.kde.org/r/6805/s/622/


Thanks,

Jeffrey

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20110913/5060db64/attachment.html>


More information about the kde-games-devel mailing list