Merge pull request #172 from umbraco/bugfix/dynamics-access-token-column-size

Bugfix/dynamics access token column size

Author: acoumb

src/Umbraco.Cms.Integrations.Crm.Dynamics/Constants.cs

@@ -9,7 +9,9 @@ public class Constants
 
         public const string TargetStateName = "dynamicsOAuthConfiguration-db";
 
-        
+        public const string AlterAccessTokenColumnLengthTargetStateName = "dynamicsOAuthConfiguration-alter-access-token-column-length-db";
+
+        public const int AccessTokenFieldSize = 4000;
 
         public const string AppPluginFolderPath = "~/App_Plugins/UmbracoCms.Integrations/Crm/Dynamics";
 

src/Umbraco.Cms.Integrations.Crm.Dynamics/Migrations/DynamicsMigration.cs

@@ -62,7 +62,7 @@ public class DynamicsOAuthConfigurationTable
         public int Id { get; set; }
 
         [Column("AccessToken")]
-        [Length(2048)]
+        [Length(Constants.AccessTokenFieldSize)]
         public string AccessToken { get; set; }
 
         [Column("UserId")]

src/Umbraco.Cms.Integrations.Crm.Dynamics/Migrations/DynamicsMigrationPlan.cs

@@ -11,7 +11,8 @@ public class DynamicsMigrationPlan : MigrationPlan
         public DynamicsMigrationPlan() : base(Constants.MigrationPlanName)
         {
             From(string.Empty)
-                .To<DynamicsMigration>(Constants.TargetStateName);
+                .To<DynamicsMigration>(Constants.TargetStateName)
+                .To<EnsureAccessTokenColumnLength>(Constants.AlterAccessTokenColumnLengthTargetStateName);
         }
     }
 }

src/Umbraco.Cms.Integrations.Crm.Dynamics/Migrations/EnsureAccessTokenColumnLength.cs

@@ -0,0 +1,39 @@
+#if NETCOREAPP
+using Microsoft.Extensions.Logging;
+using Umbraco.Cms.Infrastructure.Migrations;
+using Umbraco.Cms.Infrastructure.Persistence;
+#else
+using Umbraco.Core.Logging;
+using Umbraco.Core.Migrations;
+#endif
+
+namespace Umbraco.Cms.Integrations.Crm.Dynamics.Migrations
+{
+    public class EnsureAccessTokenColumnLength : MigrationBase
+    {
+        public EnsureAccessTokenColumnLength(IMigrationContext context) : base(context)
+        {
+        }
+
+#if NETCOREAPP
+        protected override void Migrate()
+#else
+        public override void Migrate()
+#endif
+        {
+#if NETCOREAPP
+            Logger.LogDebug("Running migration {0}", nameof(EnsureAccessTokenColumnLength));
+
+#else
+            Logger.Debug<EnsureAccessTokenColumnLength>("Running migration {0}", nameof(EnsureAccessTokenColumnLength));
+#endif
+
+            Alter.Table(Constants.DynamicsOAuthConfigurationTable)
+                .AlterColumn(nameof(DynamicsOAuthConfigurationTable.AccessToken))
+                .AsString(Constants.AccessTokenFieldSize)
+                .NotNullable()
+                .Do();
+            
+        }
+    }
+}

src/Umbraco.Cms.Integrations.Crm.Dynamics/Umbraco.Cms.Integrations.Crm.Dynamics.csproj

@@ -10,7 +10,7 @@
 		<PackageIconUrl></PackageIconUrl>
 		<PackageProjectUrl>https://github.com/umbraco/Umbraco.Cms.Integrations/blob/main/src/Umbraco.Cms.Integrations.Crm.Dynamics</PackageProjectUrl>
 		<RepositoryUrl>https://github.com/umbraco/Umbraco.Cms.Integrations</RepositoryUrl>
-		<Version>1.3.1</Version>
+		<Version>1.3.2</Version>
 		<Authors>Umbraco HQ</Authors>
 		<Company>Umbraco</Company>
 		<PackageTags>Umbraco;Umbraco-Marketplace</PackageTags>