CRM-13981 - Fix default value bug introduced in 133e2c9
authorColeman Watts <coleman@civicrm.org>
Mon, 6 Oct 2014 19:58:55 +0000 (15:58 -0400)
committerColeman Watts <coleman@civicrm.org>
Mon, 6 Oct 2014 19:58:55 +0000 (15:58 -0400)
commit9c80939e025d9366e1f694295b415f24c618135e
treecf006d78b97c96efdad7dfc09088214f42599cb6
parent2fd2fc793010519cb50457916a81d5624f3cc712
CRM-13981 - Fix default value bug introduced in 133e2c9

This commit cleans up a useless foreach loop that didn't do anything (the conditional would always evaluate to TRUE)
and fixes a bug with setting default values:
It seems that with Quickform, calling $form->setDefults with $defaults['foo']['bar'] = $value is "almost" equivalent to
$defaults["foo[bar]"] = $value but with one key difference: the latter always trumps the former.
So because the buildProfile function uses the latter method to set country default value to defaultContactCountry
(e.g. United States), that would always override the "correct" default value which is set here.
CRM/Contribute/Form/Contribution/Confirm.php