Minor - format code.

main/announcements/announcement_email.class.php

@@ -49,6 +49,7 @@ class AnnouncementEmail
      * Course info
      *
      * @param string $key
+     *
      * @return array
      */
     public function course($key = '')
@@ -176,6 +177,7 @@ class AnnouncementEmail
 
         //Clean users just in case
         $new_list_users = array();
+
         if (!empty($users)) {
             foreach ($users as $user) {
                 $new_list_users[$user['user_id']] = array('user_id' => $user['user_id']);
@@ -188,9 +190,11 @@ class AnnouncementEmail
      * Sender info
      *
      * @param string $key
+     *
      * @return array
      */
-    public function sender($key = '') {
+    public function sender($key = '')
+    {
         global $_user;
         return $key ? $_user[$key] : $_user;
     }
@@ -204,19 +208,19 @@ class AnnouncementEmail
     {
         $result = $this->course('title').' - '.$this->announcement('title');
         $result = stripslashes($result);
+
         return $result;
     }
 
     /**
      * Email message
      * @param int $receiverUserId
+     *
      * @return string
      */
     public function message($receiverUserId)
     {
-        $title = $this->announcement('title');
-        $title = stripslashes($title);
-
+        //$title = $this->announcement('title');
         $content = $this->announcement('content');
         $content = stripslashes($content);
         $content = AnnouncementManager::parse_content($receiverUserId, $content, $this->course('code'));
@@ -231,13 +235,17 @@ class AnnouncementEmail
         $attachment = $this->attachement();
         if (!empty($attachment)) {
             $result .= '<br />';
-            $result .= Display::url($attachment['filename'], api_get_path(WEB_CODE_PATH).'announcements/download.php?file='.basename($attachment['path']).'&'.$course_param).'<br />';
+            $result .= Display::url(
+                $attachment['filename'],
+                api_get_path(WEB_CODE_PATH).'announcements/download.php?file='.basename($attachment['path']).'&'.$course_param
+            ).'<br />';
         }
 
         $result .= '<hr />';
         $sender_name = api_get_person_name($this->sender('firstName'), $this->sender('lastName'), PERSON_NAME_EMAIL_ADDRESS);
         $result .= '<a href="mailto:'.$user_email.'">'.$sender_name.'</a><br/>';
         $result .= '<a href="'.api_get_path(WEB_CODE_PATH).'announcements/announcements.php?'.$course_param.'">'.$course_name.'</a><br/>';
+
         return $result;
     }
 
@@ -262,6 +270,7 @@ class AnnouncementEmail
         }
 
         $result = $result ? reset($result) : array();
+
         return $result;
     }
 
@@ -297,7 +306,8 @@ class AnnouncementEmail
         $course_id = $this->course('id');
 
         $tbl_announcement = Database::get_course_table(TABLE_ANNOUNCEMENT);
-        $sql = "UPDATE $tbl_announcement SET email_sent=1 WHERE c_id = $course_id AND id=$id AND session_id = {$this->session_id} ";
+        $sql = "UPDATE $tbl_announcement SET email_sent=1
+                WHERE c_id = $course_id AND id=$id AND session_id = {$this->session_id} ";
         Database::query($sql);
     }
 }

main/announcements/announcements.inc.php

@@ -1,13 +1,14 @@
 <?php
 /* For licensing terms, see /license.txt */
 /**
- * Include file with functions for the announcements module.
- * @package chamilo.announcements
- * @todo use OOP
+
  */
 
 /**
+ * Include file with functions for the announcements module.
  * @author jmontoya
+ * @package chamilo.announcements
+ * @todo use OOP
  *
  */
 class AnnouncementManager
@@ -35,8 +36,10 @@ class AnnouncementManager
     }
 
     /**
+     * @param int       $userId
      * @param string    $content
      * @param string    $course_code
+     *
      * @return mixed
      */
     public static function parse_content($userId, $content, $course_code)
@@ -53,6 +56,7 @@ class AnnouncementManager
                 break;
             }
         }
+
         $courseLink = api_get_course_url();
         $data['user_name'] = $readerInfo['username'];
         $data['user_firstname'] = $readerInfo['firstname'];
@@ -92,7 +96,6 @@ class AnnouncementManager
 				ORDER BY display_order DESC";
         $rs = Database::query($sql);
         $num_rows = Database::num_rows($rs);
-        $result = array();
         if ($num_rows > 0) {
             $list = array();
             while ($row = Database::fetch_array($rs)) {
@@ -119,13 +122,14 @@ class AnnouncementManager
         } else {
             api_item_property_update($_course, TOOL_ANNOUNCEMENT, $id, 'visible', api_get_user_id());
         }
+
         return true;
     }
 
     /**
      * Deletes an announcement
-     * @param array the course array
-     * @param int 	the announcement id
+     * @param array $_course the course array
+     * @param int 	$id the announcement id
      */
     public static function delete_announcement($_course, $id)
     {
@@ -134,7 +138,7 @@ class AnnouncementManager
 
     /**
      * Deletes all announcements by course
-     * @param array the course array
+     * @param array $_course the course array
      */
     public static function delete_all_announcements($_course)
     {
@@ -746,10 +750,6 @@ class AnnouncementManager
         echo "</select>";
     }
 
-    /*
-      DATA FUNCTIONS
-     */
-
     /**
      * Returns announcement info from its id
      *
@@ -782,7 +782,7 @@ class AnnouncementManager
     /**
      * this function gets all the users of the course,
      * including users from linked courses
-     * @todo deprecate this function, use CourseManager class
+     * @deprecate use CourseManager class
      */
     public static function get_course_users()
     {

tests/main/inc/lib/message.lib.test.php

@@ -5,39 +5,6 @@ class TestMessage extends UnitTestCase {
 
     public function __construct() {
         $this->UnitTestCase('Messages library - main/inc/lib/message.lib.test.php');
-    }
-
-	function testGetNumberOfMessagesMask() {
-		$res=get_number_of_messages_mask();
-		$this->assertTrue(is_string($res));
-        //var_dump($res);
-	}
-
-	function testGetMessageDataMask() {
-		$from='';
-		$number_of_items=get_number_of_messages_mask();
-		$column='3';
-		$direction='';
-		$res=get_message_data_mask($from, $number_of_items, $column, $direction);
-		$this->assertTrue(is_array($res));
-		//var_dump($res);
-	}
-
-	function testGetMessageDataSendMask() {
-		$from='';
-		$number_of_items=get_number_of_messages_send_mask();
-		$column= '3';
-		$direction='';
-		$res=get_message_data_send_mask($from, $number_of_items, $column, $direction);
-		$this->assertTrue(is_array($res));
-		//var_dump($res);
-	}
-
-	function testGetNumberOfMessagesSendMask() {
-		$res=get_number_of_messages_send_mask();
-		$this->assertTrue(is_string($res));
-        //var_dump($res);
-
     }
 
 	function testInboxDisplay() {

vendor/doctrine/cache/lib/Doctrine/Common/Cache/MemcachedCache.php

@@ -24,8 +24,7 @@ use \Memcached;
 /**
  * Memcached cache provider.
  *
- * @link   www.doctrine-project.org
- * @since  2.2
+
  * @author Benjamin Eberlei <kontakt@beberlei.de>
  * @author Guilherme Blanco <guilhermeblanco@hotmail.com>
  * @author Jonathan Wage <jonwage@gmail.com>