Merge pull request #63 from niels-nijens/non-model-formfields

Fix adding formfields not available within the object model

Author: niels-nijens

Form/Type/FlexModelFormType.php

@@ -58,16 +58,19 @@ public function buildForm(FormBuilderInterface $builder, array $options)
             if (is_array($formConfiguration)) {
                 foreach ($formConfiguration['fields'] as $formFieldConfiguration) {
                     $fieldConfiguration = $this->flexModel->getField($objectName, $formFieldConfiguration['name']);
+                    if (is_array($fieldConfiguration) === false) {
+                        $fieldConfiguration = array();
+                    }
 
                     $fieldType = $this->getFieldType($formFieldConfiguration, $fieldConfiguration);
                     $fieldOptions = $this->getFieldOptions($formFieldConfiguration, $fieldConfiguration);
-                    $fieldName = $fieldConfiguration['name'];
+                    $fieldName = $formFieldConfiguration['name'];
                     if ($fieldType === FileType::class) {
                         $fieldName .= '_upload';
                     }
 
                     $builder->add($fieldName, $fieldType, $fieldOptions);
-                    if ($fieldConfiguration['datatype'] === 'HTML') {
+                    if (isset($fieldConfiguration['datatype']) && $fieldConfiguration['datatype'] === 'HTML') {
                         $builder->get($fieldName)->addModelTransformer(new HTMLPurifierTransformer());
                     }
                 }
@@ -156,13 +159,22 @@ private function getFieldType(array $formFieldConfiguration, array $fieldConfigu
     protected function getFieldOptions(array $formFieldConfiguration, array $fieldConfiguration)
     {
         $options = array(
-            'label' => $fieldConfiguration['label'],
+            'label' => '',
             'required' => false,
             'constraints' => array(),
         );
+        if (empty($fieldConfiguration)) {
+            $options['mapped'] = false;
+        }
+        if (isset($fieldConfiguration['label'])) {
+            $options['label'] = $fieldConfiguration['label'];
+        }
         if (isset($fieldConfiguration['required'])) {
             $options['required'] = $fieldConfiguration['required'];
         }
+        if (isset($formFieldConfiguration['label'])) {
+            $options['label'] = $formFieldConfiguration['label'];
+        }
         if (isset($formFieldConfiguration['widget'])) {
             $options['widget'] = $formFieldConfiguration['widget'];
         }
@@ -206,6 +218,10 @@ public function addFieldPlaceholder(array &$options, array $formFieldConfigurati
      */
     private function addFieldOptionsByDatatype(array &$options, array $fieldConfiguration)
     {
+        if (isset($fieldConfiguration['datatype']) === false) {
+            return;
+        }
+
         switch ($fieldConfiguration['datatype']) {
             case 'SET':
                 $options['multiple'] = true;