Fixed an issue with boolean values on local DB sync.

Added support for SYNC_CONFIGURATION table on for local DB sync.
Fixed an issue with multiple row updates on remote DB sync.
Added warning for missing destination property on remote DB sync.
Throw exception when no remote machine found on remote DB sync.

8 files changed, 156 insertions, 20 deletions

diff --git a/Software/Visual_Studio/Tango.Synchronization/IDBComparer.cs b/Software/Visual_Studio/Tango.Synchronization/IDBComparer.cs
new file mode 100644
index 000000000..7d7ae4af9
--- /dev/null
+++ b/Software/Visual_Studio/Tango.Synchronization/IDBComparer.cs
@@ -0,0 +1,13 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace Tango.Synchronization
+{
+    public interface IDBComparer : IDisposable
+    {
+        List<Diff> Compare();
+    }
+}
diff --git a/Software/Visual_Studio/Tango.Synchronization/Local/Constants.cs b/Software/Visual_Studio/Tango.Synchronization/Local/Constants.cs
index ed8084301..5557c43c8 100644
--- a/Software/Visual_Studio/Tango.Synchronization/Local/Constants.cs
+++ b/Software/Visual_Studio/Tango.Synchronization/Local/Constants.cs
@@ -14,6 +14,9 @@ namespace Tango.Synchronization.Local
         public const String COLUMN_TYPE = "type";
         public const String IS_NOT_NULL = "notnull";
         public const String DEFAULT_VALUE = "dflt_value";
+        public const String SYNC_CONFIGURATIONS_TABLE_NAME = "SYNC_CONFIGURATION";
+        public const String SYNC_CONFIGURATION_TABLE_NAME_COLUMN = "TABLE_NAME";
+        public const String SYNC_CONFIGURATION_SYNC_TYPE_COLUMN = "SYNC_TYPE";
 
         //Must Column Names.
 
diff --git a/Software/Visual_Studio/Tango.Synchronization/Local/ILocalDataBase.cs b/Software/Visual_Studio/Tango.Synchronization/Local/ILocalDataBase.cs
index 7e53aeb1c..66b111467 100644
--- a/Software/Visual_Studio/Tango.Synchronization/Local/ILocalDataBase.cs
+++ b/Software/Visual_Studio/Tango.Synchronization/Local/ILocalDataBase.cs
@@ -37,6 +37,18 @@ namespace Tango.Synchronization.Local
         DataTable CloneTableFrom(ILocalDataBase otherDB, DataTable otherTable);
 
         /// <summary>
+        /// Replaces the table data with the data from another table.
+        /// </summary>
+        /// <param name="otherTable">The other table.</param>
+        void ReplaceTableData(ILocalDataBase otherDB, DataTable otherTable);
+
+        /// <summary>
+        /// Replaces the table data with the data from another table.
+        /// </summary>
+        /// <param name="otherTable">The other table.</param>
+        String GetReplaceTableDataCommand(ILocalDataBase otherDB, DataTable otherTable);
+
+        /// <summary>
         /// Gets the SQL command for <see cref="CloneTableFrom(ILocalDataBase, DataTable)"/>.
         /// </summary>
         /// <param name="otherDB">The other database.</param>
diff --git a/Software/Visual_Studio/Tango.Synchronization/Local/LocalDBComparer.cs b/Software/Visual_Studio/Tango.Synchronization/Local/LocalDBComparer.cs
index 63f2ecfaa..7883f9cc8 100644
--- a/Software/Visual_Studio/Tango.Synchronization/Local/LocalDBComparer.cs
+++ b/Software/Visual_Studio/Tango.Synchronization/Local/LocalDBComparer.cs
@@ -13,7 +13,7 @@ namespace Tango.Synchronization.Local
     /// Represents an <see cref="ILocalDataBase"/> synchronization engine.
     /// </summary>
     /// <seealso cref="System.IDisposable" />
-    public class LocalDBComparer : IDisposable
+    public class LocalDBComparer : IDBComparer
     {
         /// <summary>
         /// Gets the master SQL.
@@ -52,7 +52,35 @@ namespace Tango.Synchronization.Local
 
             List<Diff> diffs = new List<Diff>();
 
-            foreach (var masterTable in MasterSQL.Tables)
+            DataTable sync_table = MasterSQL.Tables.Single(x => x.TableName == Constants.SYNC_CONFIGURATIONS_TABLE_NAME);
+
+            List<DataTable> sync_tables = new List<DataTable>();
+            List<DataTable> overwrite_tables = new List<DataTable>();
+
+            foreach (var row in sync_table.AsEnumerable())
+            {
+                if ((SyncConfiguration)row.Field<Int64>(Constants.SYNC_CONFIGURATION_SYNC_TYPE_COLUMN) == SyncConfiguration.Synchronize)
+                {
+                    sync_tables.Add(MasterSQL.Tables.Single(x => x.TableName == row.Field<String>(Constants.SYNC_CONFIGURATION_TABLE_NAME_COLUMN)));
+                }
+                else
+                {
+                    overwrite_tables.Add(MasterSQL.Tables.Single(x => x.TableName == row.Field<String>(Constants.SYNC_CONFIGURATION_TABLE_NAME_COLUMN)));
+                }
+            }
+
+            foreach (var masterTable in overwrite_tables)
+            {
+                LogManager.Log("Generating table overwrite difference for table " + masterTable.TableName);
+                diffs.Add(new Diff(DiffAction.ReplaceTableDataInSlave, "Replace all rows on slave table " + masterTable.TableName, () =>
+                {
+
+                    SlaveSQL.ReplaceTableData(MasterSQL, masterTable);
+
+                }, SlaveSQL.GetReplaceTableDataCommand(MasterSQL, masterTable)));
+            }
+
+            foreach (var masterTable in sync_tables)
             {
                 LogManager.Log("Comparing table " + masterTable.TableName);
                 LogManager.Log("Searching table " + masterTable.TableName + " on slave...");
diff --git a/Software/Visual_Studio/Tango.Synchronization/Local/SqliteDataBase.cs b/Software/Visual_Studio/Tango.Synchronization/Local/SqliteDataBase.cs
index 35e54c66a..83a14b493 100644
--- a/Software/Visual_Studio/Tango.Synchronization/Local/SqliteDataBase.cs
+++ b/Software/Visual_Studio/Tango.Synchronization/Local/SqliteDataBase.cs
@@ -148,6 +148,44 @@ namespace Tango.Synchronization.Local
         }
 
         /// <summary>
+        /// Replaces the table data with the data from another table.
+        /// </summary>
+        /// <param name="otherTable">The other table.</param>
+        public void ReplaceTableData(ILocalDataBase otherDB, DataTable otherTable)
+        {
+            var dropCommand = _connection.CreateCommand();
+            dropCommand.CommandText = String.Format("DELETE FROM {0};", otherTable.TableName);
+            dropCommand.ExecuteNonQuery();
+
+            var attacheCommand = _connection.CreateCommand();
+            attacheCommand.CommandText = String.Format("ATTACH DATABASE '{0}' AS other;", otherDB.Source);
+            attacheCommand.ExecuteNonQuery();
+
+            var copyCommand = _connection.CreateCommand();
+            copyCommand.CommandText = String.Format("INSERT INTO main.{0} SELECT * FROM {1}.{0};", otherTable.TableName, "other");
+            copyCommand.ExecuteNonQuery();
+
+            var detacheCommand = _connection.CreateCommand();
+            detacheCommand.CommandText = "DETACH other;";
+            detacheCommand.ExecuteNonQuery();
+        }
+
+        /// <summary>
+        /// Gets the SQL command for <see cref="ReplaceTableData(ILocalDataBase, DataTable)"/>.
+        /// </summary>
+        /// <param name="otherDB"></param>
+        /// <param name="otherTable">The other table.</param>
+        /// <returns></returns>
+        public string GetReplaceTableDataCommand(ILocalDataBase otherDB, DataTable otherTable)
+        {
+            String cmd = String.Format("DELETE FROM {0};", otherTable.TableName) + Environment.NewLine;
+            cmd += String.Format("ATTACH DATABASE '{0}' AS other;", otherDB.Source) + Environment.NewLine;
+            cmd += String.Format("INSERT INTO main.{0} SELECT * FROM {1}.{0};", otherTable.TableName, "other") + Environment.NewLine;
+            cmd += "DETACH other;" + Environment.NewLine;
+            return cmd;
+        }
+
+        /// <summary>
         /// Gets the SQL command for <see cref="CloneTableFrom(ILocalDataBase, DataTable)" />.
         /// </summary>
         /// <param name="otherDB">The other database.</param>
@@ -252,6 +290,11 @@ namespace Tango.Synchronization.Local
 
                 double num = 0;
 
+                if (row.ItemArray[i].GetType() == typeof(bool))
+                {
+                    value = ((bool)row.ItemArray[i]) ? 1.ToString() : 0.ToString();
+                }
+
                 if (row.ItemArray[i].GetType() == typeof(DateTime))
                 {
                     value = ((DateTime)row.ItemArray[i]).ToSQLiteDateString();
@@ -288,6 +331,11 @@ namespace Tango.Synchronization.Local
 
                 double num = 0;
 
+                if (row.ItemArray[i].GetType() == typeof(bool))
+                {
+                    value = ((bool)row.ItemArray[i]) ? 1.ToString() : 0.ToString();
+                }
+
                 if (row.ItemArray[i].GetType() == typeof(DateTime))
                 {
                     value = ((DateTime)row.ItemArray[i]).ToSQLiteDateString();
@@ -322,6 +370,11 @@ namespace Tango.Synchronization.Local
 
                 double num = 0;
 
+                if (row.ItemArray[i].GetType() == typeof(bool))
+                {
+                    value = ((bool)row.ItemArray[i]) ? 1.ToString() : 0.ToString();
+                }
+
                 if (row.ItemArray[i].GetType() == typeof(DateTime))
                 {
                     value = ((DateTime)row.ItemArray[i]).ToSQLiteDateString();
@@ -382,6 +435,11 @@ namespace Tango.Synchronization.Local
 
                 double num = 0;
 
+                if (row.ItemArray[i].GetType() == typeof(bool))
+                {
+                    value = ((bool)row.ItemArray[i]) ? 1.ToString() : 0.ToString();
+                }
+
                 if (row.ItemArray[i].GetType() == typeof(DateTime))
                 {
                     value = ((DateTime)row.ItemArray[i]).ToSQLiteDateString();
diff --git a/Software/Visual_Studio/Tango.Synchronization/Remote/RemoteDBComparer.cs b/Software/Visual_Studio/Tango.Synchronization/Remote/RemoteDBComparer.cs
index 49555f75c..0bd76dbd1 100644
--- a/Software/Visual_Studio/Tango.Synchronization/Remote/RemoteDBComparer.cs
+++ b/Software/Visual_Studio/Tango.Synchronization/Remote/RemoteDBComparer.cs
@@ -11,7 +11,7 @@ using System.Collections;
 
 namespace Tango.Synchronization.Remote
 {
-    public class RemoteDBComparer : IDisposable
+    public class RemoteDBComparer : IDBComparer
     {
         private remote.RemoteDB _remoteDB;
         private local.LocalDB _localDB;
@@ -65,6 +65,11 @@ namespace Tango.Synchronization.Remote
             LogManager.Log("Querying all remote machines...");
             var remote_machines = _remoteDB.MACHINES.Where(x => x.SERIAL_NUMBER == _machineSerial).ToList();
 
+            if (remote_machines.Count == 0)
+            {
+                throw LogManager.Log(new DataBaseComparisonException("Could not locate machine on remote server " + _machineSerial));
+            }
+
             LogManager.Log("Querying all remote organizations...");
             guids = remote_machines.Select(x => x.ORGANIZATION_GUID).ToList();
             var remote_organizations = _remoteDB.ORGANIZATIONS.Where(x => guids.Contains(x.GUID)).ToList();
@@ -114,7 +119,7 @@ namespace Tango.Synchronization.Remote
             }
 
 
-            LogManager.Log("Comparing addresses tables");
+            LogManager.Log("Comparing addresses");
             CompareCollections(remote_addresses, local_addresses, _remoteDB.ADDRESSES, _localDB.ADDRESSES);
 
             LogManager.Log("Comparing cartridges");
@@ -150,6 +155,8 @@ namespace Tango.Synchronization.Remote
             LogManager.Log("Comparing users roles");
             CompareCollections(remote_users_roles, local_users_roles, _remoteDB.USERS_ROLES, _localDB.USERS_ROLES);
 
+            LogManager.Log("Comparison done!");
+
             return _diffs;
         }
 
@@ -163,6 +170,12 @@ namespace Tango.Synchronization.Remote
         {
             foreach (var prop in source.GetType().GetProperties().Where(x => x.PropertyType.IsPrimitive || x.PropertyType == typeof(String) || x.PropertyType == typeof(DateTime)))
             {
+                if (destination.GetType().GetProperty(prop.Name) == null)
+                {
+                    LogManager.Log(String.Format("Warning: property {0} not found on destination entity {1}!", prop.Name, destination.GetType().Name));
+                    continue;
+                }
+
                 if (prop.PropertyType == typeof(Int64))
                 {
                     destination.GetType().GetProperty(prop.Name).SetValue(destination, Convert.ToInt32((Int64)prop.GetValue(source)));
@@ -179,25 +192,25 @@ namespace Tango.Synchronization.Remote
             var slaveProp = typeof(Slave).GetProperty("GUID");
             var masterProp = typeof(Master).GetProperty("GUID");
 
+            List<Slave> compared = new List<Slave>();
+
             foreach (var masterRow in masterCollection)
             {
-                CompareEntities(masterRow, slaveCollection.SingleOrDefault(x => slaveProp.GetValue(x).ToString() == masterProp.GetValue(masterRow).ToString()), masterSet, slaveSet);
+                Slave slaveRow = slaveCollection.SingleOrDefault(x => slaveProp.GetValue(x).ToString() == masterProp.GetValue(masterRow).ToString());
+                CompareEntities(masterRow, slaveRow, masterSet, slaveSet);
+
+                if (slaveRow != null)
+                {
+                    compared.Add(slaveRow);
+                }
             }
 
-            foreach (var slaveRow in slaveCollection)
+            foreach (var slaveRow in slaveCollection.Where(x => !compared.Contains(x)))
             {
                 CompareEntities(masterCollection.SingleOrDefault(x => masterProp.GetValue(x).ToString() == slaveProp.GetValue(slaveRow).ToString()), slaveRow, masterSet, slaveSet);
             }
         }
 
-        //private Destination TransferEntity<Source, Destination>(Source entity, List<Destination> destinationSet) where Source : class where Destination : class
-        //{
-        //    var cloned = Activator.CreateInstance<Destination>();
-        //    CopyEntity(entity, cloned);
-        //    destinationSet.Add(cloned);
-        //    return cloned;
-        //}
-
         private void OverrideTable(remote.SYNC_CONFIGURATION config)
         {
             LogManager.Log("Generating table override difference for " + config.TABLE_NAME + "...");
@@ -230,11 +243,19 @@ namespace Tango.Synchronization.Remote
 
             if (masterDate > slaveDate)
             {
-                diff = new Diff(DiffAction.UpdateRowInSlave, "Update row in slave table " + typeof(Master).Name, () => CopyEntity(master, slave), null);
+                diff = new Diff(DiffAction.UpdateRowInSlave, "Update row in slave table " + typeof(Master).Name, () => 
+                {
+                    LogManager.Log("Updating row in slave table " + typeof(Master).Name);
+                    CopyEntity(master, slave);
+                }, null);
             }
             else if (slaveDate > masterDate)
             {
-                diff = new Diff(DiffAction.UpdateRowInMaster, "Update row in master table " + typeof(Master).Name, () => CopyEntity(slave, master), null);
+                diff = new Diff(DiffAction.UpdateRowInMaster, "Update row in master table " + typeof(Master).Name, () => 
+                {
+                    LogManager.Log("Updating row in master table " + typeof(Master).Name);
+                    CopyEntity(slave, master);
+                }, null);
             }
 
             if (diff != null)
@@ -249,7 +270,7 @@ namespace Tango.Synchronization.Remote
             {
                 _diffs.Add(new Diff(DiffAction.AddRowToSlave, "Add row to slave table " + typeof(Master).Name, () =>
                {
-
+                   LogManager.Log("Adding row to slave table " + typeof(Master).Name);
                    Slave newRow = slaveSet.Create();
                    CopyEntity(master, newRow);
                    slaveSet.Add(newRow);
@@ -263,7 +284,7 @@ namespace Tango.Synchronization.Remote
             {
                 _diffs.Add(new Diff(DiffAction.AddRowToMaster, "Add row to master table " + typeof(Master).Name, () =>
                 {
-
+                    LogManager.Log("Adding row to master table " + typeof(Master).Name);
                     Master newRow = masterSet.Create();
                     CopyEntity(slave, newRow);
                     masterSet.Add(newRow);
diff --git a/Software/Visual_Studio/Tango.Synchronization/Remote/SyncConfiguration.cs b/Software/Visual_Studio/Tango.Synchronization/SyncConfiguration.cs
index 7e5a60f07..6082db15b 100644
--- a/Software/Visual_Studio/Tango.Synchronization/Remote/SyncConfiguration.cs
+++ b/Software/Visual_Studio/Tango.Synchronization/SyncConfiguration.cs
@@ -4,7 +4,7 @@ using System.Linq;
 using System.Text;
 using System.Threading.Tasks;
 
-namespace Tango.Synchronization.Remote
+namespace Tango.Synchronization
 {
     public enum SyncConfiguration
     {
diff --git a/Software/Visual_Studio/Tango.Synchronization/Tango.Synchronization.csproj b/Software/Visual_Studio/Tango.Synchronization/Tango.Synchronization.csproj
index 97227a183..36719516a 100644
--- a/Software/Visual_Studio/Tango.Synchronization/Tango.Synchronization.csproj
+++ b/Software/Visual_Studio/Tango.Synchronization/Tango.Synchronization.csproj
@@ -65,6 +65,7 @@
       <Link>GlobalVersionInfo.cs</Link>
     </Compile>
     <Compile Include="DataBaseComparisonException.cs" />
+    <Compile Include="IDBComparer.cs" />
     <Compile Include="Local\Constants.cs" />
     <Compile Include="Local\ExtensionMethods.cs" />
     <Compile Include="Local\ILocalDataBase.cs" />
@@ -74,7 +75,7 @@
     <Compile Include="DiffAction.cs" />
     <Compile Include="Local\SQLiteDataBase.cs" />
     <Compile Include="Remote\RemoteDBComparer.cs" />
-    <Compile Include="Remote\SyncConfiguration.cs" />
+    <Compile Include="SyncConfiguration.cs" />
   </ItemGroup>
   <ItemGroup>
     <None Include="App.config" />