From 37c6edfe52fe4e00f3499cb697b678c89716ffe3 Mon Sep 17 00:00:00 2001
From: jmontoyaa <gugli100@gmail.com>
Date: Thu, 29 Mar 2018 08:04:15 +0200
Subject: [PATCH] Minor - format code

---
 main/inc/lib/TicketManager.php         | 2 +-
 main/inc/lib/sortable_table.class.php  | 8 ++++++--
 main/lp/learnpath.class.php            | 3 ---
 plugin/dictionary/DictionaryPlugin.php | 4 ++--
 plugin/sepe/src/sepe_plugin.class.php  | 2 +-
 5 files changed, 10 insertions(+), 9 deletions(-)

diff --git a/main/inc/lib/TicketManager.php b/main/inc/lib/TicketManager.php
index de5257d862..8691297cf6 100644
--- a/main/inc/lib/TicketManager.php
+++ b/main/inc/lib/TicketManager.php
@@ -1541,7 +1541,7 @@ class TicketManager
     }
 
     /**
-     * Close old tickets
+     * Close old tickets.
      */
     public static function close_old_tickets()
     {
diff --git a/main/inc/lib/sortable_table.class.php b/main/inc/lib/sortable_table.class.php
index e2e877942e..d35ab2c076 100755
--- a/main/inc/lib/sortable_table.class.php
+++ b/main/inc/lib/sortable_table.class.php
@@ -235,8 +235,12 @@ class SortableTable extends HTML_Table
             $params['spacesBeforeSeparator'] = '';
             $params['spacesAfterSeparator'] = '';
             $query_vars = array_keys($_GET);
-            $query_vars_needed = [$this->param_prefix.'column', $this->param_prefix.'direction', $this->param_prefix.'per_page'];
-            if (count($this->additional_parameters) > 0) {
+            $query_vars_needed = [
+                $this->param_prefix.'column',
+                $this->param_prefix.'direction',
+                $this->param_prefix.'per_page',
+            ];
+            if (!empty($this->additional_parameters) && count($this->additional_parameters) > 0) {
                 $query_vars_needed = array_merge(
                     $query_vars_needed,
                     array_keys($this->additional_parameters)
diff --git a/main/lp/learnpath.class.php b/main/lp/learnpath.class.php
index 6eb9d1d075..f1335292e9 100755
--- a/main/lp/learnpath.class.php
+++ b/main/lp/learnpath.class.php
@@ -100,7 +100,6 @@ class learnpath
      * @param string $course  Course code
      * @param int    $lp_id
      * @param int    $user_id
-     *
      */
     public function __construct($course, $lp_id, $user_id)
     {
@@ -11673,8 +11672,6 @@ EOD;
         Database::query($sql);
     }
 
-    /**
-     */
     public function set_previous_step_as_prerequisite_for_all_items()
     {
         $tbl_lp_item = Database::get_course_table(TABLE_LP_ITEM);
diff --git a/plugin/dictionary/DictionaryPlugin.php b/plugin/dictionary/DictionaryPlugin.php
index 920b36cede..f534a0bcc2 100644
--- a/plugin/dictionary/DictionaryPlugin.php
+++ b/plugin/dictionary/DictionaryPlugin.php
@@ -31,7 +31,7 @@ class DictionaryPlugin extends Plugin
     }
 
     /**
-     * Installation process
+     * Installation process.
      */
     public function install()
     {
@@ -45,7 +45,7 @@ class DictionaryPlugin extends Plugin
     }
 
     /**
-     * Uninstall process
+     * Uninstall process.
      */
     public function uninstall()
     {
diff --git a/plugin/sepe/src/sepe_plugin.class.php b/plugin/sepe/src/sepe_plugin.class.php
index 7dfa9945b4..f501a6d7a8 100644
--- a/plugin/sepe/src/sepe_plugin.class.php
+++ b/plugin/sepe/src/sepe_plugin.class.php
@@ -120,7 +120,7 @@ class SepePlugin extends Plugin
     }
 
     /**
-     * Update
+     * Update.
      */
     public function update()
     {