chamilo / chamilo-lms

main/inc/lib/pear/Text/Diff/Renderer/context.php

Spacing +2 added lines, -2 removed lines

@@ -35,10 +35,10 @@
     function _blockHeader($xbeg, $xlen, $ybeg, $ylen)
     {
         if ($xlen != 1) {
-            $xbeg .= ',' . $xlen;
+            $xbeg .= ','.$xlen;
         }
         if ($ylen != 1) {
-            $ybeg .= ',' . $ylen;
+            $ybeg .= ','.$ylen;
         }
         $this->_second_block = "--- $ybeg ----\n";
         return "***************\n*** $xbeg ****";

main/inc/lib/pear/Text/Diff/Renderer/unified.php

Spacing +3 added lines, -3 removed lines

@@ -34,10 +34,10 @@ 
     function _blockHeader($xbeg, $xlen, $ybeg, $ylen)
     {
         if ($xlen != 1) {
-            $xbeg .= ',' . $xlen;
+            $xbeg .= ','.$xlen;
         }
         if ($ylen != 1) {
-            $ybeg .= ',' . $ylen;
+            $ybeg .= ','.$ylen;
         }
         return "@@ -$xbeg +$ybeg @@";
     }
@@ -59,7 +59,7 @@ 
 
     function _changed($orig, $final)
     {
-        return $this->_deleted($orig) . $this->_added($final);
+        return $this->_deleted($orig).$this->_added($final);
     }
 
 }

main/inc/lib/pear/Text/Diff/Mapped.php

Indentation +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
      *                                  of elements as $to_lines.
      */
     function Text_Diff_Mapped($from_lines, $to_lines,
-                              $mapped_from_lines, $mapped_to_lines)
+                                $mapped_from_lines, $mapped_to_lines)
     {
         assert(count($from_lines) == count($mapped_from_lines));
         assert(count($to_lines) == count($mapped_to_lines));

main/inc/lib/pear/Text/Diff/Engine/xdiff.php

Switch Indentation +9 added lines, -9 removed lines

@@ -46,17 +46,17 @@
                 continue;
             }
             switch ($line[0]) {
-            case ' ':
-                $edits[] = &new Text_Diff_Op_copy(array(substr($line, 1)));
-                break;
+                case ' ':
+                    $edits[] = &new Text_Diff_Op_copy(array(substr($line, 1)));
+                    break;
 
-            case '+':
-                $edits[] = &new Text_Diff_Op_add(array(substr($line, 1)));
-                break;
+                case '+':
+                    $edits[] = &new Text_Diff_Op_add(array(substr($line, 1)));
+                    break;
 
-            case '-':
-                $edits[] = &new Text_Diff_Op_delete(array(substr($line, 1)));
-                break;
+                case '-':
+                    $edits[] = &new Text_Diff_Op_delete(array(substr($line, 1)));
+                    break;
             }
         }
 

main/inc/lib/pear/Text/Diff/Engine/shell.php

Indentation +2 added lines, -2 removed lines

@@ -124,9 +124,9 @@
             array_push($edits,
                 new Text_Diff_Op_copy(
                     $this->_getLines($from_lines, $from_line_no,
-                                     $from_line_no + count($from_lines) - 1),
+                                        $from_line_no + count($from_lines) - 1),
                     $this->_getLines($to_lines, $to_line_no,
-                                     $to_line_no + count($to_lines) - 1)));
+                                        $to_line_no + count($to_lines) - 1)));
         }
 
         return $edits;

Switch Indentation +23 added lines, -23 removed lines

@@ -93,29 +93,29 @@
             }
 
             switch ($match[3]) {
-            case 'd':
-                // deleted lines
-                array_push($edits,
-                    new Text_Diff_Op_delete(
-                        $this->_getLines($from_lines, $from_line_no, $match[2])));
-                $to_line_no++;
-                break;
-
-            case 'c':
-                // changed lines
-                array_push($edits,
-                    new Text_Diff_Op_change(
-                        $this->_getLines($from_lines, $from_line_no, $match[2]),
-                        $this->_getLines($to_lines, $to_line_no, $match[5])));
-                break;
-
-            case 'a':
-                // added lines
-                array_push($edits,
-                    new Text_Diff_Op_add(
-                        $this->_getLines($to_lines, $to_line_no, $match[5])));
-                $from_line_no++;
-                break;
+                case 'd':
+                    // deleted lines
+                    array_push($edits,
+                        new Text_Diff_Op_delete(
+                            $this->_getLines($from_lines, $from_line_no, $match[2])));
+                    $to_line_no++;
+                    break;
+
+                case 'c':
+                    // changed lines
+                    array_push($edits,
+                        new Text_Diff_Op_change(
+                            $this->_getLines($from_lines, $from_line_no, $match[2]),
+                            $this->_getLines($to_lines, $to_line_no, $match[5])));
+                    break;
+
+                case 'a':
+                    // added lines
+                    array_push($edits,
+                        new Text_Diff_Op_add(
+                            $this->_getLines($to_lines, $to_line_no, $match[5])));
+                    $from_line_no++;
+                    break;
             }
         }
 

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
         $fp = fopen($to_file, 'w');
         fwrite($fp, implode("\n", $to_lines));
         fclose($fp);
-        $diff = shell_exec($this->_diffCommand . ' ' . $from_file . ' ' . $to_file);
+        $diff = shell_exec($this->_diffCommand.' '.$from_file.' '.$to_file);
         unlink($from_file);
         unlink($to_file);
 

main/inc/lib/pear/PEAR5.php

Indentation +11 added lines, -11 removed lines

@@ -6,17 +6,17 @@
 class PEAR5
 {
     /**
-    * If you have a class that's mostly/entirely static, and you need static
-    * properties, you can use this method to simulate them. Eg. in your method(s)
-    * do this: $myVar = &PEAR5::getStaticProperty('myclass', 'myVar');
-    * You MUST use a reference, or they will not persist!
-    *
-    * @access public
-    * @param  string $class  The calling classname, to prevent clashes
-    * @param  string $var    The variable to retrieve.
-    * @return mixed   A reference to the variable. If not set it will be
-    *                 auto initialised to NULL.
-    */
+     * If you have a class that's mostly/entirely static, and you need static
+     * properties, you can use this method to simulate them. Eg. in your method(s)
+     * do this: $myVar = &PEAR5::getStaticProperty('myclass', 'myVar');
+     * You MUST use a reference, or they will not persist!
+     *
+     * @access public
+     * @param  string $class  The calling classname, to prevent clashes
+     * @param  string $var    The variable to retrieve.
+     * @return mixed   A reference to the variable. If not set it will be
+     *                 auto initialised to NULL.
+     */
     static function &getStaticProperty($class, $var)
     {
         static $properties;

main/inc/lib/pear/Pager/Jumping.php

Indentation +1 added lines, -1 removed lines

@@ -250,7 +250,7 @@
                 $links .= $this->_curPageSpanPre . $i . $this->_curPageSpanPost;
             }
             $links .= $this->_spacesBefore
-                   . (($i != $this->_totalPages) ? $this->_separator.$this->_spacesAfter : '');
+                    . (($i != $this->_totalPages) ? $this->_separator.$this->_spacesAfter : '');
         }
         return $links;
     }

Spacing +7 added lines, -7 removed lines

@@ -91,7 +91,7 @@ 
         }
 
         if (($index % $this->_perPage) > 0) {
-            $pageID = ceil((float)$index / (float)$this->_perPage);
+            $pageID = ceil((float) $index / (float) $this->_perPage);
         } else {
             $pageID = $index / $this->_perPage;
         }
@@ -117,14 +117,14 @@ 
      */
     function getPageRangeByPageId($pageid = null)
     {
-        $pageid = isset($pageid) ? (int)$pageid : $this->_currentPage;
+        $pageid = isset($pageid) ? (int) $pageid : $this->_currentPage;
         if (isset($this->_pageData[$pageid]) || is_null($this->_itemData)) {
             // I'm sure I'm missing something here, but this formula works
             // so I'm using it until I find something simpler.
-            $start = ((($pageid + (($this->_delta - ($pageid % $this->_delta))) % $this->_delta) / $this->_delta) - 1) * $this->_delta +1;
+            $start = ((($pageid + (($this->_delta - ($pageid % $this->_delta))) % $this->_delta) / $this->_delta) - 1) * $this->_delta + 1;
             return array(
                 max($start, 1),
-                min($start+$this->_delta-1, $this->_totalPages)
+                min($start + $this->_delta - 1, $this->_totalPages)
             );
         } else {
             return array(0, 0);
@@ -153,7 +153,7 @@ 
      *
      * @return array Back/pages/next links
      */
-    function getLinks($pageID=null, $next_html='')
+    function getLinks($pageID = null, $next_html = '')
     {
         //BC hack
         if (!empty($next_html)) {
@@ -240,14 +240,14 @@ 
         $links = '';
         $limits = $this->getPageRangeByPageId($this->_currentPage);
 
-        for ($i=$limits[0]; $i<=min($limits[1], $this->_totalPages); $i++) {
+        for ($i = $limits[0]; $i <= min($limits[1], $this->_totalPages); $i++) {
             if ($i != $this->_currentPage) {
                 $this->range[$i] = false;
                 $this->_linkData[$this->_urlVar] = $i;
                 $links .= $this->_renderLink(str_replace('%d', $i, $this->_altPage), $i);
             } else {
                 $this->range[$i] = true;
-                $links .= $this->_curPageSpanPre . $i . $this->_curPageSpanPost;
+                $links .= $this->_curPageSpanPre.$i.$this->_curPageSpanPost;
             }
             $links .= $this->_spacesBefore
                    . (($i != $this->_totalPages) ? $this->_separator.$this->_spacesAfter : '');

Doc Comments +1 added lines, -1 removed lines

@@ -107,7 +107,7 @@
      *
      * @param integer $pageid PageID to get offsets for
      *
-     * @return array  First and last offsets
+     * @return integer[]  First and last offsets
      * @access public
      */
     function getPageRangeByPageId($pageid = null)

main/inc/lib/pear/Pager/Pager.php

Spacing +2 added lines, -2 removed lines

@@ -169,8 +169,8 @@
     static function &factory($options = array())
     {
         $mode = (isset($options['mode']) ? ucfirst($options['mode']) : 'Jumping');
-        $classname = 'Pager_' . $mode;
-        $classfile = 'Pager' . DIRECTORY_SEPARATOR . $mode . '.php';
+        $classname = 'Pager_'.$mode;
+        $classfile = 'Pager'.DIRECTORY_SEPARATOR.$mode.'.php';
 
         // Attempt to include a custom version of the named class, but don't treat
         // a failure as fatal.  The caller may have already included their own

main/inc/lib/events_email.class.php

Spacing +14 added lines, -14 removed lines

@@ -124,7 +124,7 @@ 
 
                 // If the mail only need to be send once (we know that thanks to the events.conf), we log it in the table
                 if ($event_config[$event_name]["sending_mail_once"]) {
-                    $sql = 'INSERT INTO ' . Database::get_main_table(TABLE_EVENT_SENT) . ' (user_from, user_to, event_type_name)
+                    $sql = 'INSERT INTO '.Database::get_main_table(TABLE_EVENT_SENT).' (user_from, user_to, event_type_name)
                             VALUES ('.$event_data["user_id"].', '.$id.' ,"'.Database::escape_string($event_name).'")
                     ';
                     Database::query($sql);
@@ -135,9 +135,9 @@ 
         // Second, we send to people linked to the event
         // So, we get everyone
         $sql = 'SELECT u.user_id, u.language, u.email, u.firstname, u.lastname
-                FROM ' . Database::get_main_table(TABLE_EVENT_TYPE_REL_USER) . ' ue
+                FROM ' . Database::get_main_table(TABLE_EVENT_TYPE_REL_USER).' ue
                 INNER JOIN '.Database::get_main_table(TABLE_MAIN_USER).' u ON u.user_id = ue.user_id
-                WHERE event_type_name = "' . $event_name . '"';
+                WHERE event_type_name = "' . $event_name.'"';
         $result = Database::store_result(Database::query($sql), 'ASSOC');
         // for each of the linked users
         foreach ($result as $key => $value) {
@@ -177,7 +177,7 @@ 
 
             // If the mail only need to be send once (we know that thanks to the events.conf, we log it in the table
             if ($event_config[$event_name]["sending_mail_once"]) {
-                $sql = 'INSERT INTO ' . Database::get_main_table(TABLE_EVENT_SENT) . '
+                $sql = 'INSERT INTO '.Database::get_main_table(TABLE_EVENT_SENT).'
                     (user_from, user_to, event_type_name)
                     VALUES ('.$event_data["user_id"].', '.$value["user_id"].' , "'.Database::escape_string($event_name).'");
                     ';
@@ -202,16 +202,16 @@ 
         $current_language = api_get_interface_language();
 
         $sql = 'SELECT COUNT(*) as total
-                FROM ' . Database::get_main_table(TABLE_EVENT_EMAIL_TEMPLATE) . ' em
-                INNER JOIN ' . Database::get_main_table(TABLE_MAIN_LANGUAGE) . ' l
+                FROM ' . Database::get_main_table(TABLE_EVENT_EMAIL_TEMPLATE).' em
+                INNER JOIN ' . Database::get_main_table(TABLE_MAIN_LANGUAGE).' l
                 ON em.language_id = l.id
                 WHERE
-                    em.event_type_name = "' . $event_name . '" and
+                    em.event_type_name = "' . $event_name.'" and
                     l.dokeos_folder = "'.$current_language.'" and
                     em.activated = 1';
 
         $exists = Database::store_result(Database::query($sql), 'ASSOC');
-        if ($exists[0]["total"])  {
+        if ($exists[0]["total"]) {
             return true;
         } else {
             return false;
@@ -228,12 +228,12 @@ 
     private static function getMessage($event_name, $language)
     {
         $sql = 'SELECT message, subject, l.dokeos_folder
-                FROM ' . Database::get_main_table(TABLE_EVENT_EMAIL_TEMPLATE) . ' em
-                INNER JOIN ' . Database::get_main_table(TABLE_MAIN_LANGUAGE) . ' l
+                FROM ' . Database::get_main_table(TABLE_EVENT_EMAIL_TEMPLATE).' em
+                INNER JOIN ' . Database::get_main_table(TABLE_MAIN_LANGUAGE).' l
                 ON em.language_id = l.id
                 WHERE
-                    em.event_type_name = "' . $event_name . '" AND
-                    (l.dokeos_folder = "' . $language . '" OR l.dokeos_folder = "english") AND
+                    em.event_type_name = "' . $event_name.'" AND
+                    (l.dokeos_folder = "' . $language.'" OR l.dokeos_folder = "english") AND
                     em.message <> ""
                 ';
         return Database::store_result(Database::query($sql), 'ASSOC');
@@ -274,8 +274,8 @@ 
     private static function formatMessage(&$message, &$subject, $event_config, $event_name, &$event_data)
     {
         foreach ($event_config[$event_name]["available_keyvars"] as $key => $word) {
-            $message = str_replace('((' . $key . '))', $event_data[$word], $message);
-            $subject = str_replace('((' . $key . '))', $event_data[$word], $subject);
+            $message = str_replace('(('.$key.'))', $event_data[$word], $message);
+            $subject = str_replace('(('.$key.'))', $event_data[$word], $subject);
         }
     }
 }