From: eileenmcnaugton <eileen@fuzion.co.nz>
Date: Tue, 25 Aug 2015 22:32:20 +0000 (+1200)
Subject: CRM-16996 missing parenteses on if when validating financial_type_id
X-Git-Url: https://vcs.fsf.org/?a=commitdiff_plain;h=356b5786e5a9f3f9b9eb8093ec161bcb25b8c750;p=civicrm-core.git

CRM-16996 missing parenteses on if when validating financial_type_id
---

diff --git a/CRM/Member/Form/Membership.php b/CRM/Member/Form/Membership.php
index 6da29b322f..6ac490245d 100644
--- a/CRM/Member/Form/Membership.php
+++ b/CRM/Member/Form/Membership.php
@@ -779,7 +779,7 @@ class CRM_Member_Form_Membership extends CRM_Member_Form {
         $errors['num_terms'] = ts('Please enter an integer for the number of terms.');
       }
 
-      if ($self->_mode || isset($params['record_contribution']) && empty($params['financial_type_id'])) {
+      if (($self->_mode || isset($params['record_contribution'])) && empty($params['financial_type_id'])) {
         $errors['financial_type_id'] = ts('Please enter the financial Type.');
       }
     }