SilverStripe\Forms\Schema\FormSchema

Complexity

Total Complexity

9

Size/Duplication

Total Lines	64
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes	3
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
B	getSchema()	0	28	6
A	getState()	0	20	3

<?php

namespace SilverStripe\Forms\Schema;

use Convert;
use Form;

/**
 * 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

		$params = $request->AllParams();

		$schema = [
			'name' => $form->getName(),
			'id' => isset($params['ID']) ? $params['ID'] : null,
			'action' => isset($params['Action']) ? $params['Action'] : null,
			'method' => $form->controller()->getRequest()->HttpMethod(),

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

			'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?

			$schema['actions'][] = $action->getSchemaData();
		}

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

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

			foreach ($fieldList->getForm()->fields()->dataFields() as $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->getName(),
			'fields' => [],
			'messages' => []
		];

		foreach ($form->Fields()->dataFields() as $field) {
			$state['fields'][] = $field->getSchemaState();
		}

		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;
	}
}