widelands-dev team mailing list archive

["Jens Beyer \(Qcumber-some\)" <Qcumber-some@xxxxxxxxxxxxx>]

Jens Beyer (Qcumber-some) has proposed merging lp:~qcumber-some/widelands/bug1167242 into lp:widelands.

Requested reviews:
  Widelands Developers (widelands-dev)
Related bugs:
  Bug #1167242 in widelands: "Ctrl+destroying a flag with a building does not destroy the whole road"
  https://bugs.launchpad.net/widelands/+bug/1167242

For more details, see:
https://code.launchpad.net/~qcumber-some/widelands/bug1167242/+merge/210000

Was this really this easy?
I did not find a problem although I tried to test every possible combination.
-- 
https://code.launchpad.net/~qcumber-some/widelands/bug1167242/+merge/210000
Your team Widelands Developers is requested to review the proposed merge of lp:~qcumber-some/widelands/bug1167242 into lp:widelands.

=== modified file 'src/wui/fieldaction.cc'
--- src/wui/fieldaction.cc	2014-03-03 20:50:30 +0000
+++ src/wui/fieldaction.cc	2014-03-07 19:16:40 +0000
@@ -750,7 +750,7 @@
 			if (building->get_playercaps() & Building::PCap_Bulldoze) {
 				if (get_key_state(SDLK_LCTRL) or get_key_state(SDLK_RCTRL)) {
 					ref_cast<Game, Editor_Game_Base>(egbase).send_player_bulldoze
-						(*flag);
+						(*flag, get_key_state(SDLK_LCTRL) or get_key_state(SDLK_RCTRL));
 				}
 				else {
 					show_bulldoze_confirm