Selaa lähdekoodia

Se reemplaza el mÃtodo getAddress sobre el nas por un getData('address'). Se simplifican unas condiciones que aparecían en el méto.

Maxi Schvindt 7 vuotta sitten
vanhempi
commit
89a0a63f54
1 muutettua tiedostoa jossa 19 lisäystä ja 20 poistoa
  1. 19 20
      src/FTTHBundle/Controller/ONUController.php

+ 19 - 20
src/FTTHBundle/Controller/ONUController.php

@@ -206,39 +206,38 @@ class ONUController extends Controller
         foreach ($napsQuery as $nap) {
             if ($nap->getExtraData() != null) {
                 $nap->setContainer($this->container);
+                
+                // Seteamos la dirección - dirección o null
+                $address = $nap->getData('address');
+                
                 $dataNap = [
                     'id' => $nap->getId(),
                     'name' => $nap->getName(),
                     'distance' => -1,
                     'freePort' => $nap->getFreePort(),
-                    'address' => '',
+                    'address' => $address,
                     'olt' => ($nap->getOlt() != null ? $nap->getOlt()->getName() : "SIN OLT"),
                     'slot' => $nap->getSlot(),
                     'link' => $nap->getLink()
                 ];
+                
                 if ($nap->getlat() != null && $nap->getlng() != null) {
+
                     $distance = $geodecode->distanceGeoPoints($lat, $lng, $nap->getlat(), $nap->getlng(), true);
-                    if ($this->container->hasParameter('onu.distance.nap')) {
-                        if ($this->container->getParameter('onu.distance.nap') >= $distance ||
-                            $napId == $nap->getId()) {
-                            // controlo la distancia y el codigo de nap
-                            $dataNap['distance'] = $distance;
-                            $dataNap['address'] = $nap->getAddress();
-                            $naps[$nap->getId()] = $dataNap;
-                        }
-                    } else {
-                        if ($napId == $nap->getId()) {
-                            // si el codigo de napId coincide tengo que cargar la nap
-                            $dataNap['distance'] = $distance;
-                            $dataNap['address'] = $nap->getAddress();
-                            $naps[$nap->getId()] = $dataNap;
-                        }
-                    }
-                } else {
-                    if ($napId == $nap->getId()) {
-                        // si el codigo de napId coincide tengo que cargar la nap
+                    $dataNap['distance'] = $distance; //Seteamos la distancia
+
+                    $d = $this->container->hasParameter('onu.distance.nap');
+
+                    // Si tenemos distancia de referencia y se encuentra en rango, seteamos y continuamos
+                    if($d && ($this->container->getParameter('onu.distance.nap') >= $distance)) {
                         $naps[$nap->getId()] = $dataNap;
+                        continue;
                     }
+                } 
+
+                // Si tenemos un nap seleccionado de manera forzada
+                if ($napId == $nap->getId()) {
+                    $naps[$nap->getId()] = $dataNap;
                 }
             }
         }