From b4d57a7e6b4e8f1dc540d6b0ed651f990e5c3213 Mon Sep 17 00:00:00 2001 From: Dave Lawrence Date: Tue, 11 Dec 2012 18:48:00 -0800 Subject: Bug 819022 - Project Kickoff Form: Add "Vendor" and "Line Item in Budget?" in the Finance portion --- extensions/MozProjectReview/web/js/moz_project_review.js | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'extensions/MozProjectReview/web/js') diff --git a/extensions/MozProjectReview/web/js/moz_project_review.js b/extensions/MozProjectReview/web/js/moz_project_review.js index 8aea8796b..29e765ad2 100644 --- a/extensions/MozProjectReview/web/js/moz_project_review.js +++ b/extensions/MozProjectReview/web/js/moz_project_review.js @@ -23,6 +23,7 @@ MPR.required_fields = { "separate_party": "Please select a value for separate party in the initial questions section" }, "finance_questions": { + "finance_purchase_vendor": "Please enter a value for vendor in the finance questions section", "finance_purchase_what": "Please enter a value for what in the finance questions section", "finance_purchase_why": "Please enter a value for why in the finance questions section", "finance_purchase_risk": "Please enter a value for risk in the finance questions section", @@ -121,6 +122,11 @@ MPR.validateAndSubmit = function () { if (!MPR.isFilledOut('vendor_cost')) alert_text += "Please select a value for vendor cost\n"; } + if (Dom.get('finance_purchase_inbudget').value == 'No') { + if (!MPR.isFilledOut('finance_purchase_notinbudget_why')) + alert_text += "Please include additional description for the out of budget line item\n"; + } + if (alert_text) { alert(alert_text); return false; -- cgit v1.2.3-24-g4f1b