From 4652a2170320e067946db03c8f767bd304a3a543 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Nicolas=20Hord=C3=A9?= <nicolas@palon.fr>
Date: Mon, 27 Jun 2016 13:41:06 +0200
Subject: [PATCH] fix: correction importante du bogue d'enregistrement sous
 android (execsql au lieu de rawqsql) ajout de la suppression dynamique des
 liens correction de bogues pour la duplication de mondes

---
 core/src/fr/evolving/automata/Level.java      |  2 +
 core/src/fr/evolving/automata/Worlds.java     | 75 +++++++++++++++++--
 core/src/fr/evolving/database/LocalBase.java  | 18 ++---
 core/src/fr/evolving/screens/LevelScreen.java | 40 +++-------
 4 files changed, 89 insertions(+), 46 deletions(-)

diff --git a/core/src/fr/evolving/automata/Level.java b/core/src/fr/evolving/automata/Level.java
index 607b52c..14f6061 100644
--- a/core/src/fr/evolving/automata/Level.java
+++ b/core/src/fr/evolving/automata/Level.java
@@ -55,6 +55,7 @@ public class Level implements Serializable,Cloneable {
 		this.Element = Element;
 		this.rewards = Current;
 		this.Victory_orig = Victory;
+		this.Victory = Victory;
 		this.X = X;
 		this.Y = Y;
 		this.Tech = Tech;
@@ -72,6 +73,7 @@ public class Level implements Serializable,Cloneable {
 		this.Special = Special;
 		this.Tuto = Tuto;
 		this.Link = Link;
+		this.Locked=true;
 	}
 	
 	public Object clone() {
diff --git a/core/src/fr/evolving/automata/Worlds.java b/core/src/fr/evolving/automata/Worlds.java
index b603a02..b25113b 100644
--- a/core/src/fr/evolving/automata/Worlds.java
+++ b/core/src/fr/evolving/automata/Worlds.java
@@ -22,6 +22,7 @@ public class Worlds extends Actor {
 	private Level lastchange;
 	
 	public enum State {pause,simulating,notloaded,databasefailed};
+	public enum LinkDelMethod {all,in,out,rebase};
 	
 	public Worlds(String campaign) {
 		name=campaign;
@@ -193,17 +194,22 @@ public class Worlds extends Actor {
 		usedlevel.Grid_orig = (Grid)usedlevel.Grid.clone();
 	}
 	
-	public Level findLevel(int levelid) {
+	public Level findLevel(int worldid, int levelid) {
 		if (state!=State.notloaded) 
-			if (usedworld>=0) {
-				Array<Level> tempworld=getLevels();
-				for(Level level:tempworld)
-					if (level.aLevel==levelid)
-						return level;
+			if (worldid>=0) {
+				Array<Level> tempworld=getLevels(worldid);
+				if (tempworld!=null)
+						for(Level level:tempworld)
+							if (level.aLevel==levelid)
+								return level;
 			}
 		return null;
 	}
 	
+	public Level findLevel(int levelid) {
+		return findLevel(usedworld, levelid);
+	}
+	
 	public void setLevel(int levelid) {
 		Level level=findLevel(levelid);
 		if (level!=null)
@@ -252,6 +258,63 @@ public class Worlds extends Actor {
 		return lastchange;
 	}
 	
+	public void delLink(int levelid, LinkDelMethod atype) {
+		Level level = findLevel(levelid);
+		if (level!=null) {
+			if (atype==LinkDelMethod.all || atype==LinkDelMethod.out)
+				level.Link=new int[][] {};
+			if (atype==LinkDelMethod.all || atype==LinkDelMethod.in)
+				if (levels!=null)
+					for(Level alevel:levels)
+						if (alevel!=null) {
+							Array<int[]> links=new Array<int[]>(alevel.Link);
+							for(int[] link: links)
+								if (link.length==2 && link[0]==level.aWorld && link[1]==level.aLevel) 
+									links.removeValue(link, true);
+							alevel.Link=links.toArray();
+						}
+			if  (atype==LinkDelMethod.rebase) {
+ 				Array<int[]> links=new Array<int[]>();
+ 				Array<int[]> templinks=new Array<int[]>(level.Link);
+				for(int[] link: templinks) {
+					if (link.length==2) {
+						Level alevel = findLevel(link[0],link[1]);
+						if (alevel!=null)
+							links.add(new int[]{link[0],link[1],(int) alevel.X,(int) alevel.Y});
+					}
+				}
+				level.Link=new int[][]{};
+				for(Level blevel:levels)
+					if (blevel!=null) {
+						Array<int[]> alinks=new Array<int[]>(blevel.Link);
+						for(int[] alink: alinks)
+						if (alink.length==2 && alink[0]==level.aWorld && alink[1]==level.aLevel) {
+							int distance=1000000000;
+							int choosedlevel[]=null;
+							for(int[] link: links) {
+								int distancetemp=(int)Math.sqrt((blevel.X-link[2])*(blevel.X-link[2])+(blevel.Y-link[3])*(blevel.Y-link[3]));
+								if (distancetemp<distance)
+									choosedlevel=link;
+									distance=distancetemp;
+							}
+							if (choosedlevel!=null) {
+								alink[0]=choosedlevel[0];
+								alink[1]=choosedlevel[1];			
+							}
+							else
+								links.removeValue(alink, true);
+							blevel.Link=alinks.toArray();
+						}
+					}
+			}
+		}		
+	}
+	
+	public void delLink(LinkDelMethod atype) {
+		delLink(usedlevel.aLevel, atype);
+	}
+	
+	
 	public void addLevel(Level level) {
 		levels.add(level);
 		onchanged(level);
diff --git a/core/src/fr/evolving/database/LocalBase.java b/core/src/fr/evolving/database/LocalBase.java
index e030ceb..072d26a 100644
--- a/core/src/fr/evolving/database/LocalBase.java
+++ b/core/src/fr/evolving/database/LocalBase.java
@@ -76,7 +76,7 @@ public class LocalBase extends Base {
 				dbHandler.openOrCreateDatabase();
 			} catch (SQLiteGdxException e) {
 				e.printStackTrace();
-				Gdx.app.log("wirechem-LocalBase", "Erreur à l'ouverture de la base");
+				Gdx.app.error("wirechem-LocalBase", "Erreur à l'ouverture de la base");
 			}
 		}
 		try {
@@ -140,7 +140,7 @@ public class LocalBase extends Base {
 
 	public boolean deleteCampaign(String description) {
 		try {
-			dbHandler.rawQuery("delete from worlds where desc='" + description
+			dbHandler.execSQL("delete from worlds where desc='" + description
 					+ "';");
 		} catch (SQLiteGdxException e) {
 			return false;
@@ -159,7 +159,7 @@ public class LocalBase extends Base {
 			bos.close();
 			byte[] bytes = bos.toByteArray();
 			encoded = Base64Coder.encodeLines(bytes);
-			dbHandler.rawQuery("replace into worlds (desc,object) values ('"
+			dbHandler.execSQL("replace into worlds (desc,object) values ('"
 					+ description + "','" + encoded + "');");
 		} catch (Exception e) {
 			return false;
@@ -185,7 +185,7 @@ public class LocalBase extends Base {
 
 	public boolean setLevelunlock(int user, int level) {
 		try {
-			dbHandler.rawQuery("insert into locks (user,level) values (" + user
+			dbHandler.execSQL("insert into locks (user,level) values (" + user
 					+ "," + level + ");");
 		} catch (SQLiteGdxException e) {
 			return false;
@@ -237,7 +237,7 @@ public class LocalBase extends Base {
 			byte[] bytes = bos.toByteArray();
 			encoded = Base64Coder.encodeLines(bytes);
 			dbHandler
-					.rawQuery("replace into transmuters (user,object) values ("
+					.execSQL("replace into transmuters (user,object) values ("
 							+ user + ",'" + encoded + "');");
 		} catch (Exception e) {
 			return false;
@@ -262,7 +262,7 @@ public class LocalBase extends Base {
 
 	public boolean setResearchpoint(int user, int point) {
 		try {
-			dbHandler.rawQuery("replace into research (user,value) values ("
+			dbHandler.execSQL("replace into research (user,value) values ("
 					+ user + "," + point + ");");
 		} catch (Exception e) {
 			return false;
@@ -332,7 +332,7 @@ public class LocalBase extends Base {
 			bos.close();
 			byte[] bytes = bos.toByteArray();
 			encoded = Base64Coder.encodeLines(bytes);			
-			dbHandler.rawQuery("insert into grids (user,level,object) values ("
+			dbHandler.execSQL("insert into grids (user,level,object) values ("
 					+ user + "," + level + ",'" + encoded + "');");
 		} catch (Exception e) {
 			return false;
@@ -352,12 +352,12 @@ public class LocalBase extends Base {
 			byte[] bytes = bos.toByteArray();
 			encoded = Base64Coder.encodeLines(bytes);
 			try {
-				dbHandler.rawQuery("delete from grids where user=" + user
+				dbHandler.execSQL("delete from grids where user=" + user
 						+ " and level=" + level + " and tag='" + tag + "';");
 			} catch (Exception e) {
 			}
 			dbHandler
-					.rawQuery("insert into grids (user,level,tag,object) values ("
+					.execSQL("insert into grids (user,level,tag,object) values ("
 							+ user
 							+ ","
 							+ level
diff --git a/core/src/fr/evolving/screens/LevelScreen.java b/core/src/fr/evolving/screens/LevelScreen.java
index dc850cf..0c7828e 100644
--- a/core/src/fr/evolving/screens/LevelScreen.java
+++ b/core/src/fr/evolving/screens/LevelScreen.java
@@ -42,6 +42,7 @@ import fr.evolving.automata.Grid;
 import fr.evolving.automata.Level;
 import fr.evolving.automata.Transmuter;
 import fr.evolving.automata.Worlds;
+import fr.evolving.automata.Worlds.LinkDelMethod;
 import fr.evolving.automata.Worlds.State;
 import fr.evolving.database.Base;
 import fr.evolving.dialogs.WarningDialog;
@@ -187,7 +188,11 @@ public class LevelScreen implements Screen {
 				if (level.Name.isEmpty())
 					level.Name=AssetLoader.language.get("[level"+(level.aWorld+1)+"/"+(level.aLevel+1)+"-name]");
 				if (level.Description.isEmpty())
-					level.Description=AssetLoader.language.get("[level"+(level.aWorld+1)+"/"+(level.aLevel+1)+"-desc]");		
+					try {
+						level.Description=AssetLoader.language.get("[level"+(level.aWorld+1)+"/"+(level.aLevel+1)+"-desc]");
+					}
+					catch (Exception E) {}
+					finally {level.Description="";}
 				ButtonLevel buttonlevel= new ButtonLevel(level, AssetLoader.ratio, true);
 				buttonLevels.add(buttonlevel);
 				if (worlds.isDebug()) buttonlevel.setDisabled(false);
@@ -813,25 +818,6 @@ public class LevelScreen implements Screen {
 			@Override
 			public void clicked(InputEvent event, float x, float y) {
 				if (selected!=null) {
-						for (int i=0;i<buttonLevels.size;i++) {
-							ButtonLevel button=buttonLevels.get(i);
-								Array<int[]> links=new Array<int[]>(button.level.Link);
-								for(int[] link: links)
-									if (link.length==2 && link[0]==selected.level.aWorld && link[1]==selected.level.aLevel) 
-									{
-										if (i==buttonLevels.size-1)
-											links.removeValue(link, true);
-										else {
-											for (int j=i+1;j<buttonLevels.size;j++) {
-												if (buttonLevels.get(j)!=null) {
-													link[1]=j;
-													break;
-												}
-											}
-										}
-									}
-								button.level.Link=links.toArray();
-						}
 						Gdx.app.debug("wirechem-LevelScreen", "Destruction du bouton :"+selected.level.aLevel);
 						worlds.delLevel(selected.level.aLevel);
 						selectone();
@@ -844,16 +830,8 @@ public class LevelScreen implements Screen {
 			@Override
 			public void clicked(InputEvent event, float x, float y) {
 				if (selected!=null) {
-						for (ButtonLevel button : buttonLevels) {
-								Array<int[]> links=new Array<int[]>(button.level.Link);
-								for(int[] link: links)
-									if (link.length==2 && link[0]==selected.level.aWorld && link[1]==selected.level.aLevel) 
-									{
-										Gdx.app.debug("wirechem-LevelScreen", "Destruction du lien :"+selected.level.aLevel);
-										links.removeValue(link, true);
-									}
-								button.level.Link=links.toArray();
-						}
+						Gdx.app.debug("wirechem-LevelScreen", "Destruction des liens :"+selected.level.aLevel);
+						worlds.delLink(selected.level.aLevel, LinkDelMethod.rebase);
 					}
 				}
 			});
@@ -937,7 +915,7 @@ public class LevelScreen implements Screen {
 			}
 		});
 		vertibar=new VerticalGroup();
-		vertibar.setPosition(1600, AssetLoader.height-500);
+		vertibar.setPosition(1600, AssetLoader.height-100);
 		vertibar.center();
 		vertibar.addActor(databaseSave);
 		vertibar.space(20f);