Minor - format code, remove unused code/comments

Author: jmontoyaa

main/lp/lp_ajax_initialize.php

@@ -137,20 +137,14 @@ function initialize_item($lp_id, $user_id, $view_id, $next_item)
      * -lms_view_id
      * -lms_user_id
      */
-    $mytotal = $mylp->getTotalItemsCountWithoutDirs();
-    $mycomplete = $mylp->get_complete_items_count();
-    $myprogress_mode = $mylp->get_progress_bar_mode();
-    $myprogress_mode = ($myprogress_mode == '' ? '%' : $myprogress_mode);
     $mynext = $mylp->get_next_item_id();
     $myprevious = $mylp->get_previous_item_id();
     $myitemtype = $mylpi->get_type();
     $mylesson_mode = $mylpi->get_lesson_mode();
     $mycredit = $mylpi->get_credit();
     $mylaunch_data = $mylpi->get_launch_data();
     $myinteractions_count = $mylpi->get_interactions_count();
-    $myobjectives_count = $mylpi->get_objectives_count();
     $mycore_exit = $mylpi->get_core_exit();
-
     $return .=
             "olms.lms_lp_id=".$lp_id.";".
             "olms.lms_item_id=".$next_item.";".

main/lp/openoffice_text_document.class.php

@@ -207,7 +207,6 @@ public function dealPerPage($header, $body)
         $_course = api_get_course_info();
         // Split document to pages.
         $pages = explode('||page_break||', $body);
-        $first_item = 0;
         foreach ($pages as $key => $page_content) {
             // For every pages, we create a new file.
             $key += 1;

plugin/ims_lti/install.php

@@ -6,7 +6,7 @@
  */
 
 if (!api_is_platform_admin()) {
-    die ('You must have admin permissions to install plugins');
+    die('You must have admin permissions to install plugins');
 }
 
 ImsLtiPlugin::create()->install();

plugin/ims_lti/uninstall.php

@@ -1,12 +1,13 @@
 <?php
 /* For license terms, see /license.txt */
+
 /**
  * Uninstall the MSI/LTI Plugin
  * @package chamilo.plugin.ims_lti
  */
 
 if (!api_is_platform_admin()) {
-    die ('You must have admin permissions to uninstall plugins');
+    die('You must have admin permissions to uninstall plugins');
 }
 
 ImsLtiPlugin::create()->uninstall();

src/Chamilo/CoreBundle/Entity/BranchSync.php

@@ -741,4 +741,22 @@ public function setUniqueId($uniqueId)
 
         return $this;
     }
+
+    /**
+     * @return string
+     */
+    public function getDescription()
+    {
+        return $this->description;
+    }
+
+    /**
+     * @param string $description
+     * @return BranchSync
+     */
+    public function setDescription($description)
+    {
+        $this->description = $description;
+        return $this;
+    }
 }

src/Chamilo/UserBundle/Repository/UserRepository.php

@@ -313,42 +313,4 @@ public function getStudentBosses(User $user)
 
         return $queryBuilder->getQuery()->getResult();
     }
-
-    /**
-     * Get number of users in URL
-     * @param AccessUrl $url
-     *
-     * @return int
-     */
-    public function getCountUsersByUrl(AccessUrl $url)
-    {
-        return $this->createQueryBuilder('a')
-            ->select('COUNT(a)')
-            ->innerJoin('a.portals', 'u')
-            ->where('u.portal = :u')
-            ->setParameters(['u' => $url])
-            ->getQuery()
-            ->getSingleScalarResult();
-    }
-
-    /**
-     * Get number of users in URL
-     * @param AccessUrl $url
-     *
-     * @return int
-     */
-    public function getCountTeachersByUrl(AccessUrl $url)
-    {
-        $qb = $this->createQueryBuilder('a');
-
-        return $qb
-            ->select('COUNT(a)')
-            ->innerJoin('a.portals', 'u')
-            ->where('u.portal = :u and u.group = :g')
-            ->andWhere($qb->expr()->in('a.roles', ['ROLE_TEACHER']))
-            ->setParameters(['u' => $url, 'g' => $group])
-            ->getQuery()
-            ->getSingleScalarResult()
-        ;
-    }
 }