diff --git a/src/StringTagField.php b/src/StringTagField.php index fe55c28..cab1520 100644 --- a/src/StringTagField.php +++ b/src/StringTagField.php @@ -416,4 +416,11 @@ class StringTagField extends DropdownField return $this; } + + public function performReadonlyTransformation() + { + $field = parent::performReadonlyTransformation(); + $field->setValue(implode(', ', $this->Value())); + return $field; + } } diff --git a/tests/StringTagFieldTest.php b/tests/StringTagFieldTest.php index ea35c04..589f6b0 100755 --- a/tests/StringTagFieldTest.php +++ b/tests/StringTagFieldTest.php @@ -178,22 +178,26 @@ class StringTagFieldTest extends SapphireTest $this->assertNotEmpty($attributes['data-schema']); } - /** - * Ensure a source of tags matches the given string tag names - * - * @param array $expected - * @param DataList $actualSource - */ - protected function compareTagLists(array $expected, DataList $actualSource) + public function testPerformReadonlyTransformation() { - $actual = array_keys($actualSource->map('ID', 'Title')->toArray()); - sort($expected); - sort($actual); + $field = new StringTagField('Tags'); + $field->setSource(['Test1', 'Test2', 'Test3']); - $this->assertEquals( - $expected, - $actual - ); + // Ensure a single value can be rendered + $field->setValue(['Test2']); + $field_readonly = $field->performReadonlyTransformation(); + $this->assertEquals('Test2', $field_readonly->Value()); + + // Ensure multiple valid values are rendered + $field->setValue(['Test1', 'Test2']); + $field_readonly = $field->performReadonlyTransformation(); + $this->assertEquals('Test1, Test2', $field_readonly->Value()); + + // Ensure an value not in the source array is still rendered + // (because e.g. in history view it must have been a valid value when it was set) + $field->setValue(['Test', 'Test1']); + $field_readonly = $field->performReadonlyTransformation(); + $this->assertEquals('Test, Test1', $field_readonly->Value()); } /**