akoSalman / Shorturl

src/Shorturl.php

Braces +3 added lines, -2 removed lines

@@ -53,8 +53,9 @@
 		

 		private function getDriverInstance ()

 		{

-			if (!$this->driver_instance)

-				$this->setDriverInstance();

+			if (!$this->driver_instance) {

+							$this->setDriverInstance();

+			}

 			

 			return $this->driver_instance;

 		}

src/drivers/LocalDriver.php

Braces +25 added lines, -18 removed lines

@@ -46,8 +46,9 @@ 
 

             // Check if given url has been shorten previously

             $duplicate = Link::where(['long_path' => $this->path])->first();

-            if ($duplicate)

-                return $duplicate->base_url . "/" . $duplicate->short_path;

+            if ($duplicate) {

+                            return $duplicate->base_url . "/" . $duplicate->short_path;

+            }

 

             $short_path = $this->getNextShortpath();

 

@@ -77,8 +78,9 @@ 
         {

             $min_length = $this->config['min_length'];

             $short_path = "";

-            for ($i = 0; $i < $min_length; $i++)

-                $short_path .= $this->head;

+            for ($i = 0; $i < $min_length; $i++) {

+                            $short_path .= $this->head;

+            }

             return $short_path;

         }

 

@@ -91,8 +93,9 @@ 
          */

         private function findNextPerm (string $current_perm) :string

 		{

-			if (!strlen($current_perm))

-			    return $this->head;

+			if (!strlen($current_perm)) {

+						    return $this->head;

+			}

 

 			$arr = array_reverse(str_split($current_perm));

 			foreach($arr as $key => $current_char) {

@@ -120,12 +123,15 @@ 
         {

             $parse = parse_url($url);

             $path = "";

-            if ($parse['path'] ?? null)

-                $path .= str::replaceFirst("/", "", $parse['path']);

-            if ($parse['query'] ?? null)

-                $path .= "?" . $parse['query'];

-            if ($parse['fragment'] ?? null)

-                $path .= "#" . $parse['fragment'];

+            if ($parse['path'] ?? null) {

+                            $path .= str::replaceFirst("/", "", $parse['path']);

+            }

+            if ($parse['query'] ?? null) {

+                            $path .= "?" . $parse['query'];

+            }

+            if ($parse['fragment'] ?? null) {

+                            $path .= "#" . $parse['fragment'];

+            }

 

             $this->base_url = str_replace("/" . $path, "", $url);

             $this->path = $path;

@@ -151,20 +157,21 @@ 
             // so we must find the latest one(short_path) in the permutation of the main_str

             $latest = collect(\DB::select("SELECT short_path FROM $tbl WHERE created_at = (SELECT MAX(created_at) FROM $tbl)"));

 

-            if (!$latest->count())

-                return $this->getFirstUrl();

+            if (!$latest->count()) {

+                            return $this->getFirstUrl();

+            }

 

             if ($latest->count() == 1) {

                 return $this->findNextPerm($latest->first()->short_path);

-            }

-            else {

+            } else {

                 foreach ($latest->reverse() as $key => $item) {

                     $next = $this->findNextPerm($item->short_path);

 

                     // If next permutation of current item is in fetched items

                     // find next permutation of the next fetched one

-                    if (!$latest->contains("short_path", $next))

-                        return $next;

+                    if (!$latest->contains("short_path", $next)) {

+                                            return $next;

+                    }

                 }

             }

         }