chamilo / chamilo-lms

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/Component/Filesystem/Data.php

Doc Comments +1 added lines, -1 removed lines

@@ -116,7 +116,7 @@
 
     /**
      * Create folders
-     * @param array            $folderList
+     * @param string[]            $folderList
      * @param OutputInterface  $output
      * @param string           $folderPermissions
      */

Indentation +7 added lines, -7 removed lines

@@ -86,13 +86,13 @@
         return $this->get($file);
     }
 
-     /**
-     * Gets a file from the data/courses/MATHS/scorm directory
-     * @param string $courseCode
-     * @param string $file
-      *
-     * @return SplFileInfo
-     */
+        /**
+         * Gets a file from the data/courses/MATHS/scorm directory
+         * @param string $courseCode
+         * @param string $file
+         *
+         * @return SplFileInfo
+         */
     public function getCourseScormDocument($courseCode, $file)
     {
         $file = 'courses/'.$courseCode.'/scorm/'.$file;

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/Course.php

Doc Comments +2 added lines, -2 removed lines

@@ -318,7 +318,7 @@ 
     }
 
     /**
-     * @return ArrayCollection
+     * @return CTool[]
      */
     public function getTools()
     {
@@ -1146,7 +1146,7 @@ 
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     public static function getStatusList()
     {

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(

src/Chamilo/CoreBundle/Entity/SequenceResource.php

Doc Comments +2 added lines, -2 removed lines

@@ -58,7 +58,7 @@ 
     }
 
     /**
-     * @return string
+     * @return integer
      */
     public function getType()
     {
@@ -123,7 +123,7 @@ 
     }
 
     /**
-     * @param mixed $sequence
+     * @param Sequence $sequence
      * @return $this
      */
     public function setSequence(Sequence $sequence)

src/Chamilo/CoreBundle/Entity/Session.php

Doc Comments +4 added lines, -4 removed lines

@@ -460,7 +460,7 @@ 
      * @param Course $course
      * @param string $status
      *
-     * @return \Doctrine\Common\Collections\Collection|static
+     * @return ArrayCollection
      */
     public function getUserInCourse(User $user, Course $course, $status = null)
     {
@@ -506,7 +506,7 @@ 
      * Set description
      *
      * @param string $description
-     * @return Groups
+     * @return Session
      */
     public function setDescription($description)
     {
@@ -839,7 +839,7 @@ 
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     public static function getStatusList()
     {
@@ -991,7 +991,7 @@ 
      * Get user from course by status
      * @param \Chamilo\CoreBundle\Entity\Course $course
      * @param string $status
-     * @return \Doctrine\Common\Collections\Collection|static
+     * @return ArrayCollection
      */
     public function getUserCourseSubscriptionsByStatus(Course $course, $status)
     {

Spacing +1 added lines, -1 removed lines

@@ -470,7 +470,7 @@
             Criteria::expr()->eq("user", $user)
         );
 
-        if (!is_null($status))  {
+        if (!is_null($status)) {
             $criteria->andWhere(
                 Criteria::expr()->eq("status", $status)
             );