Test Failed
Pull Request — master (#171)
by
unknown
04:28
created
examples/features/complex_usage_example.php 1 patch
Doc Comments   +7 added lines patch added patch discarded remove patch
@@ -41,10 +41,17 @@
 block discarded – undo
41 41
 		$this->method1(array(1, 2), new stdClass()); // this arguments will be displayed in error backtrace
42 42
 	}
43 43
 
44
+	/**
45
+	 * @param integer[] $a
46
+	 * @param stdClass $b
47
+	 */
44 48
 	protected function method1($a, $b) {
45 49
 		$this->method2('some long string argument');
46 50
 	}
47 51
 
52
+	/**
53
+	 * @param string $c
54
+	 */
48 55
 	public function method2($c) {
49 56
 		file_get_contents('not_existed.file'); // E_WARNING error
50 57
 	}
Please login to merge, or discard this patch.
examples/features/handle_errors.php 2 patches
Doc Comments   +7 added lines patch added patch discarded remove patch
@@ -14,10 +14,17 @@
 block discarded – undo
14 14
 		$this->method1(array(1, 2), new stdClass()); // this arguments will be displayed in error backtrace
15 15
 	}
16 16
 
17
+	/**
18
+	 * @param integer[] $a
19
+	 * @param stdClass $b
20
+	 */
17 21
 	protected function method1($a, $b) {
18 22
 		$this->method2('some long string argument');
19 23
 	}
20 24
 
25
+	/**
26
+	 * @param string $c
27
+	 */
21 28
 	public function method2($c) {
22 29
 		echo $undefinedVar; // E_NOTICE error
23 30
 		file_get_contents('not_existed.file'); // E_WARNING error
Please login to merge, or discard this patch.
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -30,8 +30,7 @@
 block discarded – undo
30 30
 
31 31
 try {
32 32
 	throw new Exception('Some caught exception');
33
-}
34
-catch(Exception $exception) {
33
+} catch(Exception $exception) {
35 34
 	$handler->handleException($exception);
36 35
 }
37 36
 
Please login to merge, or discard this patch.
src/PhpConsole/Dispatcher/Debug.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@
 block discarded – undo
23 23
 	 * Send debug data message to client
24 24
 	 * @param mixed $data
25 25
 	 * @param null|string $tags Tags separated by dot, e.g. "low.db.billing"
26
-	 * @param int|array $ignoreTraceCalls Ignore tracing classes by name prefix `array('PhpConsole')` or fixed number of calls to ignore
26
+	 * @param integer $ignoreTraceCalls Ignore tracing classes by name prefix `array('PhpConsole')` or fixed number of calls to ignore
27 27
 	 */
28 28
 	public function dispatchDebug($data, $tags = null, $ignoreTraceCalls = 0) {
29 29
 		if($this->isActive()) {
Please login to merge, or discard this patch.
src/PhpConsole/Dispatcher/Errors.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -52,7 +52,7 @@
 block discarded – undo
52 52
 	 * @param null|string $text
53 53
 	 * @param null|string $file
54 54
 	 * @param null|integer $line
55
-	 * @param int|array $ignoreTraceCalls Ignore tracing classes by name prefix `array('PhpConsole')` or fixed number of calls to ignore
55
+	 * @param integer $ignoreTraceCalls Ignore tracing classes by name prefix `array('PhpConsole')` or fixed number of calls to ignore
56 56
 	 */
57 57
 	public function dispatchError($code = null, $text = null, $file = null, $line = null, $ignoreTraceCalls = 0) {
58 58
 		if($this->isActive()) {
Please login to merge, or discard this patch.
src/PhpConsole/EvalProvider.php 2 patches
Doc Comments   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -69,8 +69,8 @@  discard block
 block discarded – undo
69 69
 
70 70
 	/**
71 71
 	 * Call added code handlers
72
-	 * @param $code
73
-	 * @return mixed
72
+	 * @param string $code
73
+	 * @return string
74 74
 	 */
75 75
 	protected function applyHandlersToCode($code) {
76 76
 		foreach($this->codeCallbackHandlers as $callback) {
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
 
108 108
 	/**
109 109
 	 * Execute code with shared vars
110
-	 * @param $_code
110
+	 * @param string $_code
111 111
 	 * @param array $_sharedVars
112 112
 	 * @return mixed
113 113
 	 */
@@ -163,7 +163,7 @@  discard block
 block discarded – undo
163 163
 	/**
164 164
 	 * Protect response code access only to specified directories using http://www.php.net/manual/en/ini.core.php#ini.open-basedir
165 165
 	 * IMPORTANT: classes autoload methods will work only for specified directories
166
-	 * @param array $openBaseDirs
166
+	 * @param string[] $openBaseDirs
167 167
 	 * @codeCoverageIgnore
168 168
 	 */
169 169
 	public function setOpenBaseDirs(array $openBaseDirs) {
Please login to merge, or discard this patch.
Braces   +3 added lines, -6 removed lines patch added patch discarded remove patch
@@ -40,11 +40,9 @@  discard block
 block discarded – undo
40 40
 		$startTime = microtime(true);
41 41
 		try {
42 42
 			$result->return = static::executeCode($code, $this->sharedVars);
43
-		}
44
-		catch(\Throwable $exception) {
43
+		} catch(\Throwable $exception) {
45 44
 			$result->exception = $exception;
46
-		}
47
-		catch(\Exception $exception) {
45
+		} catch(\Exception $exception) {
48 46
 			$result->exception = $exception;
49 47
 		}
50 48
 		$result->time = abs(microtime(true) - $startTime - $selfTime);
@@ -115,8 +113,7 @@  discard block
 block discarded – undo
115 113
 		foreach($_sharedVars as $var => $value) {
116 114
 			if(isset($GLOBALS[$var]) && $var[0] == '_') { // extract($this->sharedVars, EXTR_OVERWRITE) and $$var = $value do not overwrites global vars
117 115
 				$GLOBALS[$var] = $value;
118
-			}
119
-			elseif(!isset($$var)) {
116
+			} elseif(!isset($$var)) {
120 117
 				$$var = $value;
121 118
 			}
122 119
 		}
Please login to merge, or discard this patch.
src/PhpConsole/Handler.php 2 patches
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -174,7 +174,7 @@  discard block
 block discarded – undo
174 174
 	 * @param string|null $file
175 175
 	 * @param int|null $line
176 176
 	 * @param null $context
177
-	 * @param int|array $ignoreTraceCalls Ignore tracing classes by name prefix `array('PhpConsole')` or fixed number of calls to ignore
177
+	 * @param integer $ignoreTraceCalls Ignore tracing classes by name prefix `array('PhpConsole')` or fixed number of calls to ignore
178 178
 	 */
179 179
 	public function handleError($code = null, $text = null, $file = null, $line = null, $context = null, $ignoreTraceCalls = 0) {
180 180
 		if(!$this->isStarted || error_reporting() === 0 || $this->isHandlingDisabled() || ($this->errorsHandlerLevel && !($code & $this->errorsHandlerLevel))) {
@@ -238,7 +238,7 @@  discard block
 block discarded – undo
238 238
 	 * Handle debug data
239 239
 	 * @param mixed $data
240 240
 	 * @param string|null $tags Tags separated by dot, e.g. "low.db.billing"
241
-	 * @param int|array $ignoreTraceCalls Ignore tracing classes by name prefix `array('PhpConsole')` or fixed number of calls to ignore
241
+	 * @param integer $ignoreTraceCalls Ignore tracing classes by name prefix `array('PhpConsole')` or fixed number of calls to ignore
242 242
 	 */
243 243
 	public function debug($data, $tags = null, $ignoreTraceCalls = 0) {
244 244
 		if($this->connector->isActiveClient()) {
Please login to merge, or discard this patch.
Braces   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -224,11 +224,9 @@
 block discarded – undo
224 224
 			if($this->oldExceptionsHandler && $this->callOldHandlers) {
225 225
 				call_user_func($this->oldExceptionsHandler, $exception);
226 226
 			}
227
-		}
228
-		catch(\Throwable $internalException) {
227
+		} catch(\Throwable $internalException) {
229 228
 			$this->handleException($internalException);
230
-		}
231
-		catch(\Exception $internalException) {
229
+		} catch(\Exception $internalException) {
232 230
 			$this->handleException($internalException);
233 231
 		}
234 232
 		$this->onHandlingComplete();
Please login to merge, or discard this patch.
src/PhpConsole/Helper.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -86,7 +86,7 @@
 block discarded – undo
86 86
 		 * Analog of Handler::getInstance()->debug(...) method
87 87
 		 * @param mixed $data
88 88
 		 * @param string|null $tags Tags separated by dot, e.g. "low.db.billing"
89
-		 * @param int|array $ignoreTraceCalls Ignore tracing classes by name prefix `array('PhpConsole')` or fixed number of calls to ignore
89
+		 * @param integer $ignoreTraceCalls Ignore tracing classes by name prefix `array('PhpConsole')` or fixed number of calls to ignore
90 90
 		 */
91 91
 		public static function debug($data, $tags = null, $ignoreTraceCalls = 0) {
92 92
 			if(self::$isActive) {
Please login to merge, or discard this patch.
src/PhpConsole/Dispatcher.php 1 patch
Braces   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -89,11 +89,9 @@
 block discarded – undo
89 89
 				foreach($call['args'] as $arg) {
90 90
 					if(is_object($arg)) {
91 91
 						$args[] = get_class($arg);
92
-					}
93
-					elseif(is_array($arg)) {
92
+					} elseif(is_array($arg)) {
94 93
 						$args[] = 'Array[' . count($arg) . ']';
95
-					}
96
-					else {
94
+					} else {
97 95
 						$arg = var_export($arg, 1);
98 96
 						$args[] = strlen($arg) > 15 ? substr($arg, 0, 15) . '...\'' : $arg;
99 97
 					}
Please login to merge, or discard this patch.
src/PhpConsole/Dumper.php 1 patch
Braces   +4 added lines, -8 removed lines patch added patch discarded remove patch
@@ -80,8 +80,7 @@  discard block
 block discarded – undo
80 80
 				$nullPos = strrpos($key, chr(0));
81 81
 				if($nullPos) {
82 82
 					$dataArray[substr($key, $nullPos + 1)] = $value;
83
-				}
84
-				else {
83
+				} else {
85 84
 					$dataArray[$key] = $value;
86 85
 				}
87 86
 			}
@@ -114,18 +113,15 @@  discard block
 block discarded – undo
114 113
 						$hash = spl_object_hash($v);
115 114
 						if(in_array($hash, $objectsHashes)) {
116 115
 							$v = '*RECURSION*';
117
-						}
118
-						else {
116
+						} else {
119 117
 							$v = '(object)';
120 118
 							$objectsHashes[] = $hash;
121 119
 						}
122
-					}
123
-					else {
120
+					} else {
124 121
 						$v = '(array)';
125 122
 					}
126 123
 					$sizeLeft -= strlen($k) + strlen($v) + 8;
127
-				}
128
-				else {
124
+				} else {
129 125
 					$sizeLeft -= strlen($k) + 4;
130 126
 					$this->dumpVarData($v, $levelLimit - 1, false);
131 127
 				}
Please login to merge, or discard this patch.