Se agregan traducciones y se mejora el chequeo de url y elementos (para

no consultar de manera innecesaria)

src/FTTHBundle/Controller/ONUController.php

@@ -27,32 +27,46 @@ class ONUController extends Controller
 
     public function importAction() 
     {
-        //$em = $this->get('doctrine')->getManager();
         $adminPool = $this->get('sonata.admin.pool');
         $webservice = $this->get("webservice");
+        $translator = $this->get('translator');
+        $flashbag = $this->get('session')->getFlashBag();
 
-        $urlMaps = $this->getParameter('remote_get_map_url');
-
-        /* Chequeamos que existe el objectType ONU - Esto debe venir definido por defecto luego */
-        $urlObjectTypes = $this->getParameter('remote_get_object_type_url');
-        $filters = array('name' => 'ONU');
-        $_types = $webservice->getData($urlObjectTypes, $filters, array(), null, null);
+        $urlObjects = $this->container->hasParameter('remote_get_objects_url');
+        $urlObjectTypes = $this->container->hasParameter('remote_get_object_type_url');
+        $_types = array(); 
         $disabled = false;
-        $flashbag = $this->get('session')->getFlashBag();
-        if(empty($_types)) {
-            $flashbag->add("error", "No existen ONUs definidas en el módulo de Mapas.");
+        $objectTypeId = 0;
+
+        if(!$urlObjects) {
+            $flashbag->add("error", $translator->trans("msg_no_defined_ftth_url",array(),"FTTHBundle"));
             $disabled = true;
-            $objectTypeId = 0;
         } else {
-            $objectTypeId = $_types[0]['id'];
+            if(!$urlObjectTypes) {
+                $flashbag->add("error", $translator->trans("msg_no_defined_ftth_url",array(),"FTTHBundle"));
+                $disabled = true;
+            } else {
+                $urlObjectTypes = $this->getParameter('remote_get_object_type_url');
+                $filters = array('name' => 'ONU');
+                $_types = $webservice->getData($urlObjectTypes, $filters, array(), null, null);
+            }
+            
+            if(empty($_types)) {
+                $flashbag->add("error", $translator->trans("msg_no_onu_in_map",array(),"FTTHBundle"));
+                $disabled = true;
+                $objectTypeId = 0;
+            } else {
+                $disabled = false;
+                $objectTypeId = $_types[0]['id'];
+            }
         }
         
         $form = $this->createFormBuilder()
-        ->add('mapId', RemoteMapType::class, array('label' => 'Mapa', 'disabled' => $disabled))
-        ->add('clientId', RemoteClientType::class, array('label' => 'Cliente', 'disabled' => $disabled))
+        ->add('mapId', RemoteMapType::class, array('label' => $translator->trans("form.label_map",array(),"FTTHBundle"), 'disabled' => $disabled))
+        ->add('clientId', RemoteClientType::class, array('label' => $translator->trans("form.label_client",array(),"FTTHBundle"), 'disabled' => $disabled))
         ->getForm();
         
-        $flashbag->add("warning", "Seleccione el Mapa desde el que se importarán las ONUs y el cliente al que se asociarán.");
+        $flashbag->add("warning", $translator->trans("msg_select_map",array(),"FTTHBundle"));
         
         return $this->render('FTTHBundle:ONU:onu_import.html.twig', array(
             'base_template' => $adminPool->getTemplate('layout'),

src/FTTHBundle/Resources/translations/FTTHBundle.es.yml

@@ -98,6 +98,8 @@ form:
     label_time_pon_stats: Estadísticas de PON PORTs
     label_time_onu_stats: Estadísticas de ONUs
     label_time_olt_octets: Tráfico de OLT
+    label_map: Mapa
+    label_client: Cliente
 
 list:
     label_id: Id
@@ -179,5 +181,13 @@ Undefined State: Estado no definido
 Undefined Workflow: Workflow no definido
 Incorrect Workflow: El workflow no existe o no es compatible con este objeto
 link_action_import_from_map: Importar desde Mapa
-No existen ONUs definidas en el módulo de Mapas.: No existen ONUs definidas en el módulo de Mapas.
-Seleccione el Mapa desde el que se importarán las ONUs y el cliente al que se asociarán.: Seleccione el Mapa desde el que se importarán las ONUs y el cliente al que se asociarán.
+msg_no_defined_ftth_url: No se encuentran definidas las urls para obtener ONUs del módulo MAPA.
+msg_no_onu_in_map: No existen ONUs definidas en el módulo de Mapas.
+msg_select_map: Seleccione el Mapa desde el que se importarán las ONUs y el cliente al que se asociarán.
+import: Importador
+button_import: Importar
+alert_select_map_client: Debe seleccionar un mapa y cliente
+msg_import_onu: Importar ONUs
+msg_init_import: Iniciando importación, espere un momento.
+msg_end_import: FIN :)
+

src/FTTHBundle/Resources/views/ONU/onu_import.html.twig

@@ -20,7 +20,7 @@ footer {z-index: 1100;}
     <li>
         <a href="{{url('sonata_admin_dashboard')}}"><i class="fa fa-home"></i></a>
     </li>
-    <li class="active"><span>{{ 'Importar ONUs'|trans({}, 'FTTHBundle') }}</span></li>
+    <li class="active"><span>{{ 'msg_import_onu'|trans({}, 'FTTHBundle') }}</span></li>
 </ol>
 {% endblock %}
 
@@ -60,7 +60,7 @@ footer {z-index: 1100;}
                 <div class="box box-primary" style="border-top:0px">
                     <div class="box-header">
                         <h4 class="box-title">
-                            {{ 'Importador'|trans({}, 'FTTHBundle') }}
+                            {{ 'import'|trans({}, 'FTTHBundle') }}
                         </h4>
                     </div>
                     <div class="box-body">  
@@ -86,7 +86,7 @@ footer {z-index: 1100;}
     {% else %}
         <button class="btn btn-success" type="submit" id="btn_import">
     {% endif %}
-    <i class="fa fa-download" aria-hidden="true"></i>&nbsp;{{ 'Importar'|trans({}, 'FTTHBundle') }}
+    <i class="fa fa-download" aria-hidden="true"></i>&nbsp;{{ 'button_import'|trans({}, 'FTTHBundle') }}
     </button>
 </div>
 
@@ -109,13 +109,13 @@ $(document).ready(function() {
 
         if(!mapId || !clientId) {
             $("#btn_import").removeAttr('disabled');
-            alert("{{ 'Debe seleccionar un mapa y cliente'|trans({}, 'FTTHBundle') }}");
+            alert("{{ 'alert_select_map_client'|trans({}, 'FTTHBundle') }}");
             return;
         }
 
         count = 0;
 
-        $("#console-message").prepend("<span class='bold-title'>Iniciando importación, espere un momento.</span><br />");
+        $("#console-message").prepend("<span class='bold-title'>{{ 'msg_init_import'|trans({},'FTTHBundle') }}</span><br />");
         
         do {
             result = importData(mapId, limit, offset);
@@ -124,11 +124,11 @@ $(document).ready(function() {
             result.forEach(function(row) {
                 $("#console-message").prepend(row + "<br />");
             });
-            $("#console-message").prepend("<span class='bold-title'>Importando nuevos elementos, espere un momento.</span><br />");
+            $("#console-message").prepend("<span class='bold-title'>{{ 'msg_init_import'|trans({},'FTTHBundle') }}</span><br />");
             
         } while (result && result.length);
         
-        $("#console-message").prepend("<span class='bold-title'>Fin :)</span><br />");
+        $("#console-message").prepend("<span class='bold-title'>{{ 'msg_end_import'|trans({},'FTTHBundle') }}</span><br />");
         $("#btn_import").removeAttr('disabled');
         offset = 0;
     });