chamilo / chamilo-lms

plugin/pens/lib/tests/test_pens_request.php

Doc Comments +5 added lines

@@ -76,6 +76,11 @@
 		return PENSRequestFactory::createPENSRequest($myargs);
 	}
 	
+	/**
+	 * @param string $key
+	 * @param string|null $value
+	 * @param integer $code
+	 */
 	public function exceptionTestForValue($key, $value, $code) {
 		try {
 			$myargs = $this->args;

Indentation +163 added lines, -163 removed lines

@@ -44,170 +44,170 @@
  */
 class TestPENSRequest extends UnitTestCase {
 	
-	/**
-	 * Valid arguments to be used for tests
-	 * 
-	 * @var array
-	 */
-	private $args = null;
-	
-	public function setUp() {
-		$this->args = array(
-			"pens-version" => "1.0.0",
-			"command" => "collect",
-			"package-type" => "aicc-pkg",
-			"package-type-version" => "1.0",
-			"package-format" => "zip",
-			"package-id" => "http://myurl.com/12345",
-			"package-url" => "http://myurl.com/mypackage.zip",
-			"package-url-expiry" => "2006-04-01T06:51:29Z",
-			"client" => "Authorware7",
-			"receipt" => "mailto:name@domain.com",
-			"package-url-user-id" => "guillaumev",
-			"package-url-account" => "toto",
-			"package-url-password" => "12345",
-			"system-user-id" => "guillaumev",
-			"system-password" => "12345",
-			"alerts" => "http://myurl.com/alerts",
-			"vendor-data" => "here are my data");
-	}
-	
-	public function createObject($myargs) {
-		return PENSRequestFactory::createPENSRequest($myargs);
-	}
-	
-	public function exceptionTestForValue($key, $value, $code) {
-		try {
-			$myargs = $this->args;
-			if($value === null) {
-				unset($myargs[$key]);
-			} else {
-				$myargs[$key] = $value;
-			}
-			$object = $this->createObject($myargs);
-			$this->fail();
-		} catch(PENSException $e) {
-			$this->assertEqual($e->getCode(), $code);
-		}
-	}
-	
-	public function testPensVersionInvalid() {
-		$this->exceptionTestForValue("pens-version", "0.8.0", 2001);
-	}
-	
-	public function testPensVersionNull() {
-		$this->exceptionTestForValue("pens-version", null, 2001);
-	}
-	
-	
-	public function testCommandInvalid() {
-		$this->exceptionTestForValue("command", "testing", 2002);
-	}
-	
-	public function testCommandNull() {
-		$this->exceptionTestForValue("command", null, 2002);
-	}
-	
-	public function testPackageTypeInvalid() {
-		$this->exceptionTestForValue("package-type", "testing", 2003);
-	}
-	
-	public function testPackageTypeNull() {
-		$this->exceptionTestForValue("package-type", null, 2003);
-	}
-	
-	public function testPackageTypeVersionNull() {
-		$this->exceptionTestForValue("package-type-version", null, 2004);
-	}
-	
-	public function testPackageFormatInvalid() {
-		$this->exceptionTestForValue("package-format", "testing", 2005);
-	}
-	
-	public function testPackageFormatNull() {
-		$this->exceptionTestForValue("package-format", null, 2005);
-	}
-	
-	public function testPackageIdInvalid() {
-		$this->exceptionTestForValue("package-id", "testing", 2007);
-	}
-	
-	public function testPackageIdNull() {
-		$this->exceptionTestForValue("package-id", null, 2007);
-	}
-	
-	public function testPackageUrlInvalid() {
-		$this->exceptionTestForValue("package-url", "testing", 2008);
-	}
-	
-	public function testPackageUrlInvalid2() {
-		$this->exceptionTestForValue("package-url", "http://myurl.com/mypackage", 2008);
-	}
-	
-	public function testPackageUrlNull() {
-		$this->exceptionTestForValue("package-url", null, 2008);
-	}
-	
-	public function testPackageUrlExpiryNull() {
-		$this->exceptionTestForValue("package-url-expiry", null, 2009);
-	}
-	
-	public function testPackageUrlExpiryInvalid() {
-		$this->exceptionTestForValue("package-url-expiry", "testing", 2009);
-	}
-	
-	public function testClientNull() {
-		$this->exceptionTestForValue("client", null, 2010);
-	}
-	
-	public function testReceiptNull() {
-		$this->exceptionTestForValue("receipt", null, 2011);
-	}
-	
-	public function testReceiptInvalid() {
-		$this->exceptionTestForValue("receipt", "testing", 2011);
-	}
-	
-	public function testAlertsInvalid() {
-		$this->exceptionTestForValue("alerts", "testing", 1201);
-	}
-	
-	public function testReceiptValid() {
-		try {
-			$myargs = $this->args;
-			$myargs["receipt"] = "mailto:name@domain.com,name2@domain.com";
-			$object = $this->createObject($myargs);
-			$this->pass();
-		} catch(PENSException $e) {
-			$this->fail();
-		}
-	}
-	
-	public function testValid() {
-		try {
-			$object = $this->createObject($this->args);
-			$this->assertIsA($object, "PENSRequestCollect");
-			$this->assertEqual($object->getPensVersion(), "1.0.0");
-			$this->assertEqual($object->getPackageType(), "aicc-pkg");
-			$this->assertEqual($object->getPackageTypeVersion(), "1.0");
-			$this->assertEqual($object->getPackageFormat(), "zip");
-			$this->assertEqual($object->getPackageId(), "http://myurl.com/12345");
-			$this->assertEqual($object->getPackageUrl(), "http://myurl.com/mypackage.zip");
-			$this->assertIsA($object->getPackageUrlExpiry(), "DateTime");
-			$this->assertEqual($object->getClient(), "Authorware7");
-			$this->assertEqual($object->getReceipt(), "mailto:name@domain.com");
-			$this->assertEqual($object->getPackageUrlUserId(), "guillaumev");
-			$this->assertEqual($object->getPackageUrlAccount(), "toto");
-			$this->assertEqual($object->getPackageUrlPassword(), "12345");
-			$this->assertEqual($object->getSystemUserId(), "guillaumev");
-			$this->assertEqual($object->getSystemPassword(), "12345");
-			$this->assertEqual($object->getAlerts(), "http://myurl.com/alerts");
-			$this->assertEqual($object->getVendorData(), "here are my data");
+    /**
+     * Valid arguments to be used for tests
+     * 
+     * @var array
+     */
+    private $args = null;
+	
+    public function setUp() {
+        $this->args = array(
+            "pens-version" => "1.0.0",
+            "command" => "collect",
+            "package-type" => "aicc-pkg",
+            "package-type-version" => "1.0",
+            "package-format" => "zip",
+            "package-id" => "http://myurl.com/12345",
+            "package-url" => "http://myurl.com/mypackage.zip",
+            "package-url-expiry" => "2006-04-01T06:51:29Z",
+            "client" => "Authorware7",
+            "receipt" => "mailto:name@domain.com",
+            "package-url-user-id" => "guillaumev",
+            "package-url-account" => "toto",
+            "package-url-password" => "12345",
+            "system-user-id" => "guillaumev",
+            "system-password" => "12345",
+            "alerts" => "http://myurl.com/alerts",
+            "vendor-data" => "here are my data");
+    }
+	
+    public function createObject($myargs) {
+        return PENSRequestFactory::createPENSRequest($myargs);
+    }
+	
+    public function exceptionTestForValue($key, $value, $code) {
+        try {
+            $myargs = $this->args;
+            if($value === null) {
+                unset($myargs[$key]);
+            } else {
+                $myargs[$key] = $value;
+            }
+            $object = $this->createObject($myargs);
+            $this->fail();
+        } catch(PENSException $e) {
+            $this->assertEqual($e->getCode(), $code);
+        }
+    }
+	
+    public function testPensVersionInvalid() {
+        $this->exceptionTestForValue("pens-version", "0.8.0", 2001);
+    }
+	
+    public function testPensVersionNull() {
+        $this->exceptionTestForValue("pens-version", null, 2001);
+    }
+	
+	
+    public function testCommandInvalid() {
+        $this->exceptionTestForValue("command", "testing", 2002);
+    }
+	
+    public function testCommandNull() {
+        $this->exceptionTestForValue("command", null, 2002);
+    }
+	
+    public function testPackageTypeInvalid() {
+        $this->exceptionTestForValue("package-type", "testing", 2003);
+    }
+	
+    public function testPackageTypeNull() {
+        $this->exceptionTestForValue("package-type", null, 2003);
+    }
+	
+    public function testPackageTypeVersionNull() {
+        $this->exceptionTestForValue("package-type-version", null, 2004);
+    }
+	
+    public function testPackageFormatInvalid() {
+        $this->exceptionTestForValue("package-format", "testing", 2005);
+    }
+	
+    public function testPackageFormatNull() {
+        $this->exceptionTestForValue("package-format", null, 2005);
+    }
+	
+    public function testPackageIdInvalid() {
+        $this->exceptionTestForValue("package-id", "testing", 2007);
+    }
+	
+    public function testPackageIdNull() {
+        $this->exceptionTestForValue("package-id", null, 2007);
+    }
+	
+    public function testPackageUrlInvalid() {
+        $this->exceptionTestForValue("package-url", "testing", 2008);
+    }
+	
+    public function testPackageUrlInvalid2() {
+        $this->exceptionTestForValue("package-url", "http://myurl.com/mypackage", 2008);
+    }
+	
+    public function testPackageUrlNull() {
+        $this->exceptionTestForValue("package-url", null, 2008);
+    }
+	
+    public function testPackageUrlExpiryNull() {
+        $this->exceptionTestForValue("package-url-expiry", null, 2009);
+    }
+	
+    public function testPackageUrlExpiryInvalid() {
+        $this->exceptionTestForValue("package-url-expiry", "testing", 2009);
+    }
+	
+    public function testClientNull() {
+        $this->exceptionTestForValue("client", null, 2010);
+    }
+	
+    public function testReceiptNull() {
+        $this->exceptionTestForValue("receipt", null, 2011);
+    }
+	
+    public function testReceiptInvalid() {
+        $this->exceptionTestForValue("receipt", "testing", 2011);
+    }
+	
+    public function testAlertsInvalid() {
+        $this->exceptionTestForValue("alerts", "testing", 1201);
+    }
+	
+    public function testReceiptValid() {
+        try {
+            $myargs = $this->args;
+            $myargs["receipt"] = "mailto:name@domain.com,name2@domain.com";
+            $object = $this->createObject($myargs);
+            $this->pass();
+        } catch(PENSException $e) {
+            $this->fail();
+        }
+    }
+	
+    public function testValid() {
+        try {
+            $object = $this->createObject($this->args);
+            $this->assertIsA($object, "PENSRequestCollect");
+            $this->assertEqual($object->getPensVersion(), "1.0.0");
+            $this->assertEqual($object->getPackageType(), "aicc-pkg");
+            $this->assertEqual($object->getPackageTypeVersion(), "1.0");
+            $this->assertEqual($object->getPackageFormat(), "zip");
+            $this->assertEqual($object->getPackageId(), "http://myurl.com/12345");
+            $this->assertEqual($object->getPackageUrl(), "http://myurl.com/mypackage.zip");
+            $this->assertIsA($object->getPackageUrlExpiry(), "DateTime");
+            $this->assertEqual($object->getClient(), "Authorware7");
+            $this->assertEqual($object->getReceipt(), "mailto:name@domain.com");
+            $this->assertEqual($object->getPackageUrlUserId(), "guillaumev");
+            $this->assertEqual($object->getPackageUrlAccount(), "toto");
+            $this->assertEqual($object->getPackageUrlPassword(), "12345");
+            $this->assertEqual($object->getSystemUserId(), "guillaumev");
+            $this->assertEqual($object->getSystemPassword(), "12345");
+            $this->assertEqual($object->getAlerts(), "http://myurl.com/alerts");
+            $this->assertEqual($object->getVendorData(), "here are my data");
 			
-		} catch(PENSException $e) {
-			$this->fail();
-		}
-	}
+        } catch(PENSException $e) {
+            $this->fail();
+        }
+    }
 	
 	
 }

Spacing +4 added lines, -4 removed lines

@@ -79,14 +79,14 @@ 
 	public function exceptionTestForValue($key, $value, $code) {
 		try {
 			$myargs = $this->args;
-			if($value === null) {
+			if ($value === null) {
 				unset($myargs[$key]);
 			} else {
 				$myargs[$key] = $value;
 			}
 			$object = $this->createObject($myargs);
 			$this->fail();
-		} catch(PENSException $e) {
+		} catch (PENSException $e) {
 			$this->assertEqual($e->getCode(), $code);
 		}
 	}
@@ -178,7 +178,7 @@ 
 			$myargs["receipt"] = "mailto:name@domain.com,name2@domain.com";
 			$object = $this->createObject($myargs);
 			$this->pass();
-		} catch(PENSException $e) {
+		} catch (PENSException $e) {
 			$this->fail();
 		}
 	}
@@ -204,7 +204,7 @@ 
 			$this->assertEqual($object->getAlerts(), "http://myurl.com/alerts");
 			$this->assertEqual($object->getVendorData(), "here are my data");
 			
-		} catch(PENSException $e) {
+		} catch (PENSException $e) {
 			$this->fail();
 		}
 	}

plugin/search_course/lib/register_course_widget.class.php

Doc Comments +2 added lines, -2 removed lines

@@ -57,7 +57,7 @@ 
     /**
      * Handle the subscribe action.
      *
-     * @return bool
+     * @return null|boolean
      */
     function action_subscribe_user()
     {
@@ -151,7 +151,7 @@ 
 
     /**
      *
-     * @param type $course_code
+     * @param string $course_code
      * @return type
      */
     function retrieve_course($course_code)

plugin/search_course/lib/search_course_widget.class.php

Doc Comments +8 added lines

@@ -40,11 +40,19 @@
         return isset($_GET[$key]) ? $_GET[$key] : $default;
     }
 
+    /**
+     * @param string $key
+     *
+     * @return string
+     */
     public static function server($key, $default = '')
     {
         return isset($_SERVER[$key]) ? $_SERVER[$key] : $default;
     }
 
+    /**
+     * @param string $name
+     */
     public static function get_lang($name)
     {
         return SearchCoursePlugin::create()->get_lang($name);

Spacing +13 added lines, -13 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-require_once dirname(__FILE__) . '/register_course_widget.class.php';
+require_once dirname(__FILE__).'/register_course_widget.class.php';
 
 /**
  * Search course widget.
@@ -60,12 +60,12 @@ 
         $url = self::server('REQUEST_URI');
         $url = explode('?', $url);
         $url = reset($url);
-        $url = self::server('SERVER_NAME') . $url;
+        $url = self::server('SERVER_NAME').$url;
 
         $root = api_get_path('WEB_PATH');
         $root = str_replace('https://', '', $root);
         $root = str_replace('http://', '', $root);
-        $index_url = $root . 'index.php';
+        $index_url = $root.'index.php';
 
         return $url == $index_url || $url == $root;
     }
@@ -80,12 +80,12 @@ 
         $url = self::server('REQUEST_URI');
         $url = explode('?', $url);
         $url = reset($url);
-        $url = self::server('SERVER_NAME') . $url;
+        $url = self::server('SERVER_NAME').$url;
 
         $root = api_get_path('WEB_PATH');
         $root = str_replace('https://', '', $root);
         $root = str_replace('http://', '', $root);
-        $index_url = $root . 'user_portal.php';
+        $index_url = $root.'user_portal.php';
 
         return $url == $index_url || $url == $root;
     }
@@ -153,8 +153,8 @@ 
             $parameters[self::PARAM_ACTION] = $action;
         }
         $parameters = implode('&', $parameters);
-        $parameters = $parameters ? '?' . $parameters : '';
-        return $self . $parameters;
+        $parameters = $parameters ? '?'.$parameters : '';
+        return $self.$parameters;
     }
 
     /**
@@ -241,8 +241,8 @@ 
             $details = implode(' - ', $details);
             $title = $course['title'];
 
-            $href = api_get_path(WEB_COURSE_PATH).$course['code'] .'/index.php';
-            echo '<tr><td><b><a href="' . $href . '">' . "$title</a></b><br/>$details</td><td>";
+            $href = api_get_path(WEB_COURSE_PATH).$course['code'].'/index.php';
+            echo '<tr><td><b><a href="'.$href.'">'."$title</a></b><br/>$details</td><td>";
             if (!api_is_anonymous()) {
                 if ($course['registration_code']) {
                     Display::display_icon(
@@ -336,13 +336,13 @@ 
 
         if (api_is_anonymous())
         {
-            $course_fiter = 'visibility = ' . COURSE_VISIBILITY_OPEN_WORLD;
+            $course_fiter = 'visibility = '.COURSE_VISIBILITY_OPEN_WORLD;
         }
         else
         {
-            $course_fiter = 'visibility = ' . COURSE_VISIBILITY_OPEN_WORLD . ' OR ';
-            $course_fiter .= 'visibility = ' . COURSE_VISIBILITY_OPEN_PLATFORM . ' OR ';
-            $course_fiter .= '(visibility = ' . COURSE_VISIBILITY_REGISTERED . ' AND subscribe = 1)';
+            $course_fiter = 'visibility = '.COURSE_VISIBILITY_OPEN_WORLD.' OR ';
+            $course_fiter .= 'visibility = '.COURSE_VISIBILITY_OPEN_PLATFORM.' OR ';
+            $course_fiter .= '(visibility = '.COURSE_VISIBILITY_REGISTERED.' AND subscribe = 1)';
         }
 
         $sql = <<<EOT

Braces +3 added lines, -6 removed lines

@@ -123,8 +123,7 @@ 
         if ($has_content)
         {
             echo '<div class="list">';
-        }
-        else
+        } else
         {
             echo '<div>';
         }
@@ -132,8 +131,7 @@ 
         if (RegisterCourseWidget::factory()->run())
         {
             $result = true;
-        }
-        else
+        } else
         {
             $result = $this->action_display();
         }
@@ -337,8 +335,7 @@ 
         if (api_is_anonymous())
         {
             $course_fiter = 'visibility = ' . COURSE_VISIBILITY_OPEN_WORLD;
-        }
-        else
+        } else
         {
             $course_fiter = 'visibility = ' . COURSE_VISIBILITY_OPEN_WORLD . ' OR ';
             $course_fiter .= 'visibility = ' . COURSE_VISIBILITY_OPEN_PLATFORM . ' OR ';

src/Chamilo/CoreBundle/Component/Editor/CkEditor/CkEditor.php

Doc Comments +1 added lines, -1 removed lines

@@ -94,7 +94,7 @@
     /**
      * @param array $templates
      *
-     * @return null
+     * @return null|string
      */
     public function formatTemplates($templates)
     {

Spacing +8 added lines, -8 removed lines

@@ -55,7 +55,7 @@ 
     public function createHtmlStyle()
     {
         $style = '';
-        if (trim($this->value) == '<html><head><title></title></head><body></body></html>' || $this->value == '' ) {
+        if (trim($this->value) == '<html><head><title></title></head><body></body></html>' || $this->value == '') {
             $cssFile = api_get_path(SYS_CSS_PATH).'themes/'.api_get_visual_theme().'/editor.css';
             if (!is_file($cssFile)) {
                 $cssFile = api_get_path(WEB_CSS_PATH).'editor.css';
@@ -147,14 +147,14 @@ 
         return [[
             'title' => get_lang('EmptyTemplate'),
             'description' => null,
-            'image' => api_get_path(WEB_APP_PATH) . 'home/default_platform_document/template_thumb/empty.gif',
+            'image' => api_get_path(WEB_APP_PATH).'home/default_platform_document/template_thumb/empty.gif',
             'html' => '
                 <!DOCYTPE html>
                 <html>
                     <head>
-                        <meta charset="' . api_get_system_encoding() . '" />
+                        <meta charset="' . api_get_system_encoding().'" />
                     </head>
-                    <body  dir="' . api_get_text_direction() . '">
+                    <body  dir="' . api_get_text_direction().'">
                         <p>
                             <br/>
                         </p>
@@ -177,7 +177,7 @@ 
         $search = array('{CSS}', '{IMG_DIR}', '{REL_PATH}', '{COURSE_DIR}');
         $replace = array(
             '',
-            api_get_path(REL_CODE_PATH) . 'img/',
+            api_get_path(REL_CODE_PATH).'img/',
             api_get_path(REL_PATH),
             api_get_path(REL_DEFAULT_COURSE_DOCUMENT_PATH),
             api_get_path(REL_DEFAULT_COURSE_DOCUMENT_PATH)
@@ -188,7 +188,7 @@ 
         foreach ($systemTemplates as $template) {
             $image = $template->getImage();
             $image = !empty($image) ? $image : 'empty.gif';
-            $image = api_get_path(WEB_APP_PATH) . 'home/default_platform_document/template_thumb/' . $image;
+            $image = api_get_path(WEB_APP_PATH).'home/default_platform_document/template_thumb/'.$image;
 
             /*$image = $this->urlGenerator->generate(
                 'get_document_template_action',
@@ -240,12 +240,12 @@ 
             $templateItem['image'] = api_get_path(WEB_APP_PATH)
                 . 'home/default_platform_document/template_thumb/noimage.gif';
             $templateItem['html'] = file_get_contents(api_get_path(SYS_COURSE_PATH)
-                . $courseDirectory . '/document' . $templateData['path']);
+                . $courseDirectory.'/document'.$templateData['path']);
 
             $image = $template->getImage();
             if (!empty($image)) {
                 $templateItem['image'] = api_get_path(WEB_COURSE_PATH)
-                    . $courseDirectory . '/upload/template_thumbnails/' . $template->getImage();
+                    . $courseDirectory.'/upload/template_thumbnails/'.$template->getImage();
             }
 
             $templateList[] = $templateItem;

src/Chamilo/CoreBundle/Component/Editor/Connector.php

Doc Comments +1 added lines, -1 removed lines

@@ -84,7 +84,7 @@
 
     /**
      * Available driver list.
-     * @return array
+     * @return string[]
      */
     private function getDefaultDriverList()
     {

Indentation +4 added lines, -4 removed lines

@@ -51,7 +51,7 @@ 
             'root_sys' => api_get_path(SYS_PATH),
             'sys_root' => api_get_path(SYS_PATH), // just an alias
             'sys_course_path' => api_get_path(SYS_COURSE_PATH),
-         //   'sys_config_path' => $app['path.config'],
+            //   'sys_config_path' => $app['path.config'],
             'path.temp' => api_get_path(SYS_ARCHIVE_PATH),
             //'sys_log_path' => $app['path.logs']
         );
@@ -325,9 +325,9 @@ 
      **/
     public function access($attr, $path, $data, $volume)
     {
-    	return strpos(basename($path), '.') === 0       // if file/folder begins with '.' (dot)
-    		? !($attr == 'read' || $attr == 'write')    // set read+write to false, other (locked+hidden) set to true
-    		:  null;                                    // else elFinder decide it itself
+        return strpos(basename($path), '.') === 0       // if file/folder begins with '.' (dot)
+            ? !($attr == 'read' || $attr == 'write')    // set read+write to false, other (locked+hidden) set to true
+            :  null;                                    // else elFinder decide it itself
     }
 
     /**

Spacing +1 added lines, -1 removed lines

@@ -327,7 +327,7 @@
     {
     	return strpos(basename($path), '.') === 0       // if file/folder begins with '.' (dot)
     		? !($attr == 'read' || $attr == 'write')    // set read+write to false, other (locked+hidden) set to true
-    		:  null;                                    // else elFinder decide it itself
+    		:  null; // else elFinder decide it itself
     }
 
     /**

src/Chamilo/CoreBundle/Component/Editor/Driver/DropBoxDriver.php

Doc Comments +3 added lines, -3 removed lines

@@ -133,7 +133,7 @@ 
     /**
      * Close connection
      *
-     * @return void
+     * @return boolean
      * @author Dmitry (dio) Levashov
      **/
     public function umount() {
@@ -488,7 +488,7 @@ 
      * Close opened file
      *
      * @param  resource  $fp  file pointer
-     * @return bool
+     * @return boolean|null
      * @author Dmitry (dio) Levashov
      **/
     protected function _fclose($fp, $path='') {
@@ -505,7 +505,7 @@ 
      *
      * @param  string  $path  parent dir path
      * @param string  $name  new directory name
-     * @return string|bool
+     * @return string|false
      * @author Dmitry (dio) Levashov
      **/
     protected function _mkdir($path, $name) {

Indentation +2 added lines, -2 removed lines

@@ -633,8 +633,8 @@
     }
 
     /**
-    * {@inheritdoc}
-    */
+     * {@inheritdoc}
+     */
     protected function _archive($dir, $files, $name, $arc) {
         return false;
     }

Spacing +2 added lines, -2 removed lines

@@ -464,7 +464,7 @@ 
      * @return resource|false
      * @author Dmitry (dio) Levashov
      **/
-    protected function _fopen($path, $mode='rb') {
+    protected function _fopen($path, $mode = 'rb') {
         $fp = $this->tmbPath
             ? @fopen($this->tmpname($path), 'w+')
             : @tmpfile();
@@ -491,7 +491,7 @@ 
      * @return bool
      * @author Dmitry (dio) Levashov
      **/
-    protected function _fclose($fp, $path='') {
+    protected function _fclose($fp, $path = '') {
         @fclose($fp);
         if ($path) {
             @unlink($this->tmpname($path));

src/Chamilo/CoreBundle/Entity/BranchSync.php

Doc Comments +1 added lines, -2 removed lines

@@ -532,7 +532,6 @@ 
      /**
      * Set sslPubKey
      *
-     * @param string $sslPubKey
      * @return BranchSync
      */
     public function setBranchType($branchType)
@@ -714,7 +713,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return null|BranchSync
      */
     public function getParent()
     {

Indentation +11 added lines, -11 removed lines

@@ -136,11 +136,11 @@ 
      */
     private $lastSyncType;
 
-     /**
-     * @var string
-     *
-     * @ORM\Column(name="ssl_pub_key", type="string", length=250, nullable=true, unique=false)
-     */
+        /**
+         * @var string
+         *
+         * @ORM\Column(name="ssl_pub_key", type="string", length=250, nullable=true, unique=false)
+         */
     private $sslPubKey;
 
     /**
@@ -529,12 +529,12 @@ 
         return $this->sslPubKey;
     }
 
-     /**
-     * Set sslPubKey
-     *
-     * @param string $sslPubKey
-     * @return BranchSync
-     */
+        /**
+         * Set sslPubKey
+         *
+         * @param string $sslPubKey
+         * @return BranchSync
+         */
     public function setBranchType($branchType)
     {
         $this->branchType = $branchType;

src/Chamilo/CoreBundle/Entity/CourseRelUser.php

Doc Comments +1 added lines, -1 removed lines

@@ -139,7 +139,7 @@
     }
 
     /**
-     * @param $user
+     * @param User $user
      * @return $this
      */
     public function setUser($user)

src/Chamilo/CoreBundle/Entity/Repository/ItemPropertyRepository.php

Doc Comments +2 added lines, -2 removed lines

@@ -22,8 +22,8 @@
      * @param $tool learnpath | document | etc
      * @param $itemId
      * @param Course $course
-     * @param int $sessionId
-     * @param int $groupId
+     * @param int $session
+     * @param int $group
      *
      * @return \Doctrine\ORM\QueryBuilder
      */

Spacing +1 added lines, -1 removed lines

@@ -196,7 +196,7 @@
                     $users = \GroupManager::getStudentsAndTutors($groupId);
                     $newUserList = array();
                     if (!empty($users)) {
-                        foreach($users as $user) {
+                        foreach ($users as $user) {
                             $newUserList[] = $user['user_id'];
                         }
                         $this->unsubcribeUsersToItem(