chamilo / chamilo-lms

main/exercise/TestCategory.php

Doc Comments +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
     /**
      * Save TestCategory in the database if name doesn't exists
      * @param int $courseId
-     * @return bool
+     * @return false|string
      */
     public function save($courseId = 0)
     {

Indentation +9 added lines, -9 removed lines

@@ -503,10 +503,10 @@ 
      * @param int exercise
      * @param array $check_in_question_list
      * @param array $categoriesAddedInExercise
-    *
-    * @param int $exerciseId
-    * @return array
-    */
+     *
+     * @param int $exerciseId
+     * @return array
+     */
     public static function getQuestionsByCat(
         $exerciseId,
         $check_in_question_list = array(),
@@ -636,9 +636,9 @@ 
     }
 
     /**
-    * Display signs [+] and/or (>0) after question title if question has options
-    * scoreAlwaysPositive and/or uncheckedMayScore
-    */
+     * Display signs [+] and/or (>0) after question title if question has options
+     * scoreAlwaysPositive and/or uncheckedMayScore
+     */
     public function displayQuestionOption($in_objQuestion)
     {
         if ($in_objQuestion->type == MULTIPLE_ANSWER && $in_objQuestion->scoreAlwaysPositive) {
@@ -654,7 +654,7 @@ 
      * key of $tabCategoryQuestions are the category id (0 for not in a category)
      * value is the array of question id of this category
      * Sort question by Category
-    */
+     */
     public static function sortTabByBracketLabel($in_tab)
     {
         $tabResult = array();
@@ -715,7 +715,7 @@ 
      * @param int $exerciseId
      * @author - hubert borderiou
      * @return int
-    */
+     */
     public static function getNumberMaxQuestionByCat($exerciseId)
     {
         $res_num_max = 0;

Upper-Lower-Casing +3 added lines, -3 removed lines

@@ -80,7 +80,7 @@ 
             $newId = Database::insert($table, $params);
 
             if ($newId) {
-                $sql = "UPDATE $table SET id = iid WHERE iid = $newId";
+                $sql = "update $table SET id = iid WHERE iid = $newId";
                 Database::query($sql);
 
                 api_item_property_update(
@@ -154,7 +154,7 @@ 
         }
 
         if ($cat) {
-            $sql = "UPDATE $table SET
+            $sql = "update $table SET
                         title = '$name',
                         description = '$description'
                     WHERE id = $id AND c_id = ".$courseId;
@@ -217,7 +217,7 @@ 
                 $categories[] = $category->getCategory($row['id']);
             }
         } else {
-            $sql = "SELECT $field FROM $table
+            $sql = "select $field FROM $table
                     WHERE c_id = $courseId
                     ORDER BY $field ASC";
             $res = Database::query($sql);

plugin/pens/pens.php

Indentation +13 added lines, -13 removed lines

@@ -29,19 +29,19 @@
 require_once __DIR__.'/chamilo_pens.php';
 
 class ChamiloPackageHandler extends PENSPackageHandler {
-	public function processPackage($request, $path_to_package) {
-		$server = PENSServer::singleton();
-		// Moves the package to archive/pens
-		$path_to_archives = api_get_path(SYS_ARCHIVE_PATH).'pens';
-		if (!is_dir($path_to_archives)) {
-			mkdir($path_to_archives, 0777, true);
-		}
-		rename($path_to_package, $path_to_archives.'/'.$request->getFilename());
-		// Insert the request in the database
-		$chamilo_pens = new ChamiloPens($request);
-		$chamilo_pens->save();
-		$server->sendAlert($request, new PENSResponse(0, 'Package successfully processed'));
-	}
+    public function processPackage($request, $path_to_package) {
+        $server = PENSServer::singleton();
+        // Moves the package to archive/pens
+        $path_to_archives = api_get_path(SYS_ARCHIVE_PATH).'pens';
+        if (!is_dir($path_to_archives)) {
+            mkdir($path_to_archives, 0777, true);
+        }
+        rename($path_to_package, $path_to_archives.'/'.$request->getFilename());
+        // Insert the request in the database
+        $chamilo_pens = new ChamiloPens($request);
+        $chamilo_pens->save();
+        $server->sendAlert($request, new PENSResponse(0, 'Package successfully processed'));
+    }
 }
 
 $handler = new ChamiloPackageHandler();

Braces +4 added lines, -2 removed lines

@@ -28,8 +28,10 @@
 require_once __DIR__.'/lib/pens.php';
 require_once __DIR__.'/chamilo_pens.php';
 
-class ChamiloPackageHandler extends PENSPackageHandler {
-	public function processPackage($request, $path_to_package) {
+class ChamiloPackageHandler extends PENSPackageHandler
+{
+	public function processPackage($request, $path_to_package)
+	{
 		$server = PENSServer::singleton();
 		// Moves the package to archive/pens
 		$path_to_archives = api_get_path(SYS_ARCHIVE_PATH).'pens';

main/ticket/tickets.php

Switch Indentation +41 added lines, -41 removed lines

@@ -72,48 +72,48 @@
 }
 
 switch ($action) {
-    case 'alert':
-        if (!$isAdmin && isset($_GET['ticket_id'])) {
-            TicketManager::send_alert($_GET['ticket_id'], $user_id);
-        }
-        break;
-    case 'export':
-        $data = array(
-            array(
-                '#',
-                get_lang('Date'),
-                get_lang('LastUpdate'),
-                get_lang('Category'),
-                get_lang('User'),
-                get_lang('Program'),
-                get_lang('AssignedTo'),
-                get_lang('Status'),
-                get_lang('Description')
-            )
-        );
-        $datos = $table->get_clean_html();
-        foreach ($datos as $ticket) {
-            $ticket[0] = substr(strip_tags($ticket[0]), 0, 12);
-            $ticket_rem = array(
-                utf8_decode(strip_tags($ticket[0])),
-                utf8_decode(api_html_entity_decode($ticket[1])),
-                utf8_decode(strip_tags($ticket[2])),
-                utf8_decode(strip_tags($ticket[3])),
-                utf8_decode(strip_tags($ticket[4])),
-                utf8_decode(strip_tags($ticket[5])),
-                utf8_decode(strip_tags($ticket[6])),
-                utf8_decode(strip_tags($ticket[7]))
+        case 'alert':
+            if (!$isAdmin && isset($_GET['ticket_id'])) {
+                TicketManager::send_alert($_GET['ticket_id'], $user_id);
+            }
+            break;
+        case 'export':
+            $data = array(
+                array(
+                    '#',
+                    get_lang('Date'),
+                    get_lang('LastUpdate'),
+                    get_lang('Category'),
+                    get_lang('User'),
+                    get_lang('Program'),
+                    get_lang('AssignedTo'),
+                    get_lang('Status'),
+                    get_lang('Description')
+                )
             );
-            $data[] = $ticket_rem;
-        }
-        Export::arrayToXls($data, get_lang('Tickets'));
-        exit;
-        break;
-    case 'close_tickets':
-        TicketManager::close_old_tickets();
-        break;
-    default:
-        break;
+            $datos = $table->get_clean_html();
+            foreach ($datos as $ticket) {
+                $ticket[0] = substr(strip_tags($ticket[0]), 0, 12);
+                $ticket_rem = array(
+                    utf8_decode(strip_tags($ticket[0])),
+                    utf8_decode(api_html_entity_decode($ticket[1])),
+                    utf8_decode(strip_tags($ticket[2])),
+                    utf8_decode(strip_tags($ticket[3])),
+                    utf8_decode(strip_tags($ticket[4])),
+                    utf8_decode(strip_tags($ticket[5])),
+                    utf8_decode(strip_tags($ticket[6])),
+                    utf8_decode(strip_tags($ticket[7]))
+                );
+                $data[] = $ticket_rem;
+            }
+            Export::arrayToXls($data, get_lang('Tickets'));
+            exit;
+            break;
+        case 'close_tickets':
+            TicketManager::close_old_tickets();
+            break;
+        default:
+            break;
 }
 
 if (empty($projectId)) {

Indentation +1 added lines, -1 removed lines

@@ -343,7 +343,7 @@
         echo '<div class="actions">';
         echo '<a href="'.api_get_path(WEB_CODE_PATH).'ticket/new_ticket.php?project_id='.$projectId.'">'.
                 Display::return_icon('add.png', get_lang('Add'), '', '32').
-             '</a>';
+                '</a>';
         echo '</div>';
     }
 }

src/Chamilo/CourseBundle/Component/CourseCopy/Course.php

Indentation +3 added lines, -3 removed lines

@@ -326,9 +326,9 @@
     }
 
     /**
-    * Serialize the course with the best serializer available
-    * @return string
-    */
+     * Serialize the course with the best serializer available
+     * @return string
+     */
     public static function serialize($course)
     {
         if (extension_loaded('igbinary')) {

main/inc/lib/grade_model.lib.php

Indentation +10 added lines, -10 removed lines

@@ -51,15 +51,15 @@ 
     /**
      * Displays the title + grid
      */
-	public function display()
+    public function display()
     {
-		// action links
-		echo '<div class="actions" style="margin-bottom:20px">';
+        // action links
+        echo '<div class="actions" style="margin-bottom:20px">';
         echo '<a href="grade_models.php">'.Display::return_icon('back.png', get_lang('Back'), '', ICON_SIZE_MEDIUM).'</a>';
-		echo '<a href="'.api_get_self().'?action=add">'.Display::return_icon('add.png', get_lang('Add'), '', ICON_SIZE_MEDIUM).'</a>';
-		echo '</div>';
+        echo '<a href="'.api_get_self().'?action=add">'.Display::return_icon('add.png', get_lang('Add'), '', ICON_SIZE_MEDIUM).'</a>';
+        echo '</div>';
         echo Display::grid_html('grade_model');
-	}
+    }
 
     /**
      * Returns a Form validator Obj
@@ -254,7 +254,7 @@ 
      */
     public function delete($id)
     {
-	    parent::delete($id);
+        parent::delete($id);
     }
 
     /**
@@ -308,11 +308,11 @@ 
     /**
      * GradeModelComponents constructor.
      */
-	public function __construct()
+    public function __construct()
     {
         parent::__construct();
         $this->table = Database::get_main_table(TABLE_GRADE_MODEL_COMPONENTS);
-	}
+    }
 
     /**
      * @param array $params
@@ -321,7 +321,7 @@ 
      */
     public function save($params, $show_query = false)
     {
-	    $id = parent::save($params, $show_query);
+        $id = parent::save($params, $show_query);
 
         return $id;
     }

Braces +2 added lines, -2 removed lines

@@ -52,7 +52,7 @@ 
      * Displays the title + grid
      */
 	public function display()
-    {
+	{
 		// action links
 		echo '<div class="actions" style="margin-bottom:20px">';
         echo '<a href="grade_models.php">'.Display::return_icon('back.png', get_lang('Back'), '', ICON_SIZE_MEDIUM).'</a>';
@@ -309,7 +309,7 @@ 
      * GradeModelComponents constructor.
      */
 	public function __construct()
-    {
+	{
         parent::__construct();
         $this->table = Database::get_main_table(TABLE_GRADE_MODEL_COMPONENTS);
 	}

Doc Comments +3 added lines, -3 removed lines

@@ -258,10 +258,10 @@
     }
 
     /**
-     * @param $form
+     * @param CatForm $form
      * @param string $name
-     * @param null $default_value
-     * @return bool
+     * @param null|integer $default_value
+     * @return false|null
      */
     public function fill_grade_model_select_in_form(&$form, $name = 'gradebook_model_id', $default_value = null)
     {

main/inc/lib/formvalidator/FormValidator.class.php

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -104,7 +104,7 @@ 
 	<div class="form-group">
 		<div class="col-sm-offset-2 col-sm-10">{requiredNote}</div>
 	</div>
-EOT;
+eot;
         $renderer->setRequiredNoteTemplate($noteTemplate);
     }
 
@@ -1698,7 +1698,7 @@ 
                     $elementName = $groupObj->getElementName($elementName);
 
                     if ($elementName === false) {
-                        throw new Exception("The $groupName doesn't have the element $elementName");
+                        throw new Exception("the $groupName doesn't have the element $elementName");
                     }
 
                     $this->_rules[$elementName][] = array(

plugin/buycourses/install.php

Braces +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
  * Initialization

  */

 require_once __DIR__.'/config.php';

-if (!api_is_platform_admin()) {

+if (!api_is_platform_admin()) {
     die ('You must have admin permissions to install plugins');

 }

 BuyCoursesPlugin::create()->install();

main/ticket/report.php

Braces +3 added lines, -2 removed lines

@@ -288,8 +288,9 @@
 
     if ($start_date != '' || $end_date != '') {
         $sql .= " HAVING ";
-        if ($start_date != '')
-            $sql .= "  access_date >= '$start_date'   ";
+        if ($start_date != '') {
+                    $sql .= "  access_date >= '$start_date'   ";
+        }
         if ($end_date != '') {
             $sql = ($start_date == '') ? $sql : ($sql." AND ");
             $sql .= "  access_date <= '$end_date'   ";

main/dropbox/index.php

Indentation +1 added lines, -1 removed lines

@@ -304,7 +304,7 @@
 
         // Object initialisation
         $dropbox_person = new Dropbox_Person(api_get_user_id(), $is_courseAdmin, $is_courseTutor);
-         // note: are the $is_courseAdmin and $is_courseTutor parameters needed????
+            // note: are the $is_courseAdmin and $is_courseTutor parameters needed????
 
         // Constructing the array that contains the total number of feedback messages per document.
         $number_feedback = get_total_number_feedback();