Inspection of "Add missing phpdoc code" - rudolfcms/rudolf - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 02b993...3405ac )

by Mikołaj

created 2017-08-16 19:24 UTC

Status

Doc Comments +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -142,7 +142,7 @@
 block discarded – undo
      *
      * @param string $type
      *
-     * @return int
+     * @return string
      */
     public function modifierFullName($type = '')
     {

Please login to merge, or discard this patch.

src/module/Modules/Roll/Admin/Model.php 1 patch

Doc Comments +2 added lines, -1 removed lines patch added patch discarded remove patch

@@ -10,7 +10,6 @@  discard block
 block discarded – undo
     /**
      * Returns total number of modules items.
      *
-     * @param array|string $where
      *
      * @return int
      */
@@ -22,6 +21,8 @@  discard block
 block discarded – undo
     /**
      * Returns array with modules list.
      *
+     * @param integer $limit
+     * @param integer $onPage
      * @return array
      */
     public function getList($limit, $onPage)

Please login to merge, or discard this patch.

src/module/Pages/One/Admin/AddForm.php 1 patch

Doc Comments +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -23,7 +23,7 @@
 block discarded – undo
     }
 
     /**
-     * @return int
+     * @return false|string
      */
     public function save()
     {

Please login to merge, or discard this patch.

src/module/Pages/One/Admin/EditForm.php 1 patch

Doc Comments +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -23,7 +23,7 @@
 block discarded – undo
     }
 
     /**
-     * @return int
+     * @return boolean
      */
     public function update()
     {

Please login to merge, or discard this patch.

src/module/Pages/One/Admin/EditModel.php 1 patch

Doc Comments +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -11,7 +11,7 @@
 block discarded – undo
      *
      * @param array $f
      *
-     * @return int
+     * @return boolean
      */
     public function update($f)
     {

Please login to merge, or discard this patch.

src/module/Plugins/Roll/Admin/Model.php 1 patch

Doc Comments +2 added lines, -1 removed lines patch added patch discarded remove patch

@@ -10,7 +10,6 @@  discard block
 block discarded – undo
     /**
      * Returns total number of modules items.
      *
-     * @param array|string $where
      *
      * @return int
      */
@@ -22,6 +21,8 @@  discard block
 block discarded – undo
     /**
      * Returns array with modules list.
      *
+     * @param integer $limit
+     * @param integer $onPage
      * @return array|bool
      */
     public function getList($limit, $onPage)

Please login to merge, or discard this patch.

src/module/Tools/Admin/Roll/Model.php 1 patch

Doc Comments +2 added lines, -1 removed lines patch added patch discarded remove patch

@@ -17,7 +17,6 @@  discard block
 block discarded – undo
     /**
      * Returns total number of modules items.
      *
-     * @param array|string $where
      *
      * @return int
      */
@@ -29,6 +28,8 @@  discard block
 block discarded – undo
     /**
      * Returns array with modules list.
      *
+     * @param integer $limit
+     * @param integer $onPage
      * @return array|bool
      */
     public function getList($limit, $onPage)

Please login to merge, or discard this patch.

src/component/Logger/Logger.php 1 patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -25,7 +25,7 @@
 block discarded – undo
      */
     private function format($message)
     {
-        $request = isset($_SERVER['REQUEST_URI']) ? $_SERVER['REQUEST_URI']: '';
+        $request = isset($_SERVER['REQUEST_URI']) ? $_SERVER['REQUEST_URI'] : '';
 
         return sprintf(
             "[%s] %s | %s\n",

Please login to merge, or discard this patch.

src/component/ErrorHandler/Resources/templates/layout.html.php 1 patch

Spacing +16 added lines, -16 removed lines patch added patch discarded remove patch

@@ -2,37 +2,37 @@
 block discarded – undo
 <html>
   <head>
     <meta charset="utf-8">
-    <title><?=$this->pageTitle;?></title>
-    <style><?=$this->pageStyle;?></style>
+    <title><?=$this->pageTitle; ?></title>
+    <style><?=$this->pageStyle; ?></style>
   </head>
   <body>
     <section class="wrapper">
-      <h1><?=$this->message;?></h1>
+      <h1><?=$this->message; ?></h1>
       <p>
-        <span class="file"><?=$this->description['file'];?></span>
-        <span class="line"><?=$this->description['line'];?></span> throws
-        <span class="class"><?=$this->description['class'];?></span>
+        <span class="file"><?=$this->description['file']; ?></span>
+        <span class="line"><?=$this->description['line']; ?></span> throws
+        <span class="class"><?=$this->description['class']; ?></span>
       </p>
 
       <section class="trace">
         <h2>/* Stack trace */</h2>
         <ol><?php foreach ($this->trace as $key => $v): ?> 
           <li title="check args">
-            <span class="file"><?=$v['file'];?></span>
-            <span class="line"><?=$v['line'];?></span>
-            <span class="class"><?=$v['class'];?></span>
-            <span class="type"><?=$v['type'];?></span>
-            <span class="function"><?=$v['function'];?></span>
-            <pre><?=$v['args'];?></pre>
+            <span class="file"><?=$v['file']; ?></span>
+            <span class="line"><?=$v['line']; ?></span>
+            <span class="class"><?=$v['class']; ?></span>
+            <span class="type"><?=$v['type']; ?></span>
+            <span class="function"><?=$v['function']; ?></span>
+            <pre><?=$v['args']; ?></pre>
           </li>
-          <?php endforeach;?>
+          <?php endforeach; ?>
         </ol>
       </section>
     </section>
     <footer class="footer">
-      <span><?=VER_NAME;?></span>
-      <span>&copy; <?=date('Y');?> <?=NAME;?></span>
+      <span><?=VER_NAME; ?></span>
+      <span>&copy; <?=date('Y'); ?> <?=NAME; ?></span>
     </footer>
-    <script><?=$this->pageScript;?></script>
+    <script><?=$this->pageScript; ?></script>
   </body>
 </html>

Please login to merge, or discard this patch.

		@@ -142,7 +142,7 @@
		block discarded – undo
142	142	*
143	143	* @param string $type
144	144	*
145		- * @return int
	145	+ * @return string
146	146	*/
147	147	public function modifierFullName($type = '')
148	148	{

		@@ -10,7 +10,6 @@ discard block
		block discarded – undo
10	10	/**
11	11	* Returns total number of modules items.
12	12	*
13		- * @param array\|string $where
14	13	*
15	14	* @return int
16	15	*/
		@@ -22,6 +21,8 @@ discard block
		block discarded – undo
22	21	/**
23	22	* Returns array with modules list.
24	23	*
	24	+ * @param integer $limit
	25	+ * @param integer $onPage
25	26	* @return array
26	27	*/
27	28	public function getList($limit, $onPage)

		@@ -23,7 +23,7 @@
		block discarded – undo
23	23	}
24	24
25	25	/**
26		- * @return int
	26	+ * @return false\|string
27	27	*/
28	28	public function save()
29	29	{

		@@ -11,7 +11,7 @@
		block discarded – undo
11	11	*
12	12	* @param array $f
13	13	*
14		- * @return int
	14	+ * @return boolean
15	15	*/
16	16	public function update($f)
17	17	{

		@@ -17,7 +17,6 @@ discard block
		block discarded – undo
17	17	/**
18	18	* Returns total number of modules items.
19	19	*
20		- * @param array\|string $where
21	20	*
22	21	* @return int
23	22	*/
		@@ -29,6 +28,8 @@ discard block
		block discarded – undo
29	28	/**
30	29	* Returns array with modules list.
31	30	*
	31	+ * @param integer $limit
	32	+ * @param integer $onPage
32	33	* @return array\|bool
33	34	*/
34	35	public function getList($limit, $onPage)

		@@ -25,7 +25,7 @@
		block discarded – undo
25	25	*/
26	26	private function format($message)
27	27	{
28		- $request = isset($_SERVER['REQUEST_URI']) ? $_SERVER['REQUEST_URI']: '';
	28	+ $request = isset($_SERVER['REQUEST_URI']) ? $_SERVER['REQUEST_URI'] : '';
29	29
30	30	return sprintf(
31	31	"[%s] %s \| %s\n",

		@@ -2,37 +2,37 @@
		block discarded – undo
2	2	<html>
3	3	<head>
4	4	<meta charset="utf-8">
5		- <title><?=$this->pageTitle;?></title>
6		- <style><?=$this->pageStyle;?></style>
	5	+ <title><?=$this->pageTitle; ?></title>
	6	+ <style><?=$this->pageStyle; ?></style>
7	7	</head>
8	8	<body>
9	9	<section class="wrapper">
10		- <h1><?=$this->message;?></h1>
	10	+ <h1><?=$this->message; ?></h1>
11	11	<p>
12		- <span class="file"><?=$this->description['file'];?></span>
13		- <span class="line"><?=$this->description['line'];?></span> throws
14		- <span class="class"><?=$this->description['class'];?></span>
	12	+ <span class="file"><?=$this->description['file']; ?></span>
	13	+ <span class="line"><?=$this->description['line']; ?></span> throws
	14	+ <span class="class"><?=$this->description['class']; ?></span>
15	15	</p>
16	16
17	17	<section class="trace">
18	18	<h2>/* Stack trace */</h2>
19	19	<ol><?php foreach ($this->trace as $key => $v): ?>
20	20	<li title="check args">
21		- <span class="file"><?=$v['file'];?></span>
22		- <span class="line"><?=$v['line'];?></span>
23		- <span class="class"><?=$v['class'];?></span>
24		- <span class="type"><?=$v['type'];?></span>
25		- <span class="function"><?=$v['function'];?></span>
26		- <pre><?=$v['args'];?></pre>
	21	+ <span class="file"><?=$v['file']; ?></span>
	22	+ <span class="line"><?=$v['line']; ?></span>
	23	+ <span class="class"><?=$v['class']; ?></span>
	24	+ <span class="type"><?=$v['type']; ?></span>
	25	+ <span class="function"><?=$v['function']; ?></span>
	26	+ <pre><?=$v['args']; ?></pre>
27	27	</li>
28		- <?php endforeach;?>
	28	+ <?php endforeach; ?>
29	29	</ol>
30	30	</section>
31	31	</section>
32	32	<footer class="footer">
33		- <span><?=VER_NAME;?></span>
34		- <span>© <?=date('Y');?> <?=NAME;?></span>
	33	+ <span><?=VER_NAME; ?></span>
	34	+ <span>© <?=date('Y'); ?> <?=NAME; ?></span>
35	35	</footer>
36		- <script><?=$this->pageScript;?></script>
	36	+ <script><?=$this->pageScript; ?></script>
37	37	</body>
38	38	</html>

rudolfcms / rudolf

Push — master ( 02b993...3405ac )

Status

Category

Doc Comments +1 added lines, -1 removed lines patch added patch discarded remove patch

Doc Comments +2 added lines, -1 removed lines patch added patch discarded remove patch

Doc Comments +1 added lines, -1 removed lines patch added patch discarded remove patch

Doc Comments +1 added lines, -1 removed lines patch added patch discarded remove patch

Doc Comments +1 added lines, -1 removed lines patch added patch discarded remove patch

Doc Comments +2 added lines, -1 removed lines patch added patch discarded remove patch

Doc Comments +2 added lines, -1 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Spacing +16 added lines, -16 removed lines patch added patch discarded remove patch