silverstripe / silverstripe-tagfield

src/ReadonlyTagField.php

Indentation +21 added lines, -21 removed lines

@@ -12,29 +12,29 @@
  */
 class ReadonlyTagField extends TagField
 {
-    /**
-     * {@inheritDoc}
-     */
-    protected $readonly = true;
+	/**
+	 * {@inheritDoc}
+	 */
+	protected $readonly = true;
 
-    /**
-     * Render the readonly field as HTML.
-     *
-     * @param array $properties
-     * @return HTMLText
-     */
-    public function Field($properties = array())
-    {
-        $options = array();
+	/**
+	 * Render the readonly field as HTML.
+	 *
+	 * @param array $properties
+	 * @return HTMLText
+	 */
+	public function Field($properties = array())
+	{
+		$options = array();
 
-        foreach ($this->getOptions()->filter('Selected', true) as $option) {
-            $options[] = $option->Title;
-        }
+		foreach ($this->getOptions()->filter('Selected', true) as $option) {
+			$options[] = $option->Title;
+		}
 
-        $field = ReadonlyField::create($this->name . '_Readonly', $this->title);
+		$field = ReadonlyField::create($this->name . '_Readonly', $this->title);
 
-        $field->setForm($this->form);
-        $field->setValue(implode(', ', $options));
-        return $field->Field();
-    }
+		$field->setForm($this->form);
+		$field->setValue(implode(', ', $options));
+		return $field->Field();
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
             $options[] = $option->Title;
         }
 
-        $field = ReadonlyField::create($this->name . '_Readonly', $this->title);
+        $field = ReadonlyField::create($this->name.'_Readonly', $this->title);
 
         $field->setForm($this->form);
         $field->setValue(implode(', ', $options));

tests/Stub/StringTagFieldTestController.php

Indentation +14 added lines, -14 removed lines

@@ -12,21 +12,21 @@
 
 class StringTagFieldTestController extends Controller implements TestOnly
 {
-    public function StringTagFieldTestForm()
-    {
-        $fields = new FieldList(
-            $tagField = new StringTagField('Tags')
-        );
+	public function StringTagFieldTestForm()
+	{
+		$fields = new FieldList(
+			$tagField = new StringTagField('Tags')
+		);
 
-        $actions = new FieldList(
-            new FormAction('StringTagFieldTestFormSubmit')
-        );
+		$actions = new FieldList(
+			new FormAction('StringTagFieldTestFormSubmit')
+		);
 
-        return new Form($this, 'StringTagFieldTestForm', $fields, $actions);
-    }
+		return new Form($this, 'StringTagFieldTestForm', $fields, $actions);
+	}
 
-    public function StringTagFieldTestFormSubmit(DataObject $dataObject, Form $form)
-    {
-        $form->saveInto($dataObject);
-    }
+	public function StringTagFieldTestFormSubmit(DataObject $dataObject, Form $form)
+	{
+		$form->saveInto($dataObject);
+	}
 }

tests/Stub/TagFieldTestBlogPost.php

Indentation +8 added lines, -8 removed lines

@@ -8,14 +8,14 @@
 
 class TagFieldTestBlogPost extends DataObject implements TestOnly
 {
-    private static $table_name = 'TagFieldTestBlogPost';
+	private static $table_name = 'TagFieldTestBlogPost';
 
-    private static $db = [
-        'Title'   => 'Text',
-        'Content' => 'Text'
-    ];
+	private static $db = [
+		'Title'   => 'Text',
+		'Content' => 'Text'
+	];
 
-    private static $many_many = [
-        'Tags' => TagFieldTestBlogTag::class
-    ];
+	private static $many_many = [
+		'Tags' => TagFieldTestBlogTag::class
+	];
 }

tests/Stub/TagFieldTestController.php

Indentation +14 added lines, -14 removed lines

@@ -13,21 +13,21 @@
 
 class TagFieldTestController extends Controller implements TestOnly
 {
-    public function TagFieldTestForm()
-    {
-        $fields = new FieldList(
-            $tagField = new TagField('Tags', '', new DataList('TagFieldTestBlogTag'))
-        );
+	public function TagFieldTestForm()
+	{
+		$fields = new FieldList(
+			$tagField = new TagField('Tags', '', new DataList('TagFieldTestBlogTag'))
+		);
 
-        $actions = new FieldList(
-            new FormAction('TagFieldTestFormSubmit')
-        );
+		$actions = new FieldList(
+			new FormAction('TagFieldTestFormSubmit')
+		);
 
-        return new Form($this, 'TagFieldTestForm', $fields, $actions);
-    }
+		return new Form($this, 'TagFieldTestForm', $fields, $actions);
+	}
 
-    public function TagFieldTestFormSubmit(DataObject $dataObject, Form $form)
-    {
-        $form->saveInto($dataObject);
-    }
+	public function TagFieldTestFormSubmit(DataObject $dataObject, Form $form)
+	{
+		$form->saveInto($dataObject);
+	}
 }

tests/Stub/TagFieldTestBlogTag.php

Indentation +8 added lines, -8 removed lines

@@ -8,15 +8,15 @@
 
 class TagFieldTestBlogTag extends DataObject implements TestOnly
 {
-    private static $table_name = 'TagFieldTestBlogTag';
+	private static $table_name = 'TagFieldTestBlogTag';
 
-    private static $default_sort = '"TagFieldTestBlogTag"."ID" ASC';
+	private static $default_sort = '"TagFieldTestBlogTag"."ID" ASC';
 
-    private static $db = [
-        'Title' => 'Varchar(200)'
-    ];
+	private static $db = [
+		'Title' => 'Varchar(200)'
+	];
 
-    private static $belongs_many_many = [
-        'BlogPosts' => TagFieldTestBlogPost::class
-    ];
+	private static $belongs_many_many = [
+		'BlogPosts' => TagFieldTestBlogPost::class
+	];
 }

tests/Stub/StringTagFieldTestBlogPost.php

Indentation +6 added lines, -6 removed lines

@@ -7,11 +7,11 @@
 
 class StringTagFieldTestBlogPost extends DataObject implements TestOnly
 {
-    private static $table_name = 'StringTagFieldTestBlogPost';
+	private static $table_name = 'StringTagFieldTestBlogPost';
 
-    private static $db = [
-        'Title' => 'Text',
-        'Content' => 'Text',
-        'Tags' => 'Text',
-    ];
+	private static $db = [
+		'Title' => 'Text',
+		'Content' => 'Text',
+		'Tags' => 'Text',
+	];
 }

src/StringTagField.php

Spacing +1 added lines, -1 removed lines

@@ -161,7 +161,7 @@
         $schema = array_merge(
             parent::getSchemaDataDefaults(),
             [
-                'name' => $this->getName() . '[]',
+                'name' => $this->getName().'[]',
                 'lazyLoad' => $this->getShouldLazyLoad(),
                 'creatable' => $this->getCanCreate(),
                 'multi' => $this->getIsMultiple(),

Indentation +330 added lines, -330 removed lines

@@ -26,334 +26,334 @@
  */
 class StringTagField extends DropdownField
 {
-    /**
-     * @var array
-     */
-    private static $allowed_actions = [
-        'suggest',
-    ];
-
-    /**
-     * @var bool
-     */
-    protected $shouldLazyLoad = false;
-
-    /**
-     * @var int
-     */
-    protected $lazyLoadItemLimit = 10;
-
-    /**
-     * @var bool
-     */
-    protected $canCreate = true;
-
-    /**
-     * @var null|DataObject
-     */
-    protected $record;
-
-    /**
-     * @var bool
-     */
-    protected $isMultiple = true;
-
-    /**
-     * @return bool
-     */
-    public function getShouldLazyLoad()
-    {
-        return $this->shouldLazyLoad;
-    }
-
-    /**
-     * @param bool $shouldLazyLoad
-     * @return $this
-     */
-    public function setShouldLazyLoad($shouldLazyLoad)
-    {
-        $this->shouldLazyLoad = $shouldLazyLoad;
-
-        return $this;
-    }
-
-    /**
-     * @return int
-     */
-    public function getLazyLoadItemLimit()
-    {
-        return $this->lazyLoadItemLimit;
-    }
-
-    /**
-     * @param int $lazyLoadItemLimit
-     * @return $this
-     */
-    public function setLazyLoadItemLimit($lazyLoadItemLimit)
-    {
-        $this->lazyLoadItemLimit = $lazyLoadItemLimit;
-
-        return $this;
-    }
-
-    /**
-     * @return bool
-     */
-    public function getIsMultiple()
-    {
-        return $this->isMultiple;
-    }
-
-    /**
-     * @param bool $isMultiple
-     * @return $this
-     */
-    public function setIsMultiple($isMultiple)
-    {
-        $this->isMultiple = $isMultiple;
-
-        return $this;
-    }
-
-    /**
-     * @return null|DataObject
-     */
-    public function getRecord()
-    {
-        if ($this->record) {
-            return $this->record;
-        }
-
-        if ($form = $this->getForm()) {
-            return $form->getRecord();
-        }
-
-        return null;
-    }
-
-    /**
-     * @param DataObject $record
-     * @return $this
-     */
-    public function setRecord(DataObject $record)
-    {
-        $this->record = $record;
-
-        return $this;
-    }
-
-    public function Field($properties = [])
-    {
-        $this->addExtraClass('ss-tag-field');
-
-        return $this
-            ->customise($properties)
-            ->renderWith(TagField::class);
-    }
-
-    /**
-     * Provide TagField data to the JSON schema for the frontend component
-     *
-     * @return array
-     */
-    public function getSchemaDataDefaults()
-    {
-        $schema = array_merge(
-            parent::getSchemaDataDefaults(),
-            [
-                'name' => $this->getName() . '[]',
-                'lazyLoad' => $this->getShouldLazyLoad(),
-                'creatable' => $this->getCanCreate(),
-                'multi' => $this->getIsMultiple(),
-                'value' => $this->formatOptions($this->Value()),
-                'disabled' => $this->isDisabled() || $this->isReadonly(),
-            ]
-        );
-
-        if (!$this->getShouldLazyLoad()) {
-            $schema['options'] = $this->getOptions()->toNestedArray();
-        } else {
-            $schema['optionUrl'] = $this->getSuggestURL();
-        }
-
-        return $schema;
-    }
-
-    protected function formatOptions($fieldValue)
-    {
-        if (empty($fieldValue)) {
-            return [];
-        }
-
-        $formattedValue = [];
-        foreach ($fieldValue as $value) {
-            $formattedValue[] = [
-                'Title' => $value,
-                'Value' => $value,
-            ];
-        }
-        return $formattedValue;
-    }
-
-    /**
-     * When not used in a React form factory context, this adds the schema data to SilverStripe template
-     * rendered attributes lists
-     *
-     * @return array
-     */
-    public function getAttributes()
-    {
-        $attributes = parent::getAttributes();
-        $attributes['data-schema'] = json_encode($this->getSchemaData());
-        return $attributes;
-    }
-
-    /**
-     * @return string
-     */
-    protected function getSuggestURL()
-    {
-        return Controller::join_links($this->Link(), 'suggest');
-    }
-
-    /**
-     * @return ArrayList
-     */
-    protected function getOptions()
-    {
-        $options = ArrayList::create();
-
-        $source = $this->getSource();
-
-        if ($source instanceof Iterator) {
-            $source = iterator_to_array($source);
-        }
-
-        foreach ($source as $value) {
-            $options->push(
-                ArrayData::create([
-                    'Title' => $value,
-                    'Value' => $value,
-                ])
-            );
-        }
-
-        return $options;
-    }
-
-    public function setValue($value, $source = null)
-    {
-        if (is_string($value)) {
-            $value = explode(',', $value);
-        }
-
-        if ($source instanceof DataObject) {
-            $name = $this->getName();
-            $value = explode(',', $source->$name);
-        }
-
-        if ($source instanceof SS_List) {
-            $value = $source->column('ID');
-        }
-
-        if ($value === null) {
-            $value = [];
-        }
-
-        return parent::setValue(array_filter($value));
-    }
-
-    public function saveInto(DataObjectInterface $record)
-    {
-        parent::saveInto($record);
-
-        $name = $this->getName();
-
-        $record->$name = $this->dataValue();
-        $record->write();
-    }
-
-    /**
-     * Ensure that arrays are imploded before being saved
-     *
-     * @return mixed|string
-     */
-    public function dataValue()
-    {
-        return implode(',', $this->value);
-    }
-
-    /**
-     * Returns a JSON string of tags, for lazy loading.
-     *
-     * @param  HTTPRequest $request
-     * @return HTTPResponse
-     */
-    public function suggest(HTTPRequest $request)
-    {
-        $responseBody = json_encode(
-            ['items' => $this->getTags($request->getVar('term'))]
-        );
-
-        $response = HTTPResponse::create();
-        $response->addHeader('Content-Type', 'application/json');
-        $response->setBody($responseBody);
-
-        return $response;
-    }
-
-    /**
-     * Returns array of arrays representing tags that partially match the given search term
-     *
-     * @param string $term
-     * @return array
-     */
-    protected function getTags($term)
-    {
-        $items = [];
-        foreach ($this->getOptions() as $i => $tag) {
-            /** @var ArrayData $tag */
-            $tagValue = $tag->Value;
-            // Map into a distinct list (prevent duplicates)
-            if (stripos($tagValue, $term) !== false && !array_key_exists($tagValue, $items)) {
-                $items[$tagValue] = [
-                    'id' => $tag->Title,
-                    'text' => $tag->Value,
-                ];
-            }
-        }
-        // @todo do we actually need lazy loading limits for StringTagField?
-        return array_slice(array_values($items), 0, $this->getLazyLoadItemLimit());
-    }
-
-    /**
-     * DropdownField assumes value will be a scalar so we must
-     * override validate. This only applies to Silverstripe 3.2+
-     *
-     * @param Validator $validator
-     * @return bool
-     */
-    public function validate($validator)
-    {
-        return true;
-    }
-
-    /**
-     * @return bool
-     */
-    public function getCanCreate()
-    {
-        return $this->canCreate;
-    }
-
-    /**
-     * @param bool $canCreate
-     * @return $this
-     */
-    public function setCanCreate($canCreate)
-    {
-        $this->canCreate = $canCreate;
-
-        return $this;
-    }
+	/**
+	 * @var array
+	 */
+	private static $allowed_actions = [
+		'suggest',
+	];
+
+	/**
+	 * @var bool
+	 */
+	protected $shouldLazyLoad = false;
+
+	/**
+	 * @var int
+	 */
+	protected $lazyLoadItemLimit = 10;
+
+	/**
+	 * @var bool
+	 */
+	protected $canCreate = true;
+
+	/**
+	 * @var null|DataObject
+	 */
+	protected $record;
+
+	/**
+	 * @var bool
+	 */
+	protected $isMultiple = true;
+
+	/**
+	 * @return bool
+	 */
+	public function getShouldLazyLoad()
+	{
+		return $this->shouldLazyLoad;
+	}
+
+	/**
+	 * @param bool $shouldLazyLoad
+	 * @return $this
+	 */
+	public function setShouldLazyLoad($shouldLazyLoad)
+	{
+		$this->shouldLazyLoad = $shouldLazyLoad;
+
+		return $this;
+	}
+
+	/**
+	 * @return int
+	 */
+	public function getLazyLoadItemLimit()
+	{
+		return $this->lazyLoadItemLimit;
+	}
+
+	/**
+	 * @param int $lazyLoadItemLimit
+	 * @return $this
+	 */
+	public function setLazyLoadItemLimit($lazyLoadItemLimit)
+	{
+		$this->lazyLoadItemLimit = $lazyLoadItemLimit;
+
+		return $this;
+	}
+
+	/**
+	 * @return bool
+	 */
+	public function getIsMultiple()
+	{
+		return $this->isMultiple;
+	}
+
+	/**
+	 * @param bool $isMultiple
+	 * @return $this
+	 */
+	public function setIsMultiple($isMultiple)
+	{
+		$this->isMultiple = $isMultiple;
+
+		return $this;
+	}
+
+	/**
+	 * @return null|DataObject
+	 */
+	public function getRecord()
+	{
+		if ($this->record) {
+			return $this->record;
+		}
+
+		if ($form = $this->getForm()) {
+			return $form->getRecord();
+		}
+
+		return null;
+	}
+
+	/**
+	 * @param DataObject $record
+	 * @return $this
+	 */
+	public function setRecord(DataObject $record)
+	{
+		$this->record = $record;
+
+		return $this;
+	}
+
+	public function Field($properties = [])
+	{
+		$this->addExtraClass('ss-tag-field');
+
+		return $this
+			->customise($properties)
+			->renderWith(TagField::class);
+	}
+
+	/**
+	 * Provide TagField data to the JSON schema for the frontend component
+	 *
+	 * @return array
+	 */
+	public function getSchemaDataDefaults()
+	{
+		$schema = array_merge(
+			parent::getSchemaDataDefaults(),
+			[
+				'name' => $this->getName() . '[]',
+				'lazyLoad' => $this->getShouldLazyLoad(),
+				'creatable' => $this->getCanCreate(),
+				'multi' => $this->getIsMultiple(),
+				'value' => $this->formatOptions($this->Value()),
+				'disabled' => $this->isDisabled() || $this->isReadonly(),
+			]
+		);
+
+		if (!$this->getShouldLazyLoad()) {
+			$schema['options'] = $this->getOptions()->toNestedArray();
+		} else {
+			$schema['optionUrl'] = $this->getSuggestURL();
+		}
+
+		return $schema;
+	}
+
+	protected function formatOptions($fieldValue)
+	{
+		if (empty($fieldValue)) {
+			return [];
+		}
+
+		$formattedValue = [];
+		foreach ($fieldValue as $value) {
+			$formattedValue[] = [
+				'Title' => $value,
+				'Value' => $value,
+			];
+		}
+		return $formattedValue;
+	}
+
+	/**
+	 * When not used in a React form factory context, this adds the schema data to SilverStripe template
+	 * rendered attributes lists
+	 *
+	 * @return array
+	 */
+	public function getAttributes()
+	{
+		$attributes = parent::getAttributes();
+		$attributes['data-schema'] = json_encode($this->getSchemaData());
+		return $attributes;
+	}
+
+	/**
+	 * @return string
+	 */
+	protected function getSuggestURL()
+	{
+		return Controller::join_links($this->Link(), 'suggest');
+	}
+
+	/**
+	 * @return ArrayList
+	 */
+	protected function getOptions()
+	{
+		$options = ArrayList::create();
+
+		$source = $this->getSource();
+
+		if ($source instanceof Iterator) {
+			$source = iterator_to_array($source);
+		}
+
+		foreach ($source as $value) {
+			$options->push(
+				ArrayData::create([
+					'Title' => $value,
+					'Value' => $value,
+				])
+			);
+		}
+
+		return $options;
+	}
+
+	public function setValue($value, $source = null)
+	{
+		if (is_string($value)) {
+			$value = explode(',', $value);
+		}
+
+		if ($source instanceof DataObject) {
+			$name = $this->getName();
+			$value = explode(',', $source->$name);
+		}
+
+		if ($source instanceof SS_List) {
+			$value = $source->column('ID');
+		}
+
+		if ($value === null) {
+			$value = [];
+		}
+
+		return parent::setValue(array_filter($value));
+	}
+
+	public function saveInto(DataObjectInterface $record)
+	{
+		parent::saveInto($record);
+
+		$name = $this->getName();
+
+		$record->$name = $this->dataValue();
+		$record->write();
+	}
+
+	/**
+	 * Ensure that arrays are imploded before being saved
+	 *
+	 * @return mixed|string
+	 */
+	public function dataValue()
+	{
+		return implode(',', $this->value);
+	}
+
+	/**
+	 * Returns a JSON string of tags, for lazy loading.
+	 *
+	 * @param  HTTPRequest $request
+	 * @return HTTPResponse
+	 */
+	public function suggest(HTTPRequest $request)
+	{
+		$responseBody = json_encode(
+			['items' => $this->getTags($request->getVar('term'))]
+		);
+
+		$response = HTTPResponse::create();
+		$response->addHeader('Content-Type', 'application/json');
+		$response->setBody($responseBody);
+
+		return $response;
+	}
+
+	/**
+	 * Returns array of arrays representing tags that partially match the given search term
+	 *
+	 * @param string $term
+	 * @return array
+	 */
+	protected function getTags($term)
+	{
+		$items = [];
+		foreach ($this->getOptions() as $i => $tag) {
+			/** @var ArrayData $tag */
+			$tagValue = $tag->Value;
+			// Map into a distinct list (prevent duplicates)
+			if (stripos($tagValue, $term) !== false && !array_key_exists($tagValue, $items)) {
+				$items[$tagValue] = [
+					'id' => $tag->Title,
+					'text' => $tag->Value,
+				];
+			}
+		}
+		// @todo do we actually need lazy loading limits for StringTagField?
+		return array_slice(array_values($items), 0, $this->getLazyLoadItemLimit());
+	}
+
+	/**
+	 * DropdownField assumes value will be a scalar so we must
+	 * override validate. This only applies to Silverstripe 3.2+
+	 *
+	 * @param Validator $validator
+	 * @return bool
+	 */
+	public function validate($validator)
+	{
+		return true;
+	}
+
+	/**
+	 * @return bool
+	 */
+	public function getCanCreate()
+	{
+		return $this->canCreate;
+	}
+
+	/**
+	 * @param bool $canCreate
+	 * @return $this
+	 */
+	public function setCanCreate($canCreate)
+	{
+		$this->canCreate = $canCreate;
+
+		return $this;
+	}
 }

tests/StringTagFieldTest.php

Indentation +140 added lines, -140 removed lines

@@ -15,144 +15,144 @@
  */
 class StringTagFieldTest extends SapphireTest
 {
-    /**
-     * @var string
-     */
-    protected static $fixture_file = 'StringTagFieldTest.yml';
-
-    /**
-     * @var array
-     */
-    protected static $extra_dataobjects = [
-        StringTagFieldTestBlogPost::class,
-    ];
-
-    public function testItSavesTagsOnNewRecords()
-    {
-        $record = $this->getNewStringTagFieldTestBlogPost('BlogPost1');
-
-        $field = new StringTagField('Tags');
-        $field->setValue(['Tag1', 'Tag2']);
-        $field->saveInto($record);
-
-        $record->write();
-
-        $this->assertEquals('Tag1,Tag2', $record->Tags);
-    }
-
-    /**
-     * @param string $name
-     *
-     * @return StringTagFieldTestBlogPost
-     */
-    protected function getNewStringTagFieldTestBlogPost($name)
-    {
-        return $this->objFromFixture(
-            StringTagFieldTestBlogPost::class,
-            $name
-        );
-    }
-
-    public function testItSavesTagsOnExistingRecords()
-    {
-        $record = $this->getNewStringTagFieldTestBlogPost('BlogPost1');
-        $record->write();
-
-        $field = new StringTagField('Tags');
-        $field->setValue(['Tag1', 'Tag2']);
-        $field->saveInto($record);
-
-        $this->assertEquals('Tag1,Tag2', $record->Tags);
-    }
-
-    public function testItSuggestsTags()
-    {
-        $field = new StringTagField('SomeField', 'Some field', ['Tag1', 'Tag2'], []);
-
-        /**
-         * Partial tag title match.
-         */
-        $request = $this->getNewRequest(['term' => 'Tag']);
-
-        $this->assertEquals(
-            '{"items":[{"id":"Tag1","text":"Tag1"},{"id":"Tag2","text":"Tag2"}]}',
-            $field->suggest($request)->getBody()
-        );
-
-        /**
-         * Exact tag title match.
-         */
-        $request = $this->getNewRequest(['term' => 'Tag1']);
-
-        $this->assertEquals($field->suggest($request)->getBody(), '{"items":[{"id":"Tag1","text":"Tag1"}]}');
-
-        /**
-         * Case-insensitive tag title match.
-         */
-        $request = $this->getNewRequest(['term' => 'TAG1']);
-
-        $this->assertEquals(
-            '{"items":[{"id":"Tag1","text":"Tag1"}]}',
-            $field->suggest($request)->getBody()
-        );
-
-        /**
-         * No tag title match.
-         */
-        $request = $this->getNewRequest(['term' => 'unknown']);
-
-        $this->assertEquals(
-            '{"items":[]}',
-            $field->suggest($request)->getBody()
-        );
-    }
-
-    public function testGetSchemaDataDefaults()
-    {
-        $form = new Form(null, 'Form', new FieldList(), new FieldList());
-        $field = new StringTagField('TestField', 'Test Field', ['one', 'two']);
-        $field->setForm($form);
-
-        $field
-            ->setShouldLazyLoad(false)
-            ->setCanCreate(false);
-
-        $schema = $field->getSchemaDataDefaults();
-        $this->assertSame('TestField[]', $schema['name']);
-        $this->assertFalse($schema['lazyLoad']);
-        $this->assertFalse($schema['creatable']);
-        $this->assertEquals([
-            ['Title' => 'one', 'Value' => 'one'],
-            ['Title' => 'two', 'Value' => 'two'],
-        ], $schema['options']);
-
-        $field
-            ->setShouldLazyLoad(true)
-            ->setCanCreate(true);
-
-        $schema = $field->getSchemaDataDefaults();
-        $this->assertTrue($schema['lazyLoad']);
-        $this->assertTrue($schema['creatable']);
-        $this->assertContains('suggest', $schema['optionUrl']);
-    }
-
-    public function testSchemaIsAddedToAttributes()
-    {
-        $field = new StringTagField('TestField');
-        $attributes = $field->getAttributes();
-        $this->assertNotEmpty($attributes['data-schema']);
-    }
-
-    /**
-     * @param array $parameters
-     * @return HTTPRequest
-     */
-    protected function getNewRequest(array $parameters)
-    {
-        return new HTTPRequest(
-            'get',
-            'StringTagFieldTestController/StringTagFieldTestForm/fields/Tags/suggest',
-            $parameters
-        );
-    }
+	/**
+	 * @var string
+	 */
+	protected static $fixture_file = 'StringTagFieldTest.yml';
+
+	/**
+	 * @var array
+	 */
+	protected static $extra_dataobjects = [
+		StringTagFieldTestBlogPost::class,
+	];
+
+	public function testItSavesTagsOnNewRecords()
+	{
+		$record = $this->getNewStringTagFieldTestBlogPost('BlogPost1');
+
+		$field = new StringTagField('Tags');
+		$field->setValue(['Tag1', 'Tag2']);
+		$field->saveInto($record);
+
+		$record->write();
+
+		$this->assertEquals('Tag1,Tag2', $record->Tags);
+	}
+
+	/**
+	 * @param string $name
+	 *
+	 * @return StringTagFieldTestBlogPost
+	 */
+	protected function getNewStringTagFieldTestBlogPost($name)
+	{
+		return $this->objFromFixture(
+			StringTagFieldTestBlogPost::class,
+			$name
+		);
+	}
+
+	public function testItSavesTagsOnExistingRecords()
+	{
+		$record = $this->getNewStringTagFieldTestBlogPost('BlogPost1');
+		$record->write();
+
+		$field = new StringTagField('Tags');
+		$field->setValue(['Tag1', 'Tag2']);
+		$field->saveInto($record);
+
+		$this->assertEquals('Tag1,Tag2', $record->Tags);
+	}
+
+	public function testItSuggestsTags()
+	{
+		$field = new StringTagField('SomeField', 'Some field', ['Tag1', 'Tag2'], []);
+
+		/**
+		 * Partial tag title match.
+		 */
+		$request = $this->getNewRequest(['term' => 'Tag']);
+
+		$this->assertEquals(
+			'{"items":[{"id":"Tag1","text":"Tag1"},{"id":"Tag2","text":"Tag2"}]}',
+			$field->suggest($request)->getBody()
+		);
+
+		/**
+		 * Exact tag title match.
+		 */
+		$request = $this->getNewRequest(['term' => 'Tag1']);
+
+		$this->assertEquals($field->suggest($request)->getBody(), '{"items":[{"id":"Tag1","text":"Tag1"}]}');
+
+		/**
+		 * Case-insensitive tag title match.
+		 */
+		$request = $this->getNewRequest(['term' => 'TAG1']);
+
+		$this->assertEquals(
+			'{"items":[{"id":"Tag1","text":"Tag1"}]}',
+			$field->suggest($request)->getBody()
+		);
+
+		/**
+		 * No tag title match.
+		 */
+		$request = $this->getNewRequest(['term' => 'unknown']);
+
+		$this->assertEquals(
+			'{"items":[]}',
+			$field->suggest($request)->getBody()
+		);
+	}
+
+	public function testGetSchemaDataDefaults()
+	{
+		$form = new Form(null, 'Form', new FieldList(), new FieldList());
+		$field = new StringTagField('TestField', 'Test Field', ['one', 'two']);
+		$field->setForm($form);
+
+		$field
+			->setShouldLazyLoad(false)
+			->setCanCreate(false);
+
+		$schema = $field->getSchemaDataDefaults();
+		$this->assertSame('TestField[]', $schema['name']);
+		$this->assertFalse($schema['lazyLoad']);
+		$this->assertFalse($schema['creatable']);
+		$this->assertEquals([
+			['Title' => 'one', 'Value' => 'one'],
+			['Title' => 'two', 'Value' => 'two'],
+		], $schema['options']);
+
+		$field
+			->setShouldLazyLoad(true)
+			->setCanCreate(true);
+
+		$schema = $field->getSchemaDataDefaults();
+		$this->assertTrue($schema['lazyLoad']);
+		$this->assertTrue($schema['creatable']);
+		$this->assertContains('suggest', $schema['optionUrl']);
+	}
+
+	public function testSchemaIsAddedToAttributes()
+	{
+		$field = new StringTagField('TestField');
+		$attributes = $field->getAttributes();
+		$this->assertNotEmpty($attributes['data-schema']);
+	}
+
+	/**
+	 * @param array $parameters
+	 * @return HTTPRequest
+	 */
+	protected function getNewRequest(array $parameters)
+	{
+		return new HTTPRequest(
+			'get',
+			'StringTagFieldTestController/StringTagFieldTestForm/fields/Tags/suggest',
+			$parameters
+		);
+	}
 }

tests/TagFieldTest.php

Indentation +369 added lines, -369 removed lines

@@ -20,373 +20,373 @@
  */
 class TagFieldTest extends SapphireTest
 {
-    /**
-     * @var string
-     */
-    protected static $fixture_file = 'TagFieldTest.yml';
-
-    /**
-     * @var array
-     */
-    protected static $extra_dataobjects = [
-        TagFieldTestBlogTag::class,
-        TagFieldTestBlogPost::class,
-    ];
-
-    public function testItSavesLinksToNewTagsOnNewRecords()
-    {
-        $record = $this->getNewTagFieldTestBlogPost('BlogPost1');
-        $field = new TagField('Tags', '', new DataList(TagFieldTestBlogTag::class));
-        $field->setValue(['Tag3', 'Tag4']);
-        $field->saveInto($record);
-        $record->write();
-        $this->compareExpectedAndActualTags(
-            ['Tag3', 'Tag4'],
-            $record
-        );
-    }
-
-    /**
-     * @param string $name
-     *
-     * @return TagFieldTestBlogPost
-     */
-    protected function getNewTagFieldTestBlogPost($name)
-    {
-        return $this->objFromFixture(
-            TagFieldTestBlogPost::class,
-            $name
-        );
-    }
-
-    /**
-     * @param array $expected
-     * @param TagFieldTestBlogPost $record
-     */
-    protected function compareExpectedAndActualTags(array $expected, TagFieldTestBlogPost $record)
-    {
-        $this->compareTagLists($expected, $record->Tags());
-    }
-
-    /**
-     * Ensure a source of tags matches the given string tag names
-     *
-     * @param array $expected
-     * @param DataList $actualSource
-     */
-    protected function compareTagLists(array $expected, DataList $actualSource)
-    {
-        $actual = array_values($actualSource->map('ID', 'Title')->toArray());
-        sort($expected);
-        sort($actual);
-
-        $this->assertEquals(
-            $expected,
-            $actual
-        );
-    }
-
-    public function testItSavesLinksToNewTagsOnExistingRecords()
-    {
-        $record = $this->getNewTagFieldTestBlogPost('BlogPost1');
-        $record->write();
-
-        $field = new TagField('Tags', '', new DataList(TagFieldTestBlogTag::class));
-        $field->setValue(['Tag3', 'Tag4']);
-        $field->saveInto($record);
-
-        $this->compareExpectedAndActualTags(
-            array('Tag3', 'Tag4'),
-            $record
-        );
-    }
-
-    public function testItSavesLinksToExistingTagsOnNewRecords()
-    {
-        $record = $this->getNewTagFieldTestBlogPost('BlogPost1');
-
-        $field = new TagField('Tags', '', new DataList(TagFieldTestBlogTag::class));
-        $field->setValue(['Tag1', 'Tag2']);
-        $field->saveInto($record);
-
-        $record->write();
-
-        $this->compareExpectedAndActualTags(
-            ['Tag1', 'Tag2'],
-            $record
-        );
-    }
-
-    public function testItSavesLinksToExistingTagsOnExistingRecords()
-    {
-        $record = $this->getNewTagFieldTestBlogPost('BlogPost1');
-        $record->write();
-
-        $field = new TagField('Tags', '', new DataList(TagFieldTestBlogTag::class));
-        $field->setValue(['Tag1', 'Tag2']);
-        $field->saveInto($record);
-
-        $this->compareExpectedAndActualTags(
-            ['Tag1', 'Tag2'],
-            $record
-        );
-    }
-
-    /**
-     * Ensure that {@see TagField::saveInto} respects existing tags
-     */
-    public function testSaveDuplicateTags()
-    {
-        $record = $this->getNewTagFieldTestBlogPost('BlogPost2');
-        $record->write();
-        $tag2ID = $this->idFromFixture(TagFieldTestBlogTag::class, 'Tag2');
-
-        // Check tags before write
-        $this->compareExpectedAndActualTags(
-            ['Tag1', '222'],
-            $record
-        );
-        $this->compareTagLists(
-            ['Tag1', '222'],
-            TagFieldTestBlogTag::get()
-        );
-        $this->assertContains($tag2ID, TagFieldTestBlogTag::get()->column('ID'));
-
-        // Write new tags
-        $field = new TagField('Tags', '', new DataList(TagFieldTestBlogTag::class));
-        $field->setValue(['222', 'Tag3']);
-        $field->saveInto($record);
-
-        // Check only one new tag was added
-        $this->compareExpectedAndActualTags(
-            ['222', 'Tag3'],
-            $record
-        );
-
-        // Ensure that only one new dataobject was added and that tag2s id has not changed
-        $this->compareTagLists(
-            ['Tag1', '222', 'Tag3'],
-            TagFieldTestBlogTag::get()
-        );
-        $this->assertContains($tag2ID, TagFieldTestBlogTag::get()->column('ID'));
-    }
-
-    public function testItSuggestsTags()
-    {
-        $field = new TagField('Tags', '', new DataList(TagFieldTestBlogTag::class));
-
-        /**
-         * Partial tag title match.
-         */
-        $request = $this->getNewRequest(['term' => 'Tag']);
-
-        $this->assertEquals(
-            '{"items":[{"id":"Tag1","text":"Tag1"}]}',
-            $field->suggest($request)->getBody()
-        );
-
-        /**
-         * Exact tag title match.
-         */
-        $request = $this->getNewRequest(['term' => '222']);
-
-        $this->assertEquals(
-            '{"items":[{"id":"222","text":"222"}]}',
-            $field->suggest($request)->getBody()
-        );
-
-        /**
-         * Case-insensitive tag title match.
-         */
-        $request = $this->getNewRequest(['term' => 'TAG1']);
-
-        $this->assertEquals(
-            '{"items":[{"id":"Tag1","text":"Tag1"}]}',
-            $field->suggest($request)->getBody()
-        );
-
-        /**
-         * No tag title match.
-         */
-        $request = $this->getNewRequest(['term' => 'unknown']);
-
-        $this->assertEquals(
-            '{"items":[]}',
-            $field->suggest($request)->getBody()
-        );
-    }
-
-    /**
-     * Tests that TagField supports pre-filtered data sources
-     */
-    public function testRestrictedSuggestions()
-    {
-        $source = TagFieldTestBlogTag::get()->exclude('Title', 'Tag2');
-        $field = new TagField('Tags', '', $source);
-
-        /**
-         * Partial tag title match.
-         */
-        $request = $this->getNewRequest(['term' => 'Tag']);
-
-        $this->assertEquals(
-            '{"items":[{"id":"Tag1","text":"Tag1"}]}',
-            $field->suggest($request)->getBody()
-        );
-
-        /**
-         * Exact tag title match.
-         */
-        $request = $this->getNewRequest(['term' => 'Tag1']);
-
-        $this->assertEquals(
-            '{"items":[{"id":"Tag1","text":"Tag1"}]}',
-            $field->suggest($request)->getBody()
-        );
-
-        /**
-         * Excluded item doesn't appear in matches
-         */
-        $request = $this->getNewRequest(['term' => 'Tag2']);
-
-        $this->assertEquals(
-            '{"items":[]}',
-            $field->suggest($request)->getBody()
-        );
-    }
-
-    /**
-     * @param array $parameters
-     *
-     * @return HTTPRequest
-     */
-    protected function getNewRequest(array $parameters)
-    {
-        return new HTTPRequest(
-            'get',
-            'TagFieldTestController/TagFieldTestForm/fields/Tags/suggest',
-            $parameters
-        );
-    }
-
-    public function testItDisplaysValuesFromRelations()
-    {
-        $record = $this->getNewTagFieldTestBlogPost('BlogPost1');
-        $record->write();
-
-        $form = new Form(
-            new TagFieldTestController(),
-            'Form',
-            new FieldList(
-                $field = new TagField('Tags', '', new DataList(TagFieldTestBlogTag::class))
-            ),
-            new FieldList()
-        );
-
-        $form->loadDataFrom(
-            $this->objFromFixture(TagFieldTestBlogPost::class, 'BlogPost2')
-        );
-
-        $ids = TagFieldTestBlogTag::get()->column('Title');
-
-        $this->assertEquals($field->Value(), $ids);
-    }
-
-    public function testItIgnoresNewTagsIfCannotCreate()
-    {
-        $this->markTestSkipped(
-            'This test has not been updated yet.'
-        );
-
-        $record = new TagFieldTestBlogPost();
-        $record->write();
-
-        $tag = TagFieldTestBlogTag::get()->filter('Title', 'Tag1')->first();
-
-        $field = new TagField('Tags', '', new DataList(TagFieldTestBlogTag::class), [$tag->Title, 'Tag3']);
-        $field->setCanCreate(false);
-        $field->saveInto($record);
-
-        /**
-         * @var TagFieldTestBlogPost $record
-         */
-        $record = DataObject::get_by_id(TagFieldTestBlogPost::class, $record->ID);
-
-        $this->compareExpectedAndActualTags(
-            ['Tag1'],
-            $record
-        );
-    }
-
-    /**
-     * Test you can save without a source set
-     */
-    public function testSaveEmptySource()
-    {
-        $record = new TagFieldTestBlogPost();
-        $record->write();
-
-        // Clear database of tags
-        TagFieldTestBlogTag::get()->removeAll();
-
-        $field = new TagField('Tags', '', TagFieldTestBlogTag::get());
-        $field->setValue(['New Tag']);
-        $field->setCanCreate(true);
-        $field->saveInto($record);
-
-        $tag = TagFieldTestBlogTag::get()->first();
-        $this->assertNotEmpty($tag);
-        $this->assertEquals('New Tag', $tag->Title);
-        $record = TagFieldTestBlogPost::get()->byID($record->ID);
-        $this->assertEquals(
-            $tag->ID,
-            $record->Tags()->first()->ID
-        );
-    }
-
-    /**
-     * Test read only fields are returned
-     */
-    public function testReadonlyTransformation()
-    {
-        $field = new TagField('Tags', '', TagFieldTestBlogTag::get());
-        $readOnlyField = $field->performReadonlyTransformation();
-        $this->assertInstanceOf(ReadonlyTagField::class, $readOnlyField);
-    }
-
-    public function testGetSchemaDataDefaults()
-    {
-        $form = new Form(null, 'Form', new FieldList(), new FieldList());
-        $field = new TagField('TestField', 'Test Field', TagFieldTestBlogTag::get());
-        $field->setForm($form);
-
-        $field
-            ->setShouldLazyLoad(false)
-            ->setCanCreate(false);
-
-        $schema = $field->getSchemaDataDefaults();
-        $this->assertSame('TestField[]', $schema['name']);
-        $this->assertFalse($schema['lazyLoad']);
-        $this->assertFalse($schema['creatable']);
-        $this->assertEquals([
-            ['Title' => 'Tag1', 'Value' => 'Tag1'],
-            ['Title' => '222', 'Value' => '222'],
-        ], $schema['options']);
-
-        $field
-            ->setShouldLazyLoad(true)
-            ->setCanCreate(true);
-
-        $schema = $field->getSchemaDataDefaults();
-        $this->assertTrue($schema['lazyLoad']);
-        $this->assertTrue($schema['creatable']);
-        $this->assertContains('suggest', $schema['optionUrl']);
-    }
-
-    public function testSchemaIsAddedToAttributes()
-    {
-        $field = new TagField('TestField');
-        $attributes = $field->getAttributes();
-        $this->assertNotEmpty($attributes['data-schema']);
-    }
+	/**
+	 * @var string
+	 */
+	protected static $fixture_file = 'TagFieldTest.yml';
+
+	/**
+	 * @var array
+	 */
+	protected static $extra_dataobjects = [
+		TagFieldTestBlogTag::class,
+		TagFieldTestBlogPost::class,
+	];
+
+	public function testItSavesLinksToNewTagsOnNewRecords()
+	{
+		$record = $this->getNewTagFieldTestBlogPost('BlogPost1');
+		$field = new TagField('Tags', '', new DataList(TagFieldTestBlogTag::class));
+		$field->setValue(['Tag3', 'Tag4']);
+		$field->saveInto($record);
+		$record->write();
+		$this->compareExpectedAndActualTags(
+			['Tag3', 'Tag4'],
+			$record
+		);
+	}
+
+	/**
+	 * @param string $name
+	 *
+	 * @return TagFieldTestBlogPost
+	 */
+	protected function getNewTagFieldTestBlogPost($name)
+	{
+		return $this->objFromFixture(
+			TagFieldTestBlogPost::class,
+			$name
+		);
+	}
+
+	/**
+	 * @param array $expected
+	 * @param TagFieldTestBlogPost $record
+	 */
+	protected function compareExpectedAndActualTags(array $expected, TagFieldTestBlogPost $record)
+	{
+		$this->compareTagLists($expected, $record->Tags());
+	}
+
+	/**
+	 * Ensure a source of tags matches the given string tag names
+	 *
+	 * @param array $expected
+	 * @param DataList $actualSource
+	 */
+	protected function compareTagLists(array $expected, DataList $actualSource)
+	{
+		$actual = array_values($actualSource->map('ID', 'Title')->toArray());
+		sort($expected);
+		sort($actual);
+
+		$this->assertEquals(
+			$expected,
+			$actual
+		);
+	}
+
+	public function testItSavesLinksToNewTagsOnExistingRecords()
+	{
+		$record = $this->getNewTagFieldTestBlogPost('BlogPost1');
+		$record->write();
+
+		$field = new TagField('Tags', '', new DataList(TagFieldTestBlogTag::class));
+		$field->setValue(['Tag3', 'Tag4']);
+		$field->saveInto($record);
+
+		$this->compareExpectedAndActualTags(
+			array('Tag3', 'Tag4'),
+			$record
+		);
+	}
+
+	public function testItSavesLinksToExistingTagsOnNewRecords()
+	{
+		$record = $this->getNewTagFieldTestBlogPost('BlogPost1');
+
+		$field = new TagField('Tags', '', new DataList(TagFieldTestBlogTag::class));
+		$field->setValue(['Tag1', 'Tag2']);
+		$field->saveInto($record);
+
+		$record->write();
+
+		$this->compareExpectedAndActualTags(
+			['Tag1', 'Tag2'],
+			$record
+		);
+	}
+
+	public function testItSavesLinksToExistingTagsOnExistingRecords()
+	{
+		$record = $this->getNewTagFieldTestBlogPost('BlogPost1');
+		$record->write();
+
+		$field = new TagField('Tags', '', new DataList(TagFieldTestBlogTag::class));
+		$field->setValue(['Tag1', 'Tag2']);
+		$field->saveInto($record);
+
+		$this->compareExpectedAndActualTags(
+			['Tag1', 'Tag2'],
+			$record
+		);
+	}
+
+	/**
+	 * Ensure that {@see TagField::saveInto} respects existing tags
+	 */
+	public function testSaveDuplicateTags()
+	{
+		$record = $this->getNewTagFieldTestBlogPost('BlogPost2');
+		$record->write();
+		$tag2ID = $this->idFromFixture(TagFieldTestBlogTag::class, 'Tag2');
+
+		// Check tags before write
+		$this->compareExpectedAndActualTags(
+			['Tag1', '222'],
+			$record
+		);
+		$this->compareTagLists(
+			['Tag1', '222'],
+			TagFieldTestBlogTag::get()
+		);
+		$this->assertContains($tag2ID, TagFieldTestBlogTag::get()->column('ID'));
+
+		// Write new tags
+		$field = new TagField('Tags', '', new DataList(TagFieldTestBlogTag::class));
+		$field->setValue(['222', 'Tag3']);
+		$field->saveInto($record);
+
+		// Check only one new tag was added
+		$this->compareExpectedAndActualTags(
+			['222', 'Tag3'],
+			$record
+		);
+
+		// Ensure that only one new dataobject was added and that tag2s id has not changed
+		$this->compareTagLists(
+			['Tag1', '222', 'Tag3'],
+			TagFieldTestBlogTag::get()
+		);
+		$this->assertContains($tag2ID, TagFieldTestBlogTag::get()->column('ID'));
+	}
+
+	public function testItSuggestsTags()
+	{
+		$field = new TagField('Tags', '', new DataList(TagFieldTestBlogTag::class));
+
+		/**
+		 * Partial tag title match.
+		 */
+		$request = $this->getNewRequest(['term' => 'Tag']);
+
+		$this->assertEquals(
+			'{"items":[{"id":"Tag1","text":"Tag1"}]}',
+			$field->suggest($request)->getBody()
+		);
+
+		/**
+		 * Exact tag title match.
+		 */
+		$request = $this->getNewRequest(['term' => '222']);
+
+		$this->assertEquals(
+			'{"items":[{"id":"222","text":"222"}]}',
+			$field->suggest($request)->getBody()
+		);
+
+		/**
+		 * Case-insensitive tag title match.
+		 */
+		$request = $this->getNewRequest(['term' => 'TAG1']);
+
+		$this->assertEquals(
+			'{"items":[{"id":"Tag1","text":"Tag1"}]}',
+			$field->suggest($request)->getBody()
+		);
+
+		/**
+		 * No tag title match.
+		 */
+		$request = $this->getNewRequest(['term' => 'unknown']);
+
+		$this->assertEquals(
+			'{"items":[]}',
+			$field->suggest($request)->getBody()
+		);
+	}
+
+	/**
+	 * Tests that TagField supports pre-filtered data sources
+	 */
+	public function testRestrictedSuggestions()
+	{
+		$source = TagFieldTestBlogTag::get()->exclude('Title', 'Tag2');
+		$field = new TagField('Tags', '', $source);
+
+		/**
+		 * Partial tag title match.
+		 */
+		$request = $this->getNewRequest(['term' => 'Tag']);
+
+		$this->assertEquals(
+			'{"items":[{"id":"Tag1","text":"Tag1"}]}',
+			$field->suggest($request)->getBody()
+		);
+
+		/**
+		 * Exact tag title match.
+		 */
+		$request = $this->getNewRequest(['term' => 'Tag1']);
+
+		$this->assertEquals(
+			'{"items":[{"id":"Tag1","text":"Tag1"}]}',
+			$field->suggest($request)->getBody()
+		);
+
+		/**
+		 * Excluded item doesn't appear in matches
+		 */
+		$request = $this->getNewRequest(['term' => 'Tag2']);
+
+		$this->assertEquals(
+			'{"items":[]}',
+			$field->suggest($request)->getBody()
+		);
+	}
+
+	/**
+	 * @param array $parameters
+	 *
+	 * @return HTTPRequest
+	 */
+	protected function getNewRequest(array $parameters)
+	{
+		return new HTTPRequest(
+			'get',
+			'TagFieldTestController/TagFieldTestForm/fields/Tags/suggest',
+			$parameters
+		);
+	}
+
+	public function testItDisplaysValuesFromRelations()
+	{
+		$record = $this->getNewTagFieldTestBlogPost('BlogPost1');
+		$record->write();
+
+		$form = new Form(
+			new TagFieldTestController(),
+			'Form',
+			new FieldList(
+				$field = new TagField('Tags', '', new DataList(TagFieldTestBlogTag::class))
+			),
+			new FieldList()
+		);
+
+		$form->loadDataFrom(
+			$this->objFromFixture(TagFieldTestBlogPost::class, 'BlogPost2')
+		);
+
+		$ids = TagFieldTestBlogTag::get()->column('Title');
+
+		$this->assertEquals($field->Value(), $ids);
+	}
+
+	public function testItIgnoresNewTagsIfCannotCreate()
+	{
+		$this->markTestSkipped(
+			'This test has not been updated yet.'
+		);
+
+		$record = new TagFieldTestBlogPost();
+		$record->write();
+
+		$tag = TagFieldTestBlogTag::get()->filter('Title', 'Tag1')->first();
+
+		$field = new TagField('Tags', '', new DataList(TagFieldTestBlogTag::class), [$tag->Title, 'Tag3']);
+		$field->setCanCreate(false);
+		$field->saveInto($record);
+
+		/**
+		 * @var TagFieldTestBlogPost $record
+		 */
+		$record = DataObject::get_by_id(TagFieldTestBlogPost::class, $record->ID);
+
+		$this->compareExpectedAndActualTags(
+			['Tag1'],
+			$record
+		);
+	}
+
+	/**
+	 * Test you can save without a source set
+	 */
+	public function testSaveEmptySource()
+	{
+		$record = new TagFieldTestBlogPost();
+		$record->write();
+
+		// Clear database of tags
+		TagFieldTestBlogTag::get()->removeAll();
+
+		$field = new TagField('Tags', '', TagFieldTestBlogTag::get());
+		$field->setValue(['New Tag']);
+		$field->setCanCreate(true);
+		$field->saveInto($record);
+
+		$tag = TagFieldTestBlogTag::get()->first();
+		$this->assertNotEmpty($tag);
+		$this->assertEquals('New Tag', $tag->Title);
+		$record = TagFieldTestBlogPost::get()->byID($record->ID);
+		$this->assertEquals(
+			$tag->ID,
+			$record->Tags()->first()->ID
+		);
+	}
+
+	/**
+	 * Test read only fields are returned
+	 */
+	public function testReadonlyTransformation()
+	{
+		$field = new TagField('Tags', '', TagFieldTestBlogTag::get());
+		$readOnlyField = $field->performReadonlyTransformation();
+		$this->assertInstanceOf(ReadonlyTagField::class, $readOnlyField);
+	}
+
+	public function testGetSchemaDataDefaults()
+	{
+		$form = new Form(null, 'Form', new FieldList(), new FieldList());
+		$field = new TagField('TestField', 'Test Field', TagFieldTestBlogTag::get());
+		$field->setForm($form);
+
+		$field
+			->setShouldLazyLoad(false)
+			->setCanCreate(false);
+
+		$schema = $field->getSchemaDataDefaults();
+		$this->assertSame('TestField[]', $schema['name']);
+		$this->assertFalse($schema['lazyLoad']);
+		$this->assertFalse($schema['creatable']);
+		$this->assertEquals([
+			['Title' => 'Tag1', 'Value' => 'Tag1'],
+			['Title' => '222', 'Value' => '222'],
+		], $schema['options']);
+
+		$field
+			->setShouldLazyLoad(true)
+			->setCanCreate(true);
+
+		$schema = $field->getSchemaDataDefaults();
+		$this->assertTrue($schema['lazyLoad']);
+		$this->assertTrue($schema['creatable']);
+		$this->assertContains('suggest', $schema['optionUrl']);
+	}
+
+	public function testSchemaIsAddedToAttributes()
+	{
+		$field = new TagField('TestField');
+		$attributes = $field->getAttributes();
+		$this->assertNotEmpty($attributes['data-schema']);
+	}
 }