d.agliullov/ISEbd-21_Agliullov.D.A._Confectionery
2
1
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Agliullov D. A. Lab work 1 Base #2

Closed
d.agliullov wants to merge 14 commits from Lab1_Base into main
pull from: Lab1_Base
merge into: d.agliullov:main
Conversation 0 Commits 14 Files Changed 62 +7 -4
1 changed files with 7 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 47cf1ab393 - Show all commits

11
ConfectionaryBusinessLogic/OrderLogic.cs
View File

@ -41,10 +41,8 @@ namespace ConfectioneryBusinessLogic.BusinessLogics
public bool DeliveryOrder(OrderBindingModel model) => SetOrderStatus(model, OrderStatus.Выдан);
public bool FinishOrder(OrderBindingModel model)
{
var res = SetOrderStatus(model, OrderStatus.Готов);
if (res)
model.DateImplement = DateTime.Now;
return res;
model.DateImplement = DateTime.Now;
return SetOrderStatus(model, OrderStatus.Готов);
}
public List<OrderViewModel>? ReadList(OrderSearchModel? model)
@ -99,6 +97,11 @@ namespace ConfectioneryBusinessLogic.BusinessLogics
nameof(vmodel));
}
model.Status = orderStatus;
model.DateCreate = vmodel.DateCreate;
eegov commented 2023-02-11 18:12:37 +04:00
Review
Copy Link

Требуется сменить только статус, зачем все поля передавать?

Требуется сменить только статус, зачем все поля передавать?
model.DateImplement = vmodel.DateImplement;
model.PastryId = vmodel.PastryId;
model.Sum = vmodel.Sum;
model.Count= vmodel.Count;
if (_orderStorage.Update(model) == null)
{
_logger.LogWarning("Update operation failed");