Исправления

This commit is contained in:
Данила Мочалов 2023-03-10 19:13:41 +04:00
parent 62af0c7ec5
commit 49152e95c5
6 changed files with 21 additions and 22 deletions

View File

@ -103,7 +103,6 @@ namespace LawFirmView
var operationResult = _logicO.CreateOrder(new OrderBindingModel
{
DocumentId = Convert.ToInt32(comboBoxDocument.SelectedValue),
DocumentName = comboBoxDocument.Text,
Count = Convert.ToInt32(textBoxCount.Text),
Sum = Convert.ToDouble(textBoxSum.Text)
});

View File

@ -59,7 +59,6 @@ namespace LawFirmBusinessLogic.BusinessLogics
{
Id = viewModel.Id,
DocumentId = viewModel.DocumentId,
DocumentName = viewModel.DocumentName,
Status = viewModel.Status,
DateCreate = viewModel.DateCreate,
DateImplement = viewModel.DateImplement,
@ -67,7 +66,7 @@ namespace LawFirmBusinessLogic.BusinessLogics
Sum = viewModel.Sum
};
CheckModel(model);
CheckModel(model, false);
if (model.Status + 1 != newStatus)
{
_logger.LogWarning("Status update to " + newStatus.ToString() +" operation failed. Order status incorrect.");

View File

@ -12,8 +12,6 @@ namespace LawFirmContracts.BindingModels
{
public int DocumentId { get; set; }
public string DocumentName { get; set; } = string.Empty;
public int Count { get; set; }
public double Sum { get; set; }

View File

@ -10,7 +10,6 @@ namespace LawFirmDataModels.Models
public interface IOrderModel : IId
{
int DocumentId { get; }
string DocumentName { get; }
int Count { get; }
double Sum { get; }
OrderStatus Status { get; }

View File

@ -30,7 +30,7 @@ namespace LawFirmListImplements.Implements
{
if (model.Id.HasValue && order.Id == model.Id)
{
return order.GetViewModel;
return GetViewModel(order);
}
}
return null;
@ -47,7 +47,7 @@ namespace LawFirmListImplements.Implements
{
if (order.Id == model.Id)
{
result.Add(order.GetViewModel);
result.Add(GetViewModel(order));
}
}
return result;
@ -58,7 +58,7 @@ namespace LawFirmListImplements.Implements
var result = new List<OrderViewModel>();
foreach (var order in _source.Orders)
{
result.Add(order.GetViewModel);
result.Add(GetViewModel(order));
}
return result;
}
@ -79,7 +79,7 @@ namespace LawFirmListImplements.Implements
return null;
}
_source.Orders.Add(newOrder);
return newOrder.GetViewModel;
return GetViewModel(newOrder);
}
public OrderViewModel? Update(OrderBindingModel model)
@ -89,7 +89,7 @@ namespace LawFirmListImplements.Implements
if (order.Id == model.Id)
{
order.Update(model);
return order.GetViewModel;
return GetViewModel(order);
}
}
return null;
@ -102,11 +102,25 @@ namespace LawFirmListImplements.Implements
{
var element = _source.Orders[i];
_source.Orders.RemoveAt(i);
return element.GetViewModel;
return GetViewModel(element);
}
}
return null;
}
private OrderViewModel GetViewModel(Order order)
{
var viewModel = order.GetViewModel;
foreach (var document in _source.Documents)
{
if (document.Id == order.DocumentId)
{
viewModel.DocumentName = document.DocumentName;
break;
}
}
return viewModel;
}
}
}

View File

@ -14,8 +14,6 @@ namespace LawFirmListImplements.Models
{
public int DocumentId { get; private set; }
public string DocumentName { get; private set; } = string.Empty;
public int Count { get; private set; }
public double Sum { get; private set; }
@ -37,7 +35,6 @@ namespace LawFirmListImplements.Models
return new Order
{
DocumentId = model.DocumentId,
DocumentName = model.DocumentName,
Count = model.Count,
Sum = model.Sum,
Status = model.Status,
@ -53,20 +50,13 @@ namespace LawFirmListImplements.Models
{
return;
}
DocumentId = model.DocumentId;
DocumentName = model.DocumentName;
Count = model.Count;
Sum = model.Sum;
Status = model.Status;
DateCreate = model.DateCreate;
DateImplement = model.DateImplement;
Id = model.Id;
}
public OrderViewModel GetViewModel => new()
{
DocumentId = DocumentId,
DocumentName = DocumentName,
Count = Count,
Sum = Sum,
DateCreate = DateCreate,