Skip to content
This repository has been archived by the owner on Mar 1, 2023. It is now read-only.

Commit

Permalink
[!!!][TASK] Rename form configuration to form definition
Browse files Browse the repository at this point in the history
The whole configuration objects representing all form properties have been moved
and renamed to `definition`, as this word better represents what the objects
actually are and do.

This commit is part of a bigger plan to refactor the form objects handling, that
will result in an easier way to manipulate the forms during the runtime.
  • Loading branch information
romm committed Apr 24, 2017
1 parent a353c5f commit cae979b
Show file tree
Hide file tree
Showing 95 changed files with 252 additions and 252 deletions.
4 changes: 2 additions & 2 deletions Classes/AssetHandler/Css/FieldsActivationCssAssetHandler.php
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@

use Romm\Formz\AssetHandler\AbstractAssetHandler;
use Romm\Formz\Condition\Parser\ConditionTree;
use Romm\Formz\Configuration\Form\Field\Field;
use Romm\Formz\Form\Definition\Field\Field;

/**
* This asset handler generates the CSS code which will automatically hide
Expand All @@ -37,7 +37,7 @@ class FieldsActivationCssAssetHandler extends AbstractAssetHandler
public function getFieldsActivationCss()
{
$cssBlocks = [];
$formConfiguration = $this->getFormObject()->getConfiguration();
$formConfiguration = $this->getFormObject()->getDefinition();

foreach ($formConfiguration->getFields() as $field) {
$formName = $this->getFormObject()->getName();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ class MessageContainerDisplayCssAssetHandler extends AbstractAssetHandler
public function getErrorContainerDisplayCss()
{
$cssBlocks = [];
$formConfiguration = $this->getFormObject()->getConfiguration();
$formConfiguration = $this->getFormObject()->getDefinition();

foreach ($formConfiguration->getFields() as $fieldName => $field) {
$formName = $this->getFormObject()->getName();
Expand Down
6 changes: 3 additions & 3 deletions Classes/AssetHandler/Html/DataAttributesAssetHandler.php
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ public function getFieldsValuesDataAttributes(FormResult $formResult)
$formObject = $this->getFormObject();
$formInstance = $formObject->getForm();

foreach ($formObject->getConfiguration()->getFields() as $field) {
foreach ($formObject->getDefinition()->getFields() as $field) {
$fieldName = $field->getName();

if (false === $formResult->fieldIsDeactivated($field)) {
Expand Down Expand Up @@ -83,7 +83,7 @@ public function getFieldsValuesDataAttributes(FormResult $formResult)
public function getFieldsMessagesDataAttributes()
{
$result = [];
$formConfiguration = $this->getFormObject()->getConfiguration();
$formConfiguration = $this->getFormObject()->getDefinition();
$formResult = $this->getFormObject()->getFormResult();

foreach ($formResult->getSubResults() as $fieldName => $fieldResult) {
Expand Down Expand Up @@ -165,7 +165,7 @@ protected function addFieldMessageDataAttribute($fieldName, array $messages, $ty
public function getFieldsValidDataAttributes()
{
$result = [];
$formConfiguration = $this->getFormObject()->getConfiguration();
$formConfiguration = $this->getFormObject()->getDefinition();
$formResult = $this->getFormObject()->getFormResult();

foreach ($formConfiguration->getFields() as $field) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@

use Romm\Formz\AssetHandler\AbstractAssetHandler;
use Romm\Formz\Condition\Parser\ConditionTree;
use Romm\Formz\Configuration\Form\Field\Field;
use Romm\Formz\Form\Definition\Field\Field;
use TYPO3\CMS\Core\Utility\GeneralUtility;

/**
Expand All @@ -34,7 +34,7 @@ class FieldsActivationJavaScriptAssetHandler extends AbstractAssetHandler
public function getFieldsActivationJavaScriptCode()
{
$javaScriptBlocks = [];
$formConfiguration = $this->getFormObject()->getConfiguration();
$formConfiguration = $this->getFormObject()->getDefinition();

foreach ($formConfiguration->getFields() as $field) {
$fieldConditionExpression = [];
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@

use Romm\Formz\AssetHandler\AbstractAssetHandler;
use Romm\Formz\Condition\Parser\ConditionTree;
use Romm\Formz\Configuration\Form\Field\Validation\Validation;
use Romm\Formz\Form\Definition\Field\Validation\Validation;
use TYPO3\CMS\Core\Utility\GeneralUtility;

/**
Expand All @@ -34,7 +34,7 @@ class FieldsValidationActivationJavaScriptAssetHandler extends AbstractAssetHand
public function getFieldsValidationActivationJavaScriptCode()
{
$javaScriptBlocks = [];
$formConfiguration = $this->getFormObject()->getConfiguration();
$formConfiguration = $this->getFormObject()->getDefinition();

foreach ($formConfiguration->getFields() as $field) {
foreach ($field->getValidation() as $validation) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,8 @@
namespace Romm\Formz\AssetHandler\JavaScript;

use Romm\Formz\AssetHandler\AbstractAssetHandler;
use Romm\Formz\Configuration\Form\Field\Field;
use Romm\Formz\Configuration\Form\Field\Validation\Validation;
use Romm\Formz\Form\Definition\Field\Field;
use Romm\Formz\Form\Definition\Field\Validation\Validation;
use Romm\Formz\Service\ArrayService;
use Romm\Formz\Service\ValidatorService;
use Romm\Formz\Validation\Validator\AbstractValidator;
Expand Down Expand Up @@ -44,7 +44,7 @@ class FieldsValidationJavaScriptAssetHandler extends AbstractAssetHandler
public function getJavaScriptCode()
{
$fieldsJavaScriptCode = [];
$formConfiguration = $this->getFormObject()->getConfiguration();
$formConfiguration = $this->getFormObject()->getDefinition();

foreach ($formConfiguration->getFields() as $field) {
$fieldsJavaScriptCode[] = $this->processField($field);
Expand Down Expand Up @@ -171,7 +171,7 @@ public function getJavaScriptValidationFiles()
if (null === $this->javaScriptValidationFiles) {
$this->javaScriptValidationFiles = [];

$formConfiguration = $this->getFormObject()->getConfiguration();
$formConfiguration = $this->getFormObject()->getDefinition();

foreach ($formConfiguration->getFields() as $field) {
foreach ($field->getValidation() as $validationConfiguration) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ protected function handleFormConfiguration($formConfiguration)
*/
protected function getFormConfiguration()
{
$formConfigurationArray = $this->getFormObject()->getConfiguration()->toArray();
$formConfigurationArray = $this->getFormObject()->getDefinition()->toArray();
$this->removeFieldsValidationConfiguration($formConfigurationArray)
->addClassNameProperty($formConfigurationArray);

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
namespace Romm\Formz\AssetHandler\JavaScript;

use Romm\Formz\AssetHandler\AbstractAssetHandler;
use Romm\Formz\Configuration\Form\Field\Field;
use Romm\Formz\Form\Definition\Field\Field;
use Romm\Formz\Error\FormzMessageInterface;
use Romm\Formz\Service\ArrayService;
use Romm\Formz\Service\MessageService;
Expand Down Expand Up @@ -100,7 +100,7 @@ protected function getFieldsExistingMessages()
if ($formObject->formWasSubmitted()
&& $formObject->hasFormResult()
) {
foreach ($this->getFormObject()->getConfiguration()->getFields() as $field) {
foreach ($this->getFormObject()->getDefinition()->getFields() as $field) {
$fieldsMessages[$field->getName()] = $this->getSingleFieldExistingMessages($field);
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ class FormzConfigurationJavaScriptAssetHandler extends AbstractAssetHandler
public function getJavaScriptFileName()
{
$hash = sha1($this->getFormObject()
->getConfiguration()
->getDefinition()
->getRootConfiguration()
->getHash());

Expand Down Expand Up @@ -70,7 +70,7 @@ protected function handleFormzConfiguration($formzConfiguration)
protected function getFormzConfiguration()
{
$rootConfigurationArray = $this->getFormObject()
->getConfiguration()
->getDefinition()
->getRootConfiguration()
->toArray();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
namespace Romm\Formz\AssetHandler\JavaScript;

use Romm\Formz\AssetHandler\AbstractAssetHandler;
use Romm\Formz\Configuration\Form\Field\Field;
use Romm\Formz\Form\Definition\Field\Field;
use Romm\Formz\Service\ArrayService;
use Romm\Formz\Service\HashService;
use Romm\Formz\Service\MessageService;
Expand Down Expand Up @@ -109,7 +109,7 @@ public function getTranslationKeysForFieldValidation(Field $field, $validationNa
*/
public function injectTranslationsForFormFieldsValidation()
{
$formConfiguration = $this->getFormObject()->getConfiguration();
$formConfiguration = $this->getFormObject()->getDefinition();

foreach ($formConfiguration->getFields() as $field) {
$this->storeTranslationsForFieldValidation($field);
Expand Down
10 changes: 5 additions & 5 deletions Classes/Behaviours/BehavioursManager.php
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@

namespace Romm\Formz\Behaviours;

use Romm\Formz\Configuration\Form\Form;
use Romm\Formz\Form\Definition\FormDefinition;
use Romm\Formz\Form\FormObject;
use TYPO3\CMS\Core\SingletonInterface;
use TYPO3\CMS\Core\Utility\GeneralUtility;
Expand All @@ -35,11 +35,11 @@ class BehavioursManager implements SingletonInterface
* in `$formProperties`, the behaviours are applied and the value is
* changed.
*
* @param array $formProperties Properties values of the submitted form.
* @param Form $formConfiguration Configuration object of the form.
* @param array $formProperties Properties values of the submitted form.
* @param FormDefinition $formConfiguration Configuration object of the form.
* @return array
*/
public function applyBehaviourOnPropertiesArray(array $formProperties, Form $formConfiguration)
public function applyBehaviourOnPropertiesArray(array $formProperties, FormDefinition $formConfiguration)
{
foreach ($formConfiguration->getFields() as $fieldName => $field) {
if (true === isset($formProperties[$fieldName])) {
Expand Down Expand Up @@ -67,7 +67,7 @@ public function applyBehaviourOnFormInstance(FormObject $formObject)
if ($formObject->hasForm()) {
$formInstance = $formObject->getForm();

foreach ($formObject->getConfiguration()->getFields() as $fieldName => $field) {
foreach ($formObject->getDefinition()->getFields() as $fieldName => $field) {
if (ObjectAccess::isPropertyGettable($formInstance, $fieldName)
&& ObjectAccess::isPropertySettable($formInstance, $fieldName)
) {
Expand Down
4 changes: 2 additions & 2 deletions Classes/Condition/Exceptions/InvalidConditionException.php
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,8 @@

namespace Romm\Formz\Condition\Exceptions;

use Romm\Formz\Configuration\Form\Field\Field;
use Romm\Formz\Configuration\Form\Field\Validation\Validation;
use Romm\Formz\Form\Definition\Field\Field;
use Romm\Formz\Form\Definition\Field\Validation\Validation;
use Romm\Formz\Exceptions\FormzException;

class InvalidConditionException extends FormzException
Expand Down
6 changes: 3 additions & 3 deletions Classes/Condition/Items/AbstractConditionItem.php
Original file line number Diff line number Diff line change
Expand Up @@ -16,9 +16,9 @@
use Romm\ConfigurationObject\Traits\ConfigurationObject\MagicMethodsTrait;
use Romm\Formz\Condition\Exceptions\InvalidConditionException;
use Romm\Formz\Condition\Parser\Node\ConditionNode;
use Romm\Formz\Configuration\Form\Field\Activation\ActivationInterface;
use Romm\Formz\Configuration\Form\Field\Field;
use Romm\Formz\Configuration\Form\Field\Validation\Validation;
use Romm\Formz\Form\Definition\Field\Activation\ActivationInterface;
use Romm\Formz\Form\Definition\Field\Field;
use Romm\Formz\Form\Definition\Field\Validation\Validation;
use Romm\Formz\Form\FormObject;
use Romm\Formz\Service\ArrayService;

Expand Down
2 changes: 1 addition & 1 deletion Classes/Condition/Items/ConditionItemInterface.php
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
use Romm\Formz\Condition\Exceptions\InvalidConditionException;
use Romm\Formz\Condition\Parser\Node\ConditionNode;
use Romm\Formz\Condition\Processor\DataObject\PhpConditionDataObject;
use Romm\Formz\Configuration\Form\Field\Activation\ActivationInterface;
use Romm\Formz\Form\Definition\Field\Activation\ActivationInterface;
use Romm\Formz\Form\FormObject;

interface ConditionItemInterface
Expand Down
4 changes: 2 additions & 2 deletions Classes/Condition/Items/FieldHasErrorCondition.php
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ public function getPhpResult(PhpConditionDataObject $dataObject)
{
$flag = false;
$formValidator = $dataObject->getFormValidator();
$field = $this->formObject->getConfiguration()->getField($this->fieldName);
$field = $this->formObject->getDefinition()->getField($this->fieldName);
$formValidator->validateField($field);
$result = $formValidator->getResult()->forProperty($this->fieldName);

Expand All @@ -108,7 +108,7 @@ public function getPhpResult(PhpConditionDataObject $dataObject)
*/
protected function checkConditionConfiguration()
{
$configuration = $this->formObject->getConfiguration();
$configuration = $this->formObject->getDefinition();

if (false === $configuration->hasField($this->fieldName)) {
throw InvalidConditionException::conditionFieldHasErrorFieldNotFound($this->fieldName);
Expand Down
2 changes: 1 addition & 1 deletion Classes/Condition/Items/FieldHasValueCondition.php
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ public function getPhpResult(PhpConditionDataObject $dataObject)
*/
protected function checkConditionConfiguration()
{
$configuration = $this->formObject->getConfiguration();
$configuration = $this->formObject->getDefinition();

if (false === $configuration->hasField($this->fieldName)) {
throw InvalidConditionException::conditionFieldHasValueFieldNotFound($this->fieldName);
Expand Down
2 changes: 1 addition & 1 deletion Classes/Condition/Items/FieldIsEmptyCondition.php
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ public function getPhpResult(PhpConditionDataObject $dataObject)
*/
protected function checkConditionConfiguration()
{
$configuration = $this->formObject->getConfiguration();
$configuration = $this->formObject->getDefinition();

if (false === $configuration->hasField($this->fieldName)) {
throw InvalidConditionException::conditionFieldIsEmptyFieldNotFound($this->fieldName);
Expand Down
4 changes: 2 additions & 2 deletions Classes/Condition/Items/FieldIsValidCondition.php
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ public function getJavaScriptResult()
public function getPhpResult(PhpConditionDataObject $dataObject)
{
$formValidator = $dataObject->getFormValidator();
$field = $this->formObject->getConfiguration()->getField($this->fieldName);
$field = $this->formObject->getDefinition()->getField($this->fieldName);
$formValidator->validateField($field);
$result = $formValidator->getResult();

Expand All @@ -76,7 +76,7 @@ public function getPhpResult(PhpConditionDataObject $dataObject)
*/
protected function checkConditionConfiguration()
{
$configuration = $this->formObject->getConfiguration();
$configuration = $this->formObject->getDefinition();

if (false === $configuration->hasField($this->fieldName)) {
throw InvalidConditionException::conditionFieldIsValidFieldNotFound($this->fieldName);
Expand Down
4 changes: 2 additions & 2 deletions Classes/Condition/Parser/ConditionParser.php
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,8 @@
use Romm\Formz\Condition\Parser\Node\ConditionNode;
use Romm\Formz\Condition\Parser\Node\NodeInterface;
use Romm\Formz\Condition\Parser\Node\NullNode;
use Romm\Formz\Configuration\Form\Field\Activation\ActivationInterface;
use Romm\Formz\Configuration\Form\Field\Activation\EmptyActivation;
use Romm\Formz\Form\Definition\Field\Activation\ActivationInterface;
use Romm\Formz\Form\Definition\Field\Activation\EmptyActivation;
use Romm\Formz\Service\Traits\SelfInstantiateTrait;
use TYPO3\CMS\Core\SingletonInterface;
use TYPO3\CMS\Core\Utility\GeneralUtility;
Expand Down
2 changes: 1 addition & 1 deletion Classes/Condition/Parser/ConditionParserFactory.php
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@

namespace Romm\Formz\Condition\Parser;

use Romm\Formz\Configuration\Form\Field\Activation\ActivationInterface;
use Romm\Formz\Form\Definition\Field\Activation\ActivationInterface;
use Romm\Formz\Service\CacheService;
use Romm\Formz\Service\HashService;
use Romm\Formz\Service\Traits\SelfInstantiateTrait;
Expand Down
2 changes: 1 addition & 1 deletion Classes/Condition/Parser/ConditionTree.php
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
use Romm\Formz\Condition\Parser\Node\NodeInterface;
use Romm\Formz\Condition\Processor\ConditionProcessor;
use Romm\Formz\Condition\Processor\DataObject\PhpConditionDataObject;
use Romm\Formz\Configuration\Form\Field\Activation\ActivationInterface;
use Romm\Formz\Form\Definition\Field\Activation\ActivationInterface;
use TYPO3\CMS\Extbase\Error\Result;

/**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
namespace Romm\Formz\Condition\Parser\Node;

use Romm\Formz\Condition\Processor\ConditionProcessor;
use Romm\Formz\Configuration\Form\Field\Activation\ActivationInterface;
use Romm\Formz\Form\Definition\Field\Activation\ActivationInterface;

interface ActivationDependencyAwareInterface
{
Expand Down
2 changes: 1 addition & 1 deletion Classes/Condition/Parser/Node/ConditionNode.php
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
use Romm\Formz\Condition\Items\ConditionItemInterface;
use Romm\Formz\Condition\Processor\ConditionProcessor;
use Romm\Formz\Condition\Processor\DataObject\PhpConditionDataObject;
use Romm\Formz\Configuration\Form\Field\Activation\ActivationInterface;
use Romm\Formz\Form\Definition\Field\Activation\ActivationInterface;

/**
* A condition node, which contains an instance of `ConditionItemInterface`.
Expand Down
8 changes: 4 additions & 4 deletions Classes/Condition/Processor/ConditionProcessor.php
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,9 @@
use Romm\Formz\Condition\Parser\ConditionTree;
use Romm\Formz\Condition\Parser\Node\ConditionNode;
use Romm\Formz\Condition\Parser\Node\NodeInterface;
use Romm\Formz\Configuration\Form\Field\Activation\ActivationInterface;
use Romm\Formz\Configuration\Form\Field\Field;
use Romm\Formz\Configuration\Form\Field\Validation\Validation;
use Romm\Formz\Form\Definition\Field\Activation\ActivationInterface;
use Romm\Formz\Form\Definition\Field\Field;
use Romm\Formz\Form\Definition\Field\Validation\Validation;
use Romm\Formz\Form\FormObject;

class ConditionProcessor
Expand Down Expand Up @@ -100,7 +100,7 @@ public function getActivationConditionTreeForValidation(Validation $validation)
*/
public function calculateAllTrees()
{
$fields = $this->formObject->getConfiguration()->getFields();
$fields = $this->formObject->getDefinition()->getFields();

foreach ($fields as $field) {
$this->getActivationConditionTreeForField($field);
Expand Down
6 changes: 3 additions & 3 deletions Classes/Configuration/Configuration.php
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
use Romm\ConfigurationObject\Service\ServiceInterface;
use Romm\ConfigurationObject\Traits\ConfigurationObject\ArrayConversionTrait;
use Romm\ConfigurationObject\Traits\ConfigurationObject\DefaultConfigurationObjectTrait;
use Romm\Formz\Configuration\Form\Form;
use Romm\Formz\Form\Definition\FormDefinition;
use Romm\Formz\Configuration\Settings\Settings;
use Romm\Formz\Configuration\View\View;
use Romm\Formz\Exceptions\DuplicateEntryException;
Expand Down Expand Up @@ -101,7 +101,7 @@ public function addForm(FormObject $form)
throw DuplicateEntryException::formWasAlreadyRegistered($form);
}

$form->getConfiguration()->setParents([$this]);
$form->getDefinition()->setParents([$this]);

$this->forms[$form->getClassName()][$form->getName()] = $form;
}
Expand All @@ -119,7 +119,7 @@ public function hasForm($className, $name)
/**
* @param string $className
* @param string $name
* @return null|Form
* @return null|FormDefinition
*/
public function getForm($className, $name)
{
Expand Down

0 comments on commit cae979b

Please sign in to comment.