Fixup areas expecting VehicleId
This commit is contained in:
@@ -63,6 +63,7 @@ namespace MileageTraker.Web.Controllers
|
||||
throw new SecurityException();
|
||||
viewModel.SetProperties(log);
|
||||
|
||||
log.Vehicle = DataService.GetVehicle(viewModel.VehicleId);
|
||||
if (viewModel.Purpose != null)
|
||||
log.Purpose =
|
||||
DataService.GetPurposeTypes()
|
||||
@@ -107,6 +108,7 @@ namespace MileageTraker.Web.Controllers
|
||||
|
||||
// Save the new log
|
||||
var log = model.GetLog();
|
||||
log.Vehicle = DataService.GetVehicle(model.VehicleId);
|
||||
log.User = DataService.FindUserByUsername(User.Identity.Name);
|
||||
log.Source = HttpContext.Request.Url.AbsolutePath;
|
||||
log.UserHostAddress = HttpContext.Request.UserHostAddress;
|
||||
|
||||
@@ -148,6 +148,7 @@ namespace MileageTraker.Web.Controllers
|
||||
if (ModelState.IsValid)
|
||||
{
|
||||
var log = viewModel.GetLog();
|
||||
log.Vehicle = DataService.GetVehicle(viewModel.VehicleId);
|
||||
log.User = DataService.FindUserByFullName(viewModel.UserFullName);
|
||||
log.Source = HttpContext.Request.Url.AbsolutePath;
|
||||
log.UserHostAddress = HttpContext.Request.UserHostAddress;
|
||||
@@ -201,6 +202,7 @@ namespace MileageTraker.Web.Controllers
|
||||
var log = DataService.GetLog(viewModel.LogId);
|
||||
viewModel.SetProperties(log);
|
||||
log.User = DataService.FindUserByFullName(viewModel.UserFullName);
|
||||
log.Vehicle = DataService.GetVehicle(viewModel.VehicleId);
|
||||
|
||||
if (viewModel.Purpose != null)
|
||||
log.Purpose =
|
||||
|
||||
@@ -106,10 +106,11 @@ namespace MileageTraker.Web.DAL
|
||||
fuelLog.Log = null;
|
||||
}
|
||||
|
||||
var vehicle = log.Vehicle; // save reference to vehicle
|
||||
_db.Logs.Remove(log);
|
||||
_db.SaveChanges();
|
||||
|
||||
UpdateCurrentOdometer(log.Vehicle);
|
||||
UpdateCurrentOdometer(vehicle);
|
||||
}
|
||||
|
||||
public IQueryable<Log> GetLogs()
|
||||
@@ -285,7 +286,7 @@ namespace MileageTraker.Web.DAL
|
||||
where
|
||||
dbLog.Date == log.Date &&
|
||||
dbLog.EndOdometer == log.EndOdometer &&
|
||||
dbLog.Vehicle == log.Vehicle
|
||||
dbLog.Vehicle.VehicleId == log.Vehicle.VehicleId
|
||||
select dbLog;
|
||||
}
|
||||
|
||||
@@ -442,7 +443,7 @@ namespace MileageTraker.Web.DAL
|
||||
var mostRecentOdometerQuery =
|
||||
from log in _db.Logs
|
||||
where
|
||||
log.Vehicle == vehicle
|
||||
log.Vehicle.VehicleId == vehicle.VehicleId
|
||||
orderby log.Date descending, log.EndOdometer descending
|
||||
select log;
|
||||
var mostRecentOdometer = mostRecentOdometerQuery.FirstOrDefault();
|
||||
|
||||
@@ -81,6 +81,7 @@ namespace MileageTraker.Web.ViewModels.CreateLog
|
||||
Mapper.CreateMap<EditPastViewModel, Models.Log>()
|
||||
.ForMember(u => u.Purpose, opt => opt.Ignore());
|
||||
Mapper.CreateMap<Models.Log, EditPastViewModel>()
|
||||
.ForMember(dest => dest.VehicleId, opt => opt.MapFrom(src => src.Vehicle.VehicleId))
|
||||
.ForMember(vm => vm.Date, opt => opt.MapFrom(m => m.Date.ToString("d")))
|
||||
.ForMember(u => u.Purpose, opt => opt.Ignore())
|
||||
.ForMember(vm => vm.GasPurchased, opt => opt.ResolveUsing(gasFormatter));
|
||||
|
||||
@@ -51,7 +51,8 @@ namespace MileageTraker.Web.ViewModels.FuelLog
|
||||
|
||||
static LogMatchViewModel()
|
||||
{
|
||||
Mapper.CreateMap<Models.Log, LogMatchViewModel>();
|
||||
Mapper.CreateMap<Models.Log, LogMatchViewModel>()
|
||||
.ForMember(dest => dest.VehicleId, opt => opt.MapFrom(src => src.Vehicle.VehicleId));
|
||||
}
|
||||
|
||||
public LogMatchViewModel(Models.Log log, Models.Vehicle vehicle, FuelLogViewModel fuelLog)
|
||||
|
||||
@@ -46,7 +46,8 @@ namespace MileageTraker.Web.ViewModels.Log
|
||||
|
||||
static LogIndexViewModel()
|
||||
{
|
||||
Mapper.CreateMap<Models.Log, LogIndexViewModel>();
|
||||
Mapper.CreateMap<Models.Log, LogIndexViewModel>()
|
||||
.ForMember(dest => dest.VehicleId, opt => opt.MapFrom(src => src.Vehicle.VehicleId));
|
||||
}
|
||||
|
||||
public LogIndexViewModel(Models.Log log, Models.Log previousLog)
|
||||
|
||||
@@ -38,7 +38,8 @@ namespace MileageTraker.Web.ViewModels.Log
|
||||
|
||||
static LogPartialDetails()
|
||||
{
|
||||
Mapper.CreateMap<Models.Log, LogPartialDetails>();
|
||||
Mapper.CreateMap<Models.Log, LogPartialDetails>()
|
||||
.ForMember(dest => dest.VehicleId, opt => opt.MapFrom(src => src.Vehicle.VehicleId));
|
||||
}
|
||||
|
||||
public LogPartialDetails()
|
||||
|
||||
@@ -85,6 +85,7 @@ namespace MileageTraker.Web.ViewModels.Log
|
||||
Mapper.CreateMap<LogViewModel, Models.Log>()
|
||||
.ForMember(u => u.Purpose, opt => opt.Ignore());
|
||||
Mapper.CreateMap<Models.Log, LogViewModel>()
|
||||
.ForMember(dest => dest.VehicleId, opt => opt.MapFrom(src => src.Vehicle.VehicleId))
|
||||
.ForMember(vm => vm.Date, opt => opt.MapFrom(m => m.Date.ToString("d")))
|
||||
.ForMember(u => u.Purpose, opt => opt.Ignore());
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user