# HG changeset patch
# User Wuzzy <Wuzzy2@mail.ru>
# Date 1508602887 -7200
# Node ID 745d5c9a3abc9b8137b7c0c70b776f2ece1c9571
# Parent  23fb2179945ffce9db0b7d7927dcb33774ba01b0
Sniper rifle training: Fix general camera weirdness

- Camera bounced back and forth between shots
- FollowGear was called every 20ms (way too often)
- Now camera only re-centers when using sniper rifle
- Camera also centers to dynamite once

diff -r 23fb2179945f -r 745d5c9a3abc ChangeLog.txt
--- a/ChangeLog.txt	Sat Oct 21 17:21:34 2017 +0200
+++ b/ChangeLog.txt	Sat Oct 21 18:21:27 2017 +0200
@@ -264,6 +264,7 @@
  * Climb Home: Fix game getting stuck when a player reached home in multiplayer
  * Climb Home: Fix Lua error messages and broken stats screen
  * Sniper rifle target practice: Fix targets spawning twice in dynamite scenes
+ * Sniper rifle target practice: Fix auto-camera generally behaving weird
 
 Frontend:
  + Campaign screen shows which campaigns and missions you've completed so far
diff -r 23fb2179945f -r 745d5c9a3abc share/hedgewars/Data/Missions/Challenge/Basic_Training_-_Sniper_Rifle.lua
--- a/share/hedgewars/Data/Missions/Challenge/Basic_Training_-_Sniper_Rifle.lua	Sat Oct 21 17:21:34 2017 +0200
+++ b/share/hedgewars/Data/Missions/Challenge/Basic_Training_-_Sniper_Rifle.lua	Sat Oct 21 18:21:27 2017 +0200
@@ -39,8 +39,6 @@
 -- Used to calculate final target score
 local score_bonus = 0
 
-local last_hit_time = 0
-
 local cinematic = false
 
 -- Number of dynamite gears currently in game
@@ -69,12 +67,15 @@
 end
 
 -- Cut sequence to blow up land with dynamite
-function blowUp(x, y)
+function blowUp(x, y, follow)
 	if cinematic == false then
 		cinematic = true
 		SetCinematicMode(true)
 	end
-	AddGear(x, y, gtDynamite, 0, 0, 0, 0)
+	local dyna = AddGear(x, y, gtDynamite, 0, 0, 0, 0)
+	if follow then
+		FollowGear(dyna)
+	end
 end
 
 function onNewTurn()
@@ -148,14 +149,6 @@
 	if game_lost then
 		return
 	end
-	-- after a target is destroyed, show hog, then target
-	if (target ~= nil) and (TurnTimeLeft + 1300 < last_hit_time) then
-		-- move camera to the target
-		FollowGear(target)
-	elseif TurnTimeLeft + 300 < last_hit_time then
-		-- move camera to the hog
-		FollowGear(player)
-	end
 	-- If time's up, set the game to be lost.
 	-- We actually check the time to be "1 ms" as it
 	-- will be at "0 ms" right at the start of the game.
@@ -194,6 +187,16 @@
 	SetAmmo(amSniperRifle, 9, 0, 0, 0)
 end
 
+--[[ Re-center camera to target after using sniper rifle.
+This makes it easier to find the target. If we don't
+do this, the camera would contantly bounce back to
+the hog which would be annoying. ]]
+function onAttack()
+	if target and GetCurAmmoType() == amSniperRifle then
+		FollowGear(target)
+	end
+end
+
 -- This function is called when a new gear is added.
 -- We use it to count the number of shots, which we
 -- in turn use to calculate the final score and stats
@@ -236,8 +239,6 @@
 
 	if gt == gtTarget then
 		target = nil
-		-- remember when the target was hit for adjusting the camera
-		last_hit_time = TurnTimeLeft
 		-- Add one point to our score/counter
 		score = score + 1
 		score_bonus = score_bonus + 1
@@ -258,7 +259,7 @@
 				blowUp(1440,1595)
 				blowUp(1527,1575)
 				blowUp(1614,1595)
-				blowUp(1420,1675)
+				blowUp(1420,1675, true)
 				blowUp(1527,1675)
 				blowUp(1634,1675)
 				blowUp(1440,1755)
@@ -292,7 +293,7 @@
 				blowUp(780,911)
 				blowUp(920,911)
 				blowUp(1060,913)
-				blowUp(1198,913)
+				blowUp(1198,913, true)
 				spawnTargetDelayed(1200,830)
 			elseif score == 12 then
 				spawnTarget(1430,450)
@@ -308,7 +309,7 @@
 				blowUp(2210,920)
 				blowUp(2200,305)
 				blowUp(2300,305)
-				blowUp(2300,400)
+				blowUp(2300,400, true)
 				blowUp(2300,500)
 				blowUp(2300,600)
 				blowUp(2300,700)
@@ -328,7 +329,7 @@
 				blowUp(2930,305)
 				blowUp(3060,305)
 				blowUp(3190,305)
-				blowUp(3310,305)
+				blowUp(3310,305, true)
 				blowUp(3393,613)
 				blowUp(2805,370)
 				blowUp(2805,500)