tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-10-22 12:05:37 +00:00
Code Issues Projects Releases Wiki Activity

MINOR fixed validation on CurrencyField. Will no longer parse non-numeric characters out of input. Added test coverage.

Browse Source 
git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/sapphire/trunk@97057 467b73ca-7a2a-4603-9d3b-597d59a354a9
This commit is contained in:
Tom Rix 2010-01-18 00:56:29 +00:00
parent 2996cb39e3
commit 6f9177ef43
2 changed files with 31 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
forms/CurrencyField.php
View File

@ -9,17 +9,19 @@
*/
class CurrencyField extends TextField {
/**
* allows the value to be set ( not including $ signs and number format...)
* allows the value to be set. removes the first character
* if it is not a number (probably a currency symbol)
*/
function setValue($val) {
$this->value = '$' . number_format(ereg_replace('[^0-9.]',"",$val), 2);
$this->value = preg_replace('/^[^\d]/', '', $val);
}
/**
* Overwrite the datavalue before saving to the db ;-)
* return 0.00 if no value, or value is non-numeric
*/
function dataValue() {
if($this->value){
return preg_replace('/[^0-9.]/',"", $this->value);
if($this->value && is_numeric($this->value)){
return $this->value;
}else{
return 0.00;
}

25
tests/forms/CurrencyFieldTest.php Normal file
View File

@ -0,0 +1,25 @@
<?php
class CurrencyFieldTest extends SapphireTest {
function testValidation() {
$field = new CurrencyField('cf');
$vr = new RequiredFields();
$field->setValue('$10.23');
$this->assertTrue($field->validate($vr));
$field->setValue('$1a0.23');
$this->assertFalse($field->validate($vr));
}
function testDataValues() {
$field = new CurrencyField('cf');
$field->setValue('$10.34');
$this->assertEquals($field->dataValue(), '10.34');
$field->setValue('$1s0.34');
$this->assertEquals($field->dataValue(), '0.00');
}
}
?>