Explorar el Código

Se modifican traducciones. Se mejora el validador de yaml de workflows.

Maximiliano Schvindt hace 8 años
padre
commit
a6efb73f95

+ 1 - 1
Resources/translations/WorkflowBundle.es.yml

@@ -82,7 +82,7 @@ show:
     label_actions: Actions
     label_entity_class: Entidad
 helps:
-    workflow_label_name: Nombre con el que se identificará el wWrkflow. Se convertirá a minúscula y los espacios se reemplazarán por "_"
+    workflow_label_name: Nombre con el que se identificará el workflow. Se convertirá a minúscula y los espacios se reemplazarán por "_"
     workflow_label_type: "Tipo de Workflow: state_machine o workflow"
     workflow_label_marking_type: "Tipo de Marcador: single_state o multiple_state"
     workflow_label_marking_name: "Nombre del Marcador, atributo de la entidad que tendrá el estado, por defecto: currentState"

+ 10 - 2
Validator/Constraints/ContainsYamlValidator.php

@@ -33,6 +33,11 @@ class ContainsYamlValidator extends ConstraintValidator
             return;
         }
 
+        if(!is_array($yaml)) {
+            $this->context->buildViolation("errors.workflow_template_format_error")->setParameter('%string%', "")->addViolation();
+            return;
+        }
+
         if($yaml) {
             $no_valid_key = false;
             foreach($yaml as $key => $d) {
@@ -75,8 +80,11 @@ class ContainsYamlValidator extends ConstraintValidator
         $transitions = array();
         if(isset($yaml['transitions'])) {
             foreach($yaml['transitions'] as $k => $data) {
-
-                $transitions[] = new Transition($k,$data['from'],$data['to']);
+                if(isset($data['from']) && isset($data['to'])) {
+                    $transitions[] = new Transition($k,$data['from'],$data['to']);
+                } else {
+                    $this->context->buildViolation("errors.workflow_template_transitions_format")->setParameter('%transition%', $k)->addViolation();
+                }
             }
         } else {
             $this->context->buildViolation("errors.workflow_template_transitions_undefined")->setParameter('%string%', "")->addViolation();