From 0091a163232e901b105c930853731a8d294954fb Mon Sep 17 00:00:00 2001
From: Zog
Date: Fri, 23 Feb 2018 12:31:14 +0100
Subject: Refs #5982; Show error in VJ creation modal when no JP is selected

---
 app/javascript/vehicle_journeys/actions/index.js                | 3 +--
 app/javascript/vehicle_journeys/components/tools/CreateModal.js | 8 +++++++-
 2 files changed, 8 insertions(+), 3 deletions(-)

(limited to 'app/javascript')

diff --git a/app/javascript/vehicle_journeys/actions/index.js b/app/javascript/vehicle_journeys/actions/index.js
index e67753e4b..5fb88f024 100644
--- a/app/javascript/vehicle_journeys/actions/index.js
+++ b/app/javascript/vehicle_journeys/actions/index.js
@@ -203,11 +203,10 @@ const actions = {
       let field = fields[key]
       if(field.validity && !field.validity.valid){
         valid = false
-        $(field).parent().addClass('has-error').children('.help-block').remove()
+        $(field).parent().parent().addClass('has-error').children('.help-block').remove()
         $(field).parent().append("<span class='small help-block'>" + field.validationMessage + "</span>")
       }
     })
-
     return valid
   },
   toggleArrivals : () => ({
diff --git a/app/javascript/vehicle_journeys/components/tools/CreateModal.js b/app/javascript/vehicle_journeys/components/tools/CreateModal.js
index 24d9a23c2..a60429765 100644
--- a/app/javascript/vehicle_journeys/components/tools/CreateModal.js
+++ b/app/javascript/vehicle_journeys/components/tools/CreateModal.js
@@ -12,7 +12,13 @@ export default class CreateModal extends Component {
   }
 
   handleSubmit() {
-    if (actions.validateFields(...this.refs, $('.vjCreateSelectJP')[0]) && this.props.modal.modalProps.selectedJPModal) {
+    if(!this.props.modal.modalProps.selectedJPModal){
+      let field = $('#NewVehicleJourneyModal').find(".vjCreateSelectJP")
+      field.parent().parent().addClass('has-error').children('.help-block').remove()
+      field.parent().append("<span class='small help-block'>" + I18n.t("simple_form.required.text") + "</span>")
+      return
+    }
+    if (actions.validateFields(...this.refs, $('.vjCreateSelectJP')[0])) {
       this.props.onAddVehicleJourney(_.assign({}, this.refs, {custom_fields: this.custom_fields}), this.props.modal.modalProps.selectedJPModal, this.props.stopPointsList, this.props.modal.modalProps.vehicleJourney && this.props.modal.modalProps.vehicleJourney.company)
       this.props.onModalClose()
       $('#NewVehicleJourneyModal').modal('hide')
-- 
cgit v1.2.3