Add required city field

Web/Controllers/CityController.cs

@@ -1,6 +1,7 @@
 using System;
 using System.Linq;
 using System.Web.Mvc;
+using MileageTraker.Web.Attributes;
 using MileageTraker.Web.Models;
 using MileageTraker.Web.Utility;
 
@@ -8,13 +9,13 @@ namespace MileageTraker.Web.Controllers
 {
 	[Authorize(Roles = "Administrator, Developer")]
 	public class CityController : ControllerBase
-    {
+	{
 		[Authorize(Roles = "Administrator, Developer, Driver")]
 		public JsonResult Autocomplete(string term)
-        {
+		{
 			var cities = DataService.GetCitiesAutocomplete(term);
-        	return Json(cities, JsonRequestBehavior.AllowGet);
+			return Json(cities, JsonRequestBehavior.AllowGet);
-        }
+		}
 
 		public ActionResult Index()
 		{
@@ -26,46 +27,36 @@ namespace MileageTraker.Web.Controllers
 
 		public ActionResult Create(string county)
 		{
-			var city = new City { County = county };
+			var city = new City {County = county};
 			return View(city);
 		}
 
-		//[HttpPost]
+		[ActionLog]
-		//public ActionResult Create([Bind(Exclude = "CityId")]City city)
+		[HttpPost]
-		//{
+		public ActionResult Create([Bind(Exclude = "CityId")] City city)
-		//	if (ModelState.IsValid)
+		{
-		//	{
+			if (ModelState.IsValid)
-		//		DataService.AddCity(city);
+			{
+				DataService.AddCity(city);
 
-		//		TempData["StatusMessage"] = "City " + city.Name + " created";
+				TempData["StatusMessage"] = "City " + city.Name + " created";
-		//		return RedirectToAction("Index");
+				return RedirectToAction("Index");
-		//	}
+			}
 
-		//	return View(city);
+			return View(city);
-		//}
+		}
 
-		//public ActionResult Edit(int id = 0)
+		public JsonResult CityNameAvailable(string name)
-		//{
+		{
-		//	var city = DataService.FindCity(id);
+			var city = DataService.FindCityByName(name);
-		//	if (city == null)
+			return Json(city == null, JsonRequestBehavior.AllowGet);
-		//	{
+		}
-		//		TempData["StatusMessage"] = "Purpose " + id + " not found";
-		//		return RedirectToAction("Index");
-		//	}
-		//	return View(city);
-		//}
 
-		//[HttpPost]
+		[Authorize(Roles = "Administrator, Developer, Driver")]
-		//public ActionResult Edit(City city)
+		public JsonResult CityNameExists(string cityname)
-		//{
+		{
-		//	if (ModelState.IsValid)
+			var city = DataService.FindCityByName(cityname);
-		//	{
+			return Json(city != null, JsonRequestBehavior.AllowGet);
-		//		DataService.UpdateCity(city);
+		}
-
+	}
-		//		TempData["StatusMessage"] = "Changes to Purpose  " + city.Purpose + " saved";
-		//		return RedirectToAction("Index");
-		//	}
-		//	return View(city);
-		//}
-    }
 }

Web/Controllers/CreateLogController.cs

@@ -270,6 +270,16 @@ namespace MileageTraker.Web.Controllers
 						Action = GetFixLink(viewModel)
 					}, JsonRequestBehavior.AllowGet);
 				}
+				// verify city exists
+				if (DataService.FindCityByName(log.CityName) == null)
+				{
+					return Json(new
+					{
+						Status = ImportStatus.Failure.ToString(),
+						Message = "City must exist in system",
+						Action = GetFixLink(viewModel)
+					}, JsonRequestBehavior.AllowGet);
+				}
 
 				// SAVE IT!
 				DataService.AddLog(log);

Web/Controllers/LogController.cs

@@ -350,6 +350,16 @@ namespace MileageTraker.Web.Controllers
 						Action = GetFixLink(viewModel)
 					}, JsonRequestBehavior.AllowGet);
 				}
+				// verify city exists
+				if (DataService.FindCityByName(log.CityName) == null)
+				{
+					return Json(new
+					{
+						Status = ImportStatus.Failure.ToString(),
+						Message = "City must exist in system",
+						Action = GetFixLink(viewModel)
+					}, JsonRequestBehavior.AllowGet);
+				}
 				
 				// SAVE IT!
 				DataService.AddLog(log);

Web/DAL/DataService.cs

@@ -492,6 +492,17 @@ namespace MileageTraker.Web.DAL
 			return _db.Cities;
 		}
 
+		public City FindCityByName(string name)
+		{
+			return _db.Cities.FirstOrDefault(u => name.Equals(u.Name, StringComparison.InvariantCultureIgnoreCase));
+		}
+
+		public void AddCity(City city)
+		{
+			_db.Cities.Add(city);
+			_db.SaveChanges();
+		}
+
 		#endregion
 
 		#region Membership
@@ -595,5 +606,6 @@ namespace MileageTraker.Web.DAL
 		}
 
 		#endregion
+
 	}
 }

Web/Models/City.cs

@@ -12,6 +12,7 @@ namespace MileageTraker.Web.Models
 		[Display(Name = "City Name")]
 		[InputSize("medium")]
 		[Required]
+		[Remote("CityNameAvailable", "City", ErrorMessage = "City name already in use")]
 		public string Name { get; set; }
 		[Required]
 		public string County { get; set; }

Web/ViewModels/CreateLog/CreateLogViewModel.cs

@@ -37,6 +37,7 @@ namespace MileageTraker.Web.ViewModels.CreateLog
 		[Display(Name = "Destination City")]
 		[InputSize("medium")]
 		[StringLength(64, MinimumLength = 3, ErrorMessage = "Minimum 3 characters")]
+		[Remote("CityNameExists", "City", ErrorMessage = "City must exist in system")]
 		public string CityName { get; set; }
 
 		[Display(Name = "Purpose")]

Web/ViewModels/CreateLog/EditPastViewModel.cs

@@ -39,6 +39,7 @@ namespace MileageTraker.Web.ViewModels.CreateLog
 		[Display(Name = "Destination City")]
 		[InputSize("medium")]
 		[StringLength(64, MinimumLength = 3, ErrorMessage = "Minimum 3 characters")]
+		[Remote("CityNameExists", "City", ErrorMessage = "City must exist in system")]
 		public string CityName { get; set; }
 
 		[Display(Name = "Purpose")]

Web/ViewModels/Log/LogViewModel.cs

@@ -47,6 +47,7 @@ namespace MileageTraker.Web.ViewModels.Log
 		[Display(Name = "City Name")]
 		[InputSize("medium")]
 		[StringLength(64, MinimumLength = 3, ErrorMessage = "Minimum 3 characters")]
+		[Remote("CityNameExists", "City", ErrorMessage = "City must exist in system")]
 		public string CityName { get; set; }
 
 		[Display(Name = "Purpose")]

Web/Views/City/Index.cshtml

@@ -13,13 +13,15 @@
 	{
 		<div>
 			<h3>@countyCities.Key</h3>
+			<p>
 			@foreach (var city in countyCities.Value)
 			{
 				<span class="label">
 					@city
 				</span>
 			}
-			@Html.ActionLink("Add New City", "Create", new {County=countyCities.Key}, new { @class = "btn" })
+			@Html.ActionLink("Add New City", "Create", new {County=countyCities.Key}, new { @class = "btn btn-small" })
+			</p>
 		</div>
 	}
 </div>

Web/Views/Shared/_Layout.cshtml

@@ -46,7 +46,7 @@
         @RenderBody()
     </div>
     <footer>
-         Mileage Traker &copy; 2013 James Kolpack
+         Mileage Traker &copy; 2014 Kolpack Software Consulting LLC
     </footer>
     
 	<script src="@Url.Content("~/Scripts/jquery-2.1.0.min.js")" type="text/javascript"></script>