@@ -41,23 +41,23 @@ discard block |
||
41 | 41 | * @param int $count the number other bound |
42 | 42 | * @param bool $strict if true $network |
43 | 43 | */ |
44 | - public function __construct(Address $network, Address $netmask, bool $strict=true) |
|
44 | + public function __construct(Address $network, Address $netmask, bool $strict = true) |
|
45 | 45 | { |
46 | 46 | $finalNetmask = clone($netmask); |
47 | - if (!($netmask instanceof Netmask)) |
|
47 | + if(!($netmask instanceof Netmask)) |
|
48 | 48 | { |
49 | 49 | $finalNetmask = Netmask::fromAddress($netmask); |
50 | 50 | } |
51 | 51 | |
52 | 52 | // Verify parameters validity |
53 | - if( (($network->int() & $finalNetmask->int()) != $network->int()) && $strict) |
|
53 | + if((($network->int() & $finalNetmask->int())!=$network->int()) && $strict) |
|
54 | 54 | { |
55 | 55 | throw new RangeException(sprintf("Invalid network adress %s, this address is not usable with the netmask %s", (string)$network, (string)$finalNetmask)); |
56 | 56 | } |
57 | 57 | |
58 | 58 | $finalNetwork = new Address($network->int() & $finalNetmask->int()); |
59 | 59 | |
60 | - parent::__construct($finalNetwork, $finalNetwork->shift(count($finalNetmask)-1)); |
|
60 | + parent::__construct($finalNetwork, $finalNetwork->shift(count($finalNetmask) - 1)); |
|
61 | 61 | $this->netmask = $finalNetmask; |
62 | 62 | } |
63 | 63 | |
@@ -71,7 +71,7 @@ discard block |
||
71 | 71 | * @throws RangeException when network address and netmask does not form a valid subnet |
72 | 72 | * @return Subnet |
73 | 73 | */ |
74 | - public static function fromCidr(Address $network, int $cidr, bool $strict=true) |
|
74 | + public static function fromCidr(Address $network, int $cidr, bool $strict = true) |
|
75 | 75 | { |
76 | 76 | // Ensure CIDR is valid, use Address::fromCidr to have validation logic in only one place |
77 | 77 | $netmask = Netmask::fromCidr($cidr); |
@@ -108,10 +108,10 @@ discard block |
||
108 | 108 | * @throws InvaidArgumentException when provided sting cannot be converted to a valid subnet |
109 | 109 | * @return Subnet |
110 | 110 | */ |
111 | - public static function fromString(string $subnet, bool $strict=true) |
|
111 | + public static function fromString(string $subnet, bool $strict = true) |
|
112 | 112 | { |
113 | 113 | @list($address, $netmask, $trash) = explode('/', $subnet); |
114 | - if (!is_null($trash) || is_null($netmask)) |
|
114 | + if(!is_null($trash) || is_null($netmask)) |
|
115 | 115 | { |
116 | 116 | throw new InvalidArgumentException(sprintf("%s cannot be converted to subnet. Valid formats are : x.x.x.x/cidr or x.x.x.x/y.y.y.y", $subnet)); |
117 | 117 | } |
@@ -166,7 +166,7 @@ discard block |
||
166 | 166 | public function shift(int $offset) |
167 | 167 | { |
168 | 168 | return static::fromCidr( |
169 | - $this->getNetworkAddress()->shift($offset*count($this)), |
|
169 | + $this->getNetworkAddress()->shift($offset * count($this)), |
|
170 | 170 | $this->getNetmaskAddress()->asCidr() |
171 | 171 | ); |
172 | 172 | } |