From 3407851c848f60f97309656eee20a71fcf169ed0 Mon Sep 17 00:00:00 2001
From: Lonami Exo <totufals@hotmail.com>
Date: Fri, 10 Feb 2017 13:35:14 +0100
Subject: [PATCH] Fix invalid piece locations after loading a saved game

---
 android/assets/.klooni.sav                    | Bin 446 -> 446 bytes
 .../lonamiwebs/klooni/game/PieceHolder.java   |  43 +++++++++++-------
 2 files changed, 27 insertions(+), 16 deletions(-)

diff --git a/android/assets/.klooni.sav b/android/assets/.klooni.sav
index 2b2abd0e75641d157ae5dcfdeec5111b9b3cbb9c..a62a18db5283555ff619622e3e6024533c05310a 100644
GIT binary patch
delta 78
zcmdnTypMUpat;Ot24)bRxLyH?t-=BnW1qNQ1ti7}!jlCV117HLnB2^$A<WJQ6k!Ds
LAfUj&z<3t`4~q-B

delta 56
wcmdnTypMUp@`)cfCblaelan17t$|o^ay6rYFf$_q0|S^~2QnlX7#QyY07!xlSpWb4

diff --git a/core/src/io/github/lonamiwebs/klooni/game/PieceHolder.java b/core/src/io/github/lonamiwebs/klooni/game/PieceHolder.java
index 0fc3faf..23704cc 100644
--- a/core/src/io/github/lonamiwebs/klooni/game/PieceHolder.java
+++ b/core/src/io/github/lonamiwebs/klooni/game/PieceHolder.java
@@ -90,34 +90,44 @@ public class PieceHolder implements BinSerializable {
 
     // Takes a new set of pieces. Should be called when there are no more piece left
     private void takeMore() {
-        float perPieceWidth = area.width / count;
-        for (int i = 0; i < count; ++i) {
+        for (int i = 0; i < count; ++i)
             pieces[i] = Piece.random();
+        updatePiecesStartLocation();
+
+        if (Klooni.soundsEnabled()) {
+            // Random pitch so it's not always the same sound
+            takePiecesSound.play(1, MathUtils.random(0.8f, 1.2f), 0);
+        }
+    }
+
+    private void updatePiecesStartLocation() {
+        float perPieceWidth = area.width / count;
+        Piece piece;
+        for (int i = 0; i < count; ++i) {
+            piece = pieces[i];
+            if (piece == null)
+                continue;
 
             // Set the absolute position on screen and the cells' cellSize
             // Also clamp the cell size to be the picked size as maximum, or
             // it would be too big in some cases.
-            pieces[i].pos.set(area.x + i * perPieceWidth, area.y);
-            pieces[i].cellSize = Math.min(Math.min(
-                    perPieceWidth / pieces[i].cellCols,
-                    area.height / pieces[i].cellRows), pickedCellSize);
+            piece.pos.set(area.x + i * perPieceWidth, area.y);
+            piece.cellSize = Math.min(Math.min(
+                    perPieceWidth / piece.cellCols,
+                    area.height / piece.cellRows), pickedCellSize);
 
             // Center the piece on the X and Y axes. For this we see how
             // much up we can go, this is, (area.height - piece.height) / 2
-            Rectangle rectangle = pieces[i].getRectangle();
-            pieces[i].pos.y += (area.height - rectangle.height) * 0.5f;
-            pieces[i].pos.x += (perPieceWidth - rectangle.width) * 0.5f;
+            Rectangle rectangle = piece.getRectangle();
+            piece.pos.y += (area.height - rectangle.height) * 0.5f;
+            piece.pos.x += (perPieceWidth - rectangle.width) * 0.5f;
 
             originalPositions[i] = new Rectangle(
-                    pieces[i].pos.x, pieces[i].pos.y,
-                    pieces[i].cellSize, pieces[i].cellSize);
+                    piece.pos.x, piece.pos.y,
+                    piece.cellSize, piece.cellSize);
 
             // Now that we have the original positions, reset the size so it animates and grows
-            pieces[i].cellSize = 0f;
-        }
-        if (Klooni.soundsEnabled()) {
-            // Random pitch so it's not always the same sound
-            takePiecesSound.play(1, MathUtils.random(0.8f, 1.2f), 0);
+            piece.cellSize = 0f;
         }
     }
 
@@ -261,6 +271,7 @@ public class PieceHolder implements BinSerializable {
 
         for (int i = 0; i < count; i++)
             pieces[i] = in.readBoolean() ? Piece.read(in) : null;
+        updatePiecesStartLocation();
     }
 
     //endregion