chamilo / chamilo-lms

src/Chamilo/CoreBundle/Entity/SkillRelUserComment.php

Doc Comments +1 added lines, -1 removed lines

@@ -114,7 +114,7 @@
 
     /**
      * Get feedbackDateTime
-     * @return type
+     * @return \DateTime
      */
     public function getFeedbackDateTime()
     {

main/inc/lib/surveymanager.lib.php

Spacing +9 added lines, -9 removed lines

@@ -47,20 +47,20 @@ 
         $res = Database::query($sql);
         $refs = array();
         $list = array();
-        $plain_array=array();
+        $plain_array = array();
 
         while ($survey = Database::fetch_array($res, 'ASSOC')) {
-            $plain_array[$survey['survey_id']]=$survey;
-            $surveys_parents[]=$survey['survey_version'];
-            $thisref = &$refs[ $survey['survey_id'] ];
+            $plain_array[$survey['survey_id']] = $survey;
+            $surveys_parents[] = $survey['survey_version'];
+            $thisref = &$refs[$survey['survey_id']];
             $thisref['parent_id'] = $survey['parent_id'];
             $thisref['name'] = $survey['name'];
             $thisref['id'] = $survey['survey_id'];
             $thisref['survey_version'] = $survey['survey_version'];
             if ($survey['parent_id'] == 0) {
-                $list[ $survey['survey_id'] ] = &$thisref;
+                $list[$survey['survey_id']] = &$thisref;
             } else {
-                $refs[ $survey['parent_id'] ]['children'][ $survey['survey_id'] ] = &$thisref;
+                $refs[$survey['parent_id']]['children'][$survey['survey_id']] = &$thisref;
             }
         }
         $this->surveylist = $list;
@@ -79,7 +79,7 @@ 
     public function getParentId($id)
     {
         $node = $this->plainsurveylist[$id];
-        if (is_array($node)&& !empty($node['parent_id'])) {
+        if (is_array($node) && !empty($node['parent_id'])) {
             return $node['parent_id'];
         } else {
             return -1;
@@ -99,12 +99,12 @@ 
         if (is_array($list)) {
             foreach ($list as $key => $node) {
                 if (isset($node['children']) && is_array($node['children'])) {
-                    $result[$key]= $node['name'];
+                    $result[$key] = $node['name'];
                     $re = self::createList($node['children']);
                     if (!empty($re)) {
                         if (is_array($re)) {
                             foreach ($re as $key => $r) {
-                                $result[$key] = '' . $r;
+                                $result[$key] = ''.$r;
                             }
                         } else {
                             $result[] = $re;

main/survey/preview.php

Indentation +98 added lines, -98 removed lines

@@ -44,7 +44,7 @@ 
 // We exit here if there is no valid $_GET parameter
 if (!isset($_GET['survey_id']) || !is_numeric($_GET['survey_id'])) {
     api_not_allowed(true, Display::return_message(get_lang('InvallidSurvey'), 'error', false));
-	exit;
+    exit;
 }
 
 // Getting the survey information
@@ -53,13 +53,13 @@ 
 
 if (empty($survey_data)) {
     api_not_allowed(true, Display::return_message(get_lang('InvallidSurvey'), 'error', false));
-	exit;
+    exit;
 }
 
 $urlname = strip_tags($survey_data['title']);
 if (api_is_allowed_to_edit()) {
-	// Breadcrumbs
-	$interbreadcrumb[] = array(
+    // Breadcrumbs
+    $interbreadcrumb[] = array(
         'url' => api_get_path(WEB_CODE_PATH).'survey/survey_list.php?'.api_get_cidreq(),
         'name' => get_lang('SurveyList')
     );
@@ -86,57 +86,57 @@ 
 $counter_question = 0;
 // Only a course admin is allowed to preview a survey: you are a course admin
 if (api_is_course_admin() ||
-	(api_is_course_admin() && $_GET['isStudentView'] == 'true') ||
-	api_is_allowed_to_session_edit(false, true)
+    (api_is_course_admin() && $_GET['isStudentView'] == 'true') ||
+    api_is_allowed_to_session_edit(false, true)
 ) {
-	// Survey information
-	echo '<div id="survey_title">'.$survey_data['survey_title'].'</div>';
-	echo '<div id="survey_subtitle">'.$survey_data['survey_subtitle'].'</div>';
+    // Survey information
+    echo '<div id="survey_title">'.$survey_data['survey_title'].'</div>';
+    echo '<div id="survey_subtitle">'.$survey_data['survey_subtitle'].'</div>';
 
-	// Displaying the survey introduction
-	if (!isset($_GET['show'])) {
+    // Displaying the survey introduction
+    if (!isset($_GET['show'])) {
         if (!empty($survey_data['survey_introduction'])) {
             echo '<div id="survey_content" class="survey_content">'.$survey_data['survey_introduction'].'</div>';
         }
-		$limit = 0;
-	}
+        $limit = 0;
+    }
 
-	// Displaying the survey thanks message
-	if (isset($_POST['finish_survey'])) {
-		Display::display_confirmation_message(get_lang('SurveyFinished'));
+    // Displaying the survey thanks message
+    if (isset($_POST['finish_survey'])) {
+        Display::display_confirmation_message(get_lang('SurveyFinished'));
         echo $survey_data['survey_thanks'];
-		Display :: display_footer();
-		exit;
-	}
+        Display :: display_footer();
+        exit;
+    }
 
     $questions = array();
 
-	if (isset($_GET['show'])) {
-		// Getting all the questions for this page and add them to a
-		// multidimensional array where the first index is the page.
-		// as long as there is no pagebreak fount we keep adding questions to the page
-		$questions_displayed = array();
-		$paged_questions = array();
-		$counter = 0;
-		$sql = "SELECT * FROM $table_survey_question
+    if (isset($_GET['show'])) {
+        // Getting all the questions for this page and add them to a
+        // multidimensional array where the first index is the page.
+        // as long as there is no pagebreak fount we keep adding questions to the page
+        $questions_displayed = array();
+        $paged_questions = array();
+        $counter = 0;
+        $sql = "SELECT * FROM $table_survey_question
 		        WHERE c_id = $course_id AND survey_id = '".intval($survey_id)."'
 				ORDER BY sort ASC";
-		$result = Database::query($sql);
+        $result = Database::query($sql);
         $questions_exists = true;
         if (Database::num_rows($result)) {
-    		while ($row = Database::fetch_array($result)) {
-    			if ($row['type'] == 'pagebreak') {
-    				$counter++;
-    			} else {
-    				$paged_questions[$counter][] = $row['question_id'];
-    			}
-    		}
+            while ($row = Database::fetch_array($result)) {
+                if ($row['type'] == 'pagebreak') {
+                    $counter++;
+                } else {
+                    $paged_questions[$counter][] = $row['question_id'];
+                }
+            }
         } else {
             $questions_exists = false;
         }
 
-		if (array_key_exists($_GET['show'], $paged_questions)) {
-			$sql = "SELECT
+        if (array_key_exists($_GET['show'], $paged_questions)) {
+            $sql = "SELECT
                         survey_question.question_id,
                         survey_question.survey_id,
                         survey_question.survey_question,
@@ -158,75 +158,75 @@ 
 						survey_question.c_id =  $course_id
 					ORDER BY survey_question.sort, survey_question_option.sort ASC";
 
-			$result = Database::query($sql);
-			$question_counter_max = Database::num_rows($result);
-			$limit = 0;
-			while ($row = Database::fetch_array($result)) {
-				// If the type is not a pagebreak we store it in the $questions array
-				if ($row['type'] != 'pagebreak') {
-					$questions[$row['sort']]['question_id'] = $row['question_id'];
-					$questions[$row['sort']]['survey_id'] = $row['survey_id'];
-					$questions[$row['sort']]['survey_question'] = $row['survey_question'];
-					$questions[$row['sort']]['display'] = $row['display'];
-					$questions[$row['sort']]['type'] = $row['type'];
-					$questions[$row['sort']]['options'][intval($row['option_sort'])] = $row['option_text'];
-					$questions[$row['sort']]['maximum_score'] = $row['max_value'];
-				} else {
-					// If the type is a pagebreak we are finished loading the questions for this page
-					break;
-				}
-				$counter_question++;
-			}
-		}
-	}
-
-	// Selecting the maximum number of pages
-	$sql = "SELECT * FROM $table_survey_question
+            $result = Database::query($sql);
+            $question_counter_max = Database::num_rows($result);
+            $limit = 0;
+            while ($row = Database::fetch_array($result)) {
+                // If the type is not a pagebreak we store it in the $questions array
+                if ($row['type'] != 'pagebreak') {
+                    $questions[$row['sort']]['question_id'] = $row['question_id'];
+                    $questions[$row['sort']]['survey_id'] = $row['survey_id'];
+                    $questions[$row['sort']]['survey_question'] = $row['survey_question'];
+                    $questions[$row['sort']]['display'] = $row['display'];
+                    $questions[$row['sort']]['type'] = $row['type'];
+                    $questions[$row['sort']]['options'][intval($row['option_sort'])] = $row['option_text'];
+                    $questions[$row['sort']]['maximum_score'] = $row['max_value'];
+                } else {
+                    // If the type is a pagebreak we are finished loading the questions for this page
+                    break;
+                }
+                $counter_question++;
+            }
+        }
+    }
+
+    // Selecting the maximum number of pages
+    $sql = "SELECT * FROM $table_survey_question
 	        WHERE
 	            c_id = $course_id AND
 	            type='".Database::escape_string('pagebreak')."' AND
 	            survey_id='".intval($survey_id)."'";
-	$result = Database::query($sql);
-	$numberofpages = Database::num_rows($result) + 1;
-
-	// Displaying the form with the questions
-	if (isset($_GET['show'])) {
-		$show = (int)$_GET['show'] + 1;
-	} else {
-		$show = 0;
-	}
-
-	$url = api_get_self().'?survey_id='.Security::remove_XSS($survey_id).'&show='.$show;
-	$form = new FormValidator('question', 'post', $url);
-
-	if (is_array($questions) && count($questions) > 0) {
-		foreach ($questions as $key => & $question) {
-			$ch_type = 'ch_'.$question['type'];
-			/** @var survey_question $display */
-			$display = new $ch_type;
-			$form->addHtml('<div class="survey_question_wrapper"><div class="survey_question">');
-			$form->addHtml($question['survey_question']);
-			$display->render($form, $question);
-			$form->addHtml('</div></div>');
-		}
-	}
-
-	if (($show < $numberofpages) || (!$_GET['show'] && count($questions) > 0)) {
+    $result = Database::query($sql);
+    $numberofpages = Database::num_rows($result) + 1;
+
+    // Displaying the form with the questions
+    if (isset($_GET['show'])) {
+        $show = (int)$_GET['show'] + 1;
+    } else {
+        $show = 0;
+    }
+
+    $url = api_get_self().'?survey_id='.Security::remove_XSS($survey_id).'&show='.$show;
+    $form = new FormValidator('question', 'post', $url);
+
+    if (is_array($questions) && count($questions) > 0) {
+        foreach ($questions as $key => & $question) {
+            $ch_type = 'ch_'.$question['type'];
+            /** @var survey_question $display */
+            $display = new $ch_type;
+            $form->addHtml('<div class="survey_question_wrapper"><div class="survey_question">');
+            $form->addHtml($question['survey_question']);
+            $display->render($form, $question);
+            $form->addHtml('</div></div>');
+        }
+    }
+
+    if (($show < $numberofpages) || (!$_GET['show'] && count($questions) > 0)) {
         if ($show == 0) {
-			$form->addButton('next_survey_page', get_lang('StartSurvey'), 'arrow-right', 'success', 'large');
+            $form->addButton('next_survey_page', get_lang('StartSurvey'), 'arrow-right', 'success', 'large');
         } else {
-			$form->addButton('next_survey_page', get_lang('NextQuestion'), 'arrow-right');
+            $form->addButton('next_survey_page', get_lang('NextQuestion'), 'arrow-right');
         }
-	}
-	if ($show >= $numberofpages && $_GET['show'] || (isset($_GET['show']) && count($questions) == 0)) {
-		if ($questions_exists == false) {
-			echo '<p>'.get_lang('ThereAreNotQuestionsForthisSurvey').'</p>';
-		}
-		$form->addButton('finish_survey', get_lang('FinishSurvey'), 'arrow-right');
-	}
-	$form->display();
+    }
+    if ($show >= $numberofpages && $_GET['show'] || (isset($_GET['show']) && count($questions) == 0)) {
+        if ($questions_exists == false) {
+            echo '<p>'.get_lang('ThereAreNotQuestionsForthisSurvey').'</p>';
+        }
+        $form->addButton('finish_survey', get_lang('FinishSurvey'), 'arrow-right');
+    }
+    $form->display();
 } else {
-	Display :: display_error_message(get_lang('NotAllowed'), false);
+    Display :: display_error_message(get_lang('NotAllowed'), false);
 }
 
 Display :: display_footer();

Spacing +2 added lines, -2 removed lines

@@ -154,7 +154,7 @@ 
 						survey_question_option.c_id = $course_id
 					WHERE
 					    survey_question.survey_id = '".intval($survey_id)."' AND
-						survey_question.question_id IN (".Database::escape_string(implode(',',$paged_questions[$_GET['show']]), null, false).") AND
+						survey_question.question_id IN (".Database::escape_string(implode(',', $paged_questions[$_GET['show']]), null, false).") AND
 						survey_question.c_id =  $course_id
 					ORDER BY survey_question.sort, survey_question_option.sort ASC";
 
@@ -191,7 +191,7 @@ 
 
 	// Displaying the form with the questions
 	if (isset($_GET['show'])) {
-		$show = (int)$_GET['show'] + 1;
+		$show = (int) $_GET['show'] + 1;
 	} else {
 		$show = 0;
 	}

src/Chamilo/TicketBundle/Entity/Ticket.php

Indentation +6 added lines, -6 removed lines

@@ -94,12 +94,12 @@
      */
     protected $assignedLastUser;
 
-     /**
-     * @var Status
-     *
-     * @ORM\ManyToOne(targetEntity="Chamilo\TicketBundle\Entity\Status")
-     * @ORM\JoinColumn(name="status_id", referencedColumnName="id")
-     **/
+        /**
+         * @var Status
+         *
+         * @ORM\ManyToOne(targetEntity="Chamilo\TicketBundle\Entity\Status")
+         * @ORM\JoinColumn(name="status_id", referencedColumnName="id")
+         **/
     protected $status;
 
     /**

src/Chamilo/TicketBundle/Entity/MessageAttachment.php

Indentation +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
      */
     protected $filename;
 
-   /**
+    /**
      * @var integer
      *
      * @ORM\Column(name="size", type="integer", nullable=false, unique=false)

plugin/courseblock/CourseBlockPlugin.php

Indentation +4 added lines, -4 removed lines

@@ -10,7 +10,7 @@ 
 
     // When creating a new course this settings are added to the course
     public $course_settings = array(
-         array(
+            array(
             'name' => 'course_block_pre_footer',
             'type' => 'textarea'
         ),
@@ -66,9 +66,9 @@ 
     }
 
         /**
-     * @param string $region
-     * @return string
-     */
+         * @param string $region
+         * @return string
+         */
     public function renderRegion($region)
     {
         $content = '';

src/Chamilo/TicketBundle/Entity/Message.php

Indentation +6 added lines, -6 removed lines

@@ -53,12 +53,12 @@
      */
     protected $ipAddress;
 
-     /**
-     * @var Ticket
-     *
-     * @ORM\ManyToOne(targetEntity="Chamilo\TicketBundle\Entity\Ticket")
-     * @ORM\JoinColumn(name="ticket_id", referencedColumnName="id")
-     **/
+        /**
+         * @var Ticket
+         *
+         * @ORM\ManyToOne(targetEntity="Chamilo\TicketBundle\Entity\Ticket")
+         * @ORM\JoinColumn(name="ticket_id", referencedColumnName="id")
+         **/
     protected $ticket;
 
     /**

main/ticket/tutor_report.lib.php

Doc Comments +4 added lines, -4 removed lines

@@ -73,7 +73,7 @@ 
 
 /**
  * @param $courseInfo
- * @param $weeksCount
+ * @param string $weeksCount
  * @param $page
  * @return array
  */
@@ -182,7 +182,7 @@ 
 
 /**
  * @param $datos
- * @param $pagina
+ * @param integer $pagina
  * @return string
  */
 function showStudentResult($datos, $pagina)
@@ -201,8 +201,8 @@ 
 
 /**
  * @param $data
- * @param $numero_semanas
- * @return array
+ * @param integer $numero_semanas
+ * @return string[]
  */
 function showStudentResultExport($data, $numero_semanas)
 {

Spacing +22 added lines, -22 removed lines

@@ -21,7 +21,7 @@ 
     $table_post = Database::get_course_table(TABLE_FORUM_POST);
     $table_work = Database::get_course_table(TABLE_STUDENT_PUBLICATION);
     $course_code = Database::escape_string($course_code);
-    $res = Database::query("SELECT COUNT(*) as cant FROM $table_reporte_semanas WHERE course_code = '" . $course_code . "'");
+    $res = Database::query("SELECT COUNT(*) as cant FROM $table_reporte_semanas WHERE course_code = '".$course_code."'");
     $sqlWeeks = "SELECT semanas FROM $table_semanas_curso WHERE course_code = '$course_code'";
     $resWeeks = Database::query($sqlWeeks);
     $weeks = Database::fetch_object($resWeeks);
@@ -105,52 +105,52 @@ 
     $lineHeaderExport = array(null, null);
     $lineHeaderExport2 = array(null, ull);
     while ($rowe = Database::fetch_assoc($resultHeader)) {
-        $lineHeaderExport[] = utf8_decode('Work' . $rowe['week_id']);
-        $lineHeaderExport[] = utf8_decode('Forum' . $rowe['week_id']);
+        $lineHeaderExport[] = utf8_decode('Work'.$rowe['week_id']);
+        $lineHeaderExport[] = utf8_decode('Forum'.$rowe['week_id']);
         //$fila_export_encabezado[] =  utf8_decode('Eval'.$rowe['week_id']);
         //$fila_export_encabezado[] =  utf8_decode('PC'.$rowe['week_id']);
         $lineHeaderExport2[] = utf8_decode($rowe['work_title']);
         $lineHeaderExport2[] = utf8_decode($rowe['thread_title']);
         //$fila_export_encabezado2[] = utf8_decode($rowe['eval_title']);
         //$fila_export_encabezado2[] = utf8_decode($rowe['pc_title']);
-        $fila_export = array('Work' . $rowe['week_id'], 'Forum' . $rowe['week_id'], 'Eval' . $rowe['week_id'], 'PC' . $rowe['week_id']);
+        $fila_export = array('Work'.$rowe['week_id'], 'Forum'.$rowe['week_id'], 'Eval'.$rowe['week_id'], 'PC'.$rowe['week_id']);
         if ($rowe['week_id'] > (($page - 1) * 7) && $rowe['week_id'] <= (7 * $page)) {
             $ids[$rowe['week_id']] = $rowe['id'];
-            $line.='<th>
-                <a href="#" onClick="showContent(' . "'tarea" . $rowe['week_id'] . "'" . ');">Work' . $rowe['week_id'] . '
-                        <div class="blackboard_hide" id="tarea' . $rowe['week_id'] . '">' . $rowe['work_title'] . '</div>
+            $line .= '<th>
+                <a href="#" onClick="showContent(' . "'tarea".$rowe['week_id']."'".');">Work'.$rowe['week_id'].'
+                        <div class="blackboard_hide" id="tarea' . $rowe['week_id'].'">'.$rowe['work_title'].'</div>
                 </a></th>';
-            $line.= '<th>
-                <a href="#" onClick="showContent(' . "'foro" . $rowe['week_id'] . "'" . ');">Forum' . $rowe['week_id'] . '
-                        <div class="blackboard_hide" id="foro' . $rowe['week_id'] . '">' . $rowe['thread_title'] . '</div>
+            $line .= '<th>
+                <a href="#" onClick="showContent(' . "'foro".$rowe['week_id']."'".');">Forum'.$rowe['week_id'].'
+                        <div class="blackboard_hide" id="foro' . $rowe['week_id'].'">'.$rowe['thread_title'].'</div>
                 </a>
                 </th>';
         }
     }
     $tableExport[] = $lineHeaderExport;
     $tableExport[] = $lineHeaderExport2;
-    $line.= '</tr>';
+    $line .= '</tr>';
 
     $html = '<form action="tutor.php" name="semanas" id="semanas" method="POST">
             <div class="row">
-            ' . get_lang('SelectWeeksSpan') . '
+            ' . get_lang('SelectWeeksSpan').'
             <select name="weeksNumber" id="weeksNumber" onChange="submit();">
-            <option value="7" ' . (($weeksCount == 7) ? 'selected="selected"' : "") . '>7 weeks</option>
-            <option value="14" ' . (($weeksCount == 14) ? 'selected="selected"' : "") . '>14 weeks</option>
+            <option value="7" ' . (($weeksCount == 7) ? 'selected="selected"' : "").'>7 weeks</option>
+            <option value="14" ' . (($weeksCount == 14) ? 'selected="selected"' : "").'>14 weeks</option>
             </select>';
 
 
     if ($weeksCount == 14) {
-        $html .= '<span style="float:right;"><a href="tutor.php?page=' . (($page == 1) ? 2 : 1) . '">' . (($page == 1) ? "Siguiente" : "Anterior") . '</a></span>';
+        $html .= '<span style="float:right;"><a href="tutor.php?page='.(($page == 1) ? 2 : 1).'">'.(($page == 1) ? "Siguiente" : "Anterior").'</a></span>';
     }
-    $html .= '<span style="float:right;"><a href="' . api_get_self() . '?action=export' . $get_parameter . $get_parameter2 . '">' . Display::return_icon('export_excel.png', get_lang('Export'), '', '32') . '</a></span>';
+    $html .= '<span style="float:right;"><a href="'.api_get_self().'?action=export'.$get_parameter.$get_parameter2.'">'.Display::return_icon('export_excel.png', get_lang('Export'), '', '32').'</a></span>';
 
     $html .= '</form>';
     $html .= '<table class="reports">';
     $html .= '<tr>
             <th ></th>';
     for ($i = (7 * $page - 6); $i <= $page * 7; $i++) {
-        $html .= '<th colspan="2">Week ' . $i . '<a href="assign_tickets.php?id=' . $ids[$i] . '" class="ajax">' . Display::return_icon('edit.png', get_lang('Edit'), array('width' => '16', 'height' => '16'), 22) . '</a></th>';
+        $html .= '<th colspan="2">Week '.$i.'<a href="assign_tickets.php?id='.$ids[$i].'" class="ajax">'.Display::return_icon('edit.png', get_lang('Edit'), array('width' => '16', 'height' => '16'), 22).'</a></th>';
     }
     $html .= '</tr>';
     $html .= $line;
@@ -167,7 +167,7 @@ 
         if ($row['week_id'] > (($page - 1) * 7) && $row['week_id'] <= (7 * $page)) {
             $results[$row['username']][$row['week_id']] = $row;
             if (count($results[$row['username']]) == 7) {
-                $html.= showStudentResult($results[$row['username']], $page);
+                $html .= showStudentResult($results[$row['username']], $page);
             }
         }
         if (count($resultadose[$row['username']]) == $weeksCount) {
@@ -190,12 +190,12 @@ 
     $inicio = (7 * $pagina - 6);
     $fila = '<tr>';
 
-    $fila.= '<td><a href="' . api_get_path(WEB_CODE_PATH) . 'user/userInfo.php?' . api_get_cidreq() . '&uInfo=' . $datos[$inicio]['user_id'] . '">' . $datos[$inicio]['username'] . '</a></td>';
+    $fila .= '<td><a href="'.api_get_path(WEB_CODE_PATH).'user/userInfo.php?'.api_get_cidreq().'&uInfo='.$datos[$inicio]['user_id'].'">'.$datos[$inicio]['username'].'</a></td>';
     foreach ($datos as $dato) {
-        $fila.= '<td align="center">' . (($dato['work_ok'] == 1) ? Display::return_icon('check.png') : Display::return_icon('aspa.png')) . '</td>';
-        $fila.= '<td align="center">' . (($dato['thread_ok'] == 1) ? Display::return_icon('check.png') : Display::return_icon('aspa.png')) . '</td>';
+        $fila .= '<td align="center">'.(($dato['work_ok'] == 1) ? Display::return_icon('check.png') : Display::return_icon('aspa.png')).'</td>';
+        $fila .= '<td align="center">'.(($dato['thread_ok'] == 1) ? Display::return_icon('check.png') : Display::return_icon('aspa.png')).'</td>';
     }
-    $fila.= '</tr>';
+    $fila .= '</tr>';
     return $fila;
 }
 

main/ticket/assign_tickets.php

Indentation +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
 
 api_protect_course_script();
 if (!api_is_allowed_to_edit()) {
-	api_not_allowed();
+    api_not_allowed();
 }
 $course_info = api_get_course_info();
 $course_code = $course_info['code'];

Spacing +10 added lines, -10 removed lines

@@ -20,7 +20,7 @@ 
 echo '<div id="confirmation"></div>';
 $id = intval($_GET['id']);
 $tblWeeklyReport = Database::get_main_table('rp_reporte_semanas');
-$sql ="SELECT * FROM $tblWeeklyReport WHERE id = '$id'";
+$sql = "SELECT * FROM $tblWeeklyReport WHERE id = '$id'";
 $sql_tasks = "SELECT id AS colid, title as coltitle
     FROM ".Database::get_course_table(TABLE_STUDENT_PUBLICATION)."
     WHERE parent_id = 0
@@ -45,28 +45,28 @@ 
 $result_forum = Database::query($sql_forum);
 
 echo '<div class="row">
-        <input type="hidden" id="rs_id" name ="rs_id" value="' . $id . '">
-        <div class="formw">' . get_lang('PleaseSelectTasks') . '</div>
+        <input type="hidden" id="rs_id" name ="rs_id" value="' . $id.'">
+        <div class="formw">' . get_lang('PleaseSelectTasks').'</div>
     </div>';
 echo '<div class="row"><div class="formw"><select name ="work_id" id="work_id">';
-echo '<option value="0"' . (($row['colid'] == $rs->work_id) ? "selected" : "") . '>' . get_lang('PleaseSelect') . '</option>';
+echo '<option value="0"'.(($row['colid'] == $rs->work_id) ? "selected" : "").'>'.get_lang('PleaseSelect').'</option>';
 while ($row = Database::fetch_assoc($result_tasks)) {
-    echo '<option value="' . $row['colid'] . '"' . (($row['colid'] == $rs->work_id) ? "selected" : "") . '>' . $row['coltitle'] . '</option>';
+    echo '<option value="'.$row['colid'].'"'.(($row['colid'] == $rs->work_id) ? "selected" : "").'>'.$row['coltitle'].'</option>';
 }
 echo '</select></div><div>';
 echo '<div class="row">
-        <div class="formw">' . get_lang('PleaseSelectThread') . '</div>
+        <div class="formw">' . get_lang('PleaseSelectThread').'</div>
     </div>';
 echo '<div class="row"><div class="formw"><select name ="forum_id" id="forum_id">';
-echo '<option value="0"' . (($row['colid'] == $rs->work_id) ? "forum_id" : "") . '>' . get_lang('PleaseSelect') . '</option>';
+echo '<option value="0"'.(($row['colid'] == $rs->work_id) ? "forum_id" : "").'>'.get_lang('PleaseSelect').'</option>';
 while ($row = Database::fetch_assoc($result_forum)) {
-    echo '<option value="' . $row['colid'] . '"' . (($row['colid'] == $rs->forum_id) ? "selected" : "") . '>' . $row['coltitle'] . '</option>';
+    echo '<option value="'.$row['colid'].'"'.(($row['colid'] == $rs->forum_id) ? "selected" : "").'>'.$row['coltitle'].'</option>';
 }
 echo '</select></div><div>';
 echo '<div class="row">
         <div class="formw">
-        <button class="save" name="edit" type="button" value="' . get_lang('Edit') . '" onClick="save(' . "$id" . ');">' .
-            get_lang('Edit') . '</button>
+        <button class="save" name="edit" type="button" value="' . get_lang('Edit').'" onClick="save('."$id".');">'.
+            get_lang('Edit').'</button>
         </div>
     </div>';
 echo '</form>';