Option to show/hide District, City, PostalCode (#399)

tdavisjr · thiennn · commit 1672906297d4 · 2018-02-16T17:08:55.000+07:00
* Option to show/hide District, City, PostalCode when input address information
diff --git a/src/Modules/SimplCommerce.Module.Core/Controllers/CountryApiController.cs b/src/Modules/SimplCommerce.Module.Core/Controllers/CountryApiController.cs
@@ -60,7 +60,10 @@ public IActionResult List([FromBody] SmartTableParam param)
                     c.Id,
                     c.Name,
                     c.IsShippingEnabled,
-                    c.IsBillingEnabled
+                    c.IsBillingEnabled,
+                    c.IsCityEnabled,
+                    c.IsPostalCodeEnabled,
+                    c.IsDistrictEnabled
                 });
 
             return Json(countries);
@@ -83,6 +86,9 @@ public async Task<IActionResult> Get(long id)
                 Code3 = country.Code3,
                 IsBillingEnabled = country.IsBillingEnabled,
                 IsShippingEnabled = country.IsShippingEnabled,
+                IsCityEnabled = country.IsCityEnabled,
+                IsPostalCodeEnabled = country.IsPostalCodeEnabled,
+                IsDistrictEnabled = country.IsDistrictEnabled
             };
 
             return Json(model);
@@ -108,6 +114,9 @@ public async Task<IActionResult> Put(long id, [FromBody] CountryForm model)
             country.Code3 = model.Code3;
             country.IsShippingEnabled = model.IsShippingEnabled;
             country.IsBillingEnabled = model.IsBillingEnabled;
+            country.IsCityEnabled = model.IsCityEnabled;
+            country.IsPostalCodeEnabled = model.IsPostalCodeEnabled;
+            country.IsDistrictEnabled = model.IsDistrictEnabled;
 
             await _countryRepository.SaveChangesAsync();
 
@@ -126,8 +135,12 @@ public async Task<IActionResult> Post([FromBody] CountryForm model)
                     Code2 = model.Code2,
                     Code3 = model.Code3,
                     IsBillingEnabled = model.IsBillingEnabled,
-                    IsShippingEnabled = model.IsShippingEnabled
-                };
+                    IsShippingEnabled = model.IsShippingEnabled,
+                    IsCityEnabled = model.IsCityEnabled,
+                    IsPostalCodeEnabled = model.IsPostalCodeEnabled,
+                    IsDistrictEnabled = model.IsDistrictEnabled
+
+            };
 
                 _countryRepository.Add(country);
                 await _countryRepository.SaveChangesAsync();
diff --git a/src/Modules/SimplCommerce.Module.Core/Controllers/UserAddressController.cs b/src/Modules/SimplCommerce.Module.Core/Controllers/UserAddressController.cs
@@ -49,7 +49,10 @@ public async Task<IActionResult> List()
                     AddressLine2 = x.Address.AddressLine1,
                     DistrictName = x.Address.District.Name,
                     StateOrProvinceName = x.Address.StateOrProvince.Name,
-                    CountryName = x.Address.Country.Name
+                    CountryName = x.Address.Country.Name,
+                    DisplayCity = x.Address.Country.IsCityEnabled,
+                    DisplayPostalCode = x.Address.Country.IsPostalCodeEnabled,
+                    DisplayDistrict = x.Address.Country.IsDistrictEnabled
                 }).ToList();
 
             foreach (var item in model)
@@ -207,7 +210,7 @@ public async Task<IActionResult> Delete(long id)
                 return NotFound();
             }
 
-            if(currentUser.DefaultShippingAddressId == userAddress.Id)
+            if (currentUser.DefaultShippingAddressId == userAddress.Id)
             {
                 currentUser.DefaultShippingAddressId = null;
             }
@@ -220,16 +223,29 @@ public async Task<IActionResult> Delete(long id)
 
         private void PopulateAddressFormData(UserAddressFormViewModel model)
         {
-            model.Countries = _countryRepository.Query()
+            var shippableCountries = _countryRepository.Query()
                 .Where(x => x.IsShippingEnabled)
-                .OrderBy(x => x.Name)
+                .OrderBy(x => x.Name);
+
+            model.Countries = shippableCountries
                 .Select(x => new SelectListItem
                 {
                     Text = x.Name,
                     Value = x.Id.ToString()
                 }).ToList();
 
             var onlyShipableCountryId = model.CountryId > 0 ? model.CountryId : long.Parse(model.Countries.First().Value);
+
+            var selectedCountry = shippableCountries.FirstOrDefault(c => c.Id == model.CountryId);
+
+            if (selectedCountry != null)
+            {
+                model.DisplayCity = selectedCountry.IsCityEnabled;
+                model.DisplayDistrict = selectedCountry.IsDistrictEnabled;
+                model.DisplayPostalCode = selectedCountry.IsPostalCodeEnabled;
+            }
+
+
             model.StateOrProvinces = _stateOrProvinceRepository
                 .Query()
                 .Where(x => x.CountryId == onlyShipableCountryId)
@@ -240,7 +256,7 @@ private void PopulateAddressFormData(UserAddressFormViewModel model)
                     Value = x.Id.ToString()
                 }).ToList();
 
-            if(model.StateOrProvinceId > 0)
+            if (model.StateOrProvinceId > 0)
             {
                 model.Districts = _districtRepository
                     .Query()
diff --git a/src/Modules/SimplCommerce.Module.Core/Models/Country.cs b/src/Modules/SimplCommerce.Module.Core/Models/Country.cs
@@ -13,5 +13,12 @@ public class Country : EntityBase
         public bool IsBillingEnabled { get; set; }
 
         public bool IsShippingEnabled { get; set; }
+
+        public bool IsCityEnabled { get; set; } = true;
+
+        public bool IsPostalCodeEnabled { get; set; } = true;
+
+        public bool IsDistrictEnabled { get; set; } = true;
+
     }
 }
diff --git a/src/Modules/SimplCommerce.Module.Core/ViewModels/CountryForm.cs b/src/Modules/SimplCommerce.Module.Core/ViewModels/CountryForm.cs
@@ -13,5 +13,11 @@ public class CountryForm
         public bool IsBillingEnabled { get; set; }
 
         public bool IsShippingEnabled { get; set; }
+
+        public bool IsCityEnabled { get;  set; }
+
+        public bool IsPostalCodeEnabled { get;  set; }
+
+        public bool IsDistrictEnabled { get;  set; }
     }
 }
diff --git a/src/Modules/SimplCommerce.Module.Core/ViewModels/UserAddressFormViewModel.cs b/src/Modules/SimplCommerce.Module.Core/ViewModels/UserAddressFormViewModel.cs
@@ -34,5 +34,11 @@ public class UserAddressFormViewModel
         public IList<SelectListItem> Districts { get; set; }
 
         public IList<SelectListItem> Countries { get; set; }
+
+        public bool DisplayDistrict { get; set; } = true;
+
+        public bool DisplayPostalCode { get; set; } = true;
+
+        public bool DisplayCity { get; set; } = true;
     }
 }
diff --git a/src/Modules/SimplCommerce.Module.Core/ViewModels/UserAddressListItem.cs b/src/Modules/SimplCommerce.Module.Core/ViewModels/UserAddressListItem.cs
@@ -21,5 +21,11 @@ public class UserAddressListItem
         public string CountryName { get; set; }
 
         public bool IsDefaultShippingAddress { get; set; }
+
+        public bool DisplayDistrict { get; set; }
+
+        public bool DisplayPostalCode { get; set; }
+
+        public bool DisplayCity { get; set; }
     }
 }
diff --git a/src/Modules/SimplCommerce.Module.Core/Views/UserAddress/List.cshtml b/src/Modules/SimplCommerce.Module.Core/Views/UserAddress/List.cshtml
@@ -15,7 +15,10 @@
             <div>
                 <strong>@address.ContactName </strong><br />
                 @address.AddressLine1 <br />
-                @address.DistrictName, @address.StateOrProvinceName <br />
+                @if(address.DisplayDistrict){
+                    <text>@address.DistrictName,</text>
+                }
+                @address.StateOrProvinceName <br />
                 Phone: @address.Phone
             </div>
             <div>
diff --git a/src/Modules/SimplCommerce.Module.Core/Views/UserAddress/_AddressForm.cshtml b/src/Modules/SimplCommerce.Module.Core/Views/UserAddress/_AddressForm.cshtml
@@ -14,6 +14,8 @@
         </select>
     </div>
 </div>
+@if (Model.DisplayDistrict)
+{
 <div class="form-group">
     <label class="col-sm-2 control-label">@Localizer["District"]</label>
     <div class="col-sm-10">
@@ -22,18 +24,28 @@
         </select>
     </div>
 </div>
+}
+
+@if (Model.DisplayCity)
+{
 <div class="form-group">
     <label class="col-sm-2 control-label">@Localizer["City"]</label>
     <div class="col-sm-10">
         <input asp-for="City" type="text" class="form-control">
     </div>
 </div>
+}
+
+@if (Model.DisplayPostalCode)
+{
 <div class="form-group">
     <label class="col-sm-2 control-label">@Localizer["Postal Code"]</label>
     <div class="col-sm-10">
         <input asp-for="PostalCode" type="text" class="form-control">
     </div>
 </div>
+}
+
 <div class="form-group">
     <label class="col-sm-2 control-label">@Localizer["Address"]</label>
     <div class="col-sm-10">
diff --git a/src/Modules/SimplCommerce.Module.Core/wwwroot/admin/countries/country-form.html b/src/Modules/SimplCommerce.Module.Core/wwwroot/admin/countries/country-form.html
@@ -47,6 +47,33 @@ <h2 ng-if="vm.isEditMode">{{::vm.translate.get('Edit Country')}}</h2>
                     </div>
                 </div>
             </div>
+            <div class="form-group">
+                <div class="col-sm-offset-2 col-sm-10">
+                    <div class="checkbox">
+                        <label>
+                            <input type="checkbox" ng-model="vm.country.isCityEnabled"> {{::vm.translate.get('Display City')}}
+                        </label>
+                    </div>
+                </div>
+            </div>
+            <div class="form-group">
+                <div class="col-sm-offset-2 col-sm-10">
+                    <div class="checkbox">
+                        <label>
+                            <input type="checkbox" ng-model="vm.country.isPostalCodeEnabled"> {{::vm.translate.get('Display Postal Code')}}
+                        </label>
+                    </div>
+                </div>
+            </div>
+            <div class="form-group">
+                <div class="col-sm-offset-2 col-sm-10">
+                    <div class="checkbox">
+                        <label>
+                            <input type="checkbox" ng-model="vm.country.isDistrictEnabled"> {{::vm.translate.get('Display District')}}
+                        </label>
+                    </div>
+                </div>
+            </div>
             <div class="form-group">
                 <div class="col-sm-offset-2 col-sm-10">
                     <button class="btn btn-primary" ng-click="vm.save()"><span class="glyphicon glyphicon-ok"></span> {{::vm.translate.get('Save')}}</button>
diff --git a/src/Modules/SimplCommerce.Module.Core/wwwroot/admin/countries/country-list.html b/src/Modules/SimplCommerce.Module.Core/wwwroot/admin/countries/country-list.html
@@ -15,6 +15,9 @@ <h2>{{::vm.translate.get('Countries')}}</h2>
             <th st-sort="Name" class="sortable">{{::vm.translate.get('Name')}}</th>
             <th st-sort="IsShippingEnabled" class="sortable" st-sort-default="reverse">{{::vm.translate.get('Is Shipping Enabled')}}</th>
             <th st-sort="IsBillingEnabled" class="sortable">{{::vm.translate.get('Is Billing Enabled')}}</th>
+            <th st-sort="IsBillingEnabled" class="sortable">{{::vm.translate.get('Display City')}}</th>
+            <th st-sort="IsBillingEnabled" class="sortable">{{::vm.translate.get('Display Postal Code')}}</th>
+            <th st-sort="IsBillingEnabled" class="sortable">{{::vm.translate.get('Display District')}}</th>
             <th class="sortable">&nbsp;</th>
         </tr>
         <tr>
@@ -30,10 +33,19 @@ <h2>{{::vm.translate.get('Countries')}}</h2>
         <tr ng-repeat="country in vm.countries">
             <td>{{country.name}}</td>
             <td>
-                <input type="checkbox" ng-model="country.isShippingEnabled" ng-click="vm.toggleShippingOrBilling(country)">
+                <input type="checkbox" ng-model="country.isShippingEnabled" ng-click="vm.update(country)">
             </td>
             <td>
-                <input type="checkbox" ng-model="country.isBillingEnabled" ng-click="vm.toggleShippingOrBilling(country)">
+                <input type="checkbox" ng-model="country.isBillingEnabled" ng-click="vm.update(country)">
+            </td>
+            <td>
+                <input type="checkbox" ng-model="country.isCityEnabled" ng-click="vm.update(country)">
+            </td>
+            <td>
+                <input type="checkbox" ng-model="country.isPostalCodeEnabled" ng-click="vm.update(country)">
+            </td>
+            <td>
+                <input type="checkbox" ng-model="country.isDistrictEnabled" ng-click="vm.update(country)">
             </td>
             <td>
                 <a ui-sref="country-edit({id: country.id})" title="Edit" class="btn btn-primary btn-xs"> <span class="glyphicon glyphicon-pencil"></span></a>
diff --git a/src/Modules/SimplCommerce.Module.Core/wwwroot/admin/countries/country-list.js b/src/Modules/SimplCommerce.Module.Core/wwwroot/admin/countries/country-list.js
@@ -21,7 +21,7 @@
             });
         };
 
-        vm.toggleShippingOrBilling = function (country) {
+        vm.update = function (country) {
             countryService.editCountry(country)
                 .then(function (result) {
                     toastr.success(country.name + ' has been updated');
diff --git a/src/Modules/SimplCommerce.Module.Orders/Controllers/CheckoutController.cs b/src/Modules/SimplCommerce.Module.Orders/Controllers/CheckoutController.cs
@@ -157,7 +157,10 @@ private void PopulateShippingForm(DeliveryInformationVm model, User currentUser)
                     AddressLine2 = x.Address.AddressLine1,
                     DistrictName = x.Address.District.Name,
                     StateOrProvinceName = x.Address.StateOrProvince.Name,
-                    CountryName = x.Address.Country.Name
+                    CountryName = x.Address.Country.Name,
+                    DisplayCity = x.Address.Country.IsCityEnabled,
+                    DisplayPostalCode = x.Address.Country.IsPostalCodeEnabled,
+                    DisplayDistrict = x.Address.Country.IsDistrictEnabled
                 }).ToList();
 
             model.ShippingAddressId = currentUser.DefaultShippingAddressId ?? 0;
@@ -174,6 +177,7 @@ private void PopulateShippingForm(DeliveryInformationVm model, User currentUser)
             if (model.NewAddressForm.ShipableContries.Count == 1)
             {
                 var onlyShipableCountryId = long.Parse(model.NewAddressForm.ShipableContries.First().Value);
+
                 model.NewAddressForm.StateOrProvinces = _stateOrProvinceRepository
                 .Query()
                 .Where(x => x.CountryId == onlyShipableCountryId)
diff --git a/src/Modules/SimplCommerce.Module.Orders/ViewModels/ShippingAddressVm.cs b/src/Modules/SimplCommerce.Module.Orders/ViewModels/ShippingAddressVm.cs
@@ -21,5 +21,11 @@ public class ShippingAddressVm
         public long CountryId { get; set; }
 
         public string CountryName { get; set; }
+
+        public bool DisplayDistrict { get; set; }
+
+        public bool DisplayPostalCode { get; set; }
+
+        public bool DisplayCity { get; set; }
     }
 }
diff --git a/src/Modules/SimplCommerce.Module.Orders/Views/Checkout/Shipping.cshtml b/src/Modules/SimplCommerce.Module.Orders/Views/Checkout/Shipping.cshtml
@@ -8,14 +8,18 @@
     <div class="col-md-8">
         <form asp-controller="Checkout" asp-action="Shipping" method="POST" id="checkout-delivery-address" class="form-horizontal">
             <h3>@Localizer["Shipping address"]</h3>
-            @foreach(var address in Model.ExistingShippingAddresses)
+            @foreach (var address in Model.ExistingShippingAddresses)
             {
                 <div class="radio">
                     <label>
                         <input type="radio" name="shippingAddressId" value="@address.UserAddressId" checked="@(Model.ShippingAddressId == address.UserAddressId ? "checked" : null)">
                         <strong>@address.ContactName </strong><br />
                         @address.AddressLine1 <br />
-                        @address.DistrictName, @address.StateOrProvinceName <br />
+                        @if (address.DisplayDistrict)
+                        {
+                            <text>@address.DistrictName,</text>
+                        }
+                        @address.StateOrProvinceName <br />
                         Phone: @address.Phone
                     </label>
                 </div>
@@ -49,6 +53,7 @@
                         </select>
                     </div>
                 </div>
+
                 <div class="form-group">
                     <label class="col-sm-4 control-label">@Localizer["District"]</label>
                     <div class="col-sm-8">
@@ -57,18 +62,21 @@
                         </select>
                     </div>
                 </div>
+
                 <div class="form-group">
                     <label class="col-sm-4 control-label">@Localizer["City"]</label>
                     <div class="col-sm-8">
                         <input asp-for="NewAddressForm.City" type="text" class="form-control">
                     </div>
                 </div>
+
                 <div class="form-group">
                     <label class="col-sm-4 control-label">@Localizer["Postal Code"]</label>
                     <div class="col-sm-8">
                         <input asp-for="NewAddressForm.PostalCode" type="text" class="form-control">
                     </div>
                 </div>
+
                 <div class="form-group">
                     <label class="col-sm-4 control-label">@Localizer["Address"]</label>
                     <div class="col-sm-8">
diff --git a/src/SimplCommerce.WebHost/Migrations/20180215030604_AddedCountryFieldDisplayToggles.Designer.cs b/src/SimplCommerce.WebHost/Migrations/20180215030604_AddedCountryFieldDisplayToggles.Designer.cs
diff --git a/src/SimplCommerce.WebHost/Migrations/20180215030604_AddedCountryFieldDisplayToggles.cs b/src/SimplCommerce.WebHost/Migrations/20180215030604_AddedCountryFieldDisplayToggles.cs
diff --git a/src/SimplCommerce.WebHost/Migrations/SimplDbContextModelSnapshot.cs b/src/SimplCommerce.WebHost/Migrations/SimplDbContextModelSnapshot.cs

Original file line number	Diff line number	Diff line change
`@@ -13,5 +13,12 @@ public class Country : EntityBase`
`13`	`13`	`public bool IsBillingEnabled { get; set; }`
`14`	`14`
`15`	`15`	`public bool IsShippingEnabled { get; set; }`
	`16`	`+`
	`17`	`+ public bool IsCityEnabled { get; set; } = true;`
	`18`	`+`
	`19`	`+ public bool IsPostalCodeEnabled { get; set; } = true;`
	`20`	`+`
	`21`	`+ public bool IsDistrictEnabled { get; set; } = true;`
	`22`	`+`
`16`	`23`	`}`
`17`	`24`	`}`
Original file line number	Diff line number	Diff line change
`@@ -13,5 +13,11 @@ public class CountryForm`
`13`	`13`	`public bool IsBillingEnabled { get; set; }`
`14`	`14`
`15`	`15`	`public bool IsShippingEnabled { get; set; }`
	`16`	`+`
	`17`	`+ public bool IsCityEnabled { get; set; }`
	`18`	`+`
	`19`	`+ public bool IsPostalCodeEnabled { get; set; }`
	`20`	`+`
	`21`	`+ public bool IsDistrictEnabled { get; set; }`
`16`	`22`	`}`
`17`	`23`	`}`
Original file line number	Diff line number	Diff line change
`@@ -34,5 +34,11 @@ public class UserAddressFormViewModel`
`34`	`34`	`public IList<SelectListItem> Districts { get; set; }`
`35`	`35`
`36`	`36`	`public IList<SelectListItem> Countries { get; set; }`
	`37`	`+`
	`38`	`+ public bool DisplayDistrict { get; set; } = true;`
	`39`	`+`
	`40`	`+ public bool DisplayPostalCode { get; set; } = true;`
	`41`	`+`
	`42`	`+ public bool DisplayCity { get; set; } = true;`
`37`	`43`	`}`
`38`	`44`	`}`
Original file line number	Diff line number	Diff line change
`@@ -21,5 +21,11 @@ public class UserAddressListItem`
`21`	`21`	`public string CountryName { get; set; }`
`22`	`22`
`23`	`23`	`public bool IsDefaultShippingAddress { get; set; }`
	`24`	`+`
	`25`	`+ public bool DisplayDistrict { get; set; }`
	`26`	`+`
	`27`	`+ public bool DisplayPostalCode { get; set; }`
	`28`	`+`
	`29`	`+ public bool DisplayCity { get; set; }`
`24`	`30`	`}`
`25`	`31`	`}`