multiple cases and changes (whups forgot to commit)

cases  3664, 3663, 3520 (redux)

source/Plugins/AyaNova.Plugin.QBOI/Properties/AssemblyInfo.cs

@@ -10,7 +10,7 @@ using System.Runtime.InteropServices;
 [assembly: AssemblyConfiguration("")]
 [assembly: AssemblyCompany("Ground Zero Tech-Works Inc.")]
 [assembly: AssemblyProduct("AyaNova.PlugIn.QBOI")]
-[assembly: AssemblyCopyright("Copyright © 2017-2018")]
+[assembly: AssemblyCopyright("Copyright © 2017-2019")]
 [assembly: AssemblyTrademark("AyaNova is a registered trademark of Ground Zero Tech-Works Inc.")]
 [assembly: AssemblyCulture("")]
 
@@ -33,4 +33,4 @@ using System.Runtime.InteropServices;
 // by using the '*' as shown below:
 // [assembly: AssemblyVersion("1.0.*")]
 [assembly: AssemblyVersion("7.5.0.0")]
-[assembly: AssemblyFileVersion("7.5.3.0")]
+[assembly: AssemblyFileVersion("7.5.4.0")]

source/Plugins/AyaNova.Plugin.QBOI/QBOI.cs

@@ -59,7 +59,7 @@ namespace AyaNova.PlugIn.QBOI
 
         public string PluginVersion
         {
-            get { return "7.5"; }//if patch then 7.3 (patch 1) etc
+            get { return "7.5 (patch 4)"; }//if patch then 7.3 (patch 1) etc
         }
 
         public string About

source/Plugins/AyaNova.Plugin.QBOI/Util.cs

@@ -399,8 +399,12 @@ namespace AyaNova.PlugIn.QBOI
         /// <returns></returns>
         public static pfstat PreFlightCheck()
         {
+            IntegrationLog.Log(QBID, "PFC: start");
             if (!_AuthenticationCompleted)
+            {
+                IntegrationLog.Log(QBID, "PFC: Authenticating...");
                 StartAuthorization();
+            }
 
             while (!_AuthenticationCompleted)
             {
@@ -408,6 +412,8 @@ namespace AyaNova.PlugIn.QBOI
                 //TODO: add timeout here
             }
 
+            IntegrationLog.Log(QBID, "PFC: Authentication completed, validating company data");
+
             if (QBValidate() == pfstat.Cancel)
             {
                 IntegrationLog.Log(QBID, "PFC: Unable to validate QuickBooks connection, user selected cancel");
@@ -422,6 +428,8 @@ namespace AyaNova.PlugIn.QBOI
 
             }
 
+            IntegrationLog.Log(QBID, "PFC: Checking integration object...");
+
             IntegrationObjectCheck();
 
             if (ValidateSettings(false) == pfstat.Cancel)
@@ -430,6 +438,7 @@ namespace AyaNova.PlugIn.QBOI
                 return pfstat.Cancel;
             }
 
+            IntegrationLog.Log(QBID, "PFC: Checking cached mappings for validity...");
 
             //Added: 18-Nov-2006 CASE 163
             //check that linked items in integration map exist in QB
@@ -518,6 +527,8 @@ namespace AyaNova.PlugIn.QBOI
                 }
             }
 
+            IntegrationLog.Log(QBID, "PFC: pre-flight check completed - OK");
+
             return pfstat.OK;
         }
 
@@ -600,6 +611,47 @@ namespace AyaNova.PlugIn.QBOI
         /// </summary>
         public static pfstat QBValidate()
         {
+            /**
+             12/28/2018  Error reported by user:
+             * 
+             * All my clients are set to active but when I connect QBOI to Ayanova it keeps telling me that all the clients are set to inactive.
+
+ 
+
+ 
+
+Unexpected duplicate value returned by QuickBooks Online company info query
+
+Please copy this information to AyaNova technical support (support@ayanova.com)
+
+Company info returned:
+
+KEY: NeoEnabled, VALUE: true
+
+KEY: IsQbdtMigrated, VALUE: true
+
+KEY: MigrationDate, VALUE: Nov 20, 2018 10:38:44 AM
+
+KEY: MigrationSource, VALUE: QuickBooks-Unknown ActiveX
+
+KEY: SubscriptionStatus, VALUE: PAID
+
+KEY: OfferingSku, VALUE: QuickBooks Online Essentials
+
+KEY: PayrollFeature, VALUE: false
+
+KEY: AccountantFeature, VALUE: false
+
+KEY: QBOIndustryType, VALUE: I sell products & services
+
+KEY: ItemCategoriesFeature, VALUE: true
+
+KEY: AssignedTime, VALUE: 11/09/2018 11:50:11
+             
+             
+             
+             */
+
 
             /*
              * Error reported by user:
@@ -667,6 +719,10 @@ KEY: AssignedTime, VALUE: 12/08/2017 18:52:04
                 //case 3520
                 bool dupeKeyFoundInCompanyInfo = false;
 
+                //case 3520 redux 12/31/2018
+                //working theory is that this is getting called when it's already been populated by an earlier action so it makes all items appear to be dupes because they were already set in qbcompprefs
+                QCompOtherPrefs.Clear();
+
                 //Get the country and latest version supported
                 //QB Online doesn't really have a "version", only a minor version
                 //for the sake of compatibility just setting this to 3 which is the major api version number