Move retrieval of env variable to the geofence service. #271

.env.dist

@@ -44,4 +44,4 @@ POLICY_MOBILE=insertmobilepolicyidhere
 OTP_MODE=settotestorrandom
 
 # geofence
-GEOFENCE=settotrueorfalse
+GEOFENCE_ENABLE=settotrueorfalse

config/services.yaml

@@ -87,6 +87,10 @@ services:
             $password: "%env(REDIS_CLIENT_PASSWORD)%"
             $env_flag: "dev"
 
+    App\Service\GeofenceTracker:
+        arguments:
+            $geofence_flag: "%env(GEOFENCE_ENABLE)%"
+
     Catalyst\APIBundle\Security\APIKeyUserProvider:
         arguments:
             $em: "@doctrine.orm.entity_manager"

src/Controller/APIController.php

@@ -840,14 +840,6 @@ class APIController extends Controller
         $lat = $req->request->get('lat');
 
         // geofence
-        // check if geofence is on
-        $dotenv = new Dotenv();
-        $dotenv->loadEnv(__DIR__.'/../../.env');
-
-        $geo_flag = $_ENV['GEOFENCE'];
-
-        if ($geo_flag == 'true')
-        {
         $is_covered = $geo->isCovered($long, $lat);
         if (!$is_covered)
         {
@@ -856,7 +848,6 @@ class APIController extends Controller
                 ->setErrorMessage('Oops! Our service is limited to some areas in Laguna and Baguio, and Metro Manila only. We will update you as soon as we are able to cover your area');
             return $res->getReturnResponse();
         }
-        }
 
         $jo = new JobOrder();
         $jo->setSource(TransactionOrigin::MOBILE_APP)

src/Service/GeofenceTracker.php

@@ -11,13 +11,18 @@ use CrEOF\Spatial\PHP\Types\Geometry\Point;
 class GeofenceTracker
 {
     protected $em;
+    protected $geofence_flag;
 
-    public function __construct(EntityManagerInterface $em)
+    public function __construct(EntityManagerInterface $em, $geofence_flag)
     {
         $this->em = $em;
+        $this->geofence_flag = $geofence_flag;
     }
 
     public function isCovered($long, $lat)
+    {
+        // check if geofence is enabled
+        if ($this->geofence_flag  == 'true')
         { 
             // see if the point is in any of the polygons
             $query = $this->em->createQuery('SELECT count(s) from App\Entity\SupportedArea s where st_contains(s.coverage_area, point(:long, :lat)) = true')
@@ -32,4 +37,7 @@ class GeofenceTracker
 
             return false;
         }
+
+        return true;
+    }
 }