Finalized UpdatePositioning job

PartSource.Automation/Factories/JobFactory.cs

@@ -12,7 +12,7 @@ namespace PartSource.Automation.Factories
 {
 	public class JobFactory
 	{
-       public IServiceProvider _serviceProvider;
+		private readonly IServiceProvider _serviceProvider;
 
 		public JobFactory(IServiceProvider serviceProvider)
 		{
@@ -38,6 +38,9 @@ namespace PartSource.Automation.Factories
 				case nameof(UpdatePricing):
 					return _serviceProvider.GetService<UpdatePricing>();
 
+				case nameof(UpdatePositioning):
+					return _serviceProvider.GetService<UpdatePositioning>();
+
 				case nameof(ExecuteSsisPackages):
 					return _serviceProvider.GetService<ExecuteSsisPackages>();
 

PartSource.Automation/Jobs/UpdateFitment.cs

@@ -53,8 +53,11 @@ namespace PartSource.Automation.Jobs
 						ImportData importData = _partSourceContext.ImportData.FirstOrDefault(i => i.VariantSku == product.Variants[0].Sku);
 						IList<VehicleData> vehicles = _vehicleService.GetVehiclesForPart(importData?.PartNumber, importData?.LineCode);
 
-						if (vehicles.Count > 0)
+						if (vehicles.Count == 0)
 						{
+							continue;
+						}
+
 						bool isFitment = false;
 
 						IList<int> vehicleIdFitment = _vehicleService.GetVehicleIdFitment(vehicles);
@@ -117,8 +120,6 @@ namespace PartSource.Automation.Jobs
 
 						await _shopifyClient.Metafields.Add(isFitmentMetafield);
 					}
-					}
-
 
 					catch
 					{

PartSource.Automation/Jobs/UpdatePositioning.cs

@@ -8,8 +8,10 @@ using PartSource.Services;
 using PartSource.Services.Integrations;
 using System;
 using System.Collections.Generic;
+using System.Diagnostics.CodeAnalysis;
 using System.Linq;
 using System.Text;
+using System.Text.RegularExpressions;
 using System.Threading.Tasks;
 
 namespace PartSource.Automation.Jobs
@@ -38,75 +40,47 @@ namespace PartSource.Automation.Jobs
 					try
 					{
 						ImportData importData = _partSourceContext.ImportData.FirstOrDefault(i => i.VariantSku == product.Variants[0].Sku);
+
+						if (importData?.LineCode == "SVG")  // Headlights go in front, DUH
+						{
+							continue;
+						}
+
+						IList<Fitment> fitments = GetPositionOrderedFitments(importData?.PartNumber, importData?.LineCode);
 						IList<VehicleData> vehicles = _vehicleService.GetVehiclesForPart(importData?.PartNumber, importData?.LineCode);
 
-						if (vehicles.Count > 0)
+						if (fitments.Count == 0 || vehicles.Count == 0)
-						{
-							bool isFitment = false;
-
-							IList<int> vehicleIdFitment = _vehicleService.GetVehicleIdFitment(vehicles);
-							if (vehicleIdFitment.Count > 0)
-							{
-								isFitment = true;
-
-								string json = JsonConvert.SerializeObject(vehicleIdFitment);
-								if (json.Length >= 100000)
 						{
 							continue;
 						}
 
-								Metafield vehicleMetafield = new Metafield
+						await DeletePositionMetafields(product.Id);
-								{
-									Namespace = "fitment",
-									Key = "ids",
-									Value = json,
-									ValueType = "json_string",
-									OwnerResource = "product",
-									OwnerId = product.Id
-								};
 
-								await _shopifyClient.Metafields.Add(vehicleMetafield);
+						string currentPosition = fitments[0].Position;
+						List<int> vehicleIds = new List<int>();
+
+						foreach (Fitment fitment in fitments)
+						{
+							if (fitment.Position != currentPosition)
+							{
+								await SavePositionMetafield(product, vehicleIds, currentPosition);
+
+								currentPosition = fitment.Position;
+								vehicleIds = new List<int>();
 							}
 
-							IList<string> ymmFitment = _vehicleService.GetYmmFitment(vehicles);
+							// We don't need to DCF map because these are both sourced from WHI
-							if (ymmFitment.Count > 0)
+							IList<int> fitmentVehicleIds = vehicles.Where(v => v.BaseVehicleId == fitment.BaseVehicleId && v.EngineConfigId == fitment.EngineConfigId)
-							{
+								.Select(v => v.VehicleToEngineConfigId)
-								isFitment = true;
+								.Distinct()
+								.ToList();
 
-								string json = JsonConvert.SerializeObject(ymmFitment);
+							vehicleIds.AddRange(fitmentVehicleIds);
-								if (json.Length >= 100000)
-								{
-									continue;
 						}
 
-								Metafield ymmMetafield = new Metafield
+						await SavePositionMetafield(product, vehicleIds, currentPosition);
-								{
-									Namespace = "fitment",
-									Key = "seo",
-									Value = json,
-									ValueType = "json_string",
-									OwnerResource = "product",
-									OwnerId = product.Id
-								};
-
-								await _shopifyClient.Metafields.Add(ymmMetafield);
 					}
 
-							Metafield isFitmentMetafield = new Metafield
-							{
-								Namespace = "Flags",
-								Key = "IsFitment",
-								Value = isFitment.ToString(),
-								ValueType = "string",
-								OwnerResource = "product",
-								OwnerId = product.Id
-							};
-
-							await _shopifyClient.Metafields.Add(isFitmentMetafield);
-						}
-					}
-
-
 					catch
 					{
 						Console.WriteLine(product.Id);
@@ -132,5 +106,70 @@ namespace PartSource.Automation.Jobs
 				IsSuccess = true
 			};
 		}
+
+		private IList<Fitment> GetPositionOrderedFitments(string partNumber, string lineCode)
+		{
+			partNumber = Regex.Replace(partNumber, "[^a-zA-Z0-9]", string.Empty);
+
+			IQueryable<string> whiCodes = _partSourceContext.DcfMappings
+				.Where(d => d.LineCode == lineCode)
+				.Select(d => d.WhiCode);
+
+			return _partSourceContext.Fitments
+				.Where(f => f.PartNumber == partNumber && whiCodes.Contains(f.LineCode) && !string.IsNullOrEmpty(f.Position))
+				.OrderBy(f => f.Position)
+				.ToList();
+		}
+
+		[SuppressMessage("Globalization", "CA1308:Normalize strings to uppercase", Justification = "It's a Shopify metafield key")]
+		private async Task SavePositionMetafield(Product product, IList<int> vehicleIds, string position)
+		{
+			if (vehicleIds.Count == 0)
+			{
+				return;
+			}
+
+			string json = JsonConvert.SerializeObject(vehicleIds);
+			if (json.Length >= 100000)
+			{
+				// TODO: Logging
+				return;
+			}
+
+			string key = position.ToLowerInvariant().Replace(" ", "_");
+			if (key.Length > 20)
+			{
+				key = key.Substring(0, 20);
+			}
+
+			Metafield vehicleMetafield = new Metafield
+			{
+				Namespace = "position",
+				Key = key,
+				Value = json,
+				ValueType = "json_string",
+				OwnerResource = "product",
+				OwnerId = product.Id
+			};
+
+			await _shopifyClient.Metafields.Add(vehicleMetafield);
+		}
+
+		private async Task DeletePositionMetafields(long shopifyId)
+		{
+			IDictionary<string, object> parameters = new Dictionary<string, object>
+			{
+				{ "metafield[owner_id]", shopifyId},
+				{ "metafield[owner_resource]", "product" },
+				{ "namespace", "position" },
+			};
+
+			IEnumerable<Metafield> metafields = await _shopifyClient.Metafields.Get(parameters);
+
+			foreach (Metafield metafield in metafields)
+			{
+				await _shopifyClient.Metafields.Delete(metafield);
+			}
+		}
 	}
 }

PartSource.Automation/Program.cs

@@ -110,6 +110,7 @@ namespace PartSource.Automation
                 .AddSingleton<DeleteProducts>()
                 .AddSingleton<UpdateFitment>()
                 .AddSingleton<UpdatePricing>()
+                .AddSingleton<UpdatePositioning>()
                 .AddSingleton<ExecuteSsisPackages>()
                 .AddSingleton<JobFactory>()
                 .BuildServiceProvider();

PartSource.Automation/Properties/launchSettings.json

@@ -2,7 +2,7 @@
   "profiles": {
     "PartSource.Automation": {
       "commandName": "Project",
-      "commandLineArgs": "UpdatePricing",
+      "commandLineArgs": "UpdatePositioning",
       "environmentVariables": {
         "PS_AUTOMATION_ENVIRONMENT": "development"
       }

PartSource.Services/PartService.cs

@@ -1,11 +1,13 @@
 using PartSource.Data;
 using PartSource.Data.Dtos;
 using PartSource.Data.Models;
+using System;
 using System.Collections.Generic;
 using System.Linq;
 
 namespace PartSource.Services
 {
+    [Obsolete]
     public class PartService
     {
         private readonly PartSourceContext _context;