Completed
Pull Request — master (#40)
by Mr
05:34
created
src/Client.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
     /**
54 54
      * Client constructor.
55 55
      *
56
-     * @param array|\RouterOS\Interfaces\ConfigInterface $config      Array with configuration or Config object
56
+     * @param Interfaces\ConfigInterface $config      Array with configuration or Config object
57 57
      * @param bool                                       $autoConnect If false it will skip auto-connect stage if not need to instantiate connection
58 58
      *
59 59
      * @throws \RouterOS\Exceptions\ClientException
@@ -401,7 +401,7 @@  discard block
 block discarded – undo
401 401
      * @param string     $value    Value which should be parsed
402 402
      * @param array      $result   Array with parsed response
403 403
      * @param null|array $matches  Matched words
404
-     * @param string|int $iterator Type of iterations or number of item
404
+     * @param integer $iterator Type of iterations or number of item
405 405
      */
406 406
     private function preParseResponse(string $value, array &$result, ?array &$matches, $iterator = 'after'): void
407 407
     {
Please login to merge, or discard this patch.
configs/routeros-api.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -15,11 +15,11 @@  discard block
 block discarded – undo
15 15
      */
16 16
 
17 17
     'host'     => '192.168.88.1', // Address of Mikrotik RouterOS
18
-    'user'     => 'admin',        // Username
19
-    'pass'     => null,           // Password
20
-    'port'     => 8728,           // RouterOS API port number for access (if not set use default or default with SSL if SSL enabled)
21
-    'ssl'      => false,          // Enable ssl support (if port is not set this parameter must change default port to ssl port)
22
-    'ssh_port' => 22,             // Number of SSH port
18
+    'user'     => 'admin', // Username
19
+    'pass'     => null, // Password
20
+    'port'     => 8728, // RouterOS API port number for access (if not set use default or default with SSL if SSL enabled)
21
+    'ssl'      => false, // Enable ssl support (if port is not set this parameter must change default port to ssl port)
22
+    'ssh_port' => 22, // Number of SSH port
23 23
 
24 24
     /*
25 25
      |--------------------------------------------------------------------------
@@ -32,8 +32,8 @@  discard block
 block discarded – undo
32 32
      */
33 33
 
34 34
     'legacy'   => false, // Support of legacy login scheme (true - pre 6.43, false - post 6.43)
35
-    'timeout'  => 10,    // Max timeout for answer from RouterOS
36
-    'attempts' => 10,    // Count of attempts to establish TCP session
37
-    'delay'    => 1,     // Delay between attempts in seconds
35
+    'timeout'  => 10, // Max timeout for answer from RouterOS
36
+    'attempts' => 10, // Count of attempts to establish TCP session
37
+    'delay'    => 1, // Delay between attempts in seconds
38 38
 
39 39
 ];
Please login to merge, or discard this patch.
src/Config.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -60,9 +60,9 @@
 block discarded – undo
60 60
      * List of allowed parameters of config
61 61
      */
62 62
     public const ALLOWED = [
63
-        'host'     => 'string',  // Address of Mikrotik RouterOS
64
-        'user'     => 'string',  // Username
65
-        'pass'     => 'string',  // Password
63
+        'host'     => 'string', // Address of Mikrotik RouterOS
64
+        'user'     => 'string', // Username
65
+        'pass'     => 'string', // Password
66 66
         'port'     => 'integer', // RouterOS API port number for access (if not set use default or default with SSL if SSL enabled)
67 67
         'ssl'      => 'boolean', // Enable ssl support (if port is not set this parameter must change default port to ssl port)
68 68
         'legacy'   => 'boolean', // Support of legacy login scheme (true - pre 6.43, false - post 6.43)
Please login to merge, or discard this patch.