chamilo / chamilo-lms

src/Chamilo/UserBundle/Entity/Repository/UserRepository.php

Spacing +2 added lines, -2 removed lines

@@ -258,7 +258,7 @@ 
                         INNER JOIN ChamiloUserBundle:User AS U WITH UF.friendUserId = U
                         WHERE
                             U.status != 6 AND
-                            UF.relationType NOT IN(" . USER_RELATION_TYPE_DELETED . ", " . USER_RELATION_TYPE_RRHH . ") AND
+                            UF.relationType NOT IN(" . USER_RELATION_TYPE_DELETED.", ".USER_RELATION_TYPE_RRHH.") AND
                             UF.userId = $currentUserId AND
                             UF.friendUserId != $currentUserId AND
                             U = R.user AND
@@ -282,7 +282,7 @@ 
                 $dql = "SELECT DISTINCT U
                         FROM ChamiloUserBundle:User U
                         INNER JOIN ChamiloCoreBundle:TrackEOnline T WITH U.id = T.loginUserId
-                        WHERE T.loginDate >= '" . $limit_date . "'";
+                        WHERE T.loginDate >= '" . $limit_date."'";
             }
         }
 

plugin/ims_lti/create.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
         Display::return_message($plugin->get_lang('ToolAdded'), 'success')
     );
 
-    header('Location: ' . api_get_path(WEB_PLUGIN_PATH) . 'ims_lti/list.php');
+    header('Location: '.api_get_path(WEB_PLUGIN_PATH).'ims_lti/list.php');
     exit;
 }
 

plugin/ims_lti/OAuthSimple.php

Spacing +55 added lines, -55 removed lines

@@ -25,7 +25,7 @@ 
      * @param shared_secret (String) The shared secret. This value is also usually provided by the site you wish to use.
 	 * @return OAuthSimple (Object)
      */
-    function __construct ($APIKey = "", $sharedSecret=""){
+    function __construct($APIKey = "", $sharedSecret = "") {
 
         if (!empty($APIKey))
 		{
@@ -65,7 +65,7 @@ 
 	 * @param(string, object) List of parameters for the call, this can either be a URI string (e.g. "foo=bar&gorp=banana" or an object/hash)
 	 * @return OAuthSimple (Object)
 	 */
-    public function setParameters ($parameters=Array()) {
+    public function setParameters($parameters = Array()) {
 
         if (is_string($parameters))
 		{
@@ -77,7 +77,7 @@ 
 		}
         else if (!empty($parameters))
 		{
-			$this->_parameters = array_merge($this->_parameters,$parameters);
+			$this->_parameters = array_merge($this->_parameters, $parameters);
 		}
         if (empty($this->_parameters['oauth_nonce']))
 		{
@@ -101,7 +101,7 @@ 
 		}
         if (empty($this->_parameters['oauth_version']))
 		{
-            $this->_parameters['oauth_version']="1.0";
+            $this->_parameters['oauth_version'] = "1.0";
 		}
 
         return $this;
@@ -113,7 +113,7 @@ 
 	 * @access public
 	 * @see setParameters
 	 */
-    public function setQueryString ($parameters)
+    public function setQueryString($parameters)
     {
         return $this->setParameters($parameters);
     }
@@ -124,13 +124,13 @@ 
      * @param path (String) the fully qualified URI (excluding query arguments) (e.g "http://example.org/foo")
 	 * @return OAuthSimple (Object)
      */
-    public function setURL ($path)
+    public function setURL($path)
 	{
         if (empty($path))
 		{
             throw new OAuthSimpleException('No path specified for OAuthSimple.setURL');
 		}
-        $this->_path=$path;
+        $this->_path = $path;
 
         return $this;
     }
@@ -141,9 +141,9 @@ 
      * @param path (String)
 	 * @see setURL
      */
-    public function setPath ($path)
+    public function setPath($path)
     {
-        return $this->_path=$path;
+        return $this->_path = $path;
     }
 
     /**
@@ -152,14 +152,14 @@ 
      * @param action (String) HTTP Action word.
 	 * @return OAuthSimple (Object)
      */
-    public function setAction ($action)
+    public function setAction($action)
     {
         if (empty($action))
 		{
 			$action = 'GET';
 		}
         $action = strtoupper($action);
-        if (preg_match('/[^A-Z]/',$action))
+        if (preg_match('/[^A-Z]/', $action))
 		{
             throw new OAuthSimpleException('Invalid action specified for OAuthSimple.setAction');
 		}
@@ -174,7 +174,7 @@ 
      * @param signatures (object) object/hash of the token/signature pairs {api_key:, shared_secret:, oauth_token: oauth_secret:}
 	 * @return OAuthSimple (Object)
      */
-    public function signatures ($signatures)
+    public function signatures($signatures)
     {
         if (!empty($signatures) && !is_array($signatures))
 		{
@@ -184,9 +184,9 @@ 
         {
             if (empty($this->_secrets))
             {
-                $this->_secrets=Array();
+                $this->_secrets = Array();
             }
-            $this->_secrets=array_merge($this->_secrets,$signatures);
+            $this->_secrets = array_merge($this->_secrets, $signatures);
         }
         if (isset($this->_secrets['api_key']))
 		{
@@ -231,21 +231,21 @@ 
      * @param method (String) Method of signing the transaction (only PLAINTEXT and SHA-MAC1 allowed for now)
 	 * @return OAuthSimple (Object)
     */
-    public function setSignatureMethod ($method="")
+    public function setSignatureMethod($method = "")
 	{
         if (empty($method))
 		{
             $method = $this->_default_signature_method;
 		}
         $method = strtoupper($method);
-        switch($method)
+        switch ($method)
         {
             case 'PLAINTEXT':
             case 'HMAC-SHA1':
-                $this->_parameters['oauth_signature_method']=$method;
+                $this->_parameters['oauth_signature_method'] = $method;
 				break;
             default:
-                throw new OAuthSimpleException ("Unknown signing method $method specified for OAuthSimple.setSignatureMethod");
+                throw new OAuthSimpleException("Unknown signing method $method specified for OAuthSimple.setSignatureMethod");
 				break;
         }
 
@@ -260,7 +260,7 @@ 
      * @param args (Array) hash of arguments for the call {action, path, parameters (array), method, signatures (array)} all arguments are optional.
 	 * @return (Array) signed values
      */
-    public function sign($args=array())
+    public function sign($args = array())
     {
         if (!empty($args['action']))
 		{
@@ -280,15 +280,15 @@ 
 		}
         if (empty($args['parameters']))
 		{
-            $args['parameters']=array();
+            $args['parameters'] = array();
 		}
         $this->setParameters($args['parameters']);
         $normParams = $this->_normalizedParameters();
 
-        return Array (
+        return Array(
             'parameters' => $this->_parameters,
             'signature' => self::_oauthEscape($this->_parameters['oauth_signature']),
-            'signed_url' => $this->_path . '?' . $normParams,
+            'signed_url' => $this->_path.'?'.$normParams,
             'header' => $this->getHeaderString(),
             'sbs'=> $this->sbs
             );
@@ -304,7 +304,7 @@ 
      * @param args (Array)
 	 * @return $result (String)
     */
-    public function getHeaderString ($args=array())
+    public function getHeaderString($args = array())
     {
         if (empty($this->_parameters['oauth_signature']))
 		{
@@ -314,7 +314,7 @@ 
 
         foreach ($this->_parameters as $pName => $pValue)
         {
-            if (strpos($pName,'oauth_') !== 0)
+            if (strpos($pName, 'oauth_') !== 0)
 			{
                 continue;
 			}
@@ -322,25 +322,25 @@ 
             {
                 foreach ($pValue as $val)
                 {
-                    $result .= $pName .'="' . self::_oauthEscape($val) . '", ';
+                    $result .= $pName.'="'.self::_oauthEscape($val).'", ';
                 }
             }
             else
             {
-                $result .= $pName . '="' . self::_oauthEscape($pValue) . '", ';
+                $result .= $pName.'="'.self::_oauthEscape($pValue).'", ';
             }
         }
 
-        return preg_replace('/, $/','',$result);
+        return preg_replace('/, $/', '', $result);
     }
 
-    private function _parseParameterString ($paramString)
+    private function _parseParameterString($paramString)
     {
-        $elements = explode('&',$paramString);
+        $elements = explode('&', $paramString);
         $result = array();
         foreach ($elements as $element)
         {
-            list ($key,$token) = explode('=',$element);
+            list ($key, $token) = explode('=', $element);
             if ($token)
 			{
                 $token = urldecode($token);
@@ -349,15 +349,15 @@ 
             {
                 if (!is_array($result[$key]))
 				{
-                    $result[$key] = array($result[$key],$token);
+                    $result[$key] = array($result[$key], $token);
 				}
                 else
 				{
-                    array_push($result[$key],$token);
+                    array_push($result[$key], $token);
 				}
             }
             else
-                $result[$key]=$token;
+                $result[$key] = $token;
         }
         return $result;
     }
@@ -375,21 +375,21 @@ 
 
 	    //FIX: urlencode of ~ and '+'
         $string = str_replace(
-            Array('%7E','+'  ), // Replace these
-            Array('~',  '%20'), // with these
+            Array('%7E', '+'), // Replace these
+            Array('~', '%20'), // with these
             $string);
 
         return $string;
     }
 
-    private function _getNonce($length=5)
+    private function _getNonce($length = 5)
     {
         $result = '';
         $cLength = strlen($this->_nonce_chars);
-        for ($i=0; $i < $length; $i++)
+        for ($i = 0; $i < $length; $i++)
         {
-            $rnum = rand(0,$cLength - 1);
-            $result .= substr($this->_nonce_chars,$rnum,1);
+            $rnum = rand(0, $cLength - 1);
+            $result .= substr($this->_nonce_chars, $rnum, 1);
         }
         $this->_parameters['oauth_nonce'] = $result;
 
@@ -402,7 +402,7 @@ 
         {
             throw new OAuthSimpleException('No consumer_key set for OAuthSimple');
         }
-        $this->_parameters['oauth_consumer_key']=$this->_secrets['consumer_key'];
+        $this->_parameters['oauth_consumer_key'] = $this->_secrets['consumer_key'];
 
         return $this->_parameters['oauth_consumer_key'];
     }
@@ -432,7 +432,7 @@ 
 		$normalized_keys = array();
 		$return_array = array();
 
-        foreach ( $this->_parameters as $paramName=>$paramValue) {
+        foreach ($this->_parameters as $paramName=>$paramValue) {
             if (preg_match('/w+_secret/', $paramName) OR
                 $paramName == "oauth_signature") {
                     continue;
@@ -443,9 +443,9 @@ 
 				if (is_array($paramValue))
 				{
 					$normalized_keys[self::_oauthEscape($paramName)] = array();
-					foreach($paramValue as $item)
+					foreach ($paramValue as $item)
 					{
-						array_push($normalized_keys[self::_oauthEscape($paramName)],  self::_oauthEscape($item));
+						array_push($normalized_keys[self::_oauthEscape($paramName)], self::_oauthEscape($item));
 					}
 				}
 				else
@@ -457,54 +457,54 @@ 
 
 		ksort($normalized_keys);
 
-		foreach($normalized_keys as $key=>$val)
+		foreach ($normalized_keys as $key=>$val)
 		{
 			if (is_array($val))
 			{
 				sort($val);
-				foreach($val as $element)
+				foreach ($val as $element)
 				{
-					array_push($return_array, $key . "=" . $element);
+					array_push($return_array, $key."=".$element);
 				}
 			}
 			else
 			{
-				array_push($return_array, $key .'='. $val);
+				array_push($return_array, $key.'='.$val);
 			}
 
         }
         $presig = join("&", $return_array);
         $sig = $this->_generateSignature($presig);
-        $this->_parameters['oauth_signature']=$sig;
+        $this->_parameters['oauth_signature'] = $sig;
         array_push($return_array, "oauth_signature=$sig");
 		return join("&", $return_array);
     }
 
 
-    private function _generateSignature ($parameters="")
+    private function _generateSignature($parameters = "")
     {
         $secretKey = '';
-		if(isset($this->_secrets['shared_secret']))
+		if (isset($this->_secrets['shared_secret']))
 		{
 			$secretKey = self::_oauthEscape($this->_secrets['shared_secret']);
 		}
 
 		$secretKey .= '&';
-		if(isset($this->_secrets['oauth_secret']))
+		if (isset($this->_secrets['oauth_secret']))
 		{
             $secretKey .= self::_oauthEscape($this->_secrets['oauth_secret']);
         }
-        if(!empty($parameters)){
+        if (!empty($parameters)) {
             $parameters = urlencode($parameters);
         }
-        switch($this->_parameters['oauth_signature_method'])
+        switch ($this->_parameters['oauth_signature_method'])
         {
             case 'PLAINTEXT':
-                return urlencode($secretKey);;
+                return urlencode($secretKey); ;
             case 'HMAC-SHA1':
                 $this->sbs = self::_oauthEscape($this->_action).'&'.self::_oauthEscape($this->_path).'&'.$parameters;
 
-                return base64_encode(hash_hmac('sha1',$this->sbs,$secretKey,TRUE));
+                return base64_encode(hash_hmac('sha1', $this->sbs, $secretKey, TRUE));
             default:
                 throw new OAuthSimpleException('Unknown signature method for OAuthSimple');
 				break;

plugin/ims_lti/edit.php

Spacing +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
         Display::return_message($plugin->get_lang('NoTool'), 'error')
     );
 
-    header('Location: ' . api_get_path(WEB_PLUGIN_PATH) . 'ims_lti/list.php');
+    header('Location: '.api_get_path(WEB_PLUGIN_PATH).'ims_lti/list.php');
     exit;
 }
 
@@ -58,7 +58,7 @@ 
         Display::return_message($plugin->get_lang('ToolEdited'), 'success')
     );
 
-    header('Location: ' . api_get_path(WEB_PLUGIN_PATH) . 'ims_lti/list.php');
+    header('Location: '.api_get_path(WEB_PLUGIN_PATH).'ims_lti/list.php');
     exit;
 }
 

plugin/ims_lti/ImsLtiTool.php

Spacing +1 added lines, -1 removed lines

@@ -173,7 +173,7 @@
         $foo = explode('=', $strings);
 
         return [
-            'key' => 'custom_' . $foo[0],
+            'key' => 'custom_'.$foo[0],
             'value' => $foo[1]
         ];
     }

plugin/ims_lti/ImsLtiPlugin.php

Spacing +6 added lines, -6 removed lines

@@ -111,7 +111,7 @@ 
             return false;
         }
 
-        $sql = 'DROP TABLE IF EXISTS ' . self::TABLE_TOOL;
+        $sql = 'DROP TABLE IF EXISTS '.self::TABLE_TOOL;
         Database::query($sql);
 
         return true;
@@ -124,14 +124,14 @@ 
     {
         $button = Display::toolbarButton(
             $this->get_lang('AddExternalTool'),
-            api_get_path(WEB_PLUGIN_PATH) . 'ims_lti/add.php',
+            api_get_path(WEB_PLUGIN_PATH).'ims_lti/add.php',
             'cog',
             'primary'
         );
 
         $this->course_settings = [
             [
-                'name' => $this->get_lang('ImsLtiDescription') . $button . '<hr>',
+                'name' => $this->get_lang('ImsLtiDescription').$button.'<hr>',
                 'type' => 'html'
             ]
         ];
@@ -153,8 +153,8 @@ 
             ->setId($cToolId)
             ->setCId($course->getId())
             ->setName($tool->getName())
-            ->setLink($this->get_name() . '/start.php?' . http_build_query(['id' => $tool->getId()]))
-            ->setImage($this->get_name() . '.png')
+            ->setLink($this->get_name().'/start.php?'.http_build_query(['id' => $tool->getId()]))
+            ->setImage($this->get_name().'.png')
             ->setVisibility(1)
             ->setAdmin(0)
             ->setAddress('squaregray.gif')
@@ -172,7 +172,7 @@ 
         $text = $this->get_lang('ImsLtiDescription');
         $text .= sprintf(
             $this->get_lang('ManageToolButton'),
-            api_get_path(WEB_PLUGIN_PATH) . 'ims_lti/list.php'
+            api_get_path(WEB_PLUGIN_PATH).'ims_lti/list.php'
         );
 
         return $text;

plugin/ims_lti/start.php

Spacing +2 added lines, -2 removed lines

@@ -6,7 +6,7 @@ 
 
 if (empty($toolId)) {
     if (api_is_platform_admin()) {
-        header('Location: ' . api_get_path(WEB_PLUGIN_PATH) . 'ims_lti/list.php');
+        header('Location: '.api_get_path(WEB_PLUGIN_PATH).'ims_lti/list.php');
         exit;
     }
 
@@ -25,7 +25,7 @@ 
 $template = new Template($imsLtiPlugin->get_title());
 $template->assign(
     'launch_url',
-    api_get_path(WEB_PLUGIN_PATH) . 'ims_lti/form.php?' . http_build_query(['id' => $tool->getId()])
+    api_get_path(WEB_PLUGIN_PATH).'ims_lti/form.php?'.http_build_query(['id' => $tool->getId()])
 );
 
 $content = $template->fetch('ims_lti/view/start.tpl');

main/dropbox/dropbox_download.php

Spacing +7 added lines, -7 removed lines

@@ -32,8 +32,8 @@ 
     if ($_GET['sent_received'] == 'sent') {
         // here we also incorporate the person table to make sure that deleted sent documents are not included.
         $sql = "SELECT DISTINCT file.id, file.filename, file.title
-                FROM ". Database::get_course_table(TABLE_DROPBOX_FILE) ." file
-                INNER JOIN ". Database::get_course_table(TABLE_DROPBOX_PERSON) ." person
+                FROM ". Database::get_course_table(TABLE_DROPBOX_FILE)." file
+                INNER JOIN ". Database::get_course_table(TABLE_DROPBOX_PERSON)." person
                 ON (person.file_id=file.id AND file.c_id = $course_id AND person.c_id = $course_id)
                 WHERE
                     file.uploader_id = $user_id AND
@@ -43,14 +43,14 @@ 
 
     if ($_GET['sent_received'] == 'received') {
         $sql = "SELECT DISTINCT file.id, file.filename, file.title
-                FROM ". Database::get_course_table(TABLE_DROPBOX_FILE) ." file
-                INNER JOIN ". Database::get_course_table(TABLE_DROPBOX_PERSON) ." person
+                FROM ". Database::get_course_table(TABLE_DROPBOX_FILE)." file
+                INNER JOIN ". Database::get_course_table(TABLE_DROPBOX_PERSON)." person
                 ON (person.file_id=file.id AND file.c_id = $course_id AND person.c_id = $course_id)
                 INNER JOIN ".Database::get_course_table(TABLE_DROPBOX_POST)." post
                 ON (post.file_id = file.id AND post.c_id = $course_id AND file.c_id = $course_id)
                 WHERE
                     post.cat_id = ".intval($_GET['cat_id'])." AND
-                    post.dest_user_id = $user_id" ;
+                    post.dest_user_id = $user_id";
     }
     $files_to_download = array();
     $result = Database::query($sql);
@@ -97,9 +97,9 @@ 
 
     $work = new Dropbox_Work($_GET['id']);
 
-    $path = api_get_path(SYS_COURSE_PATH) . $_course['path'] . '/dropbox/' . $work -> filename; //path to file as stored on server
+    $path = api_get_path(SYS_COURSE_PATH).$_course['path'].'/dropbox/'.$work -> filename; //path to file as stored on server
 
-    if (!Security::check_abs_path($path, api_get_path(SYS_COURSE_PATH) . $_course['path'] . '/dropbox/')) {
+    if (!Security::check_abs_path($path, api_get_path(SYS_COURSE_PATH).$_course['path'].'/dropbox/')) {
         exit;
     }
     $file = $work->title;

main/dropbox/dropbox_functions.inc.php

Spacing +42 added lines, -42 removed lines

@@ -42,7 +42,7 @@ 
 
     // STEP 2: at least one file has to be selected. If not we return an error message
     $ids = isset($_GET['id']) ? $_GET['id'] : array();
-    if (count($ids)>0) {
+    if (count($ids) > 0) {
         $checked_file_ids = $_POST['id'];
     } else {
         foreach ($_POST as $key => $value) {
@@ -133,7 +133,7 @@ 
     }
 
     $cat = get_dropbox_category($id);
-    if (count($cat)==0) {
+    if (count($cat) == 0) {
         return false;
     }
 
@@ -157,7 +157,7 @@ 
     }
 
     // step 1: delete the category
-    $sql = "DELETE FROM ". Database::get_course_table(TABLE_DROPBOX_CATEGORY) ."
+    $sql = "DELETE FROM ".Database::get_course_table(TABLE_DROPBOX_CATEGORY)."
             WHERE c_id = $course_id AND cat_id='".intval($id)."' AND $sentreceived='1'";
     Database::query($sql);
 
@@ -235,7 +235,7 @@ 
             $return_message = get_lang('ReceivedFileMoved');
         }
         if ($part == 'sent') {
-            $sql = "UPDATE ". Database::get_course_table(TABLE_DROPBOX_FILE) ."
+            $sql = "UPDATE ".Database::get_course_table(TABLE_DROPBOX_FILE)."
                     SET cat_id = ".intval($target)."
                     WHERE
                         c_id = $course_id AND
@@ -270,7 +270,7 @@ 
     $session_id = api_get_session_id();
     $condition_session = api_get_session_condition($session_id);
 
-    $sql = "SELECT * FROM ". Database::get_course_table(TABLE_DROPBOX_CATEGORY) ."
+    $sql = "SELECT * FROM ".Database::get_course_table(TABLE_DROPBOX_CATEGORY)."
             WHERE c_id = $course_id AND user_id='".$_user['user_id']."' $condition_session";
 
     $result = Database::query($sql);
@@ -292,7 +292,7 @@ 
 {
     $course_id = api_get_course_int_id();
     if (empty($id) or $id != intval($id)) { return array(); }
-    $sql = "SELECT * FROM ". Database::get_course_table(TABLE_DROPBOX_CATEGORY) ."
+    $sql = "SELECT * FROM ".Database::get_course_table(TABLE_DROPBOX_CATEGORY)."
             WHERE c_id = $course_id AND cat_id='".$id."'";
     $res = Database::query($sql);
     if ($res === false) {
@@ -341,7 +341,7 @@ 
     if (!isset($_POST['edit_id'])) {
         $session_id = api_get_session_id();
         // step 3a, we check if the category doesn't already exist
-        $sql = "SELECT * FROM ". Database::get_course_table(TABLE_DROPBOX_CATEGORY) ."
+        $sql = "SELECT * FROM ".Database::get_course_table(TABLE_DROPBOX_CATEGORY)."
                 WHERE
                     c_id = $course_id AND
                     user_id='".$_user['user_id']."' AND
@@ -364,7 +364,7 @@ 
             ];
             $id = Database::insert(Database::get_course_table(TABLE_DROPBOX_CATEGORY), $params);
             if ($id) {
-                $sql = "UPDATE ". Database::get_course_table(TABLE_DROPBOX_CATEGORY) ." SET cat_id = iid WHERE iid = $id";
+                $sql = "UPDATE ".Database::get_course_table(TABLE_DROPBOX_CATEGORY)." SET cat_id = iid WHERE iid = $id";
                 Database::query($sql);
             }
 
@@ -413,7 +413,7 @@ 
 
     if (isset($id) && $id != '') {
         // retrieve the category we are editing
-        $sql = "SELECT * FROM ". Database::get_course_table(TABLE_DROPBOX_CATEGORY) ."
+        $sql = "SELECT * FROM ".Database::get_course_table(TABLE_DROPBOX_CATEGORY)."
                 WHERE c_id = $course_id AND cat_id = ".intval($id);
         $result = Database::query($sql);
         $row = Database::fetch_array($result);
@@ -602,7 +602,7 @@ 
 
                 $full_name = $userInfo['complete_name'].$groupNameListToString;
                 $current_user_id = $current_user['user_id'];
-                $options['user_' . $current_user_id] = $full_name;
+                $options['user_'.$current_user_id] = $full_name;
             }
         }
     }
@@ -702,8 +702,8 @@ 
 
     // select all files that aren't referenced anymore
     $sql = "SELECT DISTINCT f.id, f.filename
-            FROM " . Database::get_course_table(TABLE_DROPBOX_FILE) . " f
-            LEFT JOIN " . Database::get_course_table(TABLE_DROPBOX_PERSON) . " p
+            FROM " . Database::get_course_table(TABLE_DROPBOX_FILE)." f
+            LEFT JOIN " . Database::get_course_table(TABLE_DROPBOX_PERSON)." p
             ON (f.id = p.file_id)
             WHERE p.user_id IS NULL AND
                   f.c_id = $course_id
@@ -711,14 +711,14 @@ 
     $result = Database::query($sql);
     while ($res = Database::fetch_array($result)) {
         //delete the selected files from the post and file tables
-        $sql = "DELETE FROM " . Database::get_course_table(TABLE_DROPBOX_POST) . "
-                WHERE c_id = $course_id AND file_id = '" . $res['id'] . "'";
+        $sql = "DELETE FROM ".Database::get_course_table(TABLE_DROPBOX_POST)."
+                WHERE c_id = $course_id AND file_id = '".$res['id']."'";
         Database::query($sql);
-        $sql = "DELETE FROM " . Database::get_course_table(TABLE_DROPBOX_FILE) . "
-                WHERE c_id = $course_id AND id ='" . $res['id'] . "'";
+        $sql = "DELETE FROM ".Database::get_course_table(TABLE_DROPBOX_FILE)."
+                WHERE c_id = $course_id AND id ='".$res['id']."'";
         Database::query($sql);
         //delete file from server
-        @unlink( api_get_path(SYS_COURSE_PATH) . $_course['path'] . '/dropbox/' . $res['filename']);
+        @unlink(api_get_path(SYS_COURSE_PATH).$_course['path'].'/dropbox/'.$res['filename']);
     }
 }
 
@@ -740,11 +740,11 @@ 
 
     $mailingPseudoId = intval($mailingPseudoId);
     $sql = "SELECT f.uploader_id
-            FROM " . Database::get_course_table(TABLE_DROPBOX_FILE) . " f
-            LEFT JOIN " . Database::get_course_table(TABLE_DROPBOX_POST) . " p
+            FROM " . Database::get_course_table(TABLE_DROPBOX_FILE)." f
+            LEFT JOIN " . Database::get_course_table(TABLE_DROPBOX_POST)." p
             ON (f.id = p.file_id AND f.c_id = $course_id AND p.c_id = $course_id)
             WHERE
-                p.dest_user_id = '" . $mailingPseudoId . "' AND
+                p.dest_user_id = '".$mailingPseudoId."' AND
                 p.c_id = $course_id
             ";
     $result = Database::query($sql);
@@ -770,21 +770,21 @@ 
     //    for all content files, replace mailingPseudoId by owner as uploader
     $file_id = intval($file_id);
     $sql = "SELECT p.dest_user_id
-            FROM " . Database::get_course_table(TABLE_DROPBOX_POST) . " p
-            WHERE c_id = $course_id AND p.file_id = '" . $file_id . "'";
+            FROM " . Database::get_course_table(TABLE_DROPBOX_POST)." p
+            WHERE c_id = $course_id AND p.file_id = '".$file_id."'";
     $result = Database::query($sql);
 
     if ($res = Database::fetch_array($result)) {
         $mailingPseudoId = $res['dest_user_id'];
         $mailId = get_mail_id_base();
         if ($mailingPseudoId > $mailId) {
-            $sql = "DELETE FROM " . Database::get_course_table(TABLE_DROPBOX_PERSON) . "
-                    WHERE c_id = $course_id AND user_id='" . $mailingPseudoId . "'";
+            $sql = "DELETE FROM ".Database::get_course_table(TABLE_DROPBOX_PERSON)."
+                    WHERE c_id = $course_id AND user_id='".$mailingPseudoId."'";
             Database::query($sql);
 
-            $sql = "UPDATE " . Database::get_course_table(TABLE_DROPBOX_FILE) ."
-                    SET uploader_id='" . api_get_user_id() . "'
-                    WHERE c_id = $course_id AND uploader_id='" . $mailingPseudoId . "'";
+            $sql = "UPDATE ".Database::get_course_table(TABLE_DROPBOX_FILE)."
+                    SET uploader_id='" . api_get_user_id()."'
+                    WHERE c_id = $course_id AND uploader_id='".$mailingPseudoId."'";
             Database::query($sql);
         }
     }
@@ -925,7 +925,7 @@ 
             }
         }
     } else {  // rename file to login_filename_uniqueId format
-        $dropbox_filename = $_user['username'] . "_" . $dropbox_filename . "_".uniqid('');
+        $dropbox_filename = $_user['username']."_".$dropbox_filename."_".uniqid('');
     }
 
     // creating the array that contains all the users who will receive the file
@@ -944,7 +944,7 @@ 
         }
     }
 
-    @move_uploaded_file($dropbox_filetmpname, api_get_path(SYS_COURSE_PATH) . $_course['path'] . '/dropbox/' . $dropbox_filename);
+    @move_uploaded_file($dropbox_filetmpname, api_get_path(SYS_COURSE_PATH).$_course['path'].'/dropbox/'.$dropbox_filename);
 
     $b_send_mail = api_get_course_setting('email_alert_on_new_doc_dropbox');
 
@@ -965,14 +965,14 @@ 
                 ),
                 $recipent_temp['email'],
                 get_lang('NewDropboxFileUploaded'),
-                get_lang('NewDropboxFileUploadedContent').' <a href="'.api_get_path(WEB_CODE_PATH).'dropbox/index.php?' . api_get_cidreq() . '">'.get_lang('SeeFile').'</a>'.
+                get_lang('NewDropboxFileUploadedContent').' <a href="'.api_get_path(WEB_CODE_PATH).'dropbox/index.php?'.api_get_cidreq().'">'.get_lang('SeeFile').'</a>'.
                 "\n\n".
                 api_get_person_name(
                     $_user['firstName'],
                     $_user['lastName'],
                     null,
                     PERSON_NAME_EMAIL_ADDRESS
-                )."\n".  get_lang('Email') ." : ".$_user['mail'],
+                )."\n".get_lang('Email')." : ".$_user['mail'],
                 api_get_person_name(
                     $_user['firstName'],
                     $_user['lastName'],
@@ -1068,12 +1068,12 @@ 
     $id = intval($id);
     $user_id = intval($user_id);
 
-    $sql = "SELECT file_id FROM ". Database::get_course_table(TABLE_DROPBOX_PERSON) ."
+    $sql = "SELECT file_id FROM ".Database::get_course_table(TABLE_DROPBOX_PERSON)."
             WHERE c_id = $course_id AND user_id = $user_id AND file_id = ".$id;
     $result = Database::query($sql);
     $number_users_who_see_file = Database::num_rows($result);
 
-    $sql = "SELECT file_id FROM ". Database::get_course_table(TABLE_DROPBOX_POST) ."
+    $sql = "SELECT file_id FROM ".Database::get_course_table(TABLE_DROPBOX_POST)."
             WHERE c_id = $course_id AND dest_user_id = $user_id AND file_id = ".$id;
     $result = Database::query($sql);
     $count = Database::num_rows($result);
@@ -1087,12 +1087,12 @@ 
 {
     $id = intval($id);
     $course_id = api_get_course_int_id();
-    $sql = "SELECT file_id FROM ". Database::get_course_table(TABLE_DROPBOX_PERSON) ."
+    $sql = "SELECT file_id FROM ".Database::get_course_table(TABLE_DROPBOX_PERSON)."
             WHERE c_id = $course_id AND file_id = ".$id;
     $result = Database::query($sql);
     $number_users_who_see_file = Database::num_rows($result);
 
-    $sql = "SELECT file_id FROM ". Database::get_course_table(TABLE_DROPBOX_POST) ."
+    $sql = "SELECT file_id FROM ".Database::get_course_table(TABLE_DROPBOX_POST)."
             WHERE c_id = $course_id AND file_id = ".$id;
     $result = Database::query($sql);
     $count = Database::num_rows($result);
@@ -1124,7 +1124,7 @@ 
 
         $id = Database::insert(Database::get_course_table(TABLE_DROPBOX_FEEDBACK), $params);
         if ($id) {
-            $sql = "UPDATE ". Database::get_course_table(TABLE_DROPBOX_FEEDBACK) ." SET feedback_id = iid WHERE iid = $id";
+            $sql = "UPDATE ".Database::get_course_table(TABLE_DROPBOX_FEEDBACK)." SET feedback_id = iid WHERE iid = $id";
             Database::query($sql);
         }
 
@@ -1151,10 +1151,10 @@ 
 
     // note: we also have to add the check if the user has received or sent this file.
     $sql = "SELECT DISTINCT file.filename, file.title, file.author, file.description
-            FROM ". Database::get_course_table(TABLE_DROPBOX_FILE) ." file
-            INNER JOIN ". Database::get_course_table(TABLE_DROPBOX_PERSON) ." person
+            FROM ". Database::get_course_table(TABLE_DROPBOX_FILE)." file
+            INNER JOIN ". Database::get_course_table(TABLE_DROPBOX_PERSON)." person
             ON (person.file_id=file.id AND file.c_id = $course_id AND person.c_id = $course_id)
-            INNER JOIN ". Database::get_course_table(TABLE_DROPBOX_POST) ." post
+            INNER JOIN ".Database::get_course_table(TABLE_DROPBOX_POST)." post
             ON (post.file_id = file.id AND post.c_id = $course_id AND file.c_id = $course_id)
             WHERE
                 file.id IN (".implode(', ', $fileList).") AND
@@ -1248,7 +1248,7 @@ 
         // Adding the content.
         $return .= "\n<tr>";
         foreach ($column as $column_key => $column_value) {
-            if (!in_array($column_value,$dont_show_columns)) {
+            if (!in_array($column_value, $dont_show_columns)) {
                 $return .= "\n\t<td>";
                 if (in_array($column_value, $make_link)) {
                     $return .= '<a href="'.$value[$column_value].'">'.$value[$column_value].'</a>';
@@ -1276,11 +1276,11 @@ 
 {
     $course_id = api_get_course_int_id();
     $sql = "SELECT COUNT(feedback_id) AS total, file_id
-            FROM ". Database::get_course_table(TABLE_DROPBOX_FEEDBACK) ."
+            FROM ". Database::get_course_table(TABLE_DROPBOX_FEEDBACK)."
             WHERE c_id = $course_id GROUP BY file_id";
     $result = Database::query($sql);
     $return = array();
-    while ($row=Database::fetch_array($result)) {
+    while ($row = Database::fetch_array($result)) {
         $return[$row['file_id']] = $row['total'];
     }
     return $return;