Permalink
Browse files

BUGFIX Fixed MoneyField constructor - parent (FormField) constructor …

…calls setValue() already, which needs to happen *after* setting certain field instances (from r99391)

git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/sapphire/trunk@102861 467b73ca-7a2a-4603-9d3b-597d59a354a9
  • Loading branch information...
1 parent 498c075 commit 568f0d34bc22e96ed03f0362d0627374a78f6fa7 @chillu chillu committed Apr 14, 2010
Showing with 2 additions and 3 deletions.
  1. +2 −3 forms/MoneyField.php
View
@@ -29,12 +29,11 @@ class MoneyField extends FormField {
protected $fieldCurrency = null;
function __construct($name, $title = null, $value = "", $form = null) {
- parent::__construct($name, $title, $value, $form);
-
// naming with underscores to prevent values from actually being saved somewhere
$this->fieldAmount = new NumericField("{$name}[Amount]", _t('MoneyField.FIELDLABELAMOUNT', 'Amount'));
$this->fieldCurrency = $this->FieldCurrency();
- $this->setValue($value);
+
+ parent::__construct($name, $title, $value, $form);
}
/**

0 comments on commit 568f0d3

Please sign in to comment.