SilverStripe\Forms\Schema\FormSchema

Complexity

Total Complexity

8

Size/Duplication

Total Lines	76
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes	2
Bugs	0	Features	2

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getState()	0	19	2
A	getFieldStates()	0	12	3
B	getSchema()	0	28	3

<?php

namespace SilverStripe\Forms\Schema;

use Form;
use FormField;
use CompositeField;

/**
 * Class FormSchema
 * @package SilverStripe\Forms\Schema
 *
 * Represents a {@link Form} as structured data which allows a frontend library to render it.
 * Includes information about the form as well as its fields.
 * Can create a "schema" (structure only) as well as "state" (data only).
 */
class FormSchema {

	/**
	 * Gets the schema for this form as a nested array.
	 *
	 * @param Form $form
	 * @return array
	 */
	public function getSchema(Form $form) {
		$request = $form->controller()->getRequest();

The method Form::Controller() has been deprecated with message: 4.0

		$schema = [
			'name' => $form->getName(),
			'id' => $form->FormName(),
			'action' => $form->FormAction(),
			'method' => $form->FormMethod(),
			// @todo Not really reliable. Refactor into action on $this->Link('schema')
			'schema_url' => $request->getUrl(),
			'attributes' => $form->getAttributes(),
			'data' => [],
			'fields' => [],
			'actions' => []
		];

		foreach ($form->Actions() as $action) {

The expression $form->Actions() of type object<FieldList>|null is not guaranteed to be traversable. How about adding an additional type check?

			/** @var FormField $action */
			$schema['actions'][] = $action->getSchemaData();
		}

		foreach ($form->Fields() as $field) {

The expression $form->Fields() of type object<FieldList>|null is not guaranteed to be traversable. How about adding an additional type check?

			/** @var FormField $field */
			$schema['fields'][] = $field->getSchemaData();
		}

		return $schema;
	}

	/**
	 * Gets the current state of this form as a nested array.
	 *
	 * @param Form $form
	 * @return array
	 */
	public function getState(Form $form) {
		$state = [
			'id' => $form->FormName(),
			'fields' => [],
			'messages' => []
		];

		// flattened nested fields are returned, rather than only top level fields.
		$state['fields'] = $this->getFieldStates($form->Fields());

		if($form->Message()) {

The expression $form->Message() of type string|null is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

			$state['messages'][] = [
				'value' => $form->Message(),
				'type' => $form->MessageType(),
			];
		}

		return $state;
	}

	protected function getFieldStates($fields) {
		$states = [];
		foreach ($fields as $field) {
			$states[] = $field->getSchemaState();

			if ($field instanceof CompositeField) {
				$subFields = $field->FieldList();
				array_merge($states, $this->getFieldStates($subFields));
			}
		}
		return $states;
	}
}