Compare commits
19 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 8768e9813b | |||
| 4a7087cc0c | |||
| 59b152c89f | |||
| 441898b52f | |||
| 3e30397302 | |||
| 31c5746e5b | |||
| 3f9ac27afa | |||
| df504674e9 | |||
| 07796b05c8 | |||
| 2bf8871892 | |||
| 8a0a564885 | |||
| dd4785b048 | |||
| e185e3b7e3 | |||
| 8acbc8605d | |||
| 485f0b69c8 | |||
| f380c152ca | |||
| 79c8c7e6a4 | |||
| 6cf355071b | |||
| ebd474ae81 |
@@ -137,6 +137,13 @@ public class CreateJobDto
|
||||
[Display(Name = "Oven")]
|
||||
public int? OvenCostId { get; set; }
|
||||
|
||||
[Display(Name = "Batches")]
|
||||
[Range(1, 999)]
|
||||
public int OvenBatches { get; set; } = 1;
|
||||
|
||||
[Display(Name = "Cycle Time (min)")]
|
||||
public int? OvenCycleMinutes { get; set; }
|
||||
|
||||
[Required(ErrorMessage = "Description is required")]
|
||||
[StringLength(2000, ErrorMessage = "Description cannot exceed 2000 characters")]
|
||||
[Display(Name = "Description")]
|
||||
@@ -208,6 +215,16 @@ public class UpdateJobDto
|
||||
[Display(Name = "Assigned Worker")]
|
||||
public string? AssignedUserId { get; set; }
|
||||
|
||||
[Display(Name = "Oven")]
|
||||
public int? OvenCostId { get; set; }
|
||||
|
||||
[Display(Name = "Batches")]
|
||||
[Range(1, 999)]
|
||||
public int OvenBatches { get; set; } = 1;
|
||||
|
||||
[Display(Name = "Cycle Time (min)")]
|
||||
public int? OvenCycleMinutes { get; set; }
|
||||
|
||||
[Required(ErrorMessage = "Description is required")]
|
||||
[StringLength(2000, ErrorMessage = "Description cannot exceed 2000 characters")]
|
||||
[Display(Name = "Description")]
|
||||
@@ -381,6 +398,7 @@ public class JobItemCoatDto
|
||||
public decimal? PowderCostPerLb { get; set; }
|
||||
public decimal? PowderToOrder { get; set; }
|
||||
public decimal? ActualPowderUsedLbs { get; set; } // Filled during job completion
|
||||
public bool NoExtraLayerCharge { get; set; }
|
||||
public string? Notes { get; set; }
|
||||
}
|
||||
|
||||
|
||||
@@ -801,6 +801,7 @@ public class QuoteItemCoatDto
|
||||
public decimal CoatMaterialCost { get; set; }
|
||||
public decimal CoatLaborCost { get; set; }
|
||||
public decimal CoatTotalCost { get; set; }
|
||||
public bool NoExtraLayerCharge { get; set; }
|
||||
public string? Notes { get; set; }
|
||||
}
|
||||
|
||||
|
||||
@@ -91,4 +91,11 @@ public interface INotificationService
|
||||
/// Alert company staff when a Stripe chargeback (dispute) is opened on an invoice payment.
|
||||
/// </summary>
|
||||
Task NotifyChargebackAlertAsync(Invoice invoice, string disputeId, decimal amount, string reason);
|
||||
|
||||
/// <summary>
|
||||
/// Sends an appointment reminder email to the linked customer (if opted in) and writes a
|
||||
/// notification log row. Called by <see cref="PowderCoating.Web.BackgroundServices.AppointmentReminderBackgroundService"/>
|
||||
/// when the reminder window opens. In-app bell notification is handled by the caller.
|
||||
/// </summary>
|
||||
Task NotifyAppointmentReminderAsync(Appointment appointment);
|
||||
}
|
||||
|
||||
@@ -4,8 +4,26 @@ using PowderCoating.Core.Entities;
|
||||
|
||||
namespace PowderCoating.Application.Services;
|
||||
|
||||
/// <summary>
|
||||
/// Converts quote/job data into persisted <see cref="JobItem"/>, <see cref="JobItemCoat"/>,
|
||||
/// and <see cref="JobItemPrepService"/> entities.
|
||||
///
|
||||
/// Three source types are supported, each with a matching overload:
|
||||
/// 1. <see cref="CreateQuoteItemDto"/> — quote wizard (new job from form data + fresh pricing result)
|
||||
/// 2. <see cref="QuoteItem"/> — quote-to-job conversion (copies a saved quote line)
|
||||
/// 3. <see cref="JobItem"/> — job duplication / template instantiation (copies an existing job line)
|
||||
///
|
||||
/// The private <see cref="JobItemSeed"/> / <see cref="JobItemCoatSeed"/> / <see cref="JobItemPrepServiceSeed"/>
|
||||
/// intermediary classes exist solely to give all three overload paths a single <see cref="BuildJobItem"/>
|
||||
/// construction site — avoiding subtle copy-paste drift where one overload forgets to copy a new field.
|
||||
/// </summary>
|
||||
public class JobItemAssemblyService : IJobItemAssemblyService
|
||||
{
|
||||
/// <summary>
|
||||
/// Creates a <see cref="JobItem"/> from a quote wizard DTO and a pre-calculated pricing result.
|
||||
/// Used when creating a job directly from the job form or from an approved quote via the wizard.
|
||||
/// Pricing is passed in separately because it was already computed upstream (CalculateQuoteItemPriceAsync).
|
||||
/// </summary>
|
||||
public JobItem CreateJobItem(CreateQuoteItemDto source, int jobId, int companyId, QuoteItemPricingResult pricing, DateTime createdAtUtc)
|
||||
{
|
||||
ArgumentNullException.ThrowIfNull(source);
|
||||
@@ -42,6 +60,11 @@ public class JobItemAssemblyService : IJobItemAssemblyService
|
||||
createdAtUtc);
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Builds <see cref="JobItemCoat"/> records from the coat DTOs in the quote wizard form.
|
||||
/// PowderToOrder is recalculated server-side here (not trusted from the form) using surface area,
|
||||
/// quantity, coverage, and transfer efficiency — the wizard's displayed value is for UI only.
|
||||
/// </summary>
|
||||
public IReadOnlyList<JobItemCoat> CreateJobItemCoats(CreateQuoteItemDto source, int jobItemId, int companyId, DateTime createdAtUtc)
|
||||
{
|
||||
ArgumentNullException.ThrowIfNull(source);
|
||||
@@ -62,7 +85,8 @@ public class JobItemAssemblyService : IJobItemAssemblyService
|
||||
TransferEfficiency = c.TransferEfficiency,
|
||||
PowderCostPerLb = c.PowderCostPerLb,
|
||||
PowderToOrder = CalculatePowderToOrder(c.PowderToOrder, source.SurfaceAreaSqFt, source.Quantity, c.CoverageSqFtPerLb, c.TransferEfficiency),
|
||||
Notes = c.Notes
|
||||
Notes = c.Notes,
|
||||
NoExtraLayerCharge = c.NoExtraLayerCharge
|
||||
},
|
||||
jobItemId,
|
||||
companyId,
|
||||
@@ -70,6 +94,11 @@ public class JobItemAssemblyService : IJobItemAssemblyService
|
||||
.ToList() ?? [];
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Builds <see cref="JobItemPrepService"/> records (sandblasting, masking, etc.) from the
|
||||
/// quote wizard DTO. These are per-item prep steps with individual time estimates that feed
|
||||
/// labor cost calculations and shop floor instructions.
|
||||
/// </summary>
|
||||
public IReadOnlyList<JobItemPrepService> CreateJobItemPrepServices(CreateQuoteItemDto source, int jobItemId, int companyId, DateTime createdAtUtc)
|
||||
{
|
||||
ArgumentNullException.ThrowIfNull(source);
|
||||
@@ -85,6 +114,13 @@ public class JobItemAssemblyService : IJobItemAssemblyService
|
||||
createdAtUtc);
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Creates a <see cref="JobItem"/> by copying a saved <see cref="QuoteItem"/> during quote-to-job conversion.
|
||||
/// Prices are taken directly from the quote snapshot — no repricing occurs — so the job starts with
|
||||
/// exactly the amounts that were approved by the customer.
|
||||
/// The first coat's color/finish is promoted to the job item's top-level fields for quick display
|
||||
/// (details remain in the coat records).
|
||||
/// </summary>
|
||||
public JobItem CreateJobItem(QuoteItem source, int jobId, int companyId, DateTime createdAtUtc)
|
||||
{
|
||||
ArgumentNullException.ThrowIfNull(source);
|
||||
@@ -128,6 +164,12 @@ public class JobItemAssemblyService : IJobItemAssemblyService
|
||||
createdAtUtc);
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Builds <see cref="JobItemCoat"/> records from a saved <see cref="QuoteItem"/> during quote-to-job conversion.
|
||||
/// Coat appearance (color name, code, finish) is resolved from the linked <see cref="InventoryItem"/> if available,
|
||||
/// because the inventory record is the canonical source of truth for a product's appearance —
|
||||
/// the values typed into the quote form may be incomplete or informal.
|
||||
/// </summary>
|
||||
public IReadOnlyList<JobItemCoat> CreateJobItemCoats(QuoteItem source, int jobItemId, int companyId, DateTime createdAtUtc)
|
||||
{
|
||||
ArgumentNullException.ThrowIfNull(source);
|
||||
@@ -151,7 +193,8 @@ public class JobItemAssemblyService : IJobItemAssemblyService
|
||||
TransferEfficiency = c.TransferEfficiency,
|
||||
PowderCostPerLb = c.PowderCostPerLb,
|
||||
PowderToOrder = CalculatePowderToOrder(c.PowderToOrder, source.SurfaceAreaSqFt, source.Quantity, c.CoverageSqFtPerLb, c.TransferEfficiency),
|
||||
Notes = c.Notes
|
||||
Notes = c.Notes,
|
||||
NoExtraLayerCharge = c.NoExtraLayerCharge
|
||||
},
|
||||
jobItemId,
|
||||
companyId,
|
||||
@@ -160,6 +203,9 @@ public class JobItemAssemblyService : IJobItemAssemblyService
|
||||
.ToList() ?? [];
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Copies prep service records from a <see cref="QuoteItem"/> to a new job item during quote-to-job conversion.
|
||||
/// </summary>
|
||||
public IReadOnlyList<JobItemPrepService> CreateJobItemPrepServices(QuoteItem source, int jobItemId, int companyId, DateTime createdAtUtc)
|
||||
{
|
||||
ArgumentNullException.ThrowIfNull(source);
|
||||
@@ -175,6 +221,12 @@ public class JobItemAssemblyService : IJobItemAssemblyService
|
||||
createdAtUtc);
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Creates a new <see cref="JobItem"/> by cloning an existing one — used for job templates
|
||||
/// and rework duplication where an existing job line is reused on a new job.
|
||||
/// Prices are copied as-is from the source; the job controller is responsible for repricing
|
||||
/// if operating costs have changed since the original job was created.
|
||||
/// </summary>
|
||||
public JobItem CreateJobItem(JobItem source, int jobId, int companyId, DateTime createdAtUtc)
|
||||
{
|
||||
ArgumentNullException.ThrowIfNull(source);
|
||||
@@ -214,6 +266,11 @@ public class JobItemAssemblyService : IJobItemAssemblyService
|
||||
createdAtUtc);
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Clones coat records from an existing <see cref="JobItem"/> onto a new job item.
|
||||
/// PowderToOrder is copied verbatim (not recalculated) because the original job's powder
|
||||
/// quantities may have been manually adjusted after initial calculation.
|
||||
/// </summary>
|
||||
public IReadOnlyList<JobItemCoat> CreateJobItemCoats(JobItem source, int jobItemId, int companyId, DateTime createdAtUtc)
|
||||
{
|
||||
ArgumentNullException.ThrowIfNull(source);
|
||||
@@ -234,7 +291,8 @@ public class JobItemAssemblyService : IJobItemAssemblyService
|
||||
TransferEfficiency = c.TransferEfficiency,
|
||||
PowderCostPerLb = c.PowderCostPerLb,
|
||||
PowderToOrder = c.PowderToOrder,
|
||||
Notes = c.Notes
|
||||
Notes = c.Notes,
|
||||
NoExtraLayerCharge = c.NoExtraLayerCharge
|
||||
},
|
||||
jobItemId,
|
||||
companyId,
|
||||
@@ -242,6 +300,9 @@ public class JobItemAssemblyService : IJobItemAssemblyService
|
||||
.ToList() ?? [];
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Clones prep service records from an existing <see cref="JobItem"/> onto a new job item.
|
||||
/// </summary>
|
||||
public IReadOnlyList<JobItemPrepService> CreateJobItemPrepServices(JobItem source, int jobItemId, int companyId, DateTime createdAtUtc)
|
||||
{
|
||||
ArgumentNullException.ThrowIfNull(source);
|
||||
@@ -257,6 +318,10 @@ public class JobItemAssemblyService : IJobItemAssemblyService
|
||||
createdAtUtc);
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Single construction point for all <see cref="JobItem"/> creation paths.
|
||||
/// Centralised here so that adding a new field only requires one code change, not three.
|
||||
/// </summary>
|
||||
private static JobItem BuildJobItem(JobItemSeed seed, int jobId, int companyId, DateTime createdAtUtc)
|
||||
{
|
||||
return new JobItem
|
||||
@@ -293,6 +358,9 @@ public class JobItemAssemblyService : IJobItemAssemblyService
|
||||
};
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Single construction point for all <see cref="JobItemCoat"/> creation paths.
|
||||
/// </summary>
|
||||
private static JobItemCoat BuildJobItemCoat(JobItemCoatSeed seed, int jobItemId, int companyId, DateTime createdAtUtc)
|
||||
{
|
||||
return new JobItemCoat
|
||||
@@ -310,11 +378,17 @@ public class JobItemAssemblyService : IJobItemAssemblyService
|
||||
PowderCostPerLb = seed.PowderCostPerLb,
|
||||
PowderToOrder = seed.PowderToOrder,
|
||||
Notes = seed.Notes,
|
||||
NoExtraLayerCharge = seed.NoExtraLayerCharge,
|
||||
CompanyId = companyId,
|
||||
CreatedAt = createdAtUtc
|
||||
};
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Single construction point for all <see cref="JobItemPrepService"/> creation paths.
|
||||
/// Returns an empty list (not null) when <paramref name="seeds"/> is null so callers
|
||||
/// can safely iterate without a null check.
|
||||
/// </summary>
|
||||
private static IReadOnlyList<JobItemPrepService> BuildJobItemPrepServices(IEnumerable<JobItemPrepServiceSeed>? seeds, int jobItemId, int companyId, DateTime createdAtUtc)
|
||||
{
|
||||
return seeds?
|
||||
@@ -330,6 +404,18 @@ public class JobItemAssemblyService : IJobItemAssemblyService
|
||||
.ToList() ?? [];
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Returns the pounds of powder needed to coat a batch, preferring the pre-stored value
|
||||
/// (which the user may have manually adjusted in the wizard) over a fresh recalculation.
|
||||
///
|
||||
/// Formula: (surfaceAreaSqFt × quantity) ÷ (coverageSqFtPerLb × transferEfficiency)
|
||||
///
|
||||
/// Industry defaults are applied when catalog data is missing:
|
||||
/// - Coverage: 30 sqft/lb (typical for standard powder at 2–3 mil DFT)
|
||||
/// - Transfer efficiency: 65% (industry average for electrostatic spray)
|
||||
/// These are conservative defaults that slightly overestimate powder needed — intentional,
|
||||
/// so the shop doesn't run short on a job.
|
||||
/// </summary>
|
||||
private static decimal? CalculatePowderToOrder(decimal? storedPowderToOrder, decimal surfaceAreaSqFt, decimal quantity, decimal coverageSqFtPerLb, decimal transferEfficiency)
|
||||
{
|
||||
if (storedPowderToOrder.HasValue && storedPowderToOrder.Value > 0)
|
||||
@@ -343,6 +429,12 @@ public class JobItemAssemblyService : IJobItemAssemblyService
|
||||
return Math.Round((surfaceAreaSqFt * quantity) / (coverage * efficiency), 2);
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Resolves the display appearance (color name, code, finish) for a coat, preferring the linked
|
||||
/// <see cref="InventoryItem"/>'s values over whatever was typed into the quote form.
|
||||
/// The inventory record is the canonical source of truth — the form values are used as a fallback
|
||||
/// only when no inventory item is linked (e.g. custom/one-off powder).
|
||||
/// </summary>
|
||||
private static (string? ColorName, string? ColorCode, string? Finish) ResolveCoatAppearance(
|
||||
string? colorName,
|
||||
string? colorCode,
|
||||
@@ -355,6 +447,11 @@ public class JobItemAssemblyService : IJobItemAssemblyService
|
||||
return (inventoryItem.Name, inventoryItem.ColorCode, inventoryItem.Finish);
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Intermediate value object that normalises the three different source types
|
||||
/// (DTO, QuoteItem, JobItem) into a single shape before the shared BuildJobItem factory method.
|
||||
/// Using a seed class prevents subtle bugs where an overload forgets to map a new field.
|
||||
/// </summary>
|
||||
private sealed class JobItemSeed
|
||||
{
|
||||
public string Description { get; init; } = string.Empty;
|
||||
@@ -385,6 +482,7 @@ public class JobItemAssemblyService : IJobItemAssemblyService
|
||||
public int? AiPredictionId { get; init; }
|
||||
}
|
||||
|
||||
/// <summary>Intermediate value object for coat creation — see <see cref="JobItemSeed"/> for rationale.</summary>
|
||||
private sealed class JobItemCoatSeed
|
||||
{
|
||||
public string CoatName { get; init; } = string.Empty;
|
||||
@@ -399,8 +497,10 @@ public class JobItemAssemblyService : IJobItemAssemblyService
|
||||
public decimal? PowderCostPerLb { get; init; }
|
||||
public decimal? PowderToOrder { get; init; }
|
||||
public string? Notes { get; init; }
|
||||
public bool NoExtraLayerCharge { get; init; }
|
||||
}
|
||||
|
||||
/// <summary>Intermediate value object for prep service creation — see <see cref="JobItemSeed"/> for rationale.</summary>
|
||||
private sealed class JobItemPrepServiceSeed
|
||||
{
|
||||
public int PrepServiceId { get; init; }
|
||||
|
||||
@@ -6,6 +6,20 @@ using Microsoft.Extensions.Logging;
|
||||
|
||||
namespace PowderCoating.Application.Services;
|
||||
|
||||
/// <summary>
|
||||
/// Orchestrates the full quote item assembly pipeline: pricing calculation, entity construction,
|
||||
/// AI prediction tracking, and automatic inventory record creation for incoming powder orders.
|
||||
///
|
||||
/// This service sits above <see cref="PricingCalculationService"/> — it knows HOW to build and
|
||||
/// persist quote entities, while PricingCalculationService knows HOW to compute dollar amounts.
|
||||
/// Keeping them separate means pricing logic can be unit-tested without any entity construction concerns.
|
||||
///
|
||||
/// Key responsibilities:
|
||||
/// - <see cref="ApplyPricingSnapshot"/> — stamps calculated totals onto the Quote entity so the
|
||||
/// displayed price is frozen at quote time and won't change if operating costs are updated later.
|
||||
/// - <see cref="CreateQuoteItemsAsync"/> — builds QuoteItem + coats + prep services for each DTO,
|
||||
/// records AI prediction overrides, and auto-creates incoming inventory records when needed.
|
||||
/// </summary>
|
||||
public class QuotePricingAssemblyService : IQuotePricingAssemblyService
|
||||
{
|
||||
private readonly IUnitOfWork _unitOfWork;
|
||||
@@ -25,6 +39,11 @@ public class QuotePricingAssemblyService : IQuotePricingAssemblyService
|
||||
_logger = logger;
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Writes the calculated pricing breakdown onto the <see cref="Quote"/> entity as a snapshot.
|
||||
/// Snapshots are critical: once a quote is sent to a customer, operating cost changes must NOT
|
||||
/// silently alter the quoted amounts — the snapshot preserves what was presented at the time.
|
||||
/// </summary>
|
||||
public void ApplyPricingSnapshot(Quote quote, QuotePricingResult pricingResult)
|
||||
{
|
||||
ArgumentNullException.ThrowIfNull(quote);
|
||||
@@ -56,6 +75,12 @@ public class QuotePricingAssemblyService : IQuotePricingAssemblyService
|
||||
quote.Total = pricingResult.Total;
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Builds and prices all <see cref="QuoteItem"/> entities from the incoming DTOs.
|
||||
/// For each item: constructs the entity, calculates pricing, records whether the user overrode
|
||||
/// an AI estimate, then attaches coats (including auto-creating incoming inventory entries when
|
||||
/// the user selects a catalog powder not yet in their inventory) and prep services.
|
||||
/// </summary>
|
||||
public async Task<IReadOnlyList<QuoteItem>> CreateQuoteItemsAsync(
|
||||
IEnumerable<CreateQuoteItemDto> itemDtos,
|
||||
int quoteId,
|
||||
@@ -80,6 +105,13 @@ public class QuotePricingAssemblyService : IQuotePricingAssemblyService
|
||||
return items;
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Routes a single item to the correct pricing path and stamps the result onto the entity.
|
||||
/// Priority order matches the routing table in <see cref="PricingCalculationService.CalculateQuoteItemPriceAsync"/>:
|
||||
/// AI items → Sales items → Catalog (no coats) → full calculation engine.
|
||||
/// Keeping pricing logic in PricingCalculationService means this method only decides WHICH
|
||||
/// path to take, never HOW to compute the price.
|
||||
/// </summary>
|
||||
private async Task ApplyPricingAsync(QuoteItem item, CreateQuoteItemDto itemDto, int companyId, decimal? ovenRateOverride)
|
||||
{
|
||||
if (itemDto.IsAiItem && itemDto.ManualUnitPrice.HasValue && itemDto.ManualUnitPrice.Value > 0)
|
||||
@@ -127,6 +159,12 @@ public class QuotePricingAssemblyService : IQuotePricingAssemblyService
|
||||
ApplyCalculatedPricing(item, pricing);
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Builds <see cref="QuoteItemCoat"/> entities for a single item, including per-coat pricing.
|
||||
/// If a coat has <c>AddAsIncoming = true</c> and references a catalog item but not an inventory
|
||||
/// item, an incoming <see cref="InventoryItem"/> is auto-created so the shop can track the powder
|
||||
/// order and receive it later — see <see cref="CreateIncomingInventoryItemAsync"/> for details.
|
||||
/// </summary>
|
||||
private async Task<List<QuoteItemCoat>> BuildQuoteItemCoatsAsync(CreateQuoteItemDto itemDto, int companyId, DateTime createdAtUtc)
|
||||
{
|
||||
if (itemDto.Coats == null || itemDto.Coats.Count == 0)
|
||||
@@ -158,6 +196,7 @@ public class QuotePricingAssemblyService : IQuotePricingAssemblyService
|
||||
return coats;
|
||||
}
|
||||
|
||||
/// <summary>Constructs <see cref="QuoteItemPrepService"/> entities from the item DTO's prep service list.</summary>
|
||||
private static List<QuoteItemPrepService> BuildQuoteItemPrepServices(CreateQuoteItemDto itemDto, int companyId, DateTime createdAtUtc)
|
||||
{
|
||||
if (itemDto.PrepServices == null || itemDto.PrepServices.Count == 0)
|
||||
@@ -175,6 +214,11 @@ public class QuotePricingAssemblyService : IQuotePricingAssemblyService
|
||||
.ToList();
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Constructs a bare <see cref="QuoteItem"/> entity from the DTO — no pricing or coats yet.
|
||||
/// Pricing is applied separately by <see cref="ApplyPricingAsync"/> to keep the construction
|
||||
/// and calculation steps distinct and individually testable.
|
||||
/// </summary>
|
||||
private static QuoteItem BuildQuoteItem(CreateQuoteItemDto itemDto, int quoteId, int companyId, DateTime createdAtUtc)
|
||||
{
|
||||
return new QuoteItem
|
||||
@@ -204,6 +248,7 @@ public class QuotePricingAssemblyService : IQuotePricingAssemblyService
|
||||
};
|
||||
}
|
||||
|
||||
/// <summary>Constructs a <see cref="QuoteItemCoat"/> entity from the coat DTO. Per-coat pricing is applied by the caller.</summary>
|
||||
private static QuoteItemCoat BuildQuoteItemCoat(CreateQuoteItemCoatDto coatDto, int companyId, DateTime createdAtUtc)
|
||||
{
|
||||
return new QuoteItemCoat
|
||||
@@ -219,12 +264,17 @@ public class QuotePricingAssemblyService : IQuotePricingAssemblyService
|
||||
TransferEfficiency = coatDto.TransferEfficiency,
|
||||
PowderCostPerLb = coatDto.PowderCostPerLb,
|
||||
PowderToOrder = coatDto.PowderToOrder,
|
||||
NoExtraLayerCharge = coatDto.NoExtraLayerCharge,
|
||||
Notes = coatDto.Notes,
|
||||
CompanyId = companyId,
|
||||
CreatedAt = createdAtUtc
|
||||
};
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Stamps the pricing result onto the quote item entity.
|
||||
/// Broken out as a separate method because it's called from multiple branches of ApplyPricingAsync.
|
||||
/// </summary>
|
||||
private static void ApplyCalculatedPricing(QuoteItem item, QuoteItemPricingResult pricing)
|
||||
{
|
||||
item.UnitPrice = pricing.UnitPrice;
|
||||
@@ -234,6 +284,13 @@ public class QuotePricingAssemblyService : IQuotePricingAssemblyService
|
||||
item.ItemEquipmentCost = pricing.EquipmentCost;
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Checks whether the user changed the AI's surface area or price estimates before saving,
|
||||
/// and sets <c>UserOverrodeEstimate = true</c> on the prediction record if they did.
|
||||
/// This flag feeds the AI analytics reports — over time it reveals how accurate the AI is
|
||||
/// and whether certain item types consistently need manual correction.
|
||||
/// A tolerance of $0.01 / 0.01 sqft is used to ignore floating-point rounding noise.
|
||||
/// </summary>
|
||||
private async Task UpdateAiPredictionOverrideAsync(CreateQuoteItemDto itemDto, decimal finalUnitPrice)
|
||||
{
|
||||
if (!itemDto.AiPredictionId.HasValue) return;
|
||||
@@ -247,6 +304,23 @@ public class QuotePricingAssemblyService : IQuotePricingAssemblyService
|
||||
prediction.UpdatedAt = DateTime.UtcNow;
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Auto-creates an "incoming" <see cref="InventoryItem"/> when a user selects a powder from the
|
||||
/// platform catalog that doesn't yet exist in their company's inventory.
|
||||
///
|
||||
/// WHY this exists: shops often quote jobs using powders they haven't ordered yet. Rather than
|
||||
/// forcing the user to manually add the powder to inventory before quoting, we create an
|
||||
/// IsIncoming=true record on their behalf. The shop can then receive the actual order against
|
||||
/// this record later (updating quantity + receive date) without losing the link to the original quote.
|
||||
///
|
||||
/// The AI augmentation step (LookupByUrlAsync) fills in technical specs (cure temp/time, coverage,
|
||||
/// color families, etc.) that may be missing from the scraped catalog JSON. It is best-effort —
|
||||
/// if it fails, the item is still created with whatever data the catalog has.
|
||||
///
|
||||
/// After creation, <c>coatDto.PowderCostPerLb</c> is cleared so the pricing engine treats this
|
||||
/// as an inventory-linked coat (not a custom powder), ensuring future repricings use the
|
||||
/// inventory unit cost rather than the now-stale manual price from the quote form.
|
||||
/// </summary>
|
||||
private async Task<int?> CreateIncomingInventoryItemAsync(CreateQuoteItemCoatDto coatDto, int companyId)
|
||||
{
|
||||
try
|
||||
|
||||
@@ -95,6 +95,12 @@ public class Appointment : BaseEntity
|
||||
/// </summary>
|
||||
public int ReminderMinutesBefore { get; set; } = 30;
|
||||
|
||||
/// <summary>
|
||||
/// UTC timestamp when the reminder was dispatched. Null means it hasn't fired yet.
|
||||
/// The background service uses this as a deduplication guard to prevent double-sending.
|
||||
/// </summary>
|
||||
public DateTime? ReminderSentAt { get; set; }
|
||||
|
||||
// Navigation Properties
|
||||
public virtual Customer? Customer { get; set; }
|
||||
public virtual Job? Job { get; set; }
|
||||
|
||||
@@ -42,6 +42,13 @@ public class JobItemCoat : BaseEntity
|
||||
public string? PowderReceivedByUserId { get; set; }
|
||||
public decimal? PowderReceivedLbs { get; set; }
|
||||
|
||||
// Pricing flags
|
||||
/// <summary>
|
||||
/// When true, the additional layer labor charge is not applied for this coat even if it is
|
||||
/// not the first coat in the sequence. Used for clear coats, sealers, etc.
|
||||
/// </summary>
|
||||
public bool NoExtraLayerCharge { get; set; }
|
||||
|
||||
// Notes
|
||||
public string? Notes { get; set; }
|
||||
|
||||
|
||||
@@ -33,6 +33,13 @@ public class QuoteItemCoat : BaseEntity
|
||||
public decimal CoatLaborCost { get; set; }
|
||||
public decimal CoatTotalCost { get; set; }
|
||||
|
||||
// Pricing flags
|
||||
/// <summary>
|
||||
/// When true, the additional layer labor charge is not applied for this coat even if it is
|
||||
/// not the first coat in the sequence. Used for clear coats, sealers, etc.
|
||||
/// </summary>
|
||||
public bool NoExtraLayerCharge { get; set; }
|
||||
|
||||
// Notes
|
||||
public string? Notes { get; set; }
|
||||
|
||||
|
||||
@@ -20,5 +20,7 @@ public enum NotificationType
|
||||
SmsInboundStop = 12,
|
||||
SmsInboundHelp = 13,
|
||||
AdminEmail = 14,
|
||||
SmsInboundStart = 15
|
||||
SmsInboundStart = 15,
|
||||
AppointmentReminder = 17,
|
||||
AppointmentReminderStaff = 18
|
||||
}
|
||||
|
||||
@@ -92,7 +92,11 @@ public class ApplicationDbContext : IdentityDbContext<ApplicationUser>, IDataPro
|
||||
if (companyIdClaim != null && int.TryParse(companyIdClaim, out int companyId))
|
||||
return companyId;
|
||||
|
||||
return null;
|
||||
// Authenticated but CompanyId claim is missing or invalid.
|
||||
// Return 0 (never a real company ID) so the global filter generates
|
||||
// "CompanyId = 0" which matches nothing — prevents null-comparison
|
||||
// ambiguity from leaking cross-tenant rows.
|
||||
return 0;
|
||||
}
|
||||
}
|
||||
|
||||
@@ -129,8 +133,11 @@ public class ApplicationDbContext : IdentityDbContext<ApplicationUser>, IDataPro
|
||||
{
|
||||
get
|
||||
{
|
||||
// No HTTP context means background service, hosted service, or unit test — bypass tenant filter
|
||||
if (_httpContextAccessor?.HttpContext == null) return true;
|
||||
if (!IsSuperAdmin) return false;
|
||||
return CurrentCompanyId == null || CurrentCompanyId == 1;
|
||||
// CompanyId == 0 means no claim was present (break-glass / test SuperAdmins) — treat as platform admin
|
||||
return CurrentCompanyId == null || CurrentCompanyId == 0 || CurrentCompanyId == 1;
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
+10633
File diff suppressed because it is too large
Load Diff
File diff suppressed because it is too large
Load Diff
@@ -11,26 +11,20 @@ namespace PowderCoating.Infrastructure.Migrations
|
||||
/// <inheritdoc />
|
||||
protected override void Up(MigrationBuilder migrationBuilder)
|
||||
{
|
||||
migrationBuilder.UpdateData(
|
||||
table: "PricingTiers",
|
||||
keyColumn: "Id",
|
||||
keyValue: 1,
|
||||
column: "CreatedAt",
|
||||
value: new DateTime(2026, 3, 16, 15, 49, 58, 737, DateTimeKind.Utc).AddTicks(7851));
|
||||
|
||||
migrationBuilder.UpdateData(
|
||||
table: "PricingTiers",
|
||||
keyColumn: "Id",
|
||||
keyValue: 2,
|
||||
column: "CreatedAt",
|
||||
value: new DateTime(2026, 3, 16, 15, 49, 58, 737, DateTimeKind.Utc).AddTicks(7856));
|
||||
|
||||
migrationBuilder.UpdateData(
|
||||
table: "PricingTiers",
|
||||
keyColumn: "Id",
|
||||
keyValue: 3,
|
||||
column: "CreatedAt",
|
||||
value: new DateTime(2026, 3, 16, 15, 49, 58, 737, DateTimeKind.Utc).AddTicks(7858));
|
||||
// These UpdateData calls were generated from an existing live database.
|
||||
// On a fresh install the PricingTiers table and its seed rows may not exist yet
|
||||
// (seeding is manual via Platform Management → Seed Data), so guard each update.
|
||||
migrationBuilder.Sql(@"
|
||||
IF OBJECT_ID(N'[PricingTiers]', N'U') IS NOT NULL
|
||||
BEGIN
|
||||
IF EXISTS (SELECT 1 FROM [PricingTiers] WHERE [Id] = 1)
|
||||
UPDATE [PricingTiers] SET [CreatedAt] = '2026-03-16T15:49:58.7377851Z' WHERE [Id] = 1;
|
||||
IF EXISTS (SELECT 1 FROM [PricingTiers] WHERE [Id] = 2)
|
||||
UPDATE [PricingTiers] SET [CreatedAt] = '2026-03-16T15:49:58.7377856Z' WHERE [Id] = 2;
|
||||
IF EXISTS (SELECT 1 FROM [PricingTiers] WHERE [Id] = 3)
|
||||
UPDATE [PricingTiers] SET [CreatedAt] = '2026-03-16T15:49:58.7377858Z' WHERE [Id] = 3;
|
||||
END
|
||||
");
|
||||
}
|
||||
|
||||
/// <inheritdoc />
|
||||
|
||||
Generated
+10633
File diff suppressed because it is too large
Load Diff
+217
@@ -0,0 +1,217 @@
|
||||
using System;
|
||||
using Microsoft.EntityFrameworkCore.Migrations;
|
||||
|
||||
#nullable disable
|
||||
|
||||
namespace PowderCoating.Infrastructure.Migrations
|
||||
{
|
||||
/// <inheritdoc />
|
||||
public partial class AddAppointmentReminderSentAt : Migration
|
||||
{
|
||||
/// <inheritdoc />
|
||||
protected override void Up(MigrationBuilder migrationBuilder)
|
||||
{
|
||||
// Use IF EXISTS guards for all ShopWorker drops — prod and dev diverged on whether
|
||||
// these objects exist, so unconditional drops would fail on whichever DB is missing them.
|
||||
migrationBuilder.Sql(@"
|
||||
IF EXISTS (SELECT 1 FROM sys.foreign_keys WHERE name = 'FK_Jobs_ShopWorkers_ShopWorkerId')
|
||||
ALTER TABLE [Jobs] DROP CONSTRAINT [FK_Jobs_ShopWorkers_ShopWorkerId];
|
||||
IF EXISTS (SELECT 1 FROM sys.foreign_keys WHERE name = 'FK_JobTimeEntries_ShopWorkers_ShopWorkerId')
|
||||
ALTER TABLE [JobTimeEntries] DROP CONSTRAINT [FK_JobTimeEntries_ShopWorkers_ShopWorkerId];
|
||||
IF EXISTS (SELECT 1 FROM sys.foreign_keys WHERE name = 'FK_MaintenanceRecords_ShopWorkers_ShopWorkerId')
|
||||
ALTER TABLE [MaintenanceRecords] DROP CONSTRAINT [FK_MaintenanceRecords_ShopWorkers_ShopWorkerId];
|
||||
IF EXISTS (SELECT 1 FROM sys.tables WHERE name = 'ShopWorkerRoleCosts')
|
||||
DROP TABLE [ShopWorkerRoleCosts];
|
||||
IF EXISTS (SELECT 1 FROM sys.tables WHERE name = 'ShopWorkers')
|
||||
DROP TABLE [ShopWorkers];
|
||||
IF EXISTS (SELECT 1 FROM sys.indexes WHERE name = 'IX_MaintenanceRecords_ShopWorkerId' AND object_id = OBJECT_ID('MaintenanceRecords'))
|
||||
DROP INDEX [IX_MaintenanceRecords_ShopWorkerId] ON [MaintenanceRecords];
|
||||
IF EXISTS (SELECT 1 FROM sys.indexes WHERE name = 'IX_JobTimeEntries_ShopWorkerId' AND object_id = OBJECT_ID('JobTimeEntries'))
|
||||
DROP INDEX [IX_JobTimeEntries_ShopWorkerId] ON [JobTimeEntries];
|
||||
IF EXISTS (SELECT 1 FROM sys.indexes WHERE name = 'IX_Jobs_ShopWorkerId' AND object_id = OBJECT_ID('Jobs'))
|
||||
DROP INDEX [IX_Jobs_ShopWorkerId] ON [Jobs];
|
||||
IF EXISTS (SELECT 1 FROM sys.columns WHERE name = 'ShopWorkerId' AND object_id = OBJECT_ID('MaintenanceRecords'))
|
||||
ALTER TABLE [MaintenanceRecords] DROP COLUMN [ShopWorkerId];
|
||||
IF EXISTS (SELECT 1 FROM sys.columns WHERE name = 'ShopWorkerId' AND object_id = OBJECT_ID('JobTimeEntries'))
|
||||
ALTER TABLE [JobTimeEntries] DROP COLUMN [ShopWorkerId];
|
||||
IF EXISTS (SELECT 1 FROM sys.columns WHERE name = 'ShopWorkerId' AND object_id = OBJECT_ID('Jobs'))
|
||||
ALTER TABLE [Jobs] DROP COLUMN [ShopWorkerId];
|
||||
");
|
||||
|
||||
migrationBuilder.AddColumn<DateTime>(
|
||||
name: "ReminderSentAt",
|
||||
table: "Appointments",
|
||||
type: "datetime2",
|
||||
nullable: true);
|
||||
|
||||
migrationBuilder.UpdateData(
|
||||
table: "PricingTiers",
|
||||
keyColumn: "Id",
|
||||
keyValue: 1,
|
||||
column: "CreatedAt",
|
||||
value: new DateTime(2026, 5, 19, 15, 12, 57, 355, DateTimeKind.Utc).AddTicks(2970));
|
||||
|
||||
migrationBuilder.UpdateData(
|
||||
table: "PricingTiers",
|
||||
keyColumn: "Id",
|
||||
keyValue: 2,
|
||||
column: "CreatedAt",
|
||||
value: new DateTime(2026, 5, 19, 15, 12, 57, 355, DateTimeKind.Utc).AddTicks(2976));
|
||||
|
||||
migrationBuilder.UpdateData(
|
||||
table: "PricingTiers",
|
||||
keyColumn: "Id",
|
||||
keyValue: 3,
|
||||
column: "CreatedAt",
|
||||
value: new DateTime(2026, 5, 19, 15, 12, 57, 355, DateTimeKind.Utc).AddTicks(2977));
|
||||
}
|
||||
|
||||
/// <inheritdoc />
|
||||
protected override void Down(MigrationBuilder migrationBuilder)
|
||||
{
|
||||
migrationBuilder.DropColumn(
|
||||
name: "ReminderSentAt",
|
||||
table: "Appointments");
|
||||
|
||||
migrationBuilder.AddColumn<int>(
|
||||
name: "ShopWorkerId",
|
||||
table: "MaintenanceRecords",
|
||||
type: "int",
|
||||
nullable: true);
|
||||
|
||||
migrationBuilder.AddColumn<int>(
|
||||
name: "ShopWorkerId",
|
||||
table: "JobTimeEntries",
|
||||
type: "int",
|
||||
nullable: true);
|
||||
|
||||
migrationBuilder.AddColumn<int>(
|
||||
name: "ShopWorkerId",
|
||||
table: "Jobs",
|
||||
type: "int",
|
||||
nullable: true);
|
||||
|
||||
migrationBuilder.CreateTable(
|
||||
name: "ShopWorkerRoleCosts",
|
||||
columns: table => new
|
||||
{
|
||||
Id = table.Column<int>(type: "int", nullable: false)
|
||||
.Annotation("SqlServer:Identity", "1, 1"),
|
||||
CompanyId = table.Column<int>(type: "int", nullable: false),
|
||||
CreatedAt = table.Column<DateTime>(type: "datetime2", nullable: false),
|
||||
CreatedBy = table.Column<string>(type: "nvarchar(max)", nullable: true),
|
||||
DeletedAt = table.Column<DateTime>(type: "datetime2", nullable: true),
|
||||
DeletedBy = table.Column<string>(type: "nvarchar(max)", nullable: true),
|
||||
HourlyRate = table.Column<decimal>(type: "decimal(18,2)", nullable: false),
|
||||
IsDeleted = table.Column<bool>(type: "bit", nullable: false),
|
||||
Role = table.Column<int>(type: "int", nullable: false),
|
||||
UpdatedAt = table.Column<DateTime>(type: "datetime2", nullable: true),
|
||||
UpdatedBy = table.Column<string>(type: "nvarchar(max)", nullable: true)
|
||||
},
|
||||
constraints: table =>
|
||||
{
|
||||
table.PrimaryKey("PK_ShopWorkerRoleCosts", x => x.Id);
|
||||
});
|
||||
|
||||
migrationBuilder.CreateTable(
|
||||
name: "ShopWorkers",
|
||||
columns: table => new
|
||||
{
|
||||
Id = table.Column<int>(type: "int", nullable: false)
|
||||
.Annotation("SqlServer:Identity", "1, 1"),
|
||||
CompanyId = table.Column<int>(type: "int", nullable: false),
|
||||
CreatedAt = table.Column<DateTime>(type: "datetime2", nullable: false),
|
||||
CreatedBy = table.Column<string>(type: "nvarchar(max)", nullable: true),
|
||||
DeletedAt = table.Column<DateTime>(type: "datetime2", nullable: true),
|
||||
DeletedBy = table.Column<string>(type: "nvarchar(max)", nullable: true),
|
||||
Email = table.Column<string>(type: "nvarchar(max)", nullable: true),
|
||||
IsActive = table.Column<bool>(type: "bit", nullable: false),
|
||||
IsDeleted = table.Column<bool>(type: "bit", nullable: false),
|
||||
Name = table.Column<string>(type: "nvarchar(max)", nullable: false),
|
||||
Notes = table.Column<string>(type: "nvarchar(max)", nullable: true),
|
||||
Phone = table.Column<string>(type: "nvarchar(max)", nullable: true),
|
||||
Role = table.Column<int>(type: "int", nullable: false),
|
||||
UpdatedAt = table.Column<DateTime>(type: "datetime2", nullable: true),
|
||||
UpdatedBy = table.Column<string>(type: "nvarchar(max)", nullable: true)
|
||||
},
|
||||
constraints: table =>
|
||||
{
|
||||
table.PrimaryKey("PK_ShopWorkers", x => x.Id);
|
||||
table.ForeignKey(
|
||||
name: "FK_ShopWorkers_Companies_CompanyId",
|
||||
column: x => x.CompanyId,
|
||||
principalTable: "Companies",
|
||||
principalColumn: "Id",
|
||||
onDelete: ReferentialAction.Restrict);
|
||||
});
|
||||
|
||||
migrationBuilder.UpdateData(
|
||||
table: "PricingTiers",
|
||||
keyColumn: "Id",
|
||||
keyValue: 1,
|
||||
column: "CreatedAt",
|
||||
value: new DateTime(2026, 5, 15, 23, 44, 10, 471, DateTimeKind.Utc).AddTicks(3131));
|
||||
|
||||
migrationBuilder.UpdateData(
|
||||
table: "PricingTiers",
|
||||
keyColumn: "Id",
|
||||
keyValue: 2,
|
||||
column: "CreatedAt",
|
||||
value: new DateTime(2026, 5, 15, 23, 44, 10, 471, DateTimeKind.Utc).AddTicks(3137));
|
||||
|
||||
migrationBuilder.UpdateData(
|
||||
table: "PricingTiers",
|
||||
keyColumn: "Id",
|
||||
keyValue: 3,
|
||||
column: "CreatedAt",
|
||||
value: new DateTime(2026, 5, 15, 23, 44, 10, 471, DateTimeKind.Utc).AddTicks(3138));
|
||||
|
||||
migrationBuilder.CreateIndex(
|
||||
name: "IX_MaintenanceRecords_ShopWorkerId",
|
||||
table: "MaintenanceRecords",
|
||||
column: "ShopWorkerId");
|
||||
|
||||
migrationBuilder.CreateIndex(
|
||||
name: "IX_JobTimeEntries_ShopWorkerId",
|
||||
table: "JobTimeEntries",
|
||||
column: "ShopWorkerId");
|
||||
|
||||
migrationBuilder.CreateIndex(
|
||||
name: "IX_Jobs_ShopWorkerId",
|
||||
table: "Jobs",
|
||||
column: "ShopWorkerId");
|
||||
|
||||
migrationBuilder.CreateIndex(
|
||||
name: "IX_ShopWorkerRoleCosts_CompanyId_Role",
|
||||
table: "ShopWorkerRoleCosts",
|
||||
columns: new[] { "CompanyId", "Role" },
|
||||
unique: true);
|
||||
|
||||
migrationBuilder.CreateIndex(
|
||||
name: "IX_ShopWorkers_CompanyId",
|
||||
table: "ShopWorkers",
|
||||
column: "CompanyId");
|
||||
|
||||
migrationBuilder.AddForeignKey(
|
||||
name: "FK_Jobs_ShopWorkers_ShopWorkerId",
|
||||
table: "Jobs",
|
||||
column: "ShopWorkerId",
|
||||
principalTable: "ShopWorkers",
|
||||
principalColumn: "Id");
|
||||
|
||||
migrationBuilder.AddForeignKey(
|
||||
name: "FK_JobTimeEntries_ShopWorkers_ShopWorkerId",
|
||||
table: "JobTimeEntries",
|
||||
column: "ShopWorkerId",
|
||||
principalTable: "ShopWorkers",
|
||||
principalColumn: "Id");
|
||||
|
||||
migrationBuilder.AddForeignKey(
|
||||
name: "FK_MaintenanceRecords_ShopWorkers_ShopWorkerId",
|
||||
table: "MaintenanceRecords",
|
||||
column: "ShopWorkerId",
|
||||
principalTable: "ShopWorkers",
|
||||
principalColumn: "Id");
|
||||
}
|
||||
}
|
||||
}
|
||||
Generated
+10639
File diff suppressed because it is too large
Load Diff
+83
@@ -0,0 +1,83 @@
|
||||
using System;
|
||||
using Microsoft.EntityFrameworkCore.Migrations;
|
||||
|
||||
#nullable disable
|
||||
|
||||
namespace PowderCoating.Infrastructure.Migrations
|
||||
{
|
||||
/// <inheritdoc />
|
||||
public partial class AddNoExtraLayerChargeToCoats : Migration
|
||||
{
|
||||
/// <inheritdoc />
|
||||
protected override void Up(MigrationBuilder migrationBuilder)
|
||||
{
|
||||
migrationBuilder.AddColumn<bool>(
|
||||
name: "NoExtraLayerCharge",
|
||||
table: "QuoteItemCoats",
|
||||
type: "bit",
|
||||
nullable: false,
|
||||
defaultValue: false);
|
||||
|
||||
migrationBuilder.AddColumn<bool>(
|
||||
name: "NoExtraLayerCharge",
|
||||
table: "JobItemCoats",
|
||||
type: "bit",
|
||||
nullable: false,
|
||||
defaultValue: false);
|
||||
|
||||
migrationBuilder.UpdateData(
|
||||
table: "PricingTiers",
|
||||
keyColumn: "Id",
|
||||
keyValue: 1,
|
||||
column: "CreatedAt",
|
||||
value: new DateTime(2026, 5, 19, 19, 26, 9, 226, DateTimeKind.Utc).AddTicks(5186));
|
||||
|
||||
migrationBuilder.UpdateData(
|
||||
table: "PricingTiers",
|
||||
keyColumn: "Id",
|
||||
keyValue: 2,
|
||||
column: "CreatedAt",
|
||||
value: new DateTime(2026, 5, 19, 19, 26, 9, 226, DateTimeKind.Utc).AddTicks(5190));
|
||||
|
||||
migrationBuilder.UpdateData(
|
||||
table: "PricingTiers",
|
||||
keyColumn: "Id",
|
||||
keyValue: 3,
|
||||
column: "CreatedAt",
|
||||
value: new DateTime(2026, 5, 19, 19, 26, 9, 226, DateTimeKind.Utc).AddTicks(5191));
|
||||
}
|
||||
|
||||
/// <inheritdoc />
|
||||
protected override void Down(MigrationBuilder migrationBuilder)
|
||||
{
|
||||
migrationBuilder.DropColumn(
|
||||
name: "NoExtraLayerCharge",
|
||||
table: "QuoteItemCoats");
|
||||
|
||||
migrationBuilder.DropColumn(
|
||||
name: "NoExtraLayerCharge",
|
||||
table: "JobItemCoats");
|
||||
|
||||
migrationBuilder.UpdateData(
|
||||
table: "PricingTiers",
|
||||
keyColumn: "Id",
|
||||
keyValue: 1,
|
||||
column: "CreatedAt",
|
||||
value: new DateTime(2026, 5, 19, 15, 44, 18, 742, DateTimeKind.Utc).AddTicks(3960));
|
||||
|
||||
migrationBuilder.UpdateData(
|
||||
table: "PricingTiers",
|
||||
keyColumn: "Id",
|
||||
keyValue: 2,
|
||||
column: "CreatedAt",
|
||||
value: new DateTime(2026, 5, 19, 15, 44, 18, 742, DateTimeKind.Utc).AddTicks(3966));
|
||||
|
||||
migrationBuilder.UpdateData(
|
||||
table: "PricingTiers",
|
||||
keyColumn: "Id",
|
||||
keyValue: 3,
|
||||
column: "CreatedAt",
|
||||
value: new DateTime(2026, 5, 19, 15, 44, 18, 742, DateTimeKind.Utc).AddTicks(3967));
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -716,6 +716,9 @@ namespace PowderCoating.Infrastructure.Migrations
|
||||
b.Property<int>("ReminderMinutesBefore")
|
||||
.HasColumnType("int");
|
||||
|
||||
b.Property<DateTime?>("ReminderSentAt")
|
||||
.HasColumnType("datetime2");
|
||||
|
||||
b.Property<DateTime>("ScheduledEndTime")
|
||||
.HasColumnType("datetime2");
|
||||
|
||||
@@ -4252,9 +4255,6 @@ namespace PowderCoating.Infrastructure.Migrations
|
||||
b.Property<decimal>("ShopSuppliesPercent")
|
||||
.HasColumnType("decimal(18,2)");
|
||||
|
||||
b.Property<int?>("ShopWorkerId")
|
||||
.HasColumnType("int");
|
||||
|
||||
b.Property<string>("SpecialInstructions")
|
||||
.HasColumnType("nvarchar(max)");
|
||||
|
||||
@@ -4296,8 +4296,6 @@ namespace PowderCoating.Infrastructure.Migrations
|
||||
|
||||
b.HasIndex("ScheduledDate");
|
||||
|
||||
b.HasIndex("ShopWorkerId");
|
||||
|
||||
b.HasIndex("CompanyId", "CustomerId")
|
||||
.HasDatabaseName("IX_Jobs_CompanyId_CustomerId");
|
||||
|
||||
@@ -4620,6 +4618,9 @@ namespace PowderCoating.Infrastructure.Migrations
|
||||
b.Property<int>("JobItemId")
|
||||
.HasColumnType("int");
|
||||
|
||||
b.Property<bool>("NoExtraLayerCharge")
|
||||
.HasColumnType("bit");
|
||||
|
||||
b.Property<string>("Notes")
|
||||
.HasColumnType("nvarchar(max)");
|
||||
|
||||
@@ -5439,9 +5440,6 @@ namespace PowderCoating.Infrastructure.Migrations
|
||||
b.Property<string>("Notes")
|
||||
.HasColumnType("nvarchar(max)");
|
||||
|
||||
b.Property<int?>("ShopWorkerId")
|
||||
.HasColumnType("int");
|
||||
|
||||
b.Property<string>("Stage")
|
||||
.HasColumnType("nvarchar(max)");
|
||||
|
||||
@@ -5464,8 +5462,6 @@ namespace PowderCoating.Infrastructure.Migrations
|
||||
|
||||
b.HasIndex("JobId");
|
||||
|
||||
b.HasIndex("ShopWorkerId");
|
||||
|
||||
b.ToTable("JobTimeEntries");
|
||||
});
|
||||
|
||||
@@ -5789,9 +5785,6 @@ namespace PowderCoating.Infrastructure.Migrations
|
||||
b.Property<DateTime>("ScheduledDate")
|
||||
.HasColumnType("datetime2");
|
||||
|
||||
b.Property<int?>("ShopWorkerId")
|
||||
.HasColumnType("int");
|
||||
|
||||
b.Property<int>("Status")
|
||||
.HasColumnType("int");
|
||||
|
||||
@@ -5822,8 +5815,6 @@ namespace PowderCoating.Infrastructure.Migrations
|
||||
|
||||
b.HasIndex("ScheduledDate");
|
||||
|
||||
b.HasIndex("ShopWorkerId");
|
||||
|
||||
b.HasIndex("Status");
|
||||
|
||||
b.HasIndex("CompanyId", "ScheduledDate")
|
||||
@@ -6720,7 +6711,7 @@ namespace PowderCoating.Infrastructure.Migrations
|
||||
{
|
||||
Id = 1,
|
||||
CompanyId = 0,
|
||||
CreatedAt = new DateTime(2026, 5, 15, 23, 44, 10, 471, DateTimeKind.Utc).AddTicks(3131),
|
||||
CreatedAt = new DateTime(2026, 5, 19, 19, 26, 9, 226, DateTimeKind.Utc).AddTicks(5186),
|
||||
Description = "Standard pricing for regular customers",
|
||||
DiscountPercent = 0m,
|
||||
IsActive = true,
|
||||
@@ -6731,7 +6722,7 @@ namespace PowderCoating.Infrastructure.Migrations
|
||||
{
|
||||
Id = 2,
|
||||
CompanyId = 0,
|
||||
CreatedAt = new DateTime(2026, 5, 15, 23, 44, 10, 471, DateTimeKind.Utc).AddTicks(3137),
|
||||
CreatedAt = new DateTime(2026, 5, 19, 19, 26, 9, 226, DateTimeKind.Utc).AddTicks(5190),
|
||||
Description = "5% discount for preferred customers",
|
||||
DiscountPercent = 5m,
|
||||
IsActive = true,
|
||||
@@ -6742,7 +6733,7 @@ namespace PowderCoating.Infrastructure.Migrations
|
||||
{
|
||||
Id = 3,
|
||||
CompanyId = 0,
|
||||
CreatedAt = new DateTime(2026, 5, 15, 23, 44, 10, 471, DateTimeKind.Utc).AddTicks(3138),
|
||||
CreatedAt = new DateTime(2026, 5, 19, 19, 26, 9, 226, DateTimeKind.Utc).AddTicks(5191),
|
||||
Description = "10% discount for premium customers",
|
||||
DiscountPercent = 10m,
|
||||
IsActive = true,
|
||||
@@ -7417,6 +7408,9 @@ namespace PowderCoating.Infrastructure.Migrations
|
||||
b.Property<bool>("IsDeleted")
|
||||
.HasColumnType("bit");
|
||||
|
||||
b.Property<bool>("NoExtraLayerCharge")
|
||||
.HasColumnType("bit");
|
||||
|
||||
b.Property<string>("Notes")
|
||||
.HasColumnType("nvarchar(max)");
|
||||
|
||||
@@ -8019,111 +8013,6 @@ namespace PowderCoating.Infrastructure.Migrations
|
||||
b.ToTable("ReworkRecords");
|
||||
});
|
||||
|
||||
modelBuilder.Entity("PowderCoating.Core.Entities.ShopWorker", b =>
|
||||
{
|
||||
b.Property<int>("Id")
|
||||
.ValueGeneratedOnAdd()
|
||||
.HasColumnType("int");
|
||||
|
||||
SqlServerPropertyBuilderExtensions.UseIdentityColumn(b.Property<int>("Id"));
|
||||
|
||||
b.Property<int>("CompanyId")
|
||||
.HasColumnType("int");
|
||||
|
||||
b.Property<DateTime>("CreatedAt")
|
||||
.HasColumnType("datetime2");
|
||||
|
||||
b.Property<string>("CreatedBy")
|
||||
.HasColumnType("nvarchar(max)");
|
||||
|
||||
b.Property<DateTime?>("DeletedAt")
|
||||
.HasColumnType("datetime2");
|
||||
|
||||
b.Property<string>("DeletedBy")
|
||||
.HasColumnType("nvarchar(max)");
|
||||
|
||||
b.Property<string>("Email")
|
||||
.HasColumnType("nvarchar(max)");
|
||||
|
||||
b.Property<bool>("IsActive")
|
||||
.HasColumnType("bit");
|
||||
|
||||
b.Property<bool>("IsDeleted")
|
||||
.HasColumnType("bit");
|
||||
|
||||
b.Property<string>("Name")
|
||||
.IsRequired()
|
||||
.HasColumnType("nvarchar(max)");
|
||||
|
||||
b.Property<string>("Notes")
|
||||
.HasColumnType("nvarchar(max)");
|
||||
|
||||
b.Property<string>("Phone")
|
||||
.HasColumnType("nvarchar(max)");
|
||||
|
||||
b.Property<int>("Role")
|
||||
.HasColumnType("int");
|
||||
|
||||
b.Property<DateTime?>("UpdatedAt")
|
||||
.HasColumnType("datetime2");
|
||||
|
||||
b.Property<string>("UpdatedBy")
|
||||
.HasColumnType("nvarchar(max)");
|
||||
|
||||
b.HasKey("Id");
|
||||
|
||||
b.HasIndex("CompanyId");
|
||||
|
||||
b.ToTable("ShopWorkers");
|
||||
});
|
||||
|
||||
modelBuilder.Entity("PowderCoating.Core.Entities.ShopWorkerRoleCost", b =>
|
||||
{
|
||||
b.Property<int>("Id")
|
||||
.ValueGeneratedOnAdd()
|
||||
.HasColumnType("int");
|
||||
|
||||
SqlServerPropertyBuilderExtensions.UseIdentityColumn(b.Property<int>("Id"));
|
||||
|
||||
b.Property<int>("CompanyId")
|
||||
.HasColumnType("int");
|
||||
|
||||
b.Property<DateTime>("CreatedAt")
|
||||
.HasColumnType("datetime2");
|
||||
|
||||
b.Property<string>("CreatedBy")
|
||||
.HasColumnType("nvarchar(max)");
|
||||
|
||||
b.Property<DateTime?>("DeletedAt")
|
||||
.HasColumnType("datetime2");
|
||||
|
||||
b.Property<string>("DeletedBy")
|
||||
.HasColumnType("nvarchar(max)");
|
||||
|
||||
b.Property<decimal>("HourlyRate")
|
||||
.HasColumnType("decimal(18,2)");
|
||||
|
||||
b.Property<bool>("IsDeleted")
|
||||
.HasColumnType("bit");
|
||||
|
||||
b.Property<int>("Role")
|
||||
.HasColumnType("int");
|
||||
|
||||
b.Property<DateTime?>("UpdatedAt")
|
||||
.HasColumnType("datetime2");
|
||||
|
||||
b.Property<string>("UpdatedBy")
|
||||
.HasColumnType("nvarchar(max)");
|
||||
|
||||
b.HasKey("Id");
|
||||
|
||||
b.HasIndex("CompanyId", "Role")
|
||||
.IsUnique()
|
||||
.HasDatabaseName("IX_ShopWorkerRoleCosts_CompanyId_Role");
|
||||
|
||||
b.ToTable("ShopWorkerRoleCosts");
|
||||
});
|
||||
|
||||
modelBuilder.Entity("PowderCoating.Core.Entities.StripeWebhookEvent", b =>
|
||||
{
|
||||
b.Property<long>("Id")
|
||||
@@ -9541,10 +9430,6 @@ namespace PowderCoating.Infrastructure.Migrations
|
||||
.HasForeignKey("PowderCoating.Core.Entities.Job", "QuoteId")
|
||||
.OnDelete(DeleteBehavior.SetNull);
|
||||
|
||||
b.HasOne("PowderCoating.Core.Entities.ShopWorker", null)
|
||||
.WithMany("AssignedJobs")
|
||||
.HasForeignKey("ShopWorkerId");
|
||||
|
||||
b.Navigation("AssignedUser");
|
||||
|
||||
b.Navigation("Customer");
|
||||
@@ -9847,13 +9732,7 @@ namespace PowderCoating.Infrastructure.Migrations
|
||||
.OnDelete(DeleteBehavior.Cascade)
|
||||
.IsRequired();
|
||||
|
||||
b.HasOne("PowderCoating.Core.Entities.ShopWorker", "Worker")
|
||||
.WithMany("TimeEntries")
|
||||
.HasForeignKey("ShopWorkerId");
|
||||
|
||||
b.Navigation("Job");
|
||||
|
||||
b.Navigation("Worker");
|
||||
});
|
||||
|
||||
modelBuilder.Entity("PowderCoating.Core.Entities.JournalEntry", b =>
|
||||
@@ -9924,10 +9803,6 @@ namespace PowderCoating.Infrastructure.Migrations
|
||||
.WithMany()
|
||||
.HasForeignKey("RecurrenceParentId");
|
||||
|
||||
b.HasOne("PowderCoating.Core.Entities.ShopWorker", null)
|
||||
.WithMany("AssignedMaintenanceTasks")
|
||||
.HasForeignKey("ShopWorkerId");
|
||||
|
||||
b.Navigation("AssignedUser");
|
||||
|
||||
b.Navigation("Equipment");
|
||||
@@ -10411,15 +10286,6 @@ namespace PowderCoating.Infrastructure.Migrations
|
||||
b.Navigation("ReworkJob");
|
||||
});
|
||||
|
||||
modelBuilder.Entity("PowderCoating.Core.Entities.ShopWorker", b =>
|
||||
{
|
||||
b.HasOne("PowderCoating.Core.Entities.Company", null)
|
||||
.WithMany("ShopWorkers")
|
||||
.HasForeignKey("CompanyId")
|
||||
.OnDelete(DeleteBehavior.Restrict)
|
||||
.IsRequired();
|
||||
});
|
||||
|
||||
modelBuilder.Entity("PowderCoating.Core.Entities.Vendor", b =>
|
||||
{
|
||||
b.HasOne("PowderCoating.Core.Entities.Company", null)
|
||||
@@ -10582,8 +10448,6 @@ namespace PowderCoating.Infrastructure.Migrations
|
||||
|
||||
b.Navigation("Quotes");
|
||||
|
||||
b.Navigation("ShopWorkers");
|
||||
|
||||
b.Navigation("Users");
|
||||
|
||||
b.Navigation("Vendors");
|
||||
@@ -10749,15 +10613,6 @@ namespace PowderCoating.Infrastructure.Migrations
|
||||
b.Navigation("Quotes");
|
||||
});
|
||||
|
||||
modelBuilder.Entity("PowderCoating.Core.Entities.ShopWorker", b =>
|
||||
{
|
||||
b.Navigation("AssignedJobs");
|
||||
|
||||
b.Navigation("AssignedMaintenanceTasks");
|
||||
|
||||
b.Navigation("TimeEntries");
|
||||
});
|
||||
|
||||
modelBuilder.Entity("PowderCoating.Core.Entities.Vendor", b =>
|
||||
{
|
||||
b.Navigation("BillPayments");
|
||||
|
||||
@@ -1152,6 +1152,156 @@ public class NotificationService : INotificationService
|
||||
};
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Sends appointment reminder emails when an appointment's reminder window opens.
|
||||
/// Two emails are dispatched independently:
|
||||
/// <list type="bullet">
|
||||
/// <item>Customer email — sent when a customer is linked, has an email address, and has
|
||||
/// email notifications enabled (<see cref="Customer.NotifyByEmail"/>).</item>
|
||||
/// <item>Staff email — sent to <see cref="BaseEntity.CreatedBy"/> (the user who created
|
||||
/// the appointment). This fires regardless of whether a customer is linked.</item>
|
||||
/// </list>
|
||||
/// Called exclusively by
|
||||
/// <see cref="PowderCoating.Web.BackgroundServices.AppointmentReminderBackgroundService"/>
|
||||
/// after it stamps <c>ReminderSentAt</c> — the caller owns deduplication.
|
||||
/// </summary>
|
||||
public async Task NotifyAppointmentReminderAsync(Appointment appointment)
|
||||
{
|
||||
try
|
||||
{
|
||||
var (companyName, company) = await GetCompanyAsync(appointment.CompanyId);
|
||||
var (replyToEmail, replyToName) = await GetEmailFromAsync(appointment.CompanyId);
|
||||
var baseUrl = await GetBaseUrlAsync();
|
||||
|
||||
var locationLine = !string.IsNullOrWhiteSpace(appointment.Location)
|
||||
? $"<br/><strong>Location:</strong> {WebUtility.HtmlEncode(appointment.Location)}"
|
||||
: string.Empty;
|
||||
|
||||
var appointmentDate = appointment.ScheduledStartTime.ToString("dddd, MMMM d, yyyy");
|
||||
var appointmentTime = appointment.IsAllDay
|
||||
? "All Day"
|
||||
: appointment.ScheduledStartTime.ToString("h:mm tt");
|
||||
var defaultSubject = $"Appointment Reminder — {appointment.Title} on {appointment.ScheduledStartTime:MMMM d, yyyy}";
|
||||
|
||||
// ── Customer email ────────────────────────────────────────────────
|
||||
if (appointment.CustomerId != null)
|
||||
{
|
||||
var customer = appointment.Customer
|
||||
?? await _context.Customers.FindAsync(appointment.CustomerId.Value);
|
||||
|
||||
if (customer != null)
|
||||
{
|
||||
var customerName = GetCustomerDisplayName(customer);
|
||||
var reminderEmails = ParseEmailList(customer.Email);
|
||||
|
||||
if (!customer.NotifyByEmail || reminderEmails.Count == 0)
|
||||
{
|
||||
if (reminderEmails.Count > 0)
|
||||
{
|
||||
await WriteLog(SkippedLog(NotificationChannel.Email, NotificationType.AppointmentReminder,
|
||||
customerName, string.Join(", ", reminderEmails), appointment.CompanyId,
|
||||
customerId: customer.Id));
|
||||
}
|
||||
}
|
||||
else
|
||||
{
|
||||
var customerValues = new Dictionary<string, string>
|
||||
{
|
||||
["companyName"] = companyName,
|
||||
["customerName"] = customerName,
|
||||
["appointmentTitle"] = appointment.Title,
|
||||
["appointmentDate"] = appointmentDate,
|
||||
["appointmentTime"] = appointmentTime,
|
||||
["locationLine"] = locationLine
|
||||
};
|
||||
|
||||
var (custSubject, custHtml) = await GetRenderedEmailAsync(
|
||||
appointment.CompanyId, NotificationType.AppointmentReminder, customerValues, defaultSubject);
|
||||
|
||||
var custFullHtml = AppendUnsubscribeFooterHtml(custHtml, customer.UnsubscribeToken, company, baseUrl);
|
||||
var custPlainText = StripHtml(custFullHtml);
|
||||
|
||||
var (custOk, custErr, custLog) = await SendToEmailListAsync(
|
||||
customer.Email, customerName, custSubject, custPlainText, custFullHtml,
|
||||
replyToEmail: replyToEmail, replyToName: replyToName);
|
||||
|
||||
await WriteLog(new NotificationLog
|
||||
{
|
||||
Channel = NotificationChannel.Email,
|
||||
NotificationType = NotificationType.AppointmentReminder,
|
||||
Status = custOk ? NotificationStatus.Sent : NotificationStatus.Failed,
|
||||
RecipientName = customerName,
|
||||
Recipient = custLog,
|
||||
Subject = custSubject,
|
||||
Message = custPlainText,
|
||||
ErrorMessage = custErr,
|
||||
SentAt = DateTime.UtcNow,
|
||||
CustomerId = customer.Id,
|
||||
CompanyId = appointment.CompanyId
|
||||
});
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
// ── Staff email ───────────────────────────────────────────────────
|
||||
// Send to whoever created the appointment so they get an out-of-app reminder.
|
||||
if (!string.IsNullOrWhiteSpace(appointment.CreatedBy))
|
||||
{
|
||||
// Look up the user's display name from Identity if available.
|
||||
var staffUser = await _context.Users
|
||||
.FirstOrDefaultAsync(u => u.Email == appointment.CreatedBy);
|
||||
var staffName = !string.IsNullOrWhiteSpace(staffUser?.FullName)
|
||||
? staffUser.FullName
|
||||
: appointment.CreatedBy;
|
||||
|
||||
// Include a customer line only when a customer is linked.
|
||||
var customerLine = appointment.Customer != null
|
||||
? $"<br/><strong>Customer:</strong> {WebUtility.HtmlEncode(GetCustomerDisplayName(appointment.Customer))}"
|
||||
: string.Empty;
|
||||
|
||||
var staffValues = new Dictionary<string, string>
|
||||
{
|
||||
["companyName"] = companyName,
|
||||
["staffName"] = staffName,
|
||||
["appointmentTitle"] = appointment.Title,
|
||||
["appointmentDate"] = appointmentDate,
|
||||
["appointmentTime"] = appointmentTime,
|
||||
["customerLine"] = customerLine,
|
||||
["locationLine"] = locationLine
|
||||
};
|
||||
|
||||
var staffDefaultSubject = $"[Reminder] {appointment.Title} — {appointment.ScheduledStartTime:MMMM d, yyyy 'at' h:mm tt}";
|
||||
|
||||
var (staffSubject, staffHtml) = await GetRenderedEmailAsync(
|
||||
appointment.CompanyId, NotificationType.AppointmentReminderStaff, staffValues, staffDefaultSubject);
|
||||
|
||||
var staffPlainText = StripHtml(staffHtml);
|
||||
|
||||
var (staffOk, staffErr, staffLog) = await SendToEmailListAsync(
|
||||
appointment.CreatedBy, staffName, staffSubject, staffPlainText, staffHtml,
|
||||
replyToEmail: replyToEmail, replyToName: replyToName);
|
||||
|
||||
await WriteLog(new NotificationLog
|
||||
{
|
||||
Channel = NotificationChannel.Email,
|
||||
NotificationType = NotificationType.AppointmentReminderStaff,
|
||||
Status = staffOk ? NotificationStatus.Sent : NotificationStatus.Failed,
|
||||
RecipientName = staffName,
|
||||
Recipient = staffLog,
|
||||
Subject = staffSubject,
|
||||
Message = staffPlainText,
|
||||
ErrorMessage = staffErr,
|
||||
SentAt = DateTime.UtcNow,
|
||||
CompanyId = appointment.CompanyId
|
||||
});
|
||||
}
|
||||
}
|
||||
catch (Exception ex)
|
||||
{
|
||||
_logger.LogError(ex, "NotifyAppointmentReminderAsync failed for appointment {AppointmentId}", appointment.Id);
|
||||
}
|
||||
}
|
||||
|
||||
// -----------------------------------------------------------------------
|
||||
// Fallback default templates (used when company has no DB template)
|
||||
// -----------------------------------------------------------------------
|
||||
@@ -1217,6 +1367,14 @@ public class NotificationService : INotificationService
|
||||
"Payment Reminder — Invoice {{invoiceNumber}} ({{daysOverdue}} days overdue)",
|
||||
"<p>Dear {{customerName}},</p><p>This is a friendly reminder that invoice <strong>{{invoiceNumber}}</strong> for <strong>{{invoiceTotal}}</strong> was due on <strong>{{dueDate}}</strong> and is now <strong>{{daysOverdue}} days overdue</strong>.</p><p>Outstanding balance: <strong>{{balanceDue}}</strong></p><p>Please arrange payment at your earliest convenience. If you have already sent payment, please disregard this notice.</p><p>Thank you for your business with {{companyName}}.</p>"
|
||||
),
|
||||
[(NotificationType.AppointmentReminder, NotificationChannel.Email)] = (
|
||||
"Appointment Reminder — {{appointmentTitle}} on {{appointmentDate}}",
|
||||
"<p>Dear {{customerName}},</p><p>This is a reminder that you have an upcoming appointment with <strong>{{companyName}}</strong>.</p><p><strong>Appointment:</strong> {{appointmentTitle}}<br/><strong>Date & Time:</strong> {{appointmentDate}} at {{appointmentTime}}{{locationLine}}</p><p>If you have any questions or need to reschedule, please contact us at your earliest convenience.</p><p>Thank you for choosing {{companyName}}.</p>"
|
||||
),
|
||||
[(NotificationType.AppointmentReminderStaff, NotificationChannel.Email)] = (
|
||||
"[Reminder] {{appointmentTitle}} — {{appointmentDate}}",
|
||||
"<p>Hi {{staffName}},</p><p>This is a reminder that you have an upcoming appointment.</p><p><strong>Appointment:</strong> {{appointmentTitle}}<br/><strong>Date & Time:</strong> {{appointmentDate}} at {{appointmentTime}}{{customerLine}}{{locationLine}}</p><p>— {{companyName}}</p>"
|
||||
),
|
||||
};
|
||||
|
||||
public static (string? Subject, string Body)? Get(NotificationType type, NotificationChannel channel)
|
||||
|
||||
@@ -0,0 +1,162 @@
|
||||
using Microsoft.EntityFrameworkCore;
|
||||
using Microsoft.Extensions.DependencyInjection;
|
||||
using Microsoft.Extensions.Hosting;
|
||||
using Microsoft.Extensions.Logging;
|
||||
using PowderCoating.Application.Interfaces;
|
||||
using PowderCoating.Infrastructure.Data;
|
||||
|
||||
namespace PowderCoating.Web.BackgroundServices;
|
||||
|
||||
/// <summary>
|
||||
/// Polls every 60 seconds for appointments whose reminder window has opened and dispatches
|
||||
/// an email to the linked customer plus an in-app bell notification to company staff.
|
||||
///
|
||||
/// Deduplication strategy: after selecting candidates the service immediately stamps
|
||||
/// <c>ReminderSentAt</c> on each appointment and saves before calling the notification
|
||||
/// methods. This prevents a second loop iteration from re-sending if notifications are slow
|
||||
/// or the application restarts mid-batch. A 24-hour lookback window caps the query so that
|
||||
/// appointments that slipped through (e.g., server downtime) are silently skipped rather
|
||||
/// than sending a stale reminder.
|
||||
/// </summary>
|
||||
public class AppointmentReminderBackgroundService : BackgroundService
|
||||
{
|
||||
private readonly IServiceScopeFactory _scopeFactory;
|
||||
private readonly ILogger<AppointmentReminderBackgroundService> _logger;
|
||||
|
||||
private static readonly TimeSpan PollingInterval = TimeSpan.FromMinutes(1);
|
||||
|
||||
/// <summary>
|
||||
/// Appointments whose scheduled start is more than this far in the past are ignored even
|
||||
/// if their reminder was never sent (server was down, etc.). We do not want to blast a
|
||||
/// customer with a "your appointment is in 30 minutes" email hours after it was due.
|
||||
/// </summary>
|
||||
private static readonly TimeSpan MaxLookback = TimeSpan.FromHours(24);
|
||||
|
||||
public AppointmentReminderBackgroundService(
|
||||
IServiceScopeFactory scopeFactory,
|
||||
ILogger<AppointmentReminderBackgroundService> logger)
|
||||
{
|
||||
_scopeFactory = scopeFactory;
|
||||
_logger = logger;
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Long-running loop that wakes every <see cref="PollingInterval"/> (60 s) and calls
|
||||
/// <see cref="RunAsync"/>. Uses <see cref="Task.Delay"/> with the cancellation token so
|
||||
/// the service shuts down promptly when the application stops.
|
||||
/// </summary>
|
||||
protected override async Task ExecuteAsync(CancellationToken stoppingToken)
|
||||
{
|
||||
_logger.LogInformation("AppointmentReminderBackgroundService started.");
|
||||
|
||||
while (!stoppingToken.IsCancellationRequested)
|
||||
{
|
||||
try
|
||||
{
|
||||
await Task.Delay(PollingInterval, stoppingToken);
|
||||
}
|
||||
catch (OperationCanceledException)
|
||||
{
|
||||
break;
|
||||
}
|
||||
|
||||
if (stoppingToken.IsCancellationRequested) break;
|
||||
|
||||
await RunAsync(stoppingToken);
|
||||
}
|
||||
|
||||
_logger.LogInformation("AppointmentReminderBackgroundService stopped.");
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// One poll iteration: find all appointments whose reminder window has opened, stamp them,
|
||||
/// then dispatch email + in-app notifications. A fresh DI scope is created per poll so that
|
||||
/// the DbContext change tracker is clean each time.
|
||||
/// </summary>
|
||||
private async Task RunAsync(CancellationToken ct)
|
||||
{
|
||||
try
|
||||
{
|
||||
using var scope = _scopeFactory.CreateScope();
|
||||
var db = scope.ServiceProvider.GetRequiredService<ApplicationDbContext>();
|
||||
var notificationService = scope.ServiceProvider.GetRequiredService<INotificationService>();
|
||||
var inAppService = scope.ServiceProvider.GetRequiredService<IInAppNotificationService>();
|
||||
|
||||
// ScheduledStartTime is stored as server-local time (no UTC conversion on form submit),
|
||||
// so compare against DateTime.Now rather than UtcNow to avoid a 4-hour EDT offset.
|
||||
var now = DateTime.Now;
|
||||
var lookback = now - MaxLookback;
|
||||
|
||||
// Find appointments where:
|
||||
// - Reminder is enabled and has not been sent yet
|
||||
// - The reminder window has opened: ScheduledStartTime - ReminderMinutesBefore <= now
|
||||
// - The appointment hasn't been sitting unprocessed for more than MaxLookback
|
||||
// - The appointment status is not terminal (not cancelled, completed, no-show, etc.)
|
||||
// IgnoreQueryFilters bypasses the tenant filter — no HTTP context in a background service.
|
||||
var candidates = await db.Appointments
|
||||
.IgnoreQueryFilters()
|
||||
.Include(a => a.Customer)
|
||||
.Include(a => a.AppointmentStatus)
|
||||
.Where(a =>
|
||||
!a.IsDeleted &&
|
||||
a.IsReminderEnabled &&
|
||||
a.ReminderSentAt == null &&
|
||||
a.ScheduledStartTime > lookback &&
|
||||
EF.Functions.DateDiffMinute(now, a.ScheduledStartTime) <= a.ReminderMinutesBefore &&
|
||||
!a.AppointmentStatus.IsTerminalStatus)
|
||||
.ToListAsync(ct);
|
||||
|
||||
if (candidates.Count == 0) return;
|
||||
|
||||
_logger.LogInformation(
|
||||
"AppointmentReminderBackgroundService: {Count} appointment reminder(s) to dispatch.",
|
||||
candidates.Count);
|
||||
|
||||
// Stamp ReminderSentAt before sending — prevents a restart from re-sending.
|
||||
var stampedAt = now;
|
||||
foreach (var appt in candidates)
|
||||
appt.ReminderSentAt = stampedAt;
|
||||
|
||||
await db.SaveChangesAsync(ct);
|
||||
|
||||
// Now send notifications. Failures here don't roll back the stamp because we'd
|
||||
// rather skip one reminder than spam a customer on every restart.
|
||||
foreach (var appt in candidates)
|
||||
{
|
||||
if (ct.IsCancellationRequested) break;
|
||||
|
||||
try
|
||||
{
|
||||
// Email to linked customer (no-ops internally if customer has opted out)
|
||||
await notificationService.NotifyAppointmentReminderAsync(appt);
|
||||
|
||||
// In-app bell notification for company staff
|
||||
var when = appt.IsAllDay
|
||||
? appt.ScheduledStartTime.ToString("MMMM d, yyyy")
|
||||
: appt.ScheduledStartTime.ToString("MMMM d, yyyy 'at' h:mm tt");
|
||||
|
||||
await inAppService.CreateAsync(
|
||||
companyId: appt.CompanyId,
|
||||
title: $"Appointment Reminder: {appt.Title}",
|
||||
message: $"{appt.AppointmentNumber} is scheduled for {when}.",
|
||||
notificationType: "AppointmentReminder",
|
||||
link: $"/Appointments/Details/{appt.Id}",
|
||||
customerId: appt.CustomerId);
|
||||
|
||||
_logger.LogInformation(
|
||||
"Reminder dispatched for appointment {AppointmentNumber} (id {Id}, company {CompanyId}).",
|
||||
appt.AppointmentNumber, appt.Id, appt.CompanyId);
|
||||
}
|
||||
catch (Exception ex)
|
||||
{
|
||||
_logger.LogError(ex,
|
||||
"Failed to dispatch reminder for appointment {AppointmentId}.", appt.Id);
|
||||
}
|
||||
}
|
||||
}
|
||||
catch (Exception ex)
|
||||
{
|
||||
_logger.LogError(ex, "AppointmentReminderBackgroundService poll failed.");
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -60,10 +60,11 @@ public class AccountingExportController : Controller
|
||||
{
|
||||
var start = startDate.Date;
|
||||
var end = endDate.Date.AddDays(1).AddTicks(-1);
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
|
||||
// ── Load data ─────────────────────────────────────────────────────────
|
||||
var invoices = (await _unitOfWork.Invoices.FindAsync(
|
||||
i => i.InvoiceDate >= start && i.InvoiceDate <= end,
|
||||
i => i.CompanyId == companyId && i.InvoiceDate >= start && i.InvoiceDate <= end,
|
||||
false,
|
||||
i => i.InvoiceItems,
|
||||
i => i.Payments,
|
||||
@@ -72,7 +73,7 @@ public class AccountingExportController : Controller
|
||||
.ToList();
|
||||
|
||||
var expenses = (await _unitOfWork.Expenses.FindAsync(
|
||||
e => e.Date >= start && e.Date <= end,
|
||||
e => e.CompanyId == companyId && e.Date >= start && e.Date <= end,
|
||||
false,
|
||||
e => e.Vendor,
|
||||
e => e.ExpenseAccount,
|
||||
@@ -82,7 +83,7 @@ public class AccountingExportController : Controller
|
||||
|
||||
var bills = await _unitOfWork.Bills.GetForDateRangeAsync(start, end);
|
||||
|
||||
var customers = (await _unitOfWork.Customers.GetAllAsync())
|
||||
var customers = (await _unitOfWork.Customers.FindAsync(c => c.CompanyId == companyId))
|
||||
.OrderBy(c => c.CompanyName ?? c.ContactFirstName)
|
||||
.ToList();
|
||||
|
||||
|
||||
@@ -381,9 +381,15 @@ public class AppointmentsController : Controller
|
||||
return View(dto);
|
||||
}
|
||||
|
||||
// Map changes
|
||||
// Map changes — capture old start before overwrite so we can detect a reschedule.
|
||||
var previousStart = appointment.ScheduledStartTime;
|
||||
_mapper.Map(dto, appointment);
|
||||
|
||||
// If the appointment was rescheduled, clear the reminder stamp so the background
|
||||
// service will fire again at the new time.
|
||||
if (appointment.ScheduledStartTime != previousStart)
|
||||
appointment.ReminderSentAt = null;
|
||||
|
||||
// Update
|
||||
await _unitOfWork.Appointments.UpdateAsync(appointment);
|
||||
await _unitOfWork.CompleteAsync();
|
||||
@@ -486,9 +492,12 @@ public class AppointmentsController : Controller
|
||||
try
|
||||
{
|
||||
var events = new List<CalendarEventDto>();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
|
||||
// 1. Fetch appointments in date range
|
||||
var allAppointments = await _unitOfWork.Appointments.GetAllAsync(false,
|
||||
var allAppointments = await _unitOfWork.Appointments.FindAsync(
|
||||
a => a.CompanyId == companyId,
|
||||
false,
|
||||
a => a.Customer,
|
||||
a => a.AppointmentType,
|
||||
a => a.AppointmentStatus);
|
||||
@@ -501,7 +510,9 @@ public class AppointmentsController : Controller
|
||||
events.AddRange(appointmentEvents);
|
||||
|
||||
// 2. Fetch maintenance records in date range
|
||||
var allMaintenanceRecords = await _unitOfWork.MaintenanceRecords.GetAllAsync(false,
|
||||
var allMaintenanceRecords = await _unitOfWork.MaintenanceRecords.FindAsync(
|
||||
m => m.CompanyId == companyId,
|
||||
false,
|
||||
m => m.Equipment);
|
||||
|
||||
var maintenanceRecords = allMaintenanceRecords
|
||||
@@ -539,7 +550,9 @@ public class AppointmentsController : Controller
|
||||
}
|
||||
|
||||
// 3. Fetch jobs and add as all-day events
|
||||
var allJobs = await _unitOfWork.Jobs.GetAllAsync(false,
|
||||
var allJobs = await _unitOfWork.Jobs.FindAsync(
|
||||
j => j.CompanyId == companyId,
|
||||
false,
|
||||
j => j.Customer,
|
||||
j => j.JobStatus);
|
||||
|
||||
@@ -718,6 +731,8 @@ public class AppointmentsController : Controller
|
||||
var duration = appointment.ScheduledEndTime - appointment.ScheduledStartTime;
|
||||
appointment.ScheduledStartTime = start;
|
||||
appointment.ScheduledEndTime = end;
|
||||
// Drag-drop always changes the time — reset so the reminder fires at the new time.
|
||||
appointment.ReminderSentAt = null;
|
||||
|
||||
await _unitOfWork.Appointments.UpdateAsync(appointment);
|
||||
await _unitOfWork.CompleteAsync();
|
||||
@@ -746,13 +761,16 @@ public class AppointmentsController : Controller
|
||||
try
|
||||
{
|
||||
var terminalCodes = new[] { AppConstants.StatusCodes.Job.Completed, AppConstants.StatusCodes.Job.Delivered, AppConstants.StatusCodes.Job.Cancelled };
|
||||
var allJobs = await _unitOfWork.Jobs.GetAllAsync(false,
|
||||
var calCompanyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var allJobs = await _unitOfWork.Jobs.FindAsync(
|
||||
j => j.CompanyId == calCompanyId,
|
||||
false,
|
||||
j => j.Customer, j => j.JobStatus, j => j.JobItems);
|
||||
|
||||
// Load coats separately — filter by JobItemId using already-loaded item IDs
|
||||
var jobItemIds = allJobs.SelectMany(j => j.JobItems.Select(i => i.Id)).ToList();
|
||||
var allCoats = await _unitOfWork.JobItemCoats.FindAsync(
|
||||
c => jobItemIds.Contains(c.JobItemId));
|
||||
c => jobItemIds.Contains(c.JobItemId) && c.CompanyId == calCompanyId);
|
||||
|
||||
var coatsByItemId = allCoats
|
||||
.Where(c => !c.IsDeleted)
|
||||
@@ -891,7 +909,9 @@ public class AppointmentsController : Controller
|
||||
/// </summary>
|
||||
private async Task PopulateCreateDropdowns()
|
||||
{
|
||||
var customers = await _unitOfWork.Customers.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
|
||||
var customers = await _unitOfWork.Customers.FindAsync(c => c.CompanyId == companyId);
|
||||
var customerList = customers.Select(c => new
|
||||
{
|
||||
c.Id,
|
||||
@@ -903,19 +923,16 @@ public class AppointmentsController : Controller
|
||||
.ToList();
|
||||
ViewBag.Customers = new SelectList(customerList, "Id", "DisplayName");
|
||||
|
||||
// Use cached appointment types
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var types = await _lookupCache.GetAppointmentTypeLookupsAsync(companyId);
|
||||
ViewBag.AppointmentTypes = new SelectList(types.Where(t => t.IsActive).OrderBy(t => t.DisplayOrder), "Id", "DisplayName");
|
||||
|
||||
var companyIdForWorkers = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var workers = await _userManager.Users
|
||||
.Where(u => u.CompanyId == companyIdForWorkers && u.IsActive && u.CompanyRole != null)
|
||||
.Where(u => u.CompanyId == companyId && u.IsActive && u.CompanyRole != null)
|
||||
.OrderBy(u => u.FirstName).ThenBy(u => u.LastName)
|
||||
.ToListAsync();
|
||||
ViewBag.Workers = new SelectList(workers.Select(u => new { u.Id, FullName = u.FullName }), "Id", "FullName");
|
||||
|
||||
var jobs = await _unitOfWork.Jobs.GetAllAsync();
|
||||
var jobs = await _unitOfWork.Jobs.FindAsync(j => j.CompanyId == companyId);
|
||||
ViewBag.Jobs = new SelectList(jobs.OrderBy(j => j.JobNumber), "Id", "JobNumber");
|
||||
}
|
||||
|
||||
|
||||
@@ -27,15 +27,18 @@ namespace PowderCoating.Web.Controllers
|
||||
{
|
||||
private readonly IUnitOfWork _unitOfWork;
|
||||
private readonly IMapper _mapper;
|
||||
private readonly ITenantContext _tenantContext;
|
||||
private readonly ILogger<CatalogCategoriesController> _logger;
|
||||
|
||||
public CatalogCategoriesController(
|
||||
IUnitOfWork unitOfWork,
|
||||
IMapper mapper,
|
||||
ITenantContext tenantContext,
|
||||
ILogger<CatalogCategoriesController> logger)
|
||||
{
|
||||
_unitOfWork = unitOfWork;
|
||||
_mapper = mapper;
|
||||
_tenantContext = tenantContext;
|
||||
_logger = logger;
|
||||
}
|
||||
|
||||
@@ -52,8 +55,9 @@ namespace PowderCoating.Web.Controllers
|
||||
{
|
||||
try
|
||||
{
|
||||
var indexCompanyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var categories = await _unitOfWork.CatalogCategories
|
||||
.GetAllAsync(false,
|
||||
.FindAsync(c => c.CompanyId == indexCompanyId, false,
|
||||
c => c.ParentCategory,
|
||||
c => c.SubCategories,
|
||||
c => c.Items);
|
||||
@@ -164,7 +168,8 @@ namespace PowderCoating.Web.Controllers
|
||||
if (ModelState.IsValid)
|
||||
{
|
||||
// Check for duplicate category name under the same parent (case-insensitive)
|
||||
var allCategories = await _unitOfWork.CatalogCategories.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var allCategories = await _unitOfWork.CatalogCategories.FindAsync(c => c.CompanyId == companyId);
|
||||
var existingCategory = allCategories.FirstOrDefault(c =>
|
||||
c.Name.Equals(dto.Name.Trim(), StringComparison.OrdinalIgnoreCase) &&
|
||||
c.ParentCategoryId == dto.ParentCategoryId);
|
||||
@@ -272,7 +277,8 @@ namespace PowderCoating.Web.Controllers
|
||||
|
||||
if (nameChanged || parentChanged)
|
||||
{
|
||||
var allCategories = await _unitOfWork.CatalogCategories.GetAllAsync();
|
||||
var editCompanyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var allCategories = await _unitOfWork.CatalogCategories.FindAsync(c => c.CompanyId == editCompanyId);
|
||||
var existingCategory = allCategories.FirstOrDefault(c =>
|
||||
c.Id != id &&
|
||||
c.Name.Equals(dto.Name.Trim(), StringComparison.OrdinalIgnoreCase) &&
|
||||
@@ -444,7 +450,8 @@ namespace PowderCoating.Web.Controllers
|
||||
var trimmedName = request.Name.Trim();
|
||||
|
||||
// Check for duplicate category name under the same parent (case-insensitive)
|
||||
var allCategories = await _unitOfWork.CatalogCategories.GetAllAsync();
|
||||
var quickCompanyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var allCategories = await _unitOfWork.CatalogCategories.FindAsync(c => c.CompanyId == quickCompanyId);
|
||||
var existingCategory = allCategories.FirstOrDefault(c =>
|
||||
c.Name.Equals(trimmedName, StringComparison.OrdinalIgnoreCase) &&
|
||||
c.ParentCategoryId == request.ParentCategoryId);
|
||||
@@ -500,8 +507,9 @@ namespace PowderCoating.Web.Controllers
|
||||
{
|
||||
try
|
||||
{
|
||||
var treeCompanyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var categories = await _unitOfWork.CatalogCategories
|
||||
.GetAllAsync(false, c => c.SubCategories, c => c.Items);
|
||||
.FindAsync(c => c.CompanyId == treeCompanyId, false, c => c.SubCategories, c => c.Items);
|
||||
|
||||
// Build tree from root categories
|
||||
var rootCategories = categories
|
||||
@@ -535,7 +543,8 @@ namespace PowderCoating.Web.Controllers
|
||||
{
|
||||
try
|
||||
{
|
||||
var categories = (await _unitOfWork.CatalogCategories.GetAllAsync()).ToList();
|
||||
var dropdownCompanyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var categories = (await _unitOfWork.CatalogCategories.FindAsync(c => c.CompanyId == dropdownCompanyId)).ToList();
|
||||
|
||||
// Build hierarchical list (parents before children)
|
||||
var hierarchicalList = new List<CatalogCategory>();
|
||||
@@ -573,7 +582,8 @@ namespace PowderCoating.Web.Controllers
|
||||
/// </param>
|
||||
private async Task PopulateParentCategoryDropdown(int? excludeCategoryId = null)
|
||||
{
|
||||
var categories = (await _unitOfWork.CatalogCategories.GetAllAsync()).ToList();
|
||||
var parentDropCompanyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var categories = (await _unitOfWork.CatalogCategories.FindAsync(c => c.CompanyId == parentDropCompanyId)).ToList();
|
||||
|
||||
// Exclude the current category and its descendants to prevent circular references
|
||||
var excludedIds = new HashSet<int>();
|
||||
@@ -700,7 +710,8 @@ namespace PowderCoating.Web.Controllers
|
||||
if (categoryId == newParentId)
|
||||
return true;
|
||||
|
||||
var categories = (await _unitOfWork.CatalogCategories.GetAllAsync()).ToList();
|
||||
var circleCompanyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var categories = (await _unitOfWork.CatalogCategories.FindAsync(c => c.CompanyId == circleCompanyId)).ToList();
|
||||
var current = categories.FirstOrDefault(c => c.Id == newParentId);
|
||||
|
||||
while (current != null)
|
||||
|
||||
@@ -83,7 +83,8 @@ namespace PowderCoating.Web.Controllers
|
||||
try
|
||||
{
|
||||
// Get all categories with their items
|
||||
var allCategories = (await _unitOfWork.CatalogCategories.GetAllAsync(false, c => c.Items)).ToList();
|
||||
var itemsCompanyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var allCategories = (await _unitOfWork.CatalogCategories.FindAsync(c => c.CompanyId == itemsCompanyId, false, c => c.Items)).ToList();
|
||||
var allItems = allCategories.SelectMany(c => c.Items).ToList();
|
||||
|
||||
// Apply search filter
|
||||
@@ -578,7 +579,8 @@ namespace PowderCoating.Web.Controllers
|
||||
return Json(new List<object>());
|
||||
}
|
||||
|
||||
var allItems = await _unitOfWork.CatalogItems.GetAllAsync(false, i => i.Category);
|
||||
var searchCompanyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var allItems = await _unitOfWork.CatalogItems.FindAsync(i => i.CompanyId == searchCompanyId, false, i => i.Category);
|
||||
var search = searchTerm.ToLower();
|
||||
|
||||
var items = allItems
|
||||
@@ -694,7 +696,8 @@ namespace PowderCoating.Web.Controllers
|
||||
/// </summary>
|
||||
private async Task PopulateCategoryDropdown()
|
||||
{
|
||||
var categories = (await _unitOfWork.CatalogCategories.GetAllAsync()).ToList();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var categories = (await _unitOfWork.CatalogCategories.FindAsync(c => c.CompanyId == companyId)).ToList();
|
||||
|
||||
// Build hierarchical list (parents before children)
|
||||
var hierarchicalList = new List<CatalogCategory>();
|
||||
@@ -1045,7 +1048,7 @@ namespace PowderCoating.Web.Controllers
|
||||
// Load all categories so we can build full paths (e.g. "Cerakote > Firearms").
|
||||
// The full path gives Claude the coating-type context it needs — an item in
|
||||
// "Firearms" under "Cerakote" costs very differently than one under "Powder Coat".
|
||||
var allCategories = (await _unitOfWork.CatalogCategories.GetAllAsync())
|
||||
var allCategories = (await _unitOfWork.CatalogCategories.FindAsync(c => c.CompanyId == currentUser.CompanyId))
|
||||
.ToDictionary(c => c.Id);
|
||||
|
||||
// Load company operating costs
|
||||
|
||||
@@ -142,10 +142,10 @@ public class CompanySettingsController : Controller
|
||||
&& !connectClientId.Contains("your_connect_client_id_here", StringComparison.OrdinalIgnoreCase);
|
||||
|
||||
// Load notification templates for inline tab
|
||||
var existing = await _unitOfWork.NotificationTemplates.GetAllAsync();
|
||||
var existing = await _unitOfWork.NotificationTemplates.FindAsync(t => t.CompanyId == companyId.Value);
|
||||
var seeded = await EnsureNotificationTemplatesSeededAsync(companyId.Value, existing.ToList());
|
||||
if (seeded > 0)
|
||||
existing = await _unitOfWork.NotificationTemplates.GetAllAsync();
|
||||
existing = await _unitOfWork.NotificationTemplates.FindAsync(t => t.CompanyId == companyId.Value);
|
||||
|
||||
dto.NotificationTemplates = existing
|
||||
.OrderBy(t => (int)t.NotificationType).ThenBy(t => (int)t.Channel)
|
||||
@@ -755,8 +755,8 @@ public class CompanySettingsController : Controller
|
||||
|
||||
var costs = company.OperatingCosts;
|
||||
|
||||
var ovens = (await _unitOfWork.OvenCosts.FindAsync(o => o.IsActive)).OrderBy(o => o.DisplayOrder).ToList();
|
||||
var coatingCategories = (await _unitOfWork.InventoryCategoryLookups.FindAsync(c => c.IsCoating)).ToList();
|
||||
var ovens = (await _unitOfWork.OvenCosts.FindAsync(o => o.IsActive && o.CompanyId == companyId.Value)).OrderBy(o => o.DisplayOrder).ToList();
|
||||
var coatingCategories = (await _unitOfWork.InventoryCategoryLookups.FindAsync(c => c.IsCoating && c.CompanyId == companyId.Value)).ToList();
|
||||
|
||||
var sb = new System.Text.StringBuilder();
|
||||
|
||||
@@ -920,7 +920,8 @@ public class CompanySettingsController : Controller
|
||||
{
|
||||
try
|
||||
{
|
||||
var statuses = await _unitOfWork.JobStatusLookups.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var statuses = await _unitOfWork.JobStatusLookups.FindAsync(s => s.CompanyId == companyId);
|
||||
var sortedStatuses = statuses.OrderBy(s => s.DisplayOrder).ToList();
|
||||
|
||||
var dtos = _mapper.Map<List<JobStatusLookupDto>>(sortedStatuses);
|
||||
@@ -1071,7 +1072,8 @@ public class CompanySettingsController : Controller
|
||||
if (!ModelState.IsValid)
|
||||
return Json(new { success = false, message = "Invalid data" });
|
||||
|
||||
var statuses = await _unitOfWork.JobStatusLookups.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId();
|
||||
var statuses = await _unitOfWork.JobStatusLookups.FindAsync(s => s.CompanyId == (companyId ?? 0));
|
||||
|
||||
for (int i = 0; i < dto.OrderedIds.Count; i++)
|
||||
{
|
||||
@@ -1084,7 +1086,6 @@ public class CompanySettingsController : Controller
|
||||
}
|
||||
|
||||
await _unitOfWork.CompleteAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId();
|
||||
if (companyId.HasValue) _lookupCache.InvalidateCompanyCache(companyId.Value);
|
||||
|
||||
_logger.LogInformation("Job statuses reordered");
|
||||
@@ -1113,7 +1114,8 @@ public class CompanySettingsController : Controller
|
||||
{
|
||||
try
|
||||
{
|
||||
var priorities = await _unitOfWork.JobPriorityLookups.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var priorities = await _unitOfWork.JobPriorityLookups.FindAsync(p => p.CompanyId == companyId);
|
||||
var sortedPriorities = priorities.OrderBy(p => p.DisplayOrder).ToList();
|
||||
|
||||
var dtos = _mapper.Map<List<JobPriorityLookupDto>>(sortedPriorities);
|
||||
@@ -1258,7 +1260,8 @@ public class CompanySettingsController : Controller
|
||||
if (!ModelState.IsValid)
|
||||
return Json(new { success = false, message = "Invalid data" });
|
||||
|
||||
var priorities = await _unitOfWork.JobPriorityLookups.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var priorities = await _unitOfWork.JobPriorityLookups.FindAsync(p => p.CompanyId == companyId);
|
||||
|
||||
for (int i = 0; i < dto.OrderedIds.Count; i++)
|
||||
{
|
||||
@@ -1297,7 +1300,8 @@ public class CompanySettingsController : Controller
|
||||
{
|
||||
try
|
||||
{
|
||||
var statuses = await _unitOfWork.QuoteStatusLookups.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var statuses = await _unitOfWork.QuoteStatusLookups.FindAsync(s => s.CompanyId == companyId);
|
||||
var sortedStatuses = statuses.OrderBy(s => s.DisplayOrder).ToList();
|
||||
|
||||
var dtos = _mapper.Map<List<QuoteStatusLookupDto>>(sortedStatuses);
|
||||
@@ -1478,7 +1482,8 @@ public class CompanySettingsController : Controller
|
||||
if (!ModelState.IsValid)
|
||||
return Json(new { success = false, message = "Invalid data" });
|
||||
|
||||
var statuses = await _unitOfWork.QuoteStatusLookups.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var statuses = await _unitOfWork.QuoteStatusLookups.FindAsync(s => s.CompanyId == companyId);
|
||||
|
||||
for (int i = 0; i < dto.OrderedIds.Count; i++)
|
||||
{
|
||||
@@ -1517,7 +1522,8 @@ public class CompanySettingsController : Controller
|
||||
{
|
||||
try
|
||||
{
|
||||
var services = await _unitOfWork.PrepServices.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var services = await _unitOfWork.PrepServices.FindAsync(s => s.CompanyId == companyId);
|
||||
var sortedServices = services.OrderBy(s => s.DisplayOrder).ToList();
|
||||
|
||||
var dtos = _mapper.Map<List<PrepServiceDto>>(sortedServices);
|
||||
@@ -1639,7 +1645,8 @@ public class CompanySettingsController : Controller
|
||||
if (!ModelState.IsValid)
|
||||
return Json(new { success = false, message = "Invalid data" });
|
||||
|
||||
var services = await _unitOfWork.PrepServices.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var services = await _unitOfWork.PrepServices.FindAsync(s => s.CompanyId == companyId);
|
||||
|
||||
for (int i = 0; i < dto.OrderedIds.Count; i++)
|
||||
{
|
||||
@@ -1812,7 +1819,8 @@ public class CompanySettingsController : Controller
|
||||
{
|
||||
try
|
||||
{
|
||||
var types = await _unitOfWork.AppointmentTypeLookups.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var types = await _unitOfWork.AppointmentTypeLookups.FindAsync(t => t.CompanyId == companyId);
|
||||
var sortedTypes = types.OrderBy(t => t.DisplayOrder).ToList();
|
||||
|
||||
var dtos = _mapper.Map<List<AppointmentTypeLookupDto>>(sortedTypes);
|
||||
@@ -1956,7 +1964,8 @@ public class CompanySettingsController : Controller
|
||||
if (!ModelState.IsValid)
|
||||
return Json(new { success = false, message = "Invalid data" });
|
||||
|
||||
var types = await _unitOfWork.AppointmentTypeLookups.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var types = await _unitOfWork.AppointmentTypeLookups.FindAsync(t => t.CompanyId == companyId);
|
||||
|
||||
for (int i = 0; i < dto.OrderedIds.Count; i++)
|
||||
{
|
||||
@@ -1996,7 +2005,8 @@ public class CompanySettingsController : Controller
|
||||
{
|
||||
try
|
||||
{
|
||||
var categories = await _unitOfWork.InventoryCategoryLookups.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var categories = await _unitOfWork.InventoryCategoryLookups.FindAsync(c => c.CompanyId == companyId);
|
||||
var sortedCategories = categories.OrderBy(c => c.DisplayOrder).ToList();
|
||||
|
||||
var dtos = _mapper.Map<List<InventoryCategoryLookupDto>>(sortedCategories);
|
||||
@@ -2132,7 +2142,8 @@ public class CompanySettingsController : Controller
|
||||
if (!ModelState.IsValid)
|
||||
return Json(new { success = false, message = "Invalid data" });
|
||||
|
||||
var categories = await _unitOfWork.InventoryCategoryLookups.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var categories = await _unitOfWork.InventoryCategoryLookups.FindAsync(c => c.CompanyId == companyId);
|
||||
|
||||
for (int i = 0; i < dto.OrderedIds.Count; i++)
|
||||
{
|
||||
@@ -2349,12 +2360,12 @@ public class CompanySettingsController : Controller
|
||||
if (companyId == null) return RedirectToAction(nameof(Index));
|
||||
|
||||
// Load all existing templates for this company
|
||||
var existing = await _unitOfWork.NotificationTemplates.GetAllAsync();
|
||||
var existing = await _unitOfWork.NotificationTemplates.FindAsync(t => t.CompanyId == companyId.Value);
|
||||
|
||||
// Auto-seed any missing canonical combinations
|
||||
var seeded = await EnsureNotificationTemplatesSeededAsync(companyId.Value, existing.ToList());
|
||||
if (seeded > 0)
|
||||
existing = await _unitOfWork.NotificationTemplates.GetAllAsync();
|
||||
existing = await _unitOfWork.NotificationTemplates.FindAsync(t => t.CompanyId == companyId.Value);
|
||||
|
||||
var dtos = existing.OrderBy(t => (int)t.NotificationType).ThenBy(t => (int)t.Channel)
|
||||
.Select(t => new NotificationTemplateDto
|
||||
|
||||
@@ -315,7 +315,8 @@ public class CreditMemosController : Controller
|
||||
|
||||
private async Task PopulateCustomersAsync(int? selectedId)
|
||||
{
|
||||
var customers = await _unitOfWork.Customers.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var customers = await _unitOfWork.Customers.FindAsync(c => c.CompanyId == companyId);
|
||||
ViewBag.Customers = customers
|
||||
.OrderBy(c => c.CompanyName ?? $"{c.ContactFirstName} {c.ContactLastName}".Trim())
|
||||
.Select(c => new SelectListItem
|
||||
|
||||
@@ -342,14 +342,16 @@ public class DashboardController : Controller
|
||||
TipOfTheDay = data.TipOfTheDay
|
||||
};
|
||||
|
||||
// Resolve company once so all remaining queries are explicitly scoped
|
||||
var currentCompanyId = _tenantContext.GetCurrentCompanyId();
|
||||
var companyId = currentCompanyId ?? 0;
|
||||
|
||||
// Dropdowns for the "Add Custom Powder to Inventory" modal
|
||||
var inventoryCategories = (await _unitOfWork.InventoryCategoryLookups.GetAllAsync())
|
||||
.Where(c => c.IsActive)
|
||||
var inventoryCategories = (await _unitOfWork.InventoryCategoryLookups.FindAsync(c => c.IsActive && c.CompanyId == companyId))
|
||||
.OrderBy(c => c.DisplayOrder)
|
||||
.Select(c => new { c.Id, c.DisplayName })
|
||||
.ToList();
|
||||
var vendors = (await _unitOfWork.Vendors.GetAllAsync())
|
||||
.Where(v => v.IsActive)
|
||||
var vendors = (await _unitOfWork.Vendors.FindAsync(v => v.IsActive && v.CompanyId == companyId))
|
||||
.OrderBy(v => v.CompanyName)
|
||||
.Select(v => new { v.Id, v.CompanyName })
|
||||
.ToList();
|
||||
@@ -357,7 +359,6 @@ public class DashboardController : Controller
|
||||
ViewBag.VendorList = vendors;
|
||||
|
||||
// Config health check — surface setup gaps to company admins
|
||||
var currentCompanyId = _tenantContext.GetCurrentCompanyId();
|
||||
if (currentCompanyId.HasValue)
|
||||
{
|
||||
ViewBag.ConfigHealth = await _configHealth.CheckAsync(currentCompanyId.Value);
|
||||
@@ -711,8 +712,8 @@ public class DashboardController : Controller
|
||||
i => i.Coats.Any(c => c.Id == coatId), false, i => i.Job);
|
||||
var companyId = jobItem?.Job?.CompanyId ?? _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
|
||||
// Check SKU uniqueness
|
||||
if (await _unitOfWork.InventoryItems.AnyAsync(i => i.SKU == sku.Trim()))
|
||||
// Check SKU uniqueness within this company
|
||||
if (await _unitOfWork.InventoryItems.AnyAsync(i => i.SKU == sku.Trim() && i.CompanyId == companyId))
|
||||
return Json(new { success = false, message = $"SKU '{sku}' already exists in inventory." });
|
||||
|
||||
// Determine category display name for legacy field
|
||||
|
||||
@@ -160,7 +160,8 @@ public class InventoryController : Controller
|
||||
var pagedResult = PagedResult<InventoryListDto>.From(gridRequest, itemDtos, totalCount);
|
||||
|
||||
// Load all items once to compute sidebar stats and category list in memory
|
||||
var allItems = (await _unitOfWork.InventoryItems.GetAllAsync()).ToList();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var allItems = (await _unitOfWork.InventoryItems.FindAsync(i => i.CompanyId == companyId)).ToList();
|
||||
ViewBag.Categories = allItems.Select(i => i.Category).Where(c => c != null).Distinct().OrderBy(c => c).ToList();
|
||||
ViewBag.StatsLowStockCount = allItems.Count(i => i.IsActive && i.QuantityOnHand <= i.ReorderPoint);
|
||||
ViewBag.StatsActiveCount = allItems.Count(i => i.IsActive);
|
||||
@@ -1106,7 +1107,8 @@ public class InventoryController : Controller
|
||||
|
||||
// Build a set of SKUs already in this company's inventory so we can exclude them.
|
||||
// When editing, the current item's own SKU is re-included so its catalog entry still appears.
|
||||
var existingItems = await _unitOfWork.InventoryItems.GetAllAsync();
|
||||
var skuCompanyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var existingItems = await _unitOfWork.InventoryItems.FindAsync(i => i.CompanyId == skuCompanyId);
|
||||
var existingSkus = existingItems
|
||||
.Where(i => !string.IsNullOrWhiteSpace(i.ManufacturerPartNumber) && i.Id != (currentId ?? 0))
|
||||
.Select(i => i.ManufacturerPartNumber!.Trim().ToLower())
|
||||
@@ -1182,7 +1184,7 @@ public class InventoryController : Controller
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
|
||||
// Find the default coating category to assign
|
||||
var categories = await _unitOfWork.InventoryCategoryLookups.GetAllAsync();
|
||||
var categories = await _unitOfWork.InventoryCategoryLookups.FindAsync(c => c.CompanyId == companyId);
|
||||
var coatingCategory = categories
|
||||
.Where(c => c.IsActive && c.IsCoating)
|
||||
.OrderBy(c => c.DisplayOrder)
|
||||
@@ -1369,11 +1371,11 @@ public class InventoryController : Controller
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
ViewBag.AiInventoryAssistEnabled = await _subscriptionService.IsAiInventoryAssistEnabledAsync(companyId);
|
||||
|
||||
var vendors = await _unitOfWork.Vendors.GetAllAsync();
|
||||
var vendors = await _unitOfWork.Vendors.FindAsync(v => v.CompanyId == companyId);
|
||||
ViewBag.Vendors = new SelectList(vendors.Where(s => s.IsActive).OrderBy(s => s.CompanyName), "Id", "CompanyName");
|
||||
|
||||
// Load categories from lookup table
|
||||
var allCategories = await _unitOfWork.InventoryCategoryLookups.GetAllAsync();
|
||||
var allCategories = await _unitOfWork.InventoryCategoryLookups.FindAsync(c => c.CompanyId == companyId);
|
||||
var categories = allCategories
|
||||
.Where(c => c.IsActive)
|
||||
.OrderBy(c => c.DisplayOrder)
|
||||
@@ -1738,7 +1740,8 @@ public class InventoryController : Controller
|
||||
DateTime? dateTo,
|
||||
string? typeFilter)
|
||||
{
|
||||
var allItems = await _unitOfWork.InventoryItems.GetAllAsync();
|
||||
var ledgerCompanyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var allItems = await _unitOfWork.InventoryItems.FindAsync(i => i.CompanyId == ledgerCompanyId);
|
||||
var itemList = allItems
|
||||
.Where(i => i.IsActive || i.QuantityOnHand > 0)
|
||||
.OrderBy(i => i.Name)
|
||||
|
||||
@@ -2213,7 +2213,7 @@ public class InvoicesController : Controller
|
||||
/// </summary>
|
||||
private async Task PopulateCreateViewBagAsync(int companyId, string? selectedTerms = null)
|
||||
{
|
||||
var customers = await _unitOfWork.Customers.GetAllAsync();
|
||||
var customers = await _unitOfWork.Customers.FindAsync(c => c.CompanyId == companyId);
|
||||
ViewBag.Customers = customers.Where(c => c.IsActive).OrderBy(c => c.CompanyName ?? c.ContactLastName).ToList();
|
||||
|
||||
// Expose company default tax rate and exempt customer IDs for client-side tax handling
|
||||
|
||||
@@ -36,7 +36,9 @@ public class JobTemplatesController : Controller
|
||||
/// </summary>
|
||||
public async Task<IActionResult> Index()
|
||||
{
|
||||
var templates = await _unitOfWork.JobTemplates.GetAllAsync(
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var templates = await _unitOfWork.JobTemplates.FindAsync(
|
||||
t => t.CompanyId == companyId,
|
||||
false,
|
||||
t => t.Customer,
|
||||
t => t.Items);
|
||||
|
||||
@@ -477,6 +477,7 @@ public class JobsController : Controller
|
||||
transferEfficiency = c.TransferEfficiency,
|
||||
powderCostPerLb = c.PowderCostPerLb,
|
||||
powderToOrder = c.PowderToOrder,
|
||||
noExtraLayerCharge = c.NoExtraLayerCharge,
|
||||
notes = c.Notes
|
||||
}),
|
||||
prepServices = ji.PrepServices.Select(ps => new {
|
||||
@@ -499,11 +500,21 @@ public class JobsController : Controller
|
||||
ViewBag.MaterialsUsed = allJobTransactions;
|
||||
|
||||
// Inventory items for the manual log-material modal
|
||||
var inventoryItemsForModal = (await _unitOfWork.InventoryItems.GetAllAsync())
|
||||
var inventoryItemsForModal = (await _unitOfWork.InventoryItems.FindAsync(i => i.CompanyId == job.CompanyId))
|
||||
.OrderBy(i => i.Name)
|
||||
.Select(i => new { i.Id, i.Name, i.UnitOfMeasure, i.QuantityOnHand })
|
||||
.Select(i => new { i.Id, i.Name, i.Manufacturer, i.UnitOfMeasure, i.QuantityOnHand })
|
||||
.ToList();
|
||||
ViewBag.InventoryItemsForModal = System.Text.Json.JsonSerializer.Serialize(inventoryItemsForModal);
|
||||
var jsonOpts = new System.Text.Json.JsonSerializerOptions { PropertyNamingPolicy = System.Text.Json.JsonNamingPolicy.CamelCase };
|
||||
ViewBag.InventoryItemsForModal = System.Text.Json.JsonSerializer.Serialize(inventoryItemsForModal, jsonOpts);
|
||||
|
||||
// IDs of powders already assigned to this job's coats — shown at top of log-material dropdown
|
||||
var jobPowderIds = (jobDto.Items ?? new List<PowderCoating.Application.DTOs.Job.JobItemDto>())
|
||||
.SelectMany(i => i.Coats ?? new List<PowderCoating.Application.DTOs.Job.JobItemCoatDto>())
|
||||
.Where(c => c.InventoryItemId.HasValue)
|
||||
.Select(c => c.InventoryItemId!.Value)
|
||||
.Distinct()
|
||||
.ToList();
|
||||
ViewBag.JobPowderIds = System.Text.Json.JsonSerializer.Serialize(jobPowderIds, jsonOpts);
|
||||
|
||||
// Pre-logged powder grouped by InventoryItemId (for Complete Job modal pre-fill)
|
||||
ViewBag.PreLoggedPowder = allJobTransactions
|
||||
@@ -518,7 +529,7 @@ public class JobsController : Controller
|
||||
ViewBag.JobPhotoMax = photoMax;
|
||||
|
||||
// Customer list for inline customer-change dropdown
|
||||
var allCustomers = await _unitOfWork.Customers.GetAllAsync();
|
||||
var allCustomers = await _unitOfWork.Customers.FindAsync(c => c.CompanyId == job.CompanyId);
|
||||
ViewBag.CustomerSelectList = allCustomers
|
||||
.Where(c => c.IsActive)
|
||||
.Select(c => new SelectListItem
|
||||
@@ -624,7 +635,8 @@ public class JobsController : Controller
|
||||
|
||||
if (job == null) return NotFound();
|
||||
|
||||
var allStatuses = (await _unitOfWork.JobStatusLookups.GetAllAsync())
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var allStatuses = (await _unitOfWork.JobStatusLookups.FindAsync(s => s.CompanyId == companyId))
|
||||
.OrderBy(s => s.DisplayOrder).ToList();
|
||||
|
||||
ViewBag.AllStatuses = allStatuses;
|
||||
@@ -647,7 +659,7 @@ public class JobsController : Controller
|
||||
|
||||
if (job == null) return NotFound();
|
||||
|
||||
var allStatuses = (await _unitOfWork.JobStatusLookups.GetAllAsync()).ToList();
|
||||
var allStatuses = (await _unitOfWork.JobStatusLookups.FindAsync(s => s.CompanyId == job.CompanyId)).ToList();
|
||||
var newStatus = allStatuses.FirstOrDefault(s => s.Id == newStatusId);
|
||||
if (newStatus == null) return BadRequest("Invalid status.");
|
||||
|
||||
@@ -835,7 +847,7 @@ public class JobsController : Controller
|
||||
// Optionally advance status to In Preparation
|
||||
if (advanceToInPreparation && jobToUpdate.JobStatus.StatusCode != AppConstants.StatusCodes.Job.InPreparation)
|
||||
{
|
||||
var allStatuses = await _unitOfWork.JobStatusLookups.GetAllAsync();
|
||||
var allStatuses = await _unitOfWork.JobStatusLookups.FindAsync(s => s.CompanyId == jobToUpdate.CompanyId);
|
||||
var inPrepStatus = allStatuses.FirstOrDefault(s => s.StatusCode == AppConstants.StatusCodes.Job.InPreparation);
|
||||
if (inPrepStatus != null)
|
||||
{
|
||||
@@ -892,7 +904,7 @@ public class JobsController : Controller
|
||||
|
||||
if (advanceToInPreparation && job.JobStatus.StatusCode != AppConstants.StatusCodes.Job.InPreparation && !job.JobStatus.IsTerminalStatus)
|
||||
{
|
||||
var allStatuses = await _unitOfWork.JobStatusLookups.GetAllAsync();
|
||||
var allStatuses = await _unitOfWork.JobStatusLookups.FindAsync(s => s.CompanyId == job.CompanyId);
|
||||
var inPrepStatus = allStatuses.FirstOrDefault(s => s.StatusCode == AppConstants.StatusCodes.Job.InPreparation);
|
||||
if (inPrepStatus != null)
|
||||
{
|
||||
@@ -1067,6 +1079,8 @@ public class JobsController : Controller
|
||||
QuoteId = dto.QuoteId,
|
||||
AssignedUserId = dto.AssignedUserId,
|
||||
OvenCostId = dto.OvenCostId,
|
||||
OvenBatches = dto.OvenBatches > 0 ? dto.OvenBatches : 1,
|
||||
OvenCycleMinutes = dto.OvenCycleMinutes,
|
||||
Description = dto.Description,
|
||||
JobPriorityId = dto.JobPriorityId,
|
||||
JobStatusId = pendingStatus?.Id ?? 1,
|
||||
@@ -1138,7 +1152,7 @@ public class JobsController : Controller
|
||||
var totals = await _pricingService.CalculateQuoteTotalsAsync(
|
||||
dto.JobItems, companyId, dto.CustomerId,
|
||||
await GetEffectiveTaxPercentAsync(dto.CustomerId, createCosts?.TaxPercent ?? 0m),
|
||||
dto.DiscountType, dto.DiscountValue, dto.IsRushJob, createOvenRate, job.OvenBatches, job.OvenCycleMinutes);
|
||||
dto.DiscountType, dto.DiscountValue, dto.IsRushJob, createOvenRate, dto.OvenBatches > 0 ? dto.OvenBatches : 1, dto.OvenCycleMinutes);
|
||||
|
||||
job.FinalPrice = totals.Total;
|
||||
job.OvenBatchCost = totals.OvenBatchCost;
|
||||
@@ -1206,6 +1220,9 @@ public class JobsController : Controller
|
||||
CustomerId = job.CustomerId,
|
||||
QuoteId = job.QuoteId,
|
||||
AssignedUserId = job.AssignedUserId,
|
||||
OvenCostId = job.OvenCostId,
|
||||
OvenBatches = job.OvenBatches > 0 ? job.OvenBatches : 1,
|
||||
OvenCycleMinutes = job.OvenCycleMinutes,
|
||||
Description = job.Description,
|
||||
JobStatusId = job.JobStatusId,
|
||||
JobPriorityId = job.JobPriorityId,
|
||||
@@ -1250,6 +1267,7 @@ public class JobsController : Controller
|
||||
TransferEfficiency = c.TransferEfficiency,
|
||||
PowderCostPerLb = c.PowderCostPerLb,
|
||||
PowderToOrder = c.PowderToOrder,
|
||||
NoExtraLayerCharge = c.NoExtraLayerCharge,
|
||||
Notes = c.Notes
|
||||
}).ToList(),
|
||||
PrepServices = ji.PrepServices.Select(ps => new CreateQuoteItemPrepServiceDto
|
||||
@@ -1380,6 +1398,9 @@ public class JobsController : Controller
|
||||
job.CustomerId = dto.CustomerId;
|
||||
job.QuoteId = dto.QuoteId;
|
||||
job.Description = dto.Description;
|
||||
job.OvenCostId = dto.OvenCostId;
|
||||
job.OvenBatches = dto.OvenBatches > 0 ? dto.OvenBatches : 1;
|
||||
job.OvenCycleMinutes = dto.OvenCycleMinutes;
|
||||
await RecordStatusChangeAsync(job, dto.JobStatusId);
|
||||
job.JobStatusId = dto.JobStatusId;
|
||||
job.JobPriorityId = dto.JobPriorityId;
|
||||
@@ -1606,7 +1627,7 @@ public class JobsController : Controller
|
||||
var totals = await _pricingService.CalculateQuoteTotalsAsync(
|
||||
dto.JobItems, companyId, dto.CustomerId,
|
||||
await GetEffectiveTaxPercentAsync(dto.CustomerId, editCosts?.TaxPercent ?? 0m),
|
||||
dto.DiscountType, dto.DiscountValue, dto.IsRushJob, editOvenRate, job.OvenBatches, job.OvenCycleMinutes);
|
||||
dto.DiscountType, dto.DiscountValue, dto.IsRushJob, editOvenRate, dto.OvenBatches > 0 ? dto.OvenBatches : 1, dto.OvenCycleMinutes);
|
||||
job.FinalPrice = totals.Total;
|
||||
job.OvenBatchCost = totals.OvenBatchCost;
|
||||
job.ShopSuppliesAmount = totals.ShopSuppliesAmount;
|
||||
@@ -1799,7 +1820,7 @@ public class JobsController : Controller
|
||||
ViewBag.AiPhotoQuotesEnabled = await _subscriptionService.CanUseAiPhotoQuoteAsync(companyId);
|
||||
|
||||
await PopulateDropdowns();
|
||||
await PopulatePrepServicesAsync();
|
||||
await PopulatePrepServicesAsync(companyId);
|
||||
var costs = await _pricingService.GetOperatingCostsAsync(companyId);
|
||||
await PopulateJobItemDropDownsAsync(companyId, costs?.OvenOperatingCostPerHour ?? 45m);
|
||||
ViewBag.TaxPercent = costs?.TaxPercent ?? 0m;
|
||||
@@ -1807,6 +1828,7 @@ public class JobsController : Controller
|
||||
ViewBag.ComplexityModeratePercent = costs?.ComplexityModeratePercent ?? 5m;
|
||||
ViewBag.ComplexityComplexPercent = costs?.ComplexityComplexPercent ?? 15m;
|
||||
ViewBag.ComplexityExtremePercent = costs?.ComplexityExtremePercent ?? 25m;
|
||||
ViewBag.DefaultOvenCycleMinutes = costs?.DefaultOvenCycleMinutes ?? 45;
|
||||
var useMetric = await _tenantContext.UseMetricSystemAsync();
|
||||
ViewBag.UseMetric = useMetric;
|
||||
ViewBag.AreaUnit = _measurementService.GetAreaUnitLabel(useMetric);
|
||||
@@ -1819,7 +1841,9 @@ public class JobsController : Controller
|
||||
/// </summary>
|
||||
private async Task PopulateDropdowns()
|
||||
{
|
||||
var customers = await _unitOfWork.Customers.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
|
||||
var customers = await _unitOfWork.Customers.FindAsync(c => c.CompanyId == companyId);
|
||||
ViewBag.Customers = new SelectList(
|
||||
customers.Where(c => c.IsActive).Select(c => new
|
||||
{
|
||||
@@ -1830,8 +1854,6 @@ public class JobsController : Controller
|
||||
}).OrderBy(c => c.DisplayName),
|
||||
"Id",
|
||||
"DisplayName");
|
||||
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var users = await _userManager.Users
|
||||
.Where(u => u.CompanyId == companyId && u.IsActive && u.CompanyRole != null)
|
||||
.OrderBy(u => u.FirstName).ThenBy(u => u.LastName)
|
||||
@@ -2213,13 +2235,13 @@ public class JobsController : Controller
|
||||
/// Loads all active prep services into ViewBag for the item wizard's prep services step.
|
||||
/// Prep services are ordered by DisplayOrder so they appear in the intended workflow sequence.
|
||||
/// </summary>
|
||||
private async Task PopulatePrepServicesAsync()
|
||||
private async Task PopulatePrepServicesAsync(int companyId)
|
||||
{
|
||||
var prepServices = await _unitOfWork.PrepServices.FindAsync(ps => ps.IsActive);
|
||||
var prepServices = await _unitOfWork.PrepServices.FindAsync(ps => ps.IsActive && ps.CompanyId == companyId);
|
||||
ViewBag.PrepServices = prepServices.OrderBy(ps => ps.DisplayOrder).ToList();
|
||||
_logger.LogInformation("Populated {Count} active prep services", prepServices.Count());
|
||||
|
||||
var blastSetups = await _unitOfWork.BlastSetups.FindAsync(b => b.IsActive);
|
||||
var blastSetups = await _unitOfWork.BlastSetups.FindAsync(b => b.IsActive && b.CompanyId == companyId);
|
||||
ViewBag.BlastSetups = blastSetups.OrderBy(b => b.DisplayOrder)
|
||||
.Select(b => new { id = b.Id, name = b.Name, derivedRate = ShopCapabilityCalculator.GetBlastRateSqFtPerHour(b), isDefault = b.IsDefault })
|
||||
.ToList();
|
||||
@@ -2925,6 +2947,7 @@ public class JobsController : Controller
|
||||
TransferEfficiency = c.TransferEfficiency,
|
||||
PowderCostPerLb = c.PowderCostPerLb,
|
||||
PowderToOrder = c.PowderToOrder,
|
||||
NoExtraLayerCharge = c.NoExtraLayerCharge,
|
||||
Notes = c.Notes
|
||||
}).ToList(),
|
||||
PrepServices = ji.PrepServices.Select(ps => new CreateQuoteItemPrepServiceDto
|
||||
@@ -3105,7 +3128,8 @@ public class JobsController : Controller
|
||||
InventoryItemId = c.InventoryItemId,
|
||||
CoverageSqFtPerLb = c.CoverageSqFtPerLb,
|
||||
TransferEfficiency = c.TransferEfficiency,
|
||||
PowderCostPerLb = c.PowderCostPerLb
|
||||
PowderCostPerLb = c.PowderCostPerLb,
|
||||
NoExtraLayerCharge = c.NoExtraLayerCharge
|
||||
}).ToList()
|
||||
}).ToList();
|
||||
|
||||
@@ -3156,7 +3180,7 @@ public class JobsController : Controller
|
||||
/// </summary>
|
||||
private async Task PopulateJobItemDropDownsAsync(int companyId, decimal fallbackOvenRate)
|
||||
{
|
||||
var inventory = await _unitOfWork.InventoryItems.GetAllAsync(false, i => i.InventoryCategory);
|
||||
var inventory = await _unitOfWork.InventoryItems.FindAsync(i => i.CompanyId == companyId, false, i => i.InventoryCategory);
|
||||
ViewBag.InventoryCoatings = inventory
|
||||
.Where(i => i.IsActive && i.InventoryCategory?.IsActive == true && i.InventoryCategory.IsCoating)
|
||||
.OrderBy(i => i.IsIncoming ? 1 : 0).ThenBy(i => i.InventoryCategory!.DisplayOrder).ThenBy(i => i.ColorName ?? i.Name)
|
||||
@@ -3176,12 +3200,12 @@ public class JobsController : Controller
|
||||
isIncoming = i.IsIncoming
|
||||
}).ToList();
|
||||
|
||||
var vendors = await _unitOfWork.Vendors.GetAllAsync(false);
|
||||
var vendors = await _unitOfWork.Vendors.FindAsync(s => s.CompanyId == companyId, false);
|
||||
ViewBag.Vendors = vendors
|
||||
.Where(s => s.IsActive).OrderBy(s => s.CompanyName)
|
||||
.Select(s => new { value = s.Id.ToString(), text = s.CompanyName }).ToList();
|
||||
|
||||
var catalogItems = await _unitOfWork.CatalogItems.GetAllAsync(false, i => i.Category, i => i.Category.ParentCategory);
|
||||
var catalogItems = await _unitOfWork.CatalogItems.FindAsync(i => i.CompanyId == companyId, false, i => i.Category, i => i.Category.ParentCategory);
|
||||
ViewBag.CatalogItems = catalogItems
|
||||
.Where(i => i.IsActive)
|
||||
.OrderBy(i => i.Category.DisplayOrder).ThenBy(i => i.DisplayOrder)
|
||||
@@ -3210,10 +3234,10 @@ public class JobsController : Controller
|
||||
description = i.Description
|
||||
}).ToList();
|
||||
|
||||
var prepServices = await _unitOfWork.PrepServices.FindAsync(ps => ps.IsActive);
|
||||
var prepServices = await _unitOfWork.PrepServices.FindAsync(ps => ps.IsActive && ps.CompanyId == companyId);
|
||||
ViewBag.PrepServices = prepServices.OrderBy(ps => ps.DisplayOrder).ToList();
|
||||
|
||||
var blastSetupsForEditItems = await _unitOfWork.BlastSetups.FindAsync(b => b.IsActive);
|
||||
var blastSetupsForEditItems = await _unitOfWork.BlastSetups.FindAsync(b => b.IsActive && b.CompanyId == companyId);
|
||||
ViewBag.BlastSetups = blastSetupsForEditItems.OrderBy(b => b.DisplayOrder)
|
||||
.Select(b => new { id = b.Id, name = b.Name, derivedRate = ShopCapabilityCalculator.GetBlastRateSqFtPerHour(b), isDefault = b.IsDefault })
|
||||
.ToList();
|
||||
|
||||
@@ -90,8 +90,8 @@ public class JobsPriorityController : Controller
|
||||
.ToList();
|
||||
|
||||
// Get priorities and workers for modal options
|
||||
var priorities = await _unitOfWork.JobPriorityLookups.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var priorities = await _unitOfWork.JobPriorityLookups.FindAsync(p => p.CompanyId == companyId);
|
||||
var workers = await _userManager.Users
|
||||
.Where(u => u.CompanyId == companyId && u.IsActive && u.CompanyRole != null)
|
||||
.OrderBy(u => u.FirstName).ThenBy(u => u.LastName)
|
||||
|
||||
@@ -16,15 +16,18 @@ public class MaintenanceController : Controller
|
||||
{
|
||||
private readonly IUnitOfWork _unitOfWork;
|
||||
private readonly IMapper _mapper;
|
||||
private readonly ITenantContext _tenantContext;
|
||||
private readonly ILogger<MaintenanceController> _logger;
|
||||
|
||||
public MaintenanceController(
|
||||
IUnitOfWork unitOfWork,
|
||||
IMapper mapper,
|
||||
ITenantContext tenantContext,
|
||||
ILogger<MaintenanceController> logger)
|
||||
{
|
||||
_unitOfWork = unitOfWork;
|
||||
_mapper = mapper;
|
||||
_tenantContext = tenantContext;
|
||||
_logger = logger;
|
||||
}
|
||||
|
||||
@@ -740,7 +743,8 @@ public class MaintenanceController : Controller
|
||||
/// </summary>
|
||||
private async Task PopulateViewBagAsync(int? selectedEquipmentId = null)
|
||||
{
|
||||
var equipment = await _unitOfWork.Equipment.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var equipment = await _unitOfWork.Equipment.FindAsync(e => e.CompanyId == companyId);
|
||||
ViewBag.EquipmentList = new SelectList(
|
||||
equipment.Where(e => e.IsActive).OrderBy(e => e.EquipmentName),
|
||||
"Id",
|
||||
|
||||
@@ -179,8 +179,9 @@ public class OvenSchedulerController : Controller
|
||||
public async Task<IActionResult> Suggest([FromBody] SuggestRequest req)
|
||||
{
|
||||
var goal = req?.OptimizationGoal ?? "maximize_throughput";
|
||||
var suggestCompanyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
|
||||
var equipmentList = (await _unitOfWork.OvenCosts.GetAllAsync())
|
||||
var equipmentList = (await _unitOfWork.OvenCosts.FindAsync(o => o.CompanyId == suggestCompanyId))
|
||||
.Where(o => o.IsActive)
|
||||
.OrderBy(o => o.DisplayOrder).ThenBy(o => o.Label)
|
||||
.ToList();
|
||||
@@ -188,10 +189,11 @@ public class OvenSchedulerController : Controller
|
||||
if (!equipmentList.Any())
|
||||
return Json(new { success = false, error = "No active ovens found. Add Named Ovens in Settings → Operating Costs." });
|
||||
|
||||
var companyCosts = await _unitOfWork.CompanyOperatingCosts.GetAllAsync();
|
||||
var companyCosts = await _unitOfWork.CompanyOperatingCosts.FindAsync(c => c.CompanyId == suggestCompanyId);
|
||||
var defaultCycleMinutes = companyCosts.FirstOrDefault()?.DefaultOvenCycleMinutes ?? 45;
|
||||
|
||||
var queueJobs = (await _unitOfWork.Jobs.GetAllAsync(
|
||||
var queueJobs = (await _unitOfWork.Jobs.FindAsync(
|
||||
j => j.CompanyId == suggestCompanyId,
|
||||
false,
|
||||
j => j.Customer,
|
||||
j => j.JobStatus,
|
||||
@@ -265,7 +267,8 @@ public class OvenSchedulerController : Controller
|
||||
if (req?.Batches == null || !req.Batches.Any())
|
||||
return Json(new { success = false, error = "No batches provided." });
|
||||
|
||||
var companyCosts = await _unitOfWork.CompanyOperatingCosts.GetAllAsync();
|
||||
var acceptCompanyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var companyCosts = await _unitOfWork.CompanyOperatingCosts.FindAsync(c => c.CompanyId == acceptCompanyId);
|
||||
var defaultCycleMinutes = companyCosts.FirstOrDefault()?.DefaultOvenCycleMinutes ?? 45;
|
||||
|
||||
var createdBatches = new List<object>();
|
||||
@@ -357,7 +360,8 @@ public class OvenSchedulerController : Controller
|
||||
if (oven == null)
|
||||
return Json(new { success = false, error = "Oven not found." });
|
||||
|
||||
var companyCosts = await _unitOfWork.CompanyOperatingCosts.GetAllAsync();
|
||||
var createBatchCompanyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var companyCosts = await _unitOfWork.CompanyOperatingCosts.FindAsync(c => c.CompanyId == createBatchCompanyId);
|
||||
var defaultCycleMinutes = companyCosts.FirstOrDefault()?.DefaultOvenCycleMinutes ?? 45;
|
||||
|
||||
var batchNumber = await GenerateBatchNumberAsync();
|
||||
@@ -651,7 +655,8 @@ public class OvenSchedulerController : Controller
|
||||
if (inOvenStatus != null)
|
||||
{
|
||||
var jobIds = batch.Items.Select(i => i.JobId).Distinct().ToHashSet();
|
||||
var jobs = (await _unitOfWork.Jobs.GetAllAsync()).Where(j => jobIds.Contains(j.Id));
|
||||
var startBatchCid = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var jobs = await _unitOfWork.Jobs.FindAsync(j => j.CompanyId == startBatchCid && jobIds.Contains(j.Id));
|
||||
foreach (var job in jobs)
|
||||
job.JobStatusId = inOvenStatus.Id;
|
||||
}
|
||||
|
||||
@@ -14,12 +14,14 @@ public class PricingTiersController : Controller
|
||||
private readonly IUnitOfWork _unitOfWork;
|
||||
private readonly IMapper _mapper;
|
||||
private readonly ILogger<PricingTiersController> _logger;
|
||||
private readonly ITenantContext _tenantContext;
|
||||
|
||||
public PricingTiersController(IUnitOfWork unitOfWork, IMapper mapper, ILogger<PricingTiersController> logger)
|
||||
public PricingTiersController(IUnitOfWork unitOfWork, IMapper mapper, ILogger<PricingTiersController> logger, ITenantContext tenantContext)
|
||||
{
|
||||
_unitOfWork = unitOfWork;
|
||||
_mapper = mapper;
|
||||
_logger = logger;
|
||||
_tenantContext = tenantContext;
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
@@ -27,8 +29,9 @@ public class PricingTiersController : Controller
|
||||
/// </summary>
|
||||
public async Task<IActionResult> Index()
|
||||
{
|
||||
var tiers = await _unitOfWork.PricingTiers.GetAllAsync();
|
||||
var customers = await _unitOfWork.Customers.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var tiers = await _unitOfWork.PricingTiers.FindAsync(t => t.CompanyId == companyId);
|
||||
var customers = await _unitOfWork.Customers.FindAsync(c => c.CompanyId == companyId);
|
||||
|
||||
var customerCountByTier = customers
|
||||
.Where(c => c.PricingTierId.HasValue)
|
||||
|
||||
@@ -255,7 +255,7 @@ public class QuotesController : Controller
|
||||
|
||||
// Calibration nudge — suppress when named blast setups exist OR legacy CFM is set
|
||||
var costs = (await _unitOfWork.CompanyOperatingCosts.FindAsync(c => c.CompanyId == companyId)).FirstOrDefault();
|
||||
var hasNamedSetups = (await _unitOfWork.BlastSetups.FindAsync(b => b.IsActive)).Any();
|
||||
var hasNamedSetups = (await _unitOfWork.BlastSetups.FindAsync(b => b.IsActive && b.CompanyId == companyId)).Any();
|
||||
ViewBag.QuotingNotCalibrated = costs != null
|
||||
&& !hasNamedSetups
|
||||
&& costs.CompressorCfm == 0
|
||||
@@ -441,7 +441,7 @@ public class QuotesController : Controller
|
||||
ViewBag.Deposits = quoteDeposits;
|
||||
|
||||
// Customer list for inline customer-change dropdown
|
||||
var allCustomers = await _unitOfWork.Customers.GetAllAsync();
|
||||
var allCustomers = await _unitOfWork.Customers.FindAsync(c => c.CompanyId == quote.CompanyId);
|
||||
ViewBag.CustomerSelectList = allCustomers
|
||||
.Where(c => c.IsActive)
|
||||
.Select(c => new SelectListItem
|
||||
@@ -2430,7 +2430,7 @@ public class QuotesController : Controller
|
||||
ViewBag.QuotePhotosEnabled = quotePhotoMax != 0; // 0 = feature disabled for this plan
|
||||
|
||||
// Customers
|
||||
var customers = await _unitOfWork.Customers.GetAllAsync();
|
||||
var customers = await _unitOfWork.Customers.FindAsync(c => c.CompanyId == companyId);
|
||||
ViewBag.Customers = customers
|
||||
.Select(c => new SelectListItem
|
||||
{
|
||||
@@ -2471,7 +2471,7 @@ public class QuotesController : Controller
|
||||
}
|
||||
|
||||
// Inventory coatings — include incoming items so they can be quoted while powder is in transit
|
||||
var inventory = await _unitOfWork.InventoryItems.GetAllAsync(false, i => i.InventoryCategory);
|
||||
var inventory = await _unitOfWork.InventoryItems.FindAsync(i => i.CompanyId == companyId, false, i => i.InventoryCategory);
|
||||
ViewBag.InventoryCoatings = inventory
|
||||
.Where(i => i.IsActive && i.InventoryCategory?.IsActive == true && i.InventoryCategory.IsCoating)
|
||||
.OrderBy(i => i.IsIncoming ? 1 : 0).ThenBy(i => i.InventoryCategory!.DisplayOrder).ThenBy(i => i.ColorName ?? i.Name)
|
||||
@@ -2492,13 +2492,13 @@ public class QuotesController : Controller
|
||||
}).ToList();
|
||||
|
||||
// Vendors
|
||||
var vendors = await _unitOfWork.Vendors.GetAllAsync(false);
|
||||
var vendors = await _unitOfWork.Vendors.FindAsync(s => s.CompanyId == companyId, false);
|
||||
ViewBag.Vendors = vendors
|
||||
.Where(s => s.IsActive).OrderBy(s => s.CompanyName)
|
||||
.Select(s => new { value = s.Id.ToString(), text = s.CompanyName }).ToList();
|
||||
|
||||
// Catalog items
|
||||
var catalogItems = await _unitOfWork.CatalogItems.GetAllAsync(false, i => i.Category, i => i.Category.ParentCategory);
|
||||
var catalogItems = await _unitOfWork.CatalogItems.FindAsync(i => i.CompanyId == companyId, false, i => i.Category, i => i.Category.ParentCategory);
|
||||
ViewBag.CatalogItems = catalogItems
|
||||
.Where(i => i.IsActive)
|
||||
.OrderBy(i => i.Category.DisplayOrder).ThenBy(i => i.DisplayOrder)
|
||||
@@ -2528,11 +2528,11 @@ public class QuotesController : Controller
|
||||
}).ToList();
|
||||
|
||||
// Prep services
|
||||
var prepServices = await _unitOfWork.PrepServices.FindAsync(ps => ps.IsActive);
|
||||
var prepServices = await _unitOfWork.PrepServices.FindAsync(ps => ps.IsActive && ps.CompanyId == companyId);
|
||||
ViewBag.PrepServices = prepServices.OrderBy(ps => ps.DisplayOrder).ToList();
|
||||
|
||||
// Blast setups for wizard dropdown
|
||||
var blastSetups = await _unitOfWork.BlastSetups.FindAsync(b => b.IsActive);
|
||||
var blastSetups = await _unitOfWork.BlastSetups.FindAsync(b => b.IsActive && b.CompanyId == companyId);
|
||||
ViewBag.BlastSetups = blastSetups.OrderBy(b => b.DisplayOrder)
|
||||
.Select(b => new { id = b.Id, name = b.Name, derivedRate = ShopCapabilityCalculator.GetBlastRateSqFtPerHour(b), isDefault = b.IsDefault })
|
||||
.ToList();
|
||||
@@ -2599,7 +2599,8 @@ public class QuotesController : Controller
|
||||
/// </summary>
|
||||
private async Task PopulatePricingTiersDropDownAsync()
|
||||
{
|
||||
var pricingTiers = await _unitOfWork.PricingTiers.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var pricingTiers = await _unitOfWork.PricingTiers.FindAsync(pt => pt.CompanyId == companyId);
|
||||
ViewBag.PricingTiers = pricingTiers.OrderBy(pt => pt.TierName)
|
||||
.Select(pt => new SelectListItem
|
||||
{
|
||||
@@ -2825,9 +2826,9 @@ public class QuotesController : Controller
|
||||
// Do NOT assign fullItems to quote.QuoteItems — quote is a tracked entity and assigning
|
||||
// no-tracking children (which may share InventoryItem instances) causes EF identity conflicts.
|
||||
|
||||
// Get default job statuses and priorities
|
||||
var jobStatuses = await _unitOfWork.JobStatusLookups.GetAllAsync();
|
||||
var jobPriorities = await _unitOfWork.JobPriorityLookups.GetAllAsync();
|
||||
// Get default job statuses and priorities — scope to quote's company for defense-in-depth
|
||||
var jobStatuses = await _unitOfWork.JobStatusLookups.FindAsync(s => s.CompanyId == quote.CompanyId);
|
||||
var jobPriorities = await _unitOfWork.JobPriorityLookups.FindAsync(p => p.CompanyId == quote.CompanyId);
|
||||
var approvedStatus = jobStatuses.FirstOrDefault(s => s.StatusCode == AppConstants.StatusCodes.Job.Approved);
|
||||
var normalPriority = jobPriorities.FirstOrDefault(p => p.PriorityCode == "NORMAL");
|
||||
var rushPriority = jobPriorities.FirstOrDefault(p => p.PriorityCode == "RUSH");
|
||||
@@ -3347,7 +3348,7 @@ public class QuotesController : Controller
|
||||
CompanyBlastSetup? selectedBlastSetup = null;
|
||||
if (request.BlastSetupId.HasValue)
|
||||
{
|
||||
var setups = await _unitOfWork.BlastSetups.FindAsync(b => b.Id == request.BlastSetupId.Value && b.IsActive);
|
||||
var setups = await _unitOfWork.BlastSetups.FindAsync(b => b.Id == request.BlastSetupId.Value && b.IsActive && b.CompanyId == companyId);
|
||||
selectedBlastSetup = setups.FirstOrDefault();
|
||||
}
|
||||
|
||||
|
||||
@@ -44,7 +44,8 @@ public class RecurringTemplatesController : Controller
|
||||
/// <summary>Lists all recurring templates for the current company, active first then by name.</summary>
|
||||
public async Task<IActionResult> Index()
|
||||
{
|
||||
var templates = await _unitOfWork.RecurringTemplates.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var templates = await _unitOfWork.RecurringTemplates.FindAsync(t => t.CompanyId == companyId);
|
||||
return View(templates.OrderByDescending(t => t.IsActive).ThenBy(t => t.Name).ToList());
|
||||
}
|
||||
|
||||
@@ -425,11 +426,12 @@ public class RecurringTemplatesController : Controller
|
||||
/// <summary>Loads dropdowns for vendors, accounts, and payment methods into ViewBag.</summary>
|
||||
private async Task PopulateDropDownsAsync()
|
||||
{
|
||||
var vendors = await _unitOfWork.Vendors.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var vendors = await _unitOfWork.Vendors.FindAsync(v => v.CompanyId == companyId);
|
||||
ViewBag.Vendors = vendors.OrderBy(v => v.CompanyName)
|
||||
.Select(v => new SelectListItem(v.CompanyName, v.Id.ToString())).ToList();
|
||||
|
||||
var accounts = await _unitOfWork.Accounts.GetAllAsync();
|
||||
var accounts = await _unitOfWork.Accounts.FindAsync(a => a.CompanyId == companyId);
|
||||
ViewBag.APAccounts = accounts
|
||||
.Where(a => a.AccountSubType == AccountSubType.AccountsPayable)
|
||||
.OrderBy(a => a.AccountNumber)
|
||||
|
||||
@@ -11,6 +11,7 @@ using PowderCoating.Core.Interfaces;
|
||||
using PowderCoating.Core.Entities;
|
||||
using PowderCoating.Shared.Constants;
|
||||
using PowderCoating.Web.ViewModels.Reports;
|
||||
using System.Security.Claims;
|
||||
|
||||
namespace PowderCoating.Web.Controllers;
|
||||
|
||||
@@ -25,8 +26,9 @@ public class ReportsController : Controller
|
||||
private readonly UserManager<ApplicationUser> _userManager;
|
||||
private readonly IAccountingAiService _accountingAi;
|
||||
private readonly IAiUsageLogger _usageLogger;
|
||||
private readonly ITenantContext _tenantContext;
|
||||
|
||||
public ReportsController(IUnitOfWork unitOfWork, ILogger<ReportsController> logger, IFinancialReportService financialReports, IOperationalReportService operationalReports, IPdfService pdfService, UserManager<ApplicationUser> userManager, IAccountingAiService accountingAi, IAiUsageLogger usageLogger)
|
||||
public ReportsController(IUnitOfWork unitOfWork, ILogger<ReportsController> logger, IFinancialReportService financialReports, IOperationalReportService operationalReports, IPdfService pdfService, UserManager<ApplicationUser> userManager, IAccountingAiService accountingAi, IAiUsageLogger usageLogger, ITenantContext tenantContext)
|
||||
{
|
||||
_unitOfWork = unitOfWork;
|
||||
_logger = logger;
|
||||
@@ -36,6 +38,7 @@ public class ReportsController : Controller
|
||||
_userManager = userManager;
|
||||
_accountingAi = accountingAi;
|
||||
_usageLogger = usageLogger;
|
||||
_tenantContext = tenantContext;
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
@@ -79,27 +82,26 @@ public class ReportsController : Controller
|
||||
var completedStatusCodes = new[] { "COMPLETED", "READY_FOR_PICKUP", "DELIVERED" };
|
||||
var activeStatusCodes = new[] { "PENDING", "QUOTED", "APPROVED", "IN_PREPARATION", "SANDBLASTING",
|
||||
"MASKING_TAPING", "CLEANING", "IN_OVEN", "COATING", "CURING", "QUALITY_CHECK", "ON_HOLD" };
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
|
||||
// Load only necessary data - optimized with filtering and minimal eager loading
|
||||
// Jobs: Load all jobs (we need various status filters and the collection is needed for job status distribution)
|
||||
// Note: Date filtering would exclude data needed for jobsByStatus calculation
|
||||
var jobs = (await _unitOfWork.Jobs.GetAllAsync(false, j => j.Customer, j => j.JobStatus, j => j.JobPriority, j => j.AssignedUser)).ToList();
|
||||
// Load only necessary data — all explicitly scoped to this company
|
||||
var jobs = (await _unitOfWork.Jobs.FindAsync(j => j.CompanyId == companyId, false, j => j.Customer, j => j.JobStatus, j => j.JobPriority, j => j.AssignedUser)).ToList();
|
||||
|
||||
// Quotes: Load all quotes (needed for quote status distribution and conversion funnel)
|
||||
var quotes = (await _unitOfWork.Quotes.GetAllAsync(false, q => q.Customer, q => q.QuoteStatus)).ToList();
|
||||
var quotes = (await _unitOfWork.Quotes.FindAsync(q => q.CompanyId == companyId, false, q => q.Customer, q => q.QuoteStatus)).ToList();
|
||||
|
||||
// Customers: Load all (needed for active count and customer creation trend across all months)
|
||||
var customers = (await _unitOfWork.Customers.GetAllAsync()).ToList();
|
||||
var customers = (await _unitOfWork.Customers.FindAsync(c => c.CompanyId == companyId)).ToList();
|
||||
|
||||
// Equipment: Load all for status distribution
|
||||
var equipment = (await _unitOfWork.Equipment.GetAllAsync()).ToList();
|
||||
var equipment = (await _unitOfWork.Equipment.FindAsync(e => e.CompanyId == companyId)).ToList();
|
||||
|
||||
// Inventory: Load all for low stock analysis
|
||||
var inventory = (await _unitOfWork.InventoryItems.GetAllAsync()).ToList();
|
||||
var inventory = (await _unitOfWork.InventoryItems.FindAsync(i => i.CompanyId == companyId)).ToList();
|
||||
|
||||
// Appointments: Filter to relevant date range at DB level
|
||||
var appointments = (await _unitOfWork.Appointments.FindAsync(
|
||||
a => a.ScheduledStartTime >= startDate,
|
||||
a => a.CompanyId == companyId && a.ScheduledStartTime >= startDate,
|
||||
false,
|
||||
a => a.Customer,
|
||||
a => a.AppointmentType,
|
||||
@@ -108,7 +110,7 @@ public class ReportsController : Controller
|
||||
// Users with assigned jobs/appointments will be loaded below when building worker stats
|
||||
|
||||
// CatalogItems: Load all for category distribution
|
||||
var catalogItems = (await _unitOfWork.CatalogItems.GetAllAsync(false, c => c.Category)).ToList();
|
||||
var catalogItems = (await _unitOfWork.CatalogItems.FindAsync(ci => ci.CompanyId == companyId, false, c => c.Category)).ToList();
|
||||
|
||||
// === OVERVIEW METRICS ===
|
||||
var completedJobs = jobs.Where(j => completedStatusCodes.Contains(j.JobStatus.StatusCode)).ToList();
|
||||
@@ -382,7 +384,7 @@ public class ReportsController : Controller
|
||||
.ToDictionary(g => g.Key, g => g.Count());
|
||||
|
||||
// === FINANCIAL ANALYTICS ===
|
||||
var allInvoices = (await _unitOfWork.Invoices.GetAllAsync(false, i => i.Customer, i => i.Payments)).ToList();
|
||||
var allInvoices = (await _unitOfWork.Invoices.FindAsync(i => i.CompanyId == companyId, false, i => i.Customer, i => i.Payments)).ToList();
|
||||
var activeInvoices = allInvoices.Where(i => i.Status != InvoiceStatus.Voided && i.Status != InvoiceStatus.WrittenOff).ToList();
|
||||
|
||||
var totalInvoiced = activeInvoices.Sum(i => i.Total);
|
||||
@@ -781,7 +783,7 @@ public class ReportsController : Controller
|
||||
|
||||
// === POWDER CONSUMPTION VS PURCHASE ===
|
||||
var allInventoryTransactions = (await _unitOfWork.InventoryTransactions
|
||||
.GetAllAsync(false, t => t.InventoryItem))
|
||||
.FindAsync(t => t.CompanyId == companyId, false, t => t.InventoryItem))
|
||||
.ToList();
|
||||
|
||||
var powderConsumptionItems = allInventoryTransactions
|
||||
@@ -1309,14 +1311,15 @@ public class ReportsController : Controller
|
||||
var completedStatusCodes = new[] { "COMPLETED", "READY_FOR_PICKUP", "DELIVERED" };
|
||||
var activeStatusCodes = new[] { "PENDING", "QUOTED", "APPROVED", "IN_PREPARATION", "SANDBLASTING",
|
||||
"MASKING_TAPING", "CLEANING", "IN_OVEN", "COATING", "CURING", "QUALITY_CHECK", "ON_HOLD" };
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
|
||||
var jobs = (await _unitOfWork.Jobs.GetAllAsync(false, j => j.Customer, j => j.JobStatus, j => j.JobPriority)).ToList();
|
||||
var customers = (await _unitOfWork.Customers.GetAllAsync()).ToList();
|
||||
var quotes = (await _unitOfWork.Quotes.GetAllAsync(false, q => q.QuoteStatus)).ToList();
|
||||
var equipment = (await _unitOfWork.Equipment.GetAllAsync()).ToList();
|
||||
var inventory = (await _unitOfWork.InventoryItems.GetAllAsync()).ToList();
|
||||
var allAppointments = await _unitOfWork.Appointments.GetAllAsync(false, a => a.AppointmentStatus);
|
||||
var appointments = allAppointments.Where(a => a.ScheduledStartTime >= startDate).ToList();
|
||||
var jobs = (await _unitOfWork.Jobs.FindAsync(j => j.CompanyId == companyId, false, j => j.Customer, j => j.JobStatus, j => j.JobPriority)).ToList();
|
||||
var customers = (await _unitOfWork.Customers.FindAsync(c => c.CompanyId == companyId)).ToList();
|
||||
var quotes = (await _unitOfWork.Quotes.FindAsync(q => q.CompanyId == companyId, false, q => q.QuoteStatus)).ToList();
|
||||
var equipment = (await _unitOfWork.Equipment.FindAsync(e => e.CompanyId == companyId)).ToList();
|
||||
var inventory = (await _unitOfWork.InventoryItems.FindAsync(i => i.CompanyId == companyId)).ToList();
|
||||
var allAppointments = await _unitOfWork.Appointments.FindAsync(a => a.CompanyId == companyId && a.ScheduledStartTime >= startDate, false, a => a.AppointmentStatus);
|
||||
var appointments = allAppointments.ToList();
|
||||
|
||||
var completedJobs = jobs.Where(j => completedStatusCodes.Contains(j.JobStatus.StatusCode)).ToList();
|
||||
var activeJobs = jobs.Where(j => activeStatusCodes.Contains(j.JobStatus.StatusCode)).ToList();
|
||||
@@ -1384,7 +1387,8 @@ public class ReportsController : Controller
|
||||
var now = DateTime.UtcNow;
|
||||
var startDate = now.AddMonths(-months);
|
||||
var completedStatusCodes = new[] { "COMPLETED", "READY_FOR_PICKUP", "DELIVERED" };
|
||||
var jobs = (await _unitOfWork.Jobs.GetAllAsync(false, j => j.Customer, j => j.JobStatus, j => j.JobPriority)).ToList();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var jobs = (await _unitOfWork.Jobs.FindAsync(j => j.CompanyId == companyId, false, j => j.Customer, j => j.JobStatus, j => j.JobPriority)).ToList();
|
||||
var completedJobs = jobs.Where(j => completedStatusCodes.Contains(j.JobStatus.StatusCode)).ToList();
|
||||
var inRange = completedJobs.Where(j => j.UpdatedAt >= startDate).ToList();
|
||||
var byMonth = inRange.GroupBy(j => new DateTime(j.UpdatedAt!.Value.Year, j.UpdatedAt.Value.Month, 1)).ToDictionary(g => g.Key, g => g.ToList());
|
||||
@@ -1430,12 +1434,13 @@ public class ReportsController : Controller
|
||||
var completedStatusCodes = new[] { "COMPLETED", "READY_FOR_PICKUP", "DELIVERED" };
|
||||
var activeStatusCodes = new[] { "PENDING", "QUOTED", "APPROVED", "IN_PREPARATION", "SANDBLASTING",
|
||||
"MASKING_TAPING", "CLEANING", "IN_OVEN", "COATING", "CURING", "QUALITY_CHECK", "ON_HOLD" };
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
|
||||
var jobs = (await _unitOfWork.Jobs.GetAllAsync(false, j => j.JobStatus, j => j.JobPriority, j => j.AssignedUser)).ToList();
|
||||
var equipment = (await _unitOfWork.Equipment.GetAllAsync()).ToList();
|
||||
var inventory = (await _unitOfWork.InventoryItems.GetAllAsync()).ToList();
|
||||
var allAppts = await _unitOfWork.Appointments.GetAllAsync(false, a => a.AppointmentType, a => a.AppointmentStatus);
|
||||
var appointments = allAppts.Where(a => a.ScheduledStartTime >= startDate).ToList();
|
||||
var jobs = (await _unitOfWork.Jobs.FindAsync(j => j.CompanyId == companyId, false, j => j.JobStatus, j => j.JobPriority, j => j.AssignedUser)).ToList();
|
||||
var equipment = (await _unitOfWork.Equipment.FindAsync(e => e.CompanyId == companyId)).ToList();
|
||||
var inventory = (await _unitOfWork.InventoryItems.FindAsync(i => i.CompanyId == companyId)).ToList();
|
||||
var allAppts = await _unitOfWork.Appointments.FindAsync(a => a.CompanyId == companyId && a.ScheduledStartTime >= startDate, false, a => a.AppointmentType, a => a.AppointmentStatus);
|
||||
var appointments = allAppts.ToList();
|
||||
|
||||
var activeJobs = jobs.Where(j => activeStatusCodes.Contains(j.JobStatus.StatusCode)).ToList();
|
||||
var completedJobs = jobs.Where(j => completedStatusCodes.Contains(j.JobStatus.StatusCode)).ToList();
|
||||
@@ -1483,10 +1488,11 @@ public class ReportsController : Controller
|
||||
var now = DateTime.UtcNow;
|
||||
var startDate = now.AddMonths(-months);
|
||||
var completedStatusCodes = new[] { "COMPLETED", "READY_FOR_PICKUP", "DELIVERED" };
|
||||
var customers = (await _unitOfWork.Customers.GetAllAsync()).ToList();
|
||||
var quotes = (await _unitOfWork.Quotes.GetAllAsync(false, q => q.QuoteStatus)).ToList();
|
||||
var catalogItems = (await _unitOfWork.CatalogItems.GetAllAsync(false, c => c.Category)).ToList();
|
||||
var completedJobs = (await _unitOfWork.Jobs.GetAllAsync(false, j => j.Customer, j => j.JobStatus, j => j.JobPriority))
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var customers = (await _unitOfWork.Customers.FindAsync(c => c.CompanyId == companyId)).ToList();
|
||||
var quotes = (await _unitOfWork.Quotes.FindAsync(q => q.CompanyId == companyId, false, q => q.QuoteStatus)).ToList();
|
||||
var catalogItems = (await _unitOfWork.CatalogItems.FindAsync(ci => ci.CompanyId == companyId, false, c => c.Category)).ToList();
|
||||
var completedJobs = (await _unitOfWork.Jobs.FindAsync(j => j.CompanyId == companyId, false, j => j.Customer, j => j.JobStatus, j => j.JobPriority))
|
||||
.Where(j => completedStatusCodes.Contains(j.JobStatus.StatusCode)).ToList();
|
||||
|
||||
var customersByMonth = customers.Where(c => c.CreatedAt >= startDate).GroupBy(c => new DateTime(c.CreatedAt.Year, c.CreatedAt.Month, 1)).ToDictionary(g => g.Key, g => g.Count());
|
||||
@@ -1523,7 +1529,8 @@ public class ReportsController : Controller
|
||||
if (!AllowAccounting()) return RedirectToAction(nameof(Landing));
|
||||
var now = DateTime.UtcNow;
|
||||
var today = DateTime.Today;
|
||||
var allInvoices = (await _unitOfWork.Invoices.GetAllAsync(false, i => i.Customer, i => i.Payments)).ToList();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var allInvoices = (await _unitOfWork.Invoices.FindAsync(i => i.CompanyId == companyId, false, i => i.Customer, i => i.Payments)).ToList();
|
||||
var activeInvoices = allInvoices.Where(i => i.Status != InvoiceStatus.Voided && i.Status != InvoiceStatus.WrittenOff).ToList();
|
||||
var outstandingInvoices = activeInvoices.Where(i => i.Status != InvoiceStatus.Paid && i.Total > i.AmountPaid).ToList();
|
||||
var overdueInvoices = activeInvoices.Where(i => i.Status != InvoiceStatus.Paid && i.DueDate.HasValue && i.DueDate.Value < today).ToList();
|
||||
@@ -1574,7 +1581,8 @@ public class ReportsController : Controller
|
||||
|
||||
var monthLabels = new List<string>(); var monthlyBillsPaid = new List<decimal>(); var monthlyDirectExpenses = new List<decimal>();
|
||||
// Also load collected payments for P&L comparison
|
||||
var allInvoices = (await _unitOfWork.Invoices.GetAllAsync(false, i => i.Payments)).ToList();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var allInvoices = (await _unitOfWork.Invoices.FindAsync(i => i.CompanyId == companyId, false, i => i.Payments)).ToList();
|
||||
var paymentsByMonth = allInvoices.SelectMany(i => i.Payments.Where(p => !p.IsDeleted)).GroupBy(p => new DateTime(p.PaymentDate.Year, p.PaymentDate.Month, 1)).ToDictionary(g => g.Key, g => g.ToList());
|
||||
var plRevenue = new List<decimal>(); var plExpenses = new List<decimal>(); var plNet = new List<decimal>();
|
||||
for (var i = months - 1; i >= 0; i--)
|
||||
@@ -1609,8 +1617,10 @@ public class ReportsController : Controller
|
||||
{
|
||||
var now = DateTime.UtcNow;
|
||||
var startDate = now.AddMonths(-months);
|
||||
var powderTransactions = (await _unitOfWork.InventoryTransactions.GetAllAsync(false, t => t.InventoryItem))
|
||||
.Where(t => t.TransactionType == InventoryTransactionType.JobUsage && t.TransactionDate >= startDate).ToList();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var powderTransactions = (await _unitOfWork.InventoryTransactions.FindAsync(
|
||||
t => t.CompanyId == companyId && t.TransactionType == InventoryTransactionType.JobUsage && t.TransactionDate >= startDate, false, t => t.InventoryItem))
|
||||
.ToList();
|
||||
|
||||
var topColors = powderTransactions.Where(t => t.InventoryItem != null).GroupBy(t => t.InventoryItemId)
|
||||
.Select(g => new PowderUsageByColorItem { InventoryItemId = g.Key, ColorName = g.First().InventoryItem!.ColorName ?? g.First().InventoryItem.Name, ColorCode = g.First().InventoryItem!.ColorCode, SKU = g.First().InventoryItem!.SKU, Manufacturer = g.First().InventoryItem!.Manufacturer, TotalLbsUsed = g.Sum(t => Math.Abs(t.Quantity)), TotalCost = g.Sum(t => Math.Abs(t.TotalCost)), JobCount = g.Where(t => !string.IsNullOrEmpty(t.Reference)).Select(t => t.Reference).Distinct().Count() })
|
||||
@@ -1631,7 +1641,8 @@ public class ReportsController : Controller
|
||||
/// <summary>Sales by Customer report — all active (non-voided) invoices grouped by customer, sorted by total invoiced.</summary>
|
||||
public async Task<IActionResult> SalesByCustomer(int months = 6)
|
||||
{
|
||||
var allInvoices = (await _unitOfWork.Invoices.GetAllAsync(false, i => i.Customer, i => i.Payments)).ToList();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var allInvoices = (await _unitOfWork.Invoices.FindAsync(i => i.CompanyId == companyId, false, i => i.Customer, i => i.Payments)).ToList();
|
||||
var activeInvoices = allInvoices.Where(i => i.Status != InvoiceStatus.Voided && i.Status != InvoiceStatus.WrittenOff).ToList();
|
||||
var items = activeInvoices.Where(i => i.Customer != null)
|
||||
.GroupBy(i => new { i.CustomerId, Name = i.Customer!.IsCommercial ? i.Customer.CompanyName : $"{i.Customer.ContactFirstName} {i.Customer.ContactLastName}".Trim(), i.Customer.IsCommercial })
|
||||
@@ -1650,8 +1661,9 @@ public class ReportsController : Controller
|
||||
{
|
||||
var now = DateTime.UtcNow;
|
||||
var completedStatusCodes = new[] { "COMPLETED", "READY_FOR_PICKUP", "DELIVERED" };
|
||||
var customers = (await _unitOfWork.Customers.GetAllAsync()).ToList();
|
||||
var completedJobs = (await _unitOfWork.Jobs.GetAllAsync(false, j => j.JobStatus)).Where(j => completedStatusCodes.Contains(j.JobStatus.StatusCode)).ToList();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var customers = (await _unitOfWork.Customers.FindAsync(c => c.CompanyId == companyId)).ToList();
|
||||
var completedJobs = (await _unitOfWork.Jobs.FindAsync(j => j.CompanyId == companyId, false, j => j.JobStatus)).Where(j => completedStatusCodes.Contains(j.JobStatus.StatusCode)).ToList();
|
||||
var items = customers.Where(c => c.IsActive).Select(c =>
|
||||
{
|
||||
var cJobs = completedJobs.Where(j => j.CustomerId == c.Id).ToList();
|
||||
@@ -1682,7 +1694,8 @@ public class ReportsController : Controller
|
||||
{
|
||||
var now = DateTime.UtcNow;
|
||||
var completedStatusCodes = new[] { "COMPLETED", "READY_FOR_PICKUP", "DELIVERED" };
|
||||
var completedJobs = (await _unitOfWork.Jobs.GetAllAsync(false, j => j.JobStatus)).Where(j => completedStatusCodes.Contains(j.JobStatus.StatusCode) && j.CompletedDate.HasValue).ToList();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var completedJobs = (await _unitOfWork.Jobs.FindAsync(j => j.CompanyId == companyId, false, j => j.JobStatus)).Where(j => completedStatusCodes.Contains(j.JobStatus.StatusCode) && j.CompletedDate.HasValue).ToList();
|
||||
var allStatusHistory = await _operationalReports.GetAllJobStatusHistoryAsync();
|
||||
var historyByJob = allStatusHistory.GroupBy(h => h.JobId).ToDictionary(g => g.Key, g => g.OrderBy(h => h.ChangedDate).ToList());
|
||||
var statusDisplayOrder = new[] { "PENDING", "QUOTED", "APPROVED", "IN_PREPARATION", "SANDBLASTING", "MASKING_TAPING", "CLEANING", "IN_OVEN", "COATING", "CURING", "QUALITY_CHECK" };
|
||||
@@ -1720,7 +1733,8 @@ public class ReportsController : Controller
|
||||
var now = DateTime.UtcNow;
|
||||
var today = DateTime.Today;
|
||||
var activeStatusCodes = new[] { "PENDING", "QUOTED", "APPROVED", "IN_PREPARATION", "SANDBLASTING", "MASKING_TAPING", "CLEANING", "IN_OVEN", "COATING", "CURING", "QUALITY_CHECK", "ON_HOLD" };
|
||||
var activeJobs = (await _unitOfWork.Jobs.GetAllAsync(false, j => j.Customer, j => j.JobStatus, j => j.JobPriority)).Where(j => activeStatusCodes.Contains(j.JobStatus.StatusCode)).ToList();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var activeJobs = (await _unitOfWork.Jobs.FindAsync(j => j.CompanyId == companyId, false, j => j.Customer, j => j.JobStatus, j => j.JobPriority)).Where(j => activeStatusCodes.Contains(j.JobStatus.StatusCode)).ToList();
|
||||
var items = activeJobs.Select(j => new JobStatusAgingItem
|
||||
{
|
||||
JobId = j.Id, JobNumber = j.JobNumber, CustomerName = j.Customer?.IsCommercial == true ? j.Customer.CompanyName ?? "Unknown" : $"{j.Customer?.ContactFirstName} {j.Customer?.ContactLastName}".Trim(),
|
||||
@@ -1740,7 +1754,8 @@ public class ReportsController : Controller
|
||||
{
|
||||
if (!AllowAccounting()) return RedirectToAction(nameof(Landing));
|
||||
var today = DateTime.Today;
|
||||
var allInvoices = (await _unitOfWork.Invoices.GetAllAsync(false, i => i.Customer, i => i.Payments)).ToList();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var allInvoices = (await _unitOfWork.Invoices.FindAsync(i => i.CompanyId == companyId, false, i => i.Customer, i => i.Payments)).ToList();
|
||||
var items = allInvoices.Where(i => i.Customer != null && i.Status != InvoiceStatus.Voided && i.Status != InvoiceStatus.WrittenOff && i.Status != InvoiceStatus.Paid)
|
||||
.Select(i =>
|
||||
{
|
||||
@@ -1758,7 +1773,8 @@ public class ReportsController : Controller
|
||||
/// </summary>
|
||||
public async Task<IActionResult> PowderConsumption(int months = 6)
|
||||
{
|
||||
var allTx = (await _unitOfWork.InventoryTransactions.GetAllAsync(false, t => t.InventoryItem)).ToList();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var allTx = (await _unitOfWork.InventoryTransactions.FindAsync(t => t.CompanyId == companyId, false, t => t.InventoryItem)).ToList();
|
||||
var items = allTx.Where(t => t.InventoryItem != null)
|
||||
.GroupBy(t => new { t.InventoryItemId, t.InventoryItem!.Name, t.InventoryItem.SKU, t.InventoryItem.ColorName, t.InventoryItem.ColorCode, t.InventoryItem.Manufacturer })
|
||||
.Select(g => new PowderConsumptionItem { InventoryItemId = g.Key.InventoryItemId, ItemName = g.Key.Name, SKU = g.Key.SKU, ColorName = g.Key.ColorName, ColorCode = g.Key.ColorCode, Manufacturer = g.Key.Manufacturer, TotalPurchasedLbs = g.Where(t => t.TransactionType == InventoryTransactionType.Purchase || t.TransactionType == InventoryTransactionType.Initial).Sum(t => t.Quantity), TotalConsumedLbs = g.Where(t => t.TransactionType == InventoryTransactionType.JobUsage || t.TransactionType == InventoryTransactionType.Waste).Sum(t => Math.Abs(t.Quantity)), PurchaseCount = g.Count(t => t.TransactionType == InventoryTransactionType.Purchase), UsageJobCount = g.Where(t => t.TransactionType == InventoryTransactionType.JobUsage && !string.IsNullOrEmpty(t.Reference)).Select(t => t.Reference).Distinct().Count() })
|
||||
@@ -1776,8 +1792,9 @@ public class ReportsController : Controller
|
||||
public async Task<IActionResult> InventoryTurnover(int months = 6)
|
||||
{
|
||||
var daysInPeriod = months * 30.0;
|
||||
var inventory = (await _unitOfWork.InventoryItems.GetAllAsync()).ToList();
|
||||
var allTx = (await _unitOfWork.InventoryTransactions.GetAllAsync(false, t => t.InventoryItem)).ToList();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var inventory = (await _unitOfWork.InventoryItems.FindAsync(i => i.CompanyId == companyId)).ToList();
|
||||
var allTx = (await _unitOfWork.InventoryTransactions.FindAsync(t => t.CompanyId == companyId, false, t => t.InventoryItem)).ToList();
|
||||
var items = inventory.Where(i => i.IsActive).Select(i =>
|
||||
{
|
||||
var iTx = allTx.Where(t => t.InventoryItemId == i.Id).ToList();
|
||||
@@ -1835,8 +1852,9 @@ public class ReportsController : Controller
|
||||
var now = DateTime.UtcNow;
|
||||
var today = DateTime.Today;
|
||||
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
// Load invoices for AR data
|
||||
var allInvoices = (await _unitOfWork.Invoices.GetAllAsync(false, i => i.Customer, i => i.Payments)).ToList();
|
||||
var allInvoices = (await _unitOfWork.Invoices.FindAsync(i => i.CompanyId == companyId, false, i => i.Customer, i => i.Payments)).ToList();
|
||||
var activeInvoices = allInvoices.Where(i => i.Status != InvoiceStatus.Voided && i.Status != InvoiceStatus.WrittenOff).ToList();
|
||||
var outstandingInvoices = activeInvoices.Where(i => i.BalanceDue > 0 && i.Status != InvoiceStatus.Paid).ToList();
|
||||
|
||||
@@ -1930,13 +1948,14 @@ public class ReportsController : Controller
|
||||
var companyName = await GetCompanyNameAsync();
|
||||
var today = DateTime.Today;
|
||||
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
// Open AR invoices
|
||||
var openInvoices = (await _unitOfWork.Invoices.GetAllAsync(false, i => i.Customer, i => i.Payments))
|
||||
var openInvoices = (await _unitOfWork.Invoices.FindAsync(i => i.CompanyId == companyId, false, i => i.Customer, i => i.Payments))
|
||||
.Where(i => i.BalanceDue > 0 && i.Status != InvoiceStatus.Voided && i.Status != InvoiceStatus.WrittenOff && i.Status != InvoiceStatus.Paid)
|
||||
.ToList();
|
||||
|
||||
// Compute avg days to pay per customer from paid invoices
|
||||
var paidInvoices = (await _unitOfWork.Invoices.GetAllAsync(false, i => i.Payments))
|
||||
var paidInvoices = (await _unitOfWork.Invoices.FindAsync(i => i.CompanyId == companyId, false, i => i.Payments))
|
||||
.Where(i => i.Status == InvoiceStatus.Paid && i.InvoiceDate != default)
|
||||
.ToList();
|
||||
var avgDaysByCustomer = paidInvoices
|
||||
@@ -2137,7 +2156,8 @@ public class ReportsController : Controller
|
||||
var companyName = await GetCompanyNameAsync();
|
||||
var today = DateTime.Today;
|
||||
|
||||
var allInvoices = (await _unitOfWork.Invoices.GetAllAsync(false, i => i.Customer, i => i.Payments)).ToList();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var allInvoices = (await _unitOfWork.Invoices.FindAsync(i => i.CompanyId == companyId, false, i => i.Customer, i => i.Payments)).ToList();
|
||||
var activeInvoices = allInvoices.Where(i =>
|
||||
i.Status != InvoiceStatus.Voided &&
|
||||
i.Status != InvoiceStatus.WrittenOff).ToList();
|
||||
@@ -2256,8 +2276,9 @@ public class ReportsController : Controller
|
||||
var companyName = await GetCompanyNameAsync();
|
||||
var now = DateTime.UtcNow;
|
||||
var startOfYear = new DateTime(now.Year, 1, 1);
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
|
||||
var allInvoices = (await _unitOfWork.Invoices.GetAllAsync(false, i => i.Customer, i => i.Payments))
|
||||
var allInvoices = (await _unitOfWork.Invoices.FindAsync(i => i.CompanyId == companyId, false, i => i.Customer, i => i.Payments))
|
||||
.Where(i => i.Status != InvoiceStatus.Voided && i.Status != InvoiceStatus.WrittenOff)
|
||||
.ToList();
|
||||
|
||||
|
||||
@@ -15,11 +15,13 @@ namespace PowderCoating.Web.Controllers;
|
||||
public class SmsConsentAuditController : Controller
|
||||
{
|
||||
private readonly IUnitOfWork _unitOfWork;
|
||||
private readonly ITenantContext _tenantContext;
|
||||
private readonly ILogger<SmsConsentAuditController> _logger;
|
||||
|
||||
public SmsConsentAuditController(IUnitOfWork unitOfWork, ILogger<SmsConsentAuditController> logger)
|
||||
public SmsConsentAuditController(IUnitOfWork unitOfWork, ITenantContext tenantContext, ILogger<SmsConsentAuditController> logger)
|
||||
{
|
||||
_unitOfWork = unitOfWork;
|
||||
_tenantContext = tenantContext;
|
||||
_logger = logger;
|
||||
}
|
||||
|
||||
@@ -30,7 +32,8 @@ public class SmsConsentAuditController : Controller
|
||||
{
|
||||
try
|
||||
{
|
||||
var allCustomers = await _unitOfWork.Customers.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var allCustomers = await _unitOfWork.Customers.FindAsync(c => c.CompanyId == companyId);
|
||||
|
||||
if (!string.IsNullOrWhiteSpace(search))
|
||||
{
|
||||
@@ -98,7 +101,8 @@ public class SmsConsentAuditController : Controller
|
||||
{
|
||||
try
|
||||
{
|
||||
var customers = (await _unitOfWork.Customers.GetAllAsync())
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var customers = (await _unitOfWork.Customers.FindAsync(c => c.CompanyId == companyId))
|
||||
.OrderBy(c => c.CompanyName ?? c.ContactLastName ?? c.ContactFirstName)
|
||||
.ToList();
|
||||
|
||||
|
||||
@@ -32,7 +32,8 @@ public class TaxRatesController : Controller
|
||||
[HttpGet]
|
||||
public async Task<IActionResult> Index()
|
||||
{
|
||||
var rates = await _unitOfWork.TaxRates.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var rates = await _unitOfWork.TaxRates.FindAsync(r => r.CompanyId == companyId);
|
||||
return View(rates.OrderBy(r => r.Name).ToList());
|
||||
}
|
||||
|
||||
|
||||
@@ -87,7 +87,8 @@ public class ToolsController : Controller
|
||||
[HttpGet]
|
||||
public async Task<IActionResult> GetImportAccounts()
|
||||
{
|
||||
var allAccounts = await _unitOfWork.Accounts.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var allAccounts = await _unitOfWork.Accounts.FindAsync(a => a.CompanyId == companyId);
|
||||
|
||||
var revenue = allAccounts
|
||||
.Where(a => a.AccountType == AccountType.Revenue && a.IsActive)
|
||||
@@ -123,7 +124,8 @@ public class ToolsController : Controller
|
||||
/// </summary>
|
||||
private async Task PopulateImportAccountDropdownsAsync()
|
||||
{
|
||||
var allAccounts = await _unitOfWork.Accounts.GetAllAsync();
|
||||
var companyId = _tenantContext.GetCurrentCompanyId() ?? 0;
|
||||
var allAccounts = await _unitOfWork.Accounts.FindAsync(a => a.CompanyId == companyId);
|
||||
|
||||
var revenueAccounts = allAccounts
|
||||
.Where(a => a.AccountType == AccountType.Revenue && a.IsActive)
|
||||
@@ -1102,7 +1104,7 @@ public class ToolsController : Controller
|
||||
|
||||
// Validate account IDs belong to this company — stale page load can produce IDs
|
||||
// that were valid before a data reset but no longer exist.
|
||||
var validAccountIds = (await _unitOfWork.Accounts.GetAllAsync())
|
||||
var validAccountIds = (await _unitOfWork.Accounts.FindAsync(a => a.CompanyId == companyId.Value))
|
||||
.Select(a => a.Id).ToHashSet();
|
||||
if (revenueAccountId.HasValue && !validAccountIds.Contains(revenueAccountId.Value))
|
||||
revenueAccountId = null;
|
||||
@@ -1167,7 +1169,7 @@ public class ToolsController : Controller
|
||||
|
||||
// Validate account IDs belong to this company — stale page load can produce IDs
|
||||
// that were valid before a data reset but no longer exist.
|
||||
var validAccountIds = (await _unitOfWork.Accounts.GetAllAsync())
|
||||
var validAccountIds = (await _unitOfWork.Accounts.FindAsync(a => a.CompanyId == companyId.Value))
|
||||
.Select(a => a.Id).ToHashSet();
|
||||
if (inventoryAccountId.HasValue && !validAccountIds.Contains(inventoryAccountId.Value))
|
||||
inventoryAccountId = null;
|
||||
@@ -1939,7 +1941,7 @@ public class ToolsController : Controller
|
||||
using (var archive = new System.IO.Compression.ZipArchive(memoryStream, System.IO.Compression.ZipArchiveMode.Create, true))
|
||||
{
|
||||
// 1. Customers
|
||||
var customers = await _unitOfWork.Customers.GetAllAsync();
|
||||
var customers = await _unitOfWork.Customers.FindAsync(c => c.CompanyId == companyId.Value);
|
||||
var customersCsv = GenerateCustomersCsv(customers);
|
||||
var customersEntry = archive.CreateEntry($"customers_{timestamp}.csv");
|
||||
using (var entryStream = customersEntry.Open())
|
||||
@@ -1949,7 +1951,7 @@ public class ToolsController : Controller
|
||||
}
|
||||
|
||||
// 2. Quotes
|
||||
var quotes = await _unitOfWork.Quotes.GetAllAsync(false, q => q.Customer, q => q.QuoteStatus);
|
||||
var quotes = await _unitOfWork.Quotes.FindAsync(q => q.CompanyId == companyId.Value, false, q => q.Customer, q => q.QuoteStatus);
|
||||
var quotesCsv = GenerateQuotesCsv(quotes);
|
||||
var quotesEntry = archive.CreateEntry($"quotes_{timestamp}.csv");
|
||||
using (var entryStream = quotesEntry.Open())
|
||||
@@ -1959,7 +1961,7 @@ public class ToolsController : Controller
|
||||
}
|
||||
|
||||
// 3. Jobs
|
||||
var jobs = await _unitOfWork.Jobs.GetAllAsync(false, j => j.Customer, j => j.JobStatus, j => j.JobPriority);
|
||||
var jobs = await _unitOfWork.Jobs.FindAsync(j => j.CompanyId == companyId.Value, false, j => j.Customer, j => j.JobStatus, j => j.JobPriority);
|
||||
var jobsCsv = GenerateJobsCsv(jobs);
|
||||
var jobsEntry = archive.CreateEntry($"jobs_{timestamp}.csv");
|
||||
using (var entryStream = jobsEntry.Open())
|
||||
@@ -1969,7 +1971,7 @@ public class ToolsController : Controller
|
||||
}
|
||||
|
||||
// 4. Appointments
|
||||
var appointments = await _unitOfWork.Appointments.GetAllAsync(false,
|
||||
var appointments = await _unitOfWork.Appointments.FindAsync(a => a.CompanyId == companyId.Value, false,
|
||||
a => a.Customer, a => a.AppointmentType, a => a.AppointmentStatus);
|
||||
var appointmentsCsv = GenerateAppointmentsCsv(appointments);
|
||||
var appointmentsEntry = archive.CreateEntry($"appointments_{timestamp}.csv");
|
||||
@@ -1980,9 +1982,9 @@ public class ToolsController : Controller
|
||||
}
|
||||
|
||||
// 5. Catalog
|
||||
var catalogCategories = await _unitOfWork.CatalogCategories.GetAllAsync();
|
||||
var catalogCategories = await _unitOfWork.CatalogCategories.FindAsync(cc => cc.CompanyId == companyId.Value);
|
||||
var catalogCategoryPaths = BuildCategoryPathMap(catalogCategories);
|
||||
var catalog = await _unitOfWork.CatalogItems.GetAllAsync();
|
||||
var catalog = await _unitOfWork.CatalogItems.FindAsync(ci => ci.CompanyId == companyId.Value);
|
||||
var catalogCsv = GenerateCatalogCsv(catalog, catalogCategoryPaths);
|
||||
var catalogEntry = archive.CreateEntry($"catalog_{timestamp}.csv");
|
||||
using (var entryStream = catalogEntry.Open())
|
||||
@@ -1992,7 +1994,7 @@ public class ToolsController : Controller
|
||||
}
|
||||
|
||||
// 6. Inventory
|
||||
var inventory = await _unitOfWork.InventoryItems.GetAllAsync();
|
||||
var inventory = await _unitOfWork.InventoryItems.FindAsync(i => i.CompanyId == companyId.Value);
|
||||
var inventoryCsv = GenerateInventoryCsv(inventory);
|
||||
var inventoryEntry = archive.CreateEntry($"inventory_{timestamp}.csv");
|
||||
using (var entryStream = inventoryEntry.Open())
|
||||
@@ -2002,7 +2004,7 @@ public class ToolsController : Controller
|
||||
}
|
||||
|
||||
// 7. Equipment
|
||||
var equipment = await _unitOfWork.Equipment.GetAllAsync();
|
||||
var equipment = await _unitOfWork.Equipment.FindAsync(e => e.CompanyId == companyId.Value);
|
||||
var equipmentCsv = GenerateEquipmentCsv(equipment);
|
||||
var equipmentEntry = archive.CreateEntry($"equipment_{timestamp}.csv");
|
||||
using (var entryStream = equipmentEntry.Open())
|
||||
@@ -2012,7 +2014,7 @@ public class ToolsController : Controller
|
||||
}
|
||||
|
||||
// 8. Maintenance
|
||||
var maintenance = await _unitOfWork.MaintenanceRecords.GetAllAsync(false, m => m.Equipment);
|
||||
var maintenance = await _unitOfWork.MaintenanceRecords.FindAsync(m => m.CompanyId == companyId.Value, false, m => m.Equipment);
|
||||
var maintenanceCsv = GenerateMaintenanceCsv(maintenance);
|
||||
var maintenanceEntry = archive.CreateEntry($"maintenance_{timestamp}.csv");
|
||||
using (var entryStream = maintenanceEntry.Open())
|
||||
@@ -2022,7 +2024,7 @@ public class ToolsController : Controller
|
||||
}
|
||||
|
||||
// 9. Vendors
|
||||
var vendors = await _unitOfWork.Vendors.GetAllAsync();
|
||||
var vendors = await _unitOfWork.Vendors.FindAsync(v => v.CompanyId == companyId.Value);
|
||||
var vendorsCsv = GenerateVendorsCsv(vendors);
|
||||
var vendorsEntry = archive.CreateEntry($"vendors_{timestamp}.csv");
|
||||
using (var entryStream = vendorsEntry.Open())
|
||||
@@ -2032,7 +2034,7 @@ public class ToolsController : Controller
|
||||
}
|
||||
|
||||
// 10. Prep Services
|
||||
var prepServices = await _unitOfWork.PrepServices.GetAllAsync();
|
||||
var prepServices = await _unitOfWork.PrepServices.FindAsync(ps => ps.CompanyId == companyId.Value);
|
||||
var prepServicesCsv = GeneratePrepServicesCsv(prepServices);
|
||||
var prepServicesEntry = archive.CreateEntry($"prep_services_{timestamp}.csv");
|
||||
using (var entryStream = prepServicesEntry.Open())
|
||||
@@ -2042,7 +2044,7 @@ public class ToolsController : Controller
|
||||
}
|
||||
|
||||
// 11. Invoices
|
||||
var invoices = await _unitOfWork.Invoices.GetAllAsync(false, i => i.Customer, i => i.Job);
|
||||
var invoices = await _unitOfWork.Invoices.FindAsync(i => i.CompanyId == companyId.Value, false, i => i.Customer, i => i.Job);
|
||||
var invoicesCsv = GenerateInvoicesCsv(invoices);
|
||||
var invoicesEntry = archive.CreateEntry($"invoices_{timestamp}.csv");
|
||||
using (var entryStream = invoicesEntry.Open())
|
||||
@@ -2052,7 +2054,7 @@ public class ToolsController : Controller
|
||||
}
|
||||
|
||||
// 12. Chart of Accounts
|
||||
var accounts = await _unitOfWork.Accounts.GetAllAsync();
|
||||
var accounts = await _unitOfWork.Accounts.FindAsync(a => a.CompanyId == companyId.Value);
|
||||
var accountsCsv = GenerateChartOfAccountsCsv(accounts);
|
||||
var accountsEntry = archive.CreateEntry($"chart_of_accounts_{timestamp}.csv");
|
||||
using (var entryStream = accountsEntry.Open())
|
||||
@@ -2062,7 +2064,7 @@ public class ToolsController : Controller
|
||||
}
|
||||
|
||||
// 13. Expenses
|
||||
var expenses = await _unitOfWork.Expenses.GetAllAsync(false, e => e.ExpenseAccount, e => e.PaymentAccount, e => e.Vendor, e => e.Job);
|
||||
var expenses = await _unitOfWork.Expenses.FindAsync(e => e.CompanyId == companyId.Value, false, e => e.ExpenseAccount, e => e.PaymentAccount, e => e.Vendor, e => e.Job);
|
||||
var expensesCsv = GenerateExpensesCsv(expenses);
|
||||
var expensesEntry = archive.CreateEntry($"expenses_{timestamp}.csv");
|
||||
using (var entryStream = expensesEntry.Open())
|
||||
@@ -2072,7 +2074,7 @@ public class ToolsController : Controller
|
||||
}
|
||||
|
||||
// 14. Payments
|
||||
var payments = await _unitOfWork.Payments.GetAllAsync(false, p => p.Invoice);
|
||||
var payments = await _unitOfWork.Payments.FindAsync(p => p.CompanyId == companyId.Value, false, p => p.Invoice);
|
||||
var paymentsCsv = GeneratePaymentsCsv(payments);
|
||||
var paymentsEntry = archive.CreateEntry($"payments_{timestamp}.csv");
|
||||
using (var entryStream = paymentsEntry.Open())
|
||||
@@ -2258,9 +2260,9 @@ public class ToolsController : Controller
|
||||
return RedirectToAction(nameof(Index));
|
||||
}
|
||||
|
||||
var catalogCategories = await _unitOfWork.CatalogCategories.GetAllAsync();
|
||||
var catalogCategories = await _unitOfWork.CatalogCategories.FindAsync(cc => cc.CompanyId == companyId.Value);
|
||||
var catalogCategoryPaths = BuildCategoryPathMap(catalogCategories);
|
||||
var catalogItems = await _unitOfWork.CatalogItems.GetAllAsync();
|
||||
var catalogItems = await _unitOfWork.CatalogItems.FindAsync(ci => ci.CompanyId == companyId.Value);
|
||||
var csv = GenerateCatalogCsv(catalogItems, catalogCategoryPaths);
|
||||
var fileName = $"catalog_export_{DateTime.UtcNow:yyyyMMddHHmmss}.csv";
|
||||
|
||||
@@ -2326,7 +2328,7 @@ public class ToolsController : Controller
|
||||
return RedirectToAction(nameof(Index));
|
||||
}
|
||||
|
||||
var equipment = await _unitOfWork.Equipment.GetAllAsync();
|
||||
var equipment = await _unitOfWork.Equipment.FindAsync(e => e.CompanyId == companyId.Value);
|
||||
var csv = GenerateEquipmentCsv(equipment);
|
||||
var fileName = $"equipment_export_{DateTime.UtcNow:yyyyMMddHHmmss}.csv";
|
||||
|
||||
@@ -2407,13 +2409,13 @@ public class ToolsController : Controller
|
||||
return RedirectToAction(nameof(Index));
|
||||
}
|
||||
|
||||
// Load all lookup tables
|
||||
var jobStatuses = await _unitOfWork.JobStatusLookups.GetAllAsync();
|
||||
var jobPriorities = await _unitOfWork.JobPriorityLookups.GetAllAsync();
|
||||
var quoteStatuses = await _unitOfWork.QuoteStatusLookups.GetAllAsync();
|
||||
var inventoryCategories = await _unitOfWork.InventoryCategoryLookups.GetAllAsync();
|
||||
var appointmentStatuses = await _unitOfWork.AppointmentStatusLookups.GetAllAsync();
|
||||
var appointmentTypes = await _unitOfWork.AppointmentTypeLookups.GetAllAsync();
|
||||
// Load all lookup tables — scoped to this company
|
||||
var jobStatuses = await _unitOfWork.JobStatusLookups.FindAsync(s => s.CompanyId == companyId.Value);
|
||||
var jobPriorities = await _unitOfWork.JobPriorityLookups.FindAsync(p => p.CompanyId == companyId.Value);
|
||||
var quoteStatuses = await _unitOfWork.QuoteStatusLookups.FindAsync(s => s.CompanyId == companyId.Value);
|
||||
var inventoryCategories = await _unitOfWork.InventoryCategoryLookups.FindAsync(c => c.CompanyId == companyId.Value);
|
||||
var appointmentStatuses = await _unitOfWork.AppointmentStatusLookups.FindAsync(s => s.CompanyId == companyId.Value);
|
||||
var appointmentTypes = await _unitOfWork.AppointmentTypeLookups.FindAsync(t => t.CompanyId == companyId.Value);
|
||||
|
||||
var csv = GenerateCompanySettingsCsv(company, jobStatuses, jobPriorities,
|
||||
quoteStatuses, inventoryCategories, appointmentStatuses, appointmentTypes);
|
||||
@@ -4092,7 +4094,7 @@ public class ToolsController : Controller
|
||||
return RedirectToAction(nameof(Index));
|
||||
}
|
||||
|
||||
var prepServices = await _unitOfWork.PrepServices.GetAllAsync();
|
||||
var prepServices = await _unitOfWork.PrepServices.FindAsync(ps => ps.CompanyId == companyId.Value);
|
||||
var csv = GeneratePrepServicesCsv(prepServices);
|
||||
var fileName = $"prep_services_export_{DateTime.UtcNow:yyyyMMddHHmmss}.csv";
|
||||
|
||||
@@ -4124,7 +4126,7 @@ public class ToolsController : Controller
|
||||
return RedirectToAction(nameof(Index));
|
||||
}
|
||||
|
||||
var vendors = await _unitOfWork.Vendors.GetAllAsync();
|
||||
var vendors = await _unitOfWork.Vendors.FindAsync(v => v.CompanyId == companyId.Value);
|
||||
var csv = GenerateVendorsCsv(vendors);
|
||||
var fileName = $"vendors_export_{DateTime.UtcNow:yyyyMMddHHmmss}.csv";
|
||||
|
||||
@@ -4156,7 +4158,7 @@ public class ToolsController : Controller
|
||||
return RedirectToAction(nameof(Index));
|
||||
}
|
||||
|
||||
var accounts = await _unitOfWork.Accounts.GetAllAsync();
|
||||
var accounts = await _unitOfWork.Accounts.FindAsync(a => a.CompanyId == companyId.Value);
|
||||
var csv = GenerateChartOfAccountsCsv(accounts);
|
||||
var fileName = $"chart_of_accounts_export_{DateTime.UtcNow:yyyyMMddHHmmss}.csv";
|
||||
|
||||
|
||||
@@ -240,6 +240,7 @@ builder.Services.AddHostedService<AuditLogRetentionBackgroundService>();
|
||||
builder.Services.AddHostedService<StripeWebhookRetentionBackgroundService>();
|
||||
builder.Services.AddHostedService<SetupWizardReminderBackgroundService>();
|
||||
builder.Services.AddHostedService<RecurringTransactionService>();
|
||||
builder.Services.AddHostedService<AppointmentReminderBackgroundService>();
|
||||
builder.Services.AddScoped<ISubscriptionService, SubscriptionService>();
|
||||
builder.Services.AddScoped<IStripeService, StripeService>();
|
||||
builder.Services.AddScoped<IStripeConnectService, StripeConnectService>();
|
||||
|
||||
@@ -52,12 +52,13 @@
|
||||
<div class="card-body p-0">
|
||||
@if (!Model.Items.Any())
|
||||
{
|
||||
var isCustomerListFiltered = !string.IsNullOrEmpty(ViewBag.SearchTerm as string);
|
||||
<div class="text-center py-5">
|
||||
<i class="bi bi-inbox" style="font-size: 4rem; color: #d1d5db;"></i>
|
||||
<h5 class="mt-3 text-muted">No customers found</h5>
|
||||
<p class="text-muted mb-4">Get started by adding your first customer</p>
|
||||
<p class="text-muted mb-4">@(isCustomerListFiltered ? "No customers match your search." : "Get started by adding your first customer.")</p>
|
||||
<a asp-action="Create" class="btn btn-primary">
|
||||
<i class="bi bi-plus-circle me-2"></i>Add Your First Customer
|
||||
<i class="bi bi-plus-circle me-2"></i>@(isCustomerListFiltered ? "Add Customer" : "Add Your First Customer")
|
||||
</a>
|
||||
</div>
|
||||
}
|
||||
@@ -184,12 +185,13 @@
|
||||
<div class="mobile-card-view">
|
||||
@if (!Model.Items.Any())
|
||||
{
|
||||
var isMobileCustomerListFiltered = !string.IsNullOrEmpty(ViewBag.SearchTerm as string);
|
||||
<div class="text-center py-5">
|
||||
<i class="bi bi-inbox" style="font-size: 4rem; color: #d1d5db;"></i>
|
||||
<h5 class="mt-3 text-muted">No customers found</h5>
|
||||
<p class="text-muted mb-4">Get started by adding your first customer</p>
|
||||
<p class="text-muted mb-4">@(isMobileCustomerListFiltered ? "No customers match your search." : "Get started by adding your first customer.")</p>
|
||||
<a asp-action="Create" class="btn btn-primary">
|
||||
<i class="bi bi-plus-circle me-2"></i>Add Your First Customer
|
||||
<i class="bi bi-plus-circle me-2"></i>@(isMobileCustomerListFiltered ? "Add Customer" : "Add Your First Customer")
|
||||
</a>
|
||||
</div>
|
||||
}
|
||||
|
||||
@@ -61,12 +61,13 @@
|
||||
<div class="card-body p-0">
|
||||
@if (!Model.Items.Any())
|
||||
{
|
||||
var isEquipmentListFiltered = !string.IsNullOrEmpty(ViewBag.SearchTerm as string) || ViewBag.StatusFilter != null;
|
||||
<div class="text-center py-5">
|
||||
<i class="bi bi-inbox" style="font-size: 4rem; color: #d1d5db;"></i>
|
||||
<h5 class="mt-3 text-muted">No equipment found</h5>
|
||||
<p class="text-muted mb-4">Get started by adding your first equipment</p>
|
||||
<p class="text-muted mb-4">@(isEquipmentListFiltered ? "No equipment matches your search." : "Get started by adding your first equipment.")</p>
|
||||
<a asp-action="Create" class="btn btn-primary">
|
||||
<i class="bi bi-plus-circle me-2"></i>Add Your First Equipment
|
||||
<i class="bi bi-plus-circle me-2"></i>@(isEquipmentListFiltered ? "Add Equipment" : "Add Your First Equipment")
|
||||
</a>
|
||||
</div>
|
||||
}
|
||||
|
||||
@@ -191,12 +191,13 @@
|
||||
<div class="card-body p-0">
|
||||
@if (!Model.Items.Any())
|
||||
{
|
||||
var isInventoryFiltered = !string.IsNullOrEmpty(ViewBag.SearchTerm as string) || !string.IsNullOrEmpty(ViewBag.Category as string) || lowStockOnly;
|
||||
<div class="text-center py-5">
|
||||
<i class="bi bi-inbox" style="font-size: 4rem; color: #d1d5db;"></i>
|
||||
<h5 class="mt-3 text-muted">No inventory items found</h5>
|
||||
<p class="text-muted mb-4">Get started by adding your first inventory item</p>
|
||||
<p class="text-muted mb-4">@(isInventoryFiltered ? "No items match your current filters." : "Get started by adding your first inventory item.")</p>
|
||||
<a asp-action="Create" class="btn btn-primary">
|
||||
<i class="bi bi-plus-circle me-2"></i>Add Your First Item
|
||||
<i class="bi bi-plus-circle me-2"></i>@(isInventoryFiltered ? "Add Item" : "Add Your First Item")
|
||||
</a>
|
||||
</div>
|
||||
}
|
||||
|
||||
@@ -869,42 +869,45 @@
|
||||
</div>
|
||||
</div>
|
||||
|
||||
@if (showSendModal)
|
||||
{
|
||||
<!-- Send Channel Choice Modal (shown when customer has both email + SMS) -->
|
||||
<div class="modal fade" id="sendChannelModal" tabindex="-1" aria-labelledby="sendChannelModalLabel" aria-hidden="true">
|
||||
<div class="modal-dialog modal-dialog-centered">
|
||||
<div class="modal-content">
|
||||
<div class="modal-header border-0 pb-0">
|
||||
<h5 class="modal-title" id="sendChannelModalLabel">
|
||||
<i class="bi bi-send text-primary me-2"></i>Send Invoice
|
||||
</h5>
|
||||
<button type="button" class="btn-close" data-bs-dismiss="modal" aria-label="Close"></button>
|
||||
</div>
|
||||
<div class="modal-body pt-2">
|
||||
<p class="mb-3">How would you like to send <strong>@Model.InvoiceNumber</strong> to <strong>@Model.CustomerName</strong>?</p>
|
||||
<div class="d-grid gap-2">
|
||||
<button type="button" class="btn btn-outline-primary text-start" onclick="submitSendInvoice(true, false)" data-bs-dismiss="modal">
|
||||
<i class="bi bi-envelope me-2"></i>Email only
|
||||
<small class="d-block text-muted ms-4">PDF attached · @Model.CustomerEmail</small>
|
||||
</button>
|
||||
<button type="button" class="btn btn-outline-primary text-start" onclick="submitSendInvoice(false, true)" data-bs-dismiss="modal">
|
||||
<i class="bi bi-phone me-2"></i>SMS only
|
||||
<small class="d-block text-muted ms-4">View link · @smsPhone</small>
|
||||
</button>
|
||||
<button type="button" class="btn btn-primary text-start" onclick="submitSendInvoice(true, true)" data-bs-dismiss="modal">
|
||||
<i class="bi bi-send me-2"></i>Both Email & SMS
|
||||
<small class="d-block text-muted ms-4">PDF via email + view link via SMS</small>
|
||||
</button>
|
||||
</div>
|
||||
</div>
|
||||
<div class="modal-footer border-0 pt-0">
|
||||
<button type="button" class="btn btn-outline-secondary" data-bs-dismiss="modal">Cancel</button>
|
||||
}
|
||||
|
||||
@if (showSendModal)
|
||||
{
|
||||
<!-- Send Channel Choice Modal (shown when customer has both email + SMS available).
|
||||
Lives outside the isDraft block so it also renders for Sent/Overdue invoices
|
||||
where the customer's email was added after an SMS-only initial send. -->
|
||||
<div class="modal fade" id="sendChannelModal" tabindex="-1" aria-labelledby="sendChannelModalLabel" aria-hidden="true">
|
||||
<div class="modal-dialog modal-dialog-centered">
|
||||
<div class="modal-content">
|
||||
<div class="modal-header border-0 pb-0">
|
||||
<h5 class="modal-title" id="sendChannelModalLabel">
|
||||
<i class="bi bi-send text-primary me-2"></i>Send Invoice
|
||||
</h5>
|
||||
<button type="button" class="btn-close" data-bs-dismiss="modal" aria-label="Close"></button>
|
||||
</div>
|
||||
<div class="modal-body pt-2">
|
||||
<p class="mb-3">How would you like to send <strong>@Model.InvoiceNumber</strong> to <strong>@Model.CustomerName</strong>?</p>
|
||||
<div class="d-grid gap-2">
|
||||
<button type="button" class="btn btn-outline-primary text-start" onclick="submitSendInvoice(true, false)" data-bs-dismiss="modal">
|
||||
<i class="bi bi-envelope me-2"></i>Email only
|
||||
<small class="d-block text-muted ms-4">PDF attached · @Model.CustomerEmail</small>
|
||||
</button>
|
||||
<button type="button" class="btn btn-outline-primary text-start" onclick="submitSendInvoice(false, true)" data-bs-dismiss="modal">
|
||||
<i class="bi bi-phone me-2"></i>SMS only
|
||||
<small class="d-block text-muted ms-4">View link · @smsPhone</small>
|
||||
</button>
|
||||
<button type="button" class="btn btn-primary text-start" onclick="submitSendInvoice(true, true)" data-bs-dismiss="modal">
|
||||
<i class="bi bi-send me-2"></i>Both Email & SMS
|
||||
<small class="d-block text-muted ms-4">PDF via email + view link via SMS</small>
|
||||
</button>
|
||||
</div>
|
||||
</div>
|
||||
<div class="modal-footer border-0 pt-0">
|
||||
<button type="button" class="btn btn-outline-secondary" data-bs-dismiss="modal">Cancel</button>
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
}
|
||||
</div>
|
||||
}
|
||||
|
||||
@if (canPay)
|
||||
|
||||
@@ -221,6 +221,49 @@
|
||||
</div>
|
||||
</div>
|
||||
|
||||
<!-- Oven & Batch Settings -->
|
||||
<div class="card mb-4">
|
||||
<div class="card-header">
|
||||
<div class="d-flex align-items-center gap-2">
|
||||
<h5 class="mb-0"><i class="bi bi-thermometer-half me-2"></i>Oven & Batch Settings</h5>
|
||||
<a tabindex="0" class="help-icon" role="button"
|
||||
data-bs-toggle="popover" data-bs-placement="right" data-bs-trigger="focus"
|
||||
data-bs-title="Oven & Batch Pricing"
|
||||
data-bs-content="The oven cost is charged once per batch at the job level, not per item. Estimate how many oven loads the full job will fill &mdash; for example, if you have 20 small parts and your oven fits 10, that&#39;s 2 batches. Cycle time is how long each batch runs. The cost is calculated from your oven&#39;s hourly rate in Settings.">
|
||||
<i class="bi bi-question-circle"></i>
|
||||
</a>
|
||||
</div>
|
||||
</div>
|
||||
<div class="card-body">
|
||||
<p class="text-muted small mb-3">Estimate how many oven loads the complete job will require. The oven cycle cost is added once at the job level, not per item.</p>
|
||||
<div class="row g-3 align-items-end">
|
||||
@if (ViewBag.OvenCosts != null && ((List<Microsoft.AspNetCore.Mvc.Rendering.SelectListItem>)ViewBag.OvenCosts).Count > 1)
|
||||
{
|
||||
<div class="col-md-4">
|
||||
<label asp-for="OvenCostId" class="form-label fw-semibold"><i class="bi bi-thermometer-half me-1"></i>Oven</label>
|
||||
<select asp-for="OvenCostId" asp-items="@((List<Microsoft.AspNetCore.Mvc.Rendering.SelectListItem>)ViewBag.OvenCosts)"
|
||||
class="form-select" onchange="scheduleAutoPricing()"></select>
|
||||
</div>
|
||||
}
|
||||
<div class="col-sm-3 col-md-2">
|
||||
<label asp-for="OvenBatches" class="form-label fw-semibold">Batches</label>
|
||||
<input asp-for="OvenBatches" class="form-control" type="number" min="1"
|
||||
value="@(Model.OvenBatches > 0 ? Model.OvenBatches : 1)"
|
||||
id="OvenBatches" onchange="scheduleAutoPricing()" />
|
||||
</div>
|
||||
<div class="col-sm-4 col-md-3">
|
||||
<label asp-for="OvenCycleMinutes" class="form-label fw-semibold">
|
||||
Cycle Time per Batch (min)
|
||||
<small class="text-muted fw-normal">default: @(ViewBag.DefaultOvenCycleMinutes ?? 45)</small>
|
||||
</label>
|
||||
<input asp-for="OvenCycleMinutes" class="form-control" type="number" min="1"
|
||||
id="OvenCycleMinutes" placeholder="@(ViewBag.DefaultOvenCycleMinutes ?? 45)"
|
||||
onchange="scheduleAutoPricing()" />
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
|
||||
<!-- Pricing Options (Rush / Discount) -->
|
||||
<div class="card mb-4">
|
||||
<div class="card-header">
|
||||
@@ -370,7 +413,8 @@
|
||||
transferEfficiency = c.TransferEfficiency,
|
||||
powderCostPerLb = c.PowderCostPerLb,
|
||||
powderToOrder = c.PowderToOrder,
|
||||
notes = c.Notes
|
||||
notes = c.Notes,
|
||||
noExtraLayerCharge = c.NoExtraLayerCharge
|
||||
}),
|
||||
prepServices = item.PrepServices.Select(ps => new {
|
||||
prepServiceId = ps.PrepServiceId,
|
||||
@@ -387,7 +431,7 @@
|
||||
"discountType": @Json.Serialize(Model.DiscountType),
|
||||
"discountValue": @Model.DiscountValue,
|
||||
"isRushJob": @Json.Serialize(Model.IsRushJob),
|
||||
"ovenCostId": null,
|
||||
"ovenCostId": @Json.Serialize(Model.OvenCostId),
|
||||
"areaUnit": @Json.Serialize((string?)ViewBag.AreaUnit),
|
||||
"useMetric": @Json.Serialize((bool)(ViewBag.UseMetric ?? false)),
|
||||
"pricingUrl": "@Url.Action("CalculatePricing", "Jobs")",
|
||||
|
||||
@@ -653,7 +653,10 @@
|
||||
<span class="mobile-card-value">
|
||||
@foreach (var coat in item.Coats.OrderBy(c => c.Sequence))
|
||||
{
|
||||
<small class="d-block">@coat.CoatName@if (!string.IsNullOrEmpty(coat.ColorName)) { <text> – @coat.ColorName</text> }</small>
|
||||
<small class="d-block">
|
||||
@coat.CoatName@if (!string.IsNullOrEmpty(coat.ColorName)) { <text> – @coat.ColorName</text> }
|
||||
@if (!string.IsNullOrEmpty(coat.Notes)) { <text><br /><span class="fst-italic text-muted ms-2">@coat.Notes</span></text> }
|
||||
</small>
|
||||
}
|
||||
</span>
|
||||
</div>
|
||||
@@ -1105,9 +1108,22 @@
|
||||
<div class="modal-body">
|
||||
<div class="mb-3">
|
||||
<label class="form-label fw-semibold">Inventory Item <span class="text-danger">*</span></label>
|
||||
<select id="lmInventoryItem" class="form-select">
|
||||
<option value="">-- Select item --</option>
|
||||
</select>
|
||||
<div class="position-relative">
|
||||
<div class="input-group">
|
||||
<input type="text" class="form-control" id="lmItemSearch"
|
||||
placeholder="Search by name or manufacturer…" autocomplete="off"
|
||||
oninput="lmComboInput()"
|
||||
onfocus="lmComboOpen()"
|
||||
onkeydown="lmComboKey(event)">
|
||||
<button class="btn btn-outline-secondary" type="button" tabindex="-1"
|
||||
id="lmItemDropdownToggle" onclick="lmComboToggle()">
|
||||
<i class="bi bi-chevron-down" style="font-size:.75rem;"></i>
|
||||
</button>
|
||||
</div>
|
||||
<div id="lmItemDropdown"
|
||||
style="display:none;max-height:220px;overflow-y:auto;z-index:1070;background:#fff;border:1px solid rgba(0,0,0,.15);border-radius:.375rem;box-shadow:0 4px 12px rgba(0,0,0,.12);">
|
||||
</div>
|
||||
</div>
|
||||
<div id="lmItemBalance" class="form-text text-muted d-none"></div>
|
||||
</div>
|
||||
<div class="mb-3">
|
||||
@@ -3149,10 +3165,11 @@
|
||||
<script>
|
||||
(function () {
|
||||
const inventoryItems = @Html.Raw(ViewBag.InventoryItemsForModal ?? "[]");
|
||||
const jobPowderIds = @Html.Raw(ViewBag.JobPowderIds ?? "[]");
|
||||
const jobId = @Model.Id;
|
||||
const logUrl = '@Url.Action("LogMaterial", "Jobs")';
|
||||
const token = document.querySelector('input[name="__RequestVerificationToken"]')?.value ?? '';
|
||||
window.__logMaterial = { inventoryItems, jobId, logUrl, token };
|
||||
window.__logMaterial = { inventoryItems, jobPowderIds, jobId, logUrl, token };
|
||||
})();
|
||||
</script>
|
||||
|
||||
|
||||
@@ -190,6 +190,49 @@
|
||||
</div>
|
||||
</div>
|
||||
|
||||
<!-- Oven & Batch Settings -->
|
||||
<div class="card mb-4">
|
||||
<div class="card-header">
|
||||
<div class="d-flex align-items-center gap-2">
|
||||
<h5 class="mb-0"><i class="bi bi-thermometer-half me-2"></i>Oven & Batch Settings</h5>
|
||||
<a tabindex="0" class="help-icon" role="button"
|
||||
data-bs-toggle="popover" data-bs-placement="right" data-bs-trigger="focus"
|
||||
data-bs-title="Oven & Batch Pricing"
|
||||
data-bs-content="The oven cost is charged once per batch at the job level, not per item. Estimate how many oven loads the full job will fill &mdash; for example, if you have 20 small parts and your oven fits 10, that&#39;s 2 batches. Cycle time is how long each batch runs. The cost is calculated from your oven&#39;s hourly rate in Settings.">
|
||||
<i class="bi bi-question-circle"></i>
|
||||
</a>
|
||||
</div>
|
||||
</div>
|
||||
<div class="card-body">
|
||||
<p class="text-muted small mb-3">Estimate how many oven loads the complete job will require. The oven cycle cost is added once at the job level, not per item.</p>
|
||||
<div class="row g-3 align-items-end">
|
||||
@if (ViewBag.OvenCosts != null && ((List<Microsoft.AspNetCore.Mvc.Rendering.SelectListItem>)ViewBag.OvenCosts).Count > 1)
|
||||
{
|
||||
<div class="col-md-4">
|
||||
<label asp-for="OvenCostId" class="form-label fw-semibold"><i class="bi bi-thermometer-half me-1"></i>Oven</label>
|
||||
<select asp-for="OvenCostId" asp-items="@((List<Microsoft.AspNetCore.Mvc.Rendering.SelectListItem>)ViewBag.OvenCosts)"
|
||||
class="form-select" onchange="scheduleAutoPricing()"></select>
|
||||
</div>
|
||||
}
|
||||
<div class="col-sm-3 col-md-2">
|
||||
<label asp-for="OvenBatches" class="form-label fw-semibold">Batches</label>
|
||||
<input asp-for="OvenBatches" class="form-control" type="number" min="1"
|
||||
value="@(Model.OvenBatches > 0 ? Model.OvenBatches : 1)"
|
||||
id="OvenBatches" onchange="scheduleAutoPricing()" />
|
||||
</div>
|
||||
<div class="col-sm-4 col-md-3">
|
||||
<label asp-for="OvenCycleMinutes" class="form-label fw-semibold">
|
||||
Cycle Time per Batch (min)
|
||||
<small class="text-muted fw-normal">default: @(ViewBag.DefaultOvenCycleMinutes ?? 45)</small>
|
||||
</label>
|
||||
<input asp-for="OvenCycleMinutes" class="form-control" type="number" min="1"
|
||||
id="OvenCycleMinutes" placeholder="@(ViewBag.DefaultOvenCycleMinutes ?? 45)"
|
||||
onchange="scheduleAutoPricing()" />
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
|
||||
<!-- Pricing Options (Rush / Discount) -->
|
||||
<div class="card mb-4">
|
||||
<div class="card-header">
|
||||
@@ -357,7 +400,8 @@
|
||||
transferEfficiency = c.TransferEfficiency,
|
||||
powderCostPerLb = c.PowderCostPerLb,
|
||||
powderToOrder = c.PowderToOrder,
|
||||
notes = c.Notes
|
||||
notes = c.Notes,
|
||||
noExtraLayerCharge = c.NoExtraLayerCharge
|
||||
}),
|
||||
prepServices = item.PrepServices.Select(ps => new {
|
||||
prepServiceId = ps.PrepServiceId,
|
||||
@@ -374,7 +418,7 @@
|
||||
"discountType": @Json.Serialize(Model.DiscountType),
|
||||
"discountValue": @Model.DiscountValue,
|
||||
"isRushJob": @Json.Serialize(Model.IsRushJob),
|
||||
"ovenCostId": null,
|
||||
"ovenCostId": @Json.Serialize(Model.OvenCostId),
|
||||
"areaUnit": @Json.Serialize((string?)ViewBag.AreaUnit),
|
||||
"useMetric": @Json.Serialize((bool)(ViewBag.UseMetric ?? false)),
|
||||
"pricingUrl": "@Url.Action("CalculatePricing", "Jobs")",
|
||||
|
||||
@@ -153,7 +153,8 @@
|
||||
transferEfficiency = c.TransferEfficiency,
|
||||
powderCostPerLb = c.PowderCostPerLb,
|
||||
powderToOrder = c.PowderToOrder,
|
||||
notes = c.Notes
|
||||
notes = c.Notes,
|
||||
noExtraLayerCharge = c.NoExtraLayerCharge
|
||||
}),
|
||||
prepServices = item.PrepServices.Select(ps => new {
|
||||
prepServiceId = ps.PrepServiceId,
|
||||
|
||||
@@ -479,7 +479,8 @@
|
||||
transferEfficiency = c.TransferEfficiency,
|
||||
powderCostPerLb = c.PowderCostPerLb,
|
||||
powderToOrder = c.PowderToOrder,
|
||||
notes = c.Notes
|
||||
notes = c.Notes,
|
||||
noExtraLayerCharge = c.NoExtraLayerCharge
|
||||
})
|
||||
})))
|
||||
</script>
|
||||
|
||||
@@ -525,7 +525,8 @@
|
||||
transferEfficiency = c.TransferEfficiency,
|
||||
powderCostPerLb = c.PowderCostPerLb,
|
||||
powderToOrder = c.PowderToOrder,
|
||||
notes = c.Notes
|
||||
notes = c.Notes,
|
||||
noExtraLayerCharge = c.NoExtraLayerCharge
|
||||
})
|
||||
})))
|
||||
</script>
|
||||
|
||||
@@ -82,6 +82,21 @@ document.addEventListener('DOMContentLoaded', () => {
|
||||
const ownerForm = hfc?.closest('form');
|
||||
if (ownerForm) {
|
||||
ownerForm.addEventListener('submit', writeHiddenFields, { capture: true });
|
||||
|
||||
// Save scroll position before the form causes a full-page reload so we can
|
||||
// restore it after the server redirects back to this page. Key is path-specific
|
||||
// so navigating away and back doesn't restore a stale position.
|
||||
const scrollKey = 'wizardScrollY:' + location.pathname;
|
||||
ownerForm.addEventListener('submit', () => {
|
||||
sessionStorage.setItem(scrollKey, String(Math.round(window.scrollY)));
|
||||
}, { capture: true });
|
||||
|
||||
// Restore on load — fire after layout is painted so scrollTo lands correctly.
|
||||
const savedY = sessionStorage.getItem(scrollKey);
|
||||
if (savedY !== null) {
|
||||
sessionStorage.removeItem(scrollKey);
|
||||
requestAnimationFrame(() => window.scrollTo({ top: parseInt(savedY, 10), behavior: 'instant' }));
|
||||
}
|
||||
}
|
||||
|
||||
// Close any open powder combobox or catalog lookup dropdown when clicking outside it
|
||||
@@ -2731,8 +2746,9 @@ function buildCardHtml(item, i) {
|
||||
const orderBadge = (!c.inventoryItemId && c.powderToOrder)
|
||||
? ` <span class="badge bg-warning text-dark" style="font-size:.65em;vertical-align:middle;" title="Custom powder — must be purchased before coating"><i class="bi bi-cart me-1"></i>ORDER ${parseFloat(c.powderToOrder).toFixed(2)} lbs</span>`
|
||||
: '';
|
||||
const coatNotes = c.notes ? `<span class="fst-italic ms-1 opacity-75">— ${escHtml(c.notes)}</span>` : '';
|
||||
return `<div style="font-size:.8rem;" class="text-muted mt-1">
|
||||
<i class="bi bi-layers me-1"></i><span class="fw-semibold">${escHtml(c.coatName || 'Coat')}</span>${color ? ` — ${color}${code}` : ''}${orderBadge}
|
||||
<i class="bi bi-layers me-1"></i><span class="fw-semibold">${escHtml(c.coatName || 'Coat')}</span>${color ? ` — ${color}${code}` : ''}${orderBadge}${coatNotes}
|
||||
</div>`;
|
||||
}).join('');
|
||||
|
||||
@@ -3414,7 +3430,7 @@ function loadItemsFromTemplate(templateItems) {
|
||||
coverageSqFtPerLb: c.coverageSqFtPerLb || 30,
|
||||
transferEfficiency: c.transferEfficiency || 65,
|
||||
powderCostPerLb: c.powderCostPerLb || null,
|
||||
noExtraLayerCharge: false
|
||||
noExtraLayerCharge: !!c.noExtraLayerCharge
|
||||
})),
|
||||
prepServices: (ti.prepServices || []).map(p => ({
|
||||
prepServiceId: p.prepServiceId,
|
||||
|
||||
@@ -4,62 +4,173 @@
|
||||
*/
|
||||
(function () {
|
||||
let _items = [];
|
||||
let _jobPowderIds = new Set();
|
||||
let _modal = null;
|
||||
|
||||
function init() {
|
||||
const cfg = window.__logMaterial;
|
||||
if (!cfg) return;
|
||||
// ── Combobox state ────────────────────────────────────────────────────────
|
||||
let _selectedItemId = 0;
|
||||
|
||||
_items = cfg.inventoryItems || [];
|
||||
_modal = new bootstrap.Modal(document.getElementById('logMaterialModal'));
|
||||
|
||||
const sel = document.getElementById('lmInventoryItem');
|
||||
_items.forEach(function (item) {
|
||||
const opt = document.createElement('option');
|
||||
opt.value = item.id;
|
||||
opt.textContent = item.name + (item.unitOfMeasure ? ' (' + item.unitOfMeasure + ')' : '');
|
||||
opt.dataset.qty = item.quantityOnHand;
|
||||
opt.dataset.uom = item.unitOfMeasure || '';
|
||||
sel.appendChild(opt);
|
||||
});
|
||||
|
||||
sel.addEventListener('change', lmOnItemChange);
|
||||
document.getElementById('lmQuantity').addEventListener('input', lmOnQtyInput);
|
||||
}
|
||||
|
||||
function lmOnItemChange() {
|
||||
const sel = document.getElementById('lmInventoryItem');
|
||||
const opt = sel.options[sel.selectedIndex];
|
||||
const balDiv = document.getElementById('lmItemBalance');
|
||||
if (sel.value && opt) {
|
||||
const qty = parseFloat(opt.dataset.qty) || 0;
|
||||
const uom = opt.dataset.uom;
|
||||
balDiv.textContent = 'Current stock: ' + qty.toFixed(2) + (uom ? ' ' + uom : '');
|
||||
balDiv.classList.remove('d-none');
|
||||
} else {
|
||||
balDiv.classList.add('d-none');
|
||||
}
|
||||
function lmComboInput() {
|
||||
const q = document.getElementById('lmItemSearch')?.value?.toLowerCase() || '';
|
||||
lmComboRender(q);
|
||||
lmComboShow();
|
||||
_selectedItemId = 0;
|
||||
document.getElementById('lmItemBalance').classList.add('d-none');
|
||||
lmOnQtyInput();
|
||||
}
|
||||
|
||||
function lmComboOpen() {
|
||||
const q = document.getElementById('lmItemSearch')?.value?.toLowerCase() || '';
|
||||
lmComboRender(q);
|
||||
lmComboShow();
|
||||
}
|
||||
|
||||
function lmComboToggle() {
|
||||
const dd = document.getElementById('lmItemDropdown');
|
||||
if (!dd) return;
|
||||
if (dd.style.display === 'none' || !dd.style.display) {
|
||||
lmComboOpen();
|
||||
document.getElementById('lmItemSearch')?.focus();
|
||||
} else {
|
||||
lmComboClose();
|
||||
}
|
||||
}
|
||||
|
||||
function lmMakeRow(it) {
|
||||
const display = (it.manufacturer ? escLm(it.manufacturer) + ' – ' : '') +
|
||||
escLm(it.name) +
|
||||
(it.unitOfMeasure ? ' <span class="text-muted" style="font-size:.82rem;">(' + escLm(it.unitOfMeasure) + ')</span>' : '');
|
||||
const label = (it.manufacturer ? it.manufacturer + ' - ' : '') +
|
||||
it.name +
|
||||
(it.unitOfMeasure ? ' (' + it.unitOfMeasure + ')' : '');
|
||||
return `<div class="lm-item-opt" style="padding:.35rem .75rem;font-size:.875rem;cursor:pointer;"
|
||||
data-id="${it.id}"
|
||||
data-qty="${it.quantityOnHand}"
|
||||
data-uom="${escLm(it.unitOfMeasure || '')}"
|
||||
data-label="${escLm(label)}"
|
||||
onmousedown="event.preventDefault(); lmComboSelect(this)"
|
||||
onmouseenter="this.style.background='#f0f4ff'"
|
||||
onmouseleave="this.classList.contains('lm-active') ? null : this.style.background=''">
|
||||
${display}
|
||||
</div>`;
|
||||
}
|
||||
|
||||
function lmComboRender(query) {
|
||||
const dd = document.getElementById('lmItemDropdown');
|
||||
if (!dd) return;
|
||||
const filtered = query
|
||||
? _items.filter(it => it.name.toLowerCase().includes(query) ||
|
||||
(it.manufacturer && it.manufacturer.toLowerCase().includes(query)) ||
|
||||
(it.unitOfMeasure && it.unitOfMeasure.toLowerCase().includes(query)))
|
||||
: _items;
|
||||
if (filtered.length === 0) {
|
||||
dd.innerHTML = '<div class="px-3 py-2 text-muted small">No items match.</div>';
|
||||
return;
|
||||
}
|
||||
|
||||
const jobItems = filtered.filter(it => _jobPowderIds.has(it.id));
|
||||
const otherItems = filtered.filter(it => !_jobPowderIds.has(it.id));
|
||||
|
||||
let html = '';
|
||||
if (jobItems.length > 0) {
|
||||
html += '<div class="px-3 py-1 text-muted" style="font-size:.72rem;letter-spacing:.04em;text-transform:uppercase;background:#f8f9fa;border-bottom:1px solid #dee2e6;">This Job</div>';
|
||||
html += jobItems.map(lmMakeRow).join('');
|
||||
if (otherItems.length > 0) {
|
||||
html += '<div style="height:1px;background:#dee2e6;margin:.25rem 0;"></div>';
|
||||
html += '<div class="px-3 py-1 text-muted" style="font-size:.72rem;letter-spacing:.04em;text-transform:uppercase;background:#f8f9fa;border-bottom:1px solid #dee2e6;">All Inventory</div>';
|
||||
}
|
||||
}
|
||||
html += otherItems.map(lmMakeRow).join('');
|
||||
dd.innerHTML = html;
|
||||
}
|
||||
|
||||
function lmComboShow() {
|
||||
const dd = document.getElementById('lmItemDropdown');
|
||||
const anchor = document.getElementById('lmItemSearch');
|
||||
if (!dd || !anchor) return;
|
||||
const rect = anchor.closest('.input-group').getBoundingClientRect();
|
||||
dd.style.position = 'fixed';
|
||||
dd.style.top = (rect.bottom + 2) + 'px';
|
||||
dd.style.left = rect.left + 'px';
|
||||
dd.style.width = rect.width + 'px';
|
||||
dd.style.display = 'block';
|
||||
}
|
||||
|
||||
function lmComboClose() {
|
||||
const dd = document.getElementById('lmItemDropdown');
|
||||
if (dd) dd.style.display = 'none';
|
||||
}
|
||||
|
||||
window.lmComboSelect = function (el) {
|
||||
_selectedItemId = parseInt(el.dataset.id) || 0;
|
||||
document.getElementById('lmItemSearch').value = el.dataset.label;
|
||||
lmComboClose();
|
||||
|
||||
const qty = parseFloat(el.dataset.qty) || 0;
|
||||
const uom = el.dataset.uom;
|
||||
const balDiv = document.getElementById('lmItemBalance');
|
||||
balDiv.textContent = 'Current stock: ' + qty.toFixed(2) + (uom ? ' ' + uom : '');
|
||||
balDiv.classList.remove('d-none');
|
||||
lmOnQtyInput();
|
||||
};
|
||||
|
||||
window.lmComboInput = lmComboInput;
|
||||
window.lmComboOpen = lmComboOpen;
|
||||
window.lmComboToggle = lmComboToggle;
|
||||
|
||||
window.lmComboKey = function (event) {
|
||||
const dd = document.getElementById('lmItemDropdown');
|
||||
if (!dd || dd.style.display === 'none') {
|
||||
if (event.key === 'ArrowDown' || event.key === 'Enter') {
|
||||
event.preventDefault();
|
||||
lmComboOpen();
|
||||
}
|
||||
return;
|
||||
}
|
||||
const opts = Array.from(dd.querySelectorAll('.lm-item-opt'));
|
||||
let idx = opts.findIndex(o => o.classList.contains('lm-active'));
|
||||
if (event.key === 'ArrowDown') {
|
||||
event.preventDefault();
|
||||
idx = Math.min(idx + 1, opts.length - 1);
|
||||
opts.forEach(o => { o.classList.remove('lm-active'); o.style.background = ''; });
|
||||
if (opts[idx]) { opts[idx].classList.add('lm-active'); opts[idx].style.background = '#e8eeff'; opts[idx].scrollIntoView({ block: 'nearest' }); }
|
||||
} else if (event.key === 'ArrowUp') {
|
||||
event.preventDefault();
|
||||
idx = Math.max(idx - 1, 0);
|
||||
opts.forEach(o => { o.classList.remove('lm-active'); o.style.background = ''; });
|
||||
if (opts[idx]) { opts[idx].classList.add('lm-active'); opts[idx].style.background = '#e8eeff'; opts[idx].scrollIntoView({ block: 'nearest' }); }
|
||||
} else if (event.key === 'Enter') {
|
||||
event.preventDefault();
|
||||
const active = dd.querySelector('.lm-active') || opts[0];
|
||||
if (active) active.dispatchEvent(new MouseEvent('mousedown'));
|
||||
} else if (event.key === 'Escape') {
|
||||
lmComboClose();
|
||||
}
|
||||
};
|
||||
|
||||
function escLm(s) {
|
||||
return String(s).replace(/&/g, '&').replace(/</g, '<').replace(/>/g, '>').replace(/"/g, '"');
|
||||
}
|
||||
|
||||
// ── Quantity / label logic ────────────────────────────────────────────────
|
||||
|
||||
function lmOnQtyInput() {
|
||||
const method = document.querySelector('input[name="lmEntryMethod"]:checked')?.value;
|
||||
if (method !== 'remaining') {
|
||||
document.getElementById('lmComputedUsed').classList.add('d-none');
|
||||
return;
|
||||
}
|
||||
const sel = document.getElementById('lmInventoryItem');
|
||||
const opt = sel.options[sel.selectedIndex];
|
||||
if (!_selectedItemId) {
|
||||
document.getElementById('lmComputedUsed').classList.add('d-none');
|
||||
return;
|
||||
}
|
||||
const item = _items.find(it => it.id === _selectedItemId);
|
||||
const onHand = item ? (parseFloat(item.quantityOnHand) || 0) : 0;
|
||||
const remaining = parseFloat(document.getElementById('lmQuantity').value) || 0;
|
||||
const onHand = parseFloat(opt?.dataset.qty) || 0;
|
||||
const used = onHand - remaining;
|
||||
const computedDiv = document.getElementById('lmComputedUsed');
|
||||
if (sel.value) {
|
||||
computedDiv.textContent = 'Usage = ' + onHand.toFixed(2) + ' − ' + remaining.toFixed(2) + ' = ' + used.toFixed(2) + ' ' + (opt?.dataset.uom || '');
|
||||
computedDiv.classList.remove('d-none');
|
||||
} else {
|
||||
computedDiv.classList.add('d-none');
|
||||
}
|
||||
computedDiv.textContent = 'Usage = ' + onHand.toFixed(2) + ' − ' + remaining.toFixed(2) + ' = ' + used.toFixed(2) + (item?.unitOfMeasure ? ' ' + item.unitOfMeasure : '');
|
||||
computedDiv.classList.remove('d-none');
|
||||
}
|
||||
|
||||
window.lmUpdateQuantityLabel = function () {
|
||||
@@ -70,9 +181,11 @@
|
||||
lmOnQtyInput();
|
||||
};
|
||||
|
||||
// ── Modal open / save ─────────────────────────────────────────────────────
|
||||
|
||||
window.openLogMaterialModal = function () {
|
||||
// Reset form
|
||||
document.getElementById('lmInventoryItem').value = '';
|
||||
_selectedItemId = 0;
|
||||
document.getElementById('lmItemSearch').value = '';
|
||||
document.getElementById('lmItemBalance').classList.add('d-none');
|
||||
document.getElementById('lmQuantity').value = '';
|
||||
document.getElementById('lmComputedUsed').classList.add('d-none');
|
||||
@@ -82,14 +195,12 @@
|
||||
document.getElementById('lmSaveBtn').disabled = false;
|
||||
document.getElementById('lmMethodUsed').checked = true;
|
||||
window.lmUpdateQuantityLabel();
|
||||
lmComboClose();
|
||||
if (_modal) _modal.show();
|
||||
};
|
||||
|
||||
window.lmSave = async function () {
|
||||
const cfg = window.__logMaterial;
|
||||
const itemId = parseInt(document.getElementById('lmInventoryItem').value) || 0;
|
||||
const qtyInput = parseFloat(document.getElementById('lmQuantity').value) || 0;
|
||||
const method = document.querySelector('input[name="lmEntryMethod"]:checked')?.value;
|
||||
const alertEl = document.getElementById('lmAlert');
|
||||
|
||||
function showError(msg) {
|
||||
@@ -98,13 +209,16 @@
|
||||
alertEl.classList.remove('d-none');
|
||||
}
|
||||
|
||||
if (!itemId) { showError('Please select an inventory item.'); return; }
|
||||
if (!_selectedItemId) { showError('Please select an inventory item.'); return; }
|
||||
|
||||
const qtyInput = parseFloat(document.getElementById('lmQuantity').value) || 0;
|
||||
if (qtyInput <= 0) { showError('Please enter a quantity greater than zero.'); return; }
|
||||
|
||||
const method = document.querySelector('input[name="lmEntryMethod"]:checked')?.value;
|
||||
let quantityUsed = qtyInput;
|
||||
if (method === 'remaining') {
|
||||
const sel = document.getElementById('lmInventoryItem');
|
||||
const onHand = parseFloat(sel.options[sel.selectedIndex]?.dataset.qty) || 0;
|
||||
const item = _items.find(it => it.id === _selectedItemId);
|
||||
const onHand = item ? (parseFloat(item.quantityOnHand) || 0) : 0;
|
||||
quantityUsed = onHand - qtyInput;
|
||||
if (quantityUsed <= 0) {
|
||||
showError('Remaining quantity cannot be equal to or greater than the current stock (' + onHand.toFixed(2) + ').');
|
||||
@@ -125,7 +239,7 @@
|
||||
},
|
||||
body: JSON.stringify({
|
||||
jobId: cfg.jobId,
|
||||
inventoryItemId: itemId,
|
||||
inventoryItemId: _selectedItemId,
|
||||
quantityUsed: quantityUsed,
|
||||
transactionType: document.getElementById('lmTransactionType').value,
|
||||
notes: document.getElementById('lmNotes').value.trim() || null
|
||||
@@ -134,7 +248,6 @@
|
||||
const data = await resp.json();
|
||||
if (data.success) {
|
||||
if (_modal) _modal.hide();
|
||||
// Reload page so the materials table refreshes
|
||||
window.location.reload();
|
||||
} else {
|
||||
showError(data.message || 'An error occurred.');
|
||||
@@ -146,5 +259,27 @@
|
||||
}
|
||||
};
|
||||
|
||||
// ── Init ──────────────────────────────────────────────────────────────────
|
||||
|
||||
function init() {
|
||||
const cfg = window.__logMaterial;
|
||||
if (!cfg) return;
|
||||
|
||||
_items = cfg.inventoryItems || [];
|
||||
_jobPowderIds = new Set(cfg.jobPowderIds || []);
|
||||
_modal = new bootstrap.Modal(document.getElementById('logMaterialModal'));
|
||||
|
||||
document.getElementById('lmQuantity').addEventListener('input', lmOnQtyInput);
|
||||
|
||||
// Close dropdown when clicking outside
|
||||
document.addEventListener('click', function (e) {
|
||||
if (!e.target.closest('#lmItemSearch') &&
|
||||
!e.target.closest('#lmItemDropdown') &&
|
||||
!e.target.closest('#lmItemDropdownToggle')) {
|
||||
lmComboClose();
|
||||
}
|
||||
});
|
||||
}
|
||||
|
||||
document.addEventListener('DOMContentLoaded', init);
|
||||
})();
|
||||
|
||||
@@ -0,0 +1,284 @@
|
||||
using Microsoft.AspNetCore.Http;
|
||||
using Microsoft.AspNetCore.Identity;
|
||||
using Microsoft.AspNetCore.Mvc;
|
||||
using Microsoft.EntityFrameworkCore;
|
||||
using Microsoft.Extensions.Logging;
|
||||
using Moq;
|
||||
using PowderCoating.Core.Entities;
|
||||
using PowderCoating.Core.Enums;
|
||||
using PowderCoating.Core.Interfaces;
|
||||
using PowderCoating.Infrastructure.Data;
|
||||
using PowderCoating.Infrastructure.Repositories;
|
||||
using PowderCoating.Web.Controllers;
|
||||
|
||||
namespace PowderCoating.UnitTests;
|
||||
|
||||
/// <summary>
|
||||
/// Verifies that the explicit <c>CompanyId == companyId</c> predicates added to every
|
||||
/// user-facing controller action actually prevent cross-tenant data leakage.
|
||||
///
|
||||
/// Each test seeds entities for TWO companies, creates a controller whose ITenantContext
|
||||
/// returns Company 1's ID, calls the action, and asserts that Company 2's data never
|
||||
/// appears in the result.
|
||||
///
|
||||
/// These tests validate the defense-in-depth layer (explicit predicates in controllers)
|
||||
/// independently of the EF Core global query filters, which behave differently on the
|
||||
/// in-memory provider when no HttpContext is present.
|
||||
/// </summary>
|
||||
public class MultiTenantIsolationTests
|
||||
{
|
||||
// ── Repository-level isolation ────────────────────────────────────────────
|
||||
|
||||
/// <summary>
|
||||
/// FindAsync with an explicit CompanyId predicate returns only the matching company's rows,
|
||||
/// even when rows from other companies exist in the database.
|
||||
/// </summary>
|
||||
[Fact]
|
||||
public async Task Repository_FindAsync_WithCompanyIdPredicate_ExcludesOtherTenants()
|
||||
{
|
||||
await using var context = CreateContext();
|
||||
context.Customers.Add(MakeCustomer(id: 1, companyId: 1, firstName: "Alice"));
|
||||
context.Customers.Add(MakeCustomer(id: 2, companyId: 2, firstName: "Bob"));
|
||||
context.Customers.Add(MakeCustomer(id: 3, companyId: 1, firstName: "Carol"));
|
||||
await context.SaveChangesAsync();
|
||||
|
||||
var uow = new UnitOfWork(context);
|
||||
var results = (await uow.Customers.FindAsync(c => c.CompanyId == 1)).ToList();
|
||||
|
||||
Assert.Equal(2, results.Count);
|
||||
Assert.All(results, c => Assert.Equal(1, c.CompanyId));
|
||||
Assert.DoesNotContain(results, c => c.ContactFirstName == "Bob");
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public async Task Repository_FindAsync_ReturnsEmpty_WhenNoMatchingCompanyId()
|
||||
{
|
||||
await using var context = CreateContext();
|
||||
context.Customers.Add(MakeCustomer(id: 1, companyId: 2, firstName: "Bob"));
|
||||
await context.SaveChangesAsync();
|
||||
|
||||
var uow = new UnitOfWork(context);
|
||||
var results = await uow.Customers.FindAsync(c => c.CompanyId == 1);
|
||||
|
||||
Assert.Empty(results);
|
||||
}
|
||||
|
||||
// ── SmsConsentAuditController ─────────────────────────────────────────────
|
||||
|
||||
[Fact]
|
||||
public async Task SmsConsentAudit_Index_ReturnsOnlyCurrentCompanyCustomers()
|
||||
{
|
||||
await using var context = CreateContext();
|
||||
context.Customers.Add(MakeCustomer(id: 1, companyId: 1, firstName: "Alice"));
|
||||
context.Customers.Add(MakeCustomer(id: 2, companyId: 2, firstName: "Bob")); // other company
|
||||
context.Customers.Add(MakeCustomer(id: 3, companyId: 1, firstName: "Carol"));
|
||||
await context.SaveChangesAsync();
|
||||
|
||||
var controller = new SmsConsentAuditController(
|
||||
new UnitOfWork(context),
|
||||
MockTenant(companyId: 1),
|
||||
Mock.Of<ILogger<SmsConsentAuditController>>());
|
||||
SetHttpContext(controller);
|
||||
|
||||
var result = await controller.Index();
|
||||
|
||||
var view = Assert.IsType<ViewResult>(result);
|
||||
var vm = Assert.IsType<SmsConsentAuditViewModel>(view.Model);
|
||||
Assert.Equal(2, vm.TotalCount);
|
||||
Assert.DoesNotContain(vm.Rows, r => r.CustomerName.Contains("Bob"));
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public async Task SmsConsentAudit_ExportCsv_ContainsOnlyCurrentCompanyCustomers()
|
||||
{
|
||||
await using var context = CreateContext();
|
||||
context.Customers.Add(MakeCustomer(id: 1, companyId: 1, firstName: "Alice"));
|
||||
context.Customers.Add(MakeCustomer(id: 2, companyId: 2, firstName: "Bob"));
|
||||
await context.SaveChangesAsync();
|
||||
|
||||
var controller = new SmsConsentAuditController(
|
||||
new UnitOfWork(context),
|
||||
MockTenant(companyId: 1),
|
||||
Mock.Of<ILogger<SmsConsentAuditController>>());
|
||||
SetHttpContext(controller);
|
||||
|
||||
var result = await controller.ExportCsv();
|
||||
|
||||
var file = Assert.IsType<FileContentResult>(result);
|
||||
var csv = System.Text.Encoding.UTF8.GetString(file.FileContents);
|
||||
Assert.Contains("Alice", csv);
|
||||
Assert.DoesNotContain("Bob", csv);
|
||||
}
|
||||
|
||||
// ── TaxRatesController ────────────────────────────────────────────────────
|
||||
|
||||
[Fact]
|
||||
public async Task TaxRates_Index_ReturnsOnlyCurrentCompanyRates()
|
||||
{
|
||||
await using var context = CreateContext();
|
||||
context.TaxRates.Add(MakeTaxRate(id: 1, companyId: 1, name: "State Tax"));
|
||||
context.TaxRates.Add(MakeTaxRate(id: 2, companyId: 2, name: "Foreign Tax")); // other company
|
||||
context.TaxRates.Add(MakeTaxRate(id: 3, companyId: 1, name: "Local Tax"));
|
||||
await context.SaveChangesAsync();
|
||||
|
||||
var controller = new TaxRatesController(
|
||||
new UnitOfWork(context),
|
||||
MockTenant(companyId: 1),
|
||||
Mock.Of<ILogger<TaxRatesController>>());
|
||||
SetHttpContext(controller);
|
||||
|
||||
var result = await controller.Index();
|
||||
|
||||
var view = Assert.IsType<ViewResult>(result);
|
||||
var rates = Assert.IsAssignableFrom<IEnumerable<TaxRate>>(view.Model).ToList();
|
||||
Assert.Equal(2, rates.Count);
|
||||
Assert.All(rates, r => Assert.Equal(1, r.CompanyId));
|
||||
Assert.DoesNotContain(rates, r => r.Name == "Foreign Tax");
|
||||
}
|
||||
|
||||
// ── RecurringTemplatesController ──────────────────────────────────────────
|
||||
|
||||
[Fact]
|
||||
public async Task RecurringTemplates_Index_ReturnsOnlyCurrentCompanyTemplates()
|
||||
{
|
||||
await using var context = CreateContext();
|
||||
context.RecurringTemplates.Add(MakeRecurringTemplate(id: 1, companyId: 1, name: "Monthly Rent"));
|
||||
context.RecurringTemplates.Add(MakeRecurringTemplate(id: 2, companyId: 2, name: "Other Tenant Bill")); // other company
|
||||
await context.SaveChangesAsync();
|
||||
|
||||
var controller = new RecurringTemplatesController(
|
||||
new UnitOfWork(context),
|
||||
MockTenant(companyId: 1),
|
||||
CreateUserManagerMock().Object,
|
||||
Mock.Of<ILogger<RecurringTemplatesController>>());
|
||||
SetHttpContext(controller);
|
||||
|
||||
var result = await controller.Index();
|
||||
|
||||
var view = Assert.IsType<ViewResult>(result);
|
||||
var templates = Assert.IsAssignableFrom<IEnumerable<RecurringTemplate>>(view.Model).ToList();
|
||||
Assert.Single(templates);
|
||||
Assert.Equal("Monthly Rent", templates[0].Name);
|
||||
}
|
||||
|
||||
// ── JobTemplatesController ────────────────────────────────────────────────
|
||||
|
||||
[Fact]
|
||||
public async Task JobTemplates_Index_ReturnsOnlyCurrentCompanyTemplates()
|
||||
{
|
||||
await using var context = CreateContext();
|
||||
context.JobTemplates.Add(MakeJobTemplate(id: 1, companyId: 1, name: "Standard Wheel Coat"));
|
||||
context.JobTemplates.Add(MakeJobTemplate(id: 2, companyId: 2, name: "Other Company Template")); // other company
|
||||
await context.SaveChangesAsync();
|
||||
|
||||
var controller = new JobTemplatesController(
|
||||
new UnitOfWork(context),
|
||||
MockTenant(companyId: 1));
|
||||
SetHttpContext(controller);
|
||||
|
||||
var result = await controller.Index();
|
||||
|
||||
var view = Assert.IsType<ViewResult>(result);
|
||||
var templates = Assert.IsAssignableFrom<IEnumerable<JobTemplate>>(view.Model).ToList();
|
||||
Assert.Single(templates);
|
||||
Assert.Equal("Standard Wheel Coat", templates[0].Name);
|
||||
}
|
||||
|
||||
// ── Cross-tenant write protection ─────────────────────────────────────────
|
||||
|
||||
/// <summary>
|
||||
/// Verifies that the companyId-scoped FindAsync used for SMS export returns zero
|
||||
/// rows for a company that has no customers, even when another company has many.
|
||||
/// Guards against the "empty predicate returns all" regression.
|
||||
/// </summary>
|
||||
[Fact]
|
||||
public async Task SmsConsentAudit_ExportCsv_IsEmpty_WhenCompanyHasNoCustomers()
|
||||
{
|
||||
await using var context = CreateContext();
|
||||
context.Customers.Add(MakeCustomer(id: 1, companyId: 2, firstName: "Other"));
|
||||
context.Customers.Add(MakeCustomer(id: 2, companyId: 2, firstName: "Also Other"));
|
||||
await context.SaveChangesAsync();
|
||||
|
||||
var controller = new SmsConsentAuditController(
|
||||
new UnitOfWork(context),
|
||||
MockTenant(companyId: 1), // Company 1 has no customers
|
||||
Mock.Of<ILogger<SmsConsentAuditController>>());
|
||||
SetHttpContext(controller);
|
||||
|
||||
var result = await controller.ExportCsv();
|
||||
|
||||
var file = Assert.IsType<FileContentResult>(result);
|
||||
var csv = System.Text.Encoding.UTF8.GetString(file.FileContents);
|
||||
// Only header row, no data rows
|
||||
Assert.DoesNotContain("Other", csv);
|
||||
}
|
||||
|
||||
// ── Helpers ───────────────────────────────────────────────────────────────
|
||||
|
||||
private static ApplicationDbContext CreateContext()
|
||||
{
|
||||
var options = new DbContextOptionsBuilder<ApplicationDbContext>()
|
||||
.UseInMemoryDatabase(Guid.NewGuid().ToString())
|
||||
.Options;
|
||||
return new ApplicationDbContext(options);
|
||||
}
|
||||
|
||||
/// <summary>Returns a mock ITenantContext that always yields the given companyId.</summary>
|
||||
private static ITenantContext MockTenant(int companyId)
|
||||
{
|
||||
var mock = new Mock<ITenantContext>();
|
||||
mock.Setup(t => t.GetCurrentCompanyId()).Returns(companyId);
|
||||
return mock.Object;
|
||||
}
|
||||
|
||||
private static void SetHttpContext(Controller controller)
|
||||
{
|
||||
controller.ControllerContext = new ControllerContext
|
||||
{
|
||||
HttpContext = new DefaultHttpContext()
|
||||
};
|
||||
}
|
||||
|
||||
private static Customer MakeCustomer(int id, int companyId, string firstName) => new()
|
||||
{
|
||||
Id = id,
|
||||
CompanyId = companyId,
|
||||
ContactFirstName = firstName,
|
||||
ContactLastName = "Test",
|
||||
IsCommercial = false
|
||||
};
|
||||
|
||||
private static TaxRate MakeTaxRate(int id, int companyId, string name) => new()
|
||||
{
|
||||
Id = id,
|
||||
CompanyId = companyId,
|
||||
Name = name,
|
||||
Rate = 8.5m
|
||||
};
|
||||
|
||||
private static RecurringTemplate MakeRecurringTemplate(int id, int companyId, string name) => new()
|
||||
{
|
||||
Id = id,
|
||||
CompanyId = companyId,
|
||||
Name = name,
|
||||
TemplateType = RecurringTemplateType.Bill,
|
||||
Frequency = RecurringFrequency.Monthly,
|
||||
IntervalCount = 1,
|
||||
NextFireDate = DateTime.Today,
|
||||
IsActive = true
|
||||
};
|
||||
|
||||
private static JobTemplate MakeJobTemplate(int id, int companyId, string name) => new()
|
||||
{
|
||||
Id = id,
|
||||
CompanyId = companyId,
|
||||
Name = name
|
||||
};
|
||||
|
||||
private static Mock<UserManager<ApplicationUser>> CreateUserManagerMock()
|
||||
{
|
||||
var store = new Mock<IUserStore<ApplicationUser>>();
|
||||
return new Mock<UserManager<ApplicationUser>>(
|
||||
store.Object, null!, null!, null!, null!, null!, null!, null!, null!);
|
||||
}
|
||||
}
|
||||
Reference in New Issue
Block a user