axyr / silverstripe-adminlogin

code/model/IpAccess.php

Braces +9 added lines, -3 removed lines

@@ -119,8 +119,10 @@ 
         $ipFirstPart = substr($this->ip, 0, strrpos($this->ip, '.') + 1);
         $ipLastpart  = substr(strrchr($this->ip, '.'), 1);
 
-        if (!empty($ranges)) foreach ($ranges as $range) {
+        if (!empty($ranges)) {
+        	foreach ($ranges as $range) {
             $rangeFirstPart = substr($range, 0, strrpos($range, '.') + 1);
+        }
             list ($start, $end) = explode('-', substr(strrchr($range, '.'), 1));
             if ($ipFirstPart === $rangeFirstPart && $ipLastpart >= $start && $ipLastpart <= $end) {
                 return $range;
@@ -141,8 +143,10 @@ 
             return strstr($ip, '/');
         });
 
-        if (!empty($ranges)) foreach ($ranges as $range) {
+        if (!empty($ranges)) {
+        	foreach ($ranges as $range) {
             list ($net, $mask) = explode('/', $cidr);
+        }
             if ((ip2long($this->ip) & ~((1 << (32 - $mask)) - 1)) == ip2long($net)) {
                 return $cidr;
             }
@@ -163,9 +167,11 @@ 
             return substr($ip, -1) === '*';
         });
 
-        if (!empty($ranges)) foreach ($ranges as $range) {
+        if (!empty($ranges)) {
+        	foreach ($ranges as $range) {
             if (substr($this->ip, 0, strlen(substr($range, 0, -1))) === substr($range, 0, -1)) {
                 return $range;
+        }
             }
         }
         return '';