chamilo / chamilo-lms

main/plagiarism/compilatio/compilatio.class.php

Doc Comments +4 added lines, -4 removed lines

@@ -226,7 +226,7 @@ 
 	 * @param $pour
 	 * @param $imagesPath
 	 * @param $text  Array includes the extract from the text
-	 * @return unknown_type
+	 * @return string|null
 	 */
 	function getProgressionAnalyseDocv31($status, $pour = 0, $imagesPath = '', $text = '')
 	{
@@ -281,11 +281,11 @@ 
 	 * Method for display the PomprseuilmankBar (% de plagiat)
 	 *
 	 * @param $percentagePumping
-	 * @param $weakThreshold
-	 * @param $highThreshold
+	 * @param integer $weakThreshold
+	 * @param integer $highThreshold
 	 * @param $imagesPath
 	 * @param $text : array  includes the extract from the text
-	 * @return unknown_type
+	 * @return string
 	 */
 	function getPomprankBarv31($pourcentagePompage, $weakThreshold, $highThreshold, $chemin_images='',$texte='') {
 

Indentation +300 added lines, -300 removed lines

@@ -1,19 +1,19 @@ 
 <?php
-	/**
-	 * Description:
-	 * build the comunication with the SOAP server Compilatio.net
-	 * call severals methods for the file management in Compilatio.net
-	 *
-	 * Date: 26/05/16
-	 * @version:1.0
-	 */
+    /**
+     * Description:
+     * build the comunication with the SOAP server Compilatio.net
+     * call severals methods for the file management in Compilatio.net
+     *
+     * Date: 26/05/16
+     * @version:1.0
+     */
 
-	class compilatio
-	{
-		/*Identification key for the Compilatio account*/
-		var $key  = null;
-		/*Webservice connection*/
-		var $soapcli;
+    class compilatio
+    {
+        /*Identification key for the Compilatio account*/
+        var $key  = null;
+        /*Webservice connection*/
+        var $soapcli;
 
         /**
          * compilatio constructor.
@@ -22,338 +22,338 @@ 
          * @param $proxyHost
          * @param $proxyPort
          */
-		function compilatio($key, $urlsoap, $proxyHost, $proxyPort)
-		{
-			try
-			{
-				if(!empty($key)){
-					$this->key = $key;
-					if(!empty($urlsoap)){
-						if(!empty($proxyHost)) {
-							$param = array('trace' => false,
-								'soap_version' => SOAP_1_2,
-								'exceptions' => true,
-								'proxy_host' => '"' . $proxyHost . '"',
-								'proxy_port' => $proxyPort);
-						} else {
-							$param = array('trace' => false,
-								'soap_version' => SOAP_1_2,
-								'exceptions' => true);
-						}
-						$this->soapcli = new SoapClient($urlsoap,$param);
-					}else{
-						$this->soapcli = 'WS urlsoap not available' ;
-					}
-				}else{
-					$this->soapcli ='API key not available';
-				}
-			}
-			catch (SoapFault $fault)
-			{
-				$this->soapcli = "Error constructor compilatio " . $fault->faultcode ." " .$fault->faultstring ;
-			}
-			catch (Exception $e) {
-				$this->soapcli = "Error constructor compilatio with urlsoap" . $urlsoap;
-			}
-		}
+        function compilatio($key, $urlsoap, $proxyHost, $proxyPort)
+        {
+            try
+            {
+                if(!empty($key)){
+                    $this->key = $key;
+                    if(!empty($urlsoap)){
+                        if(!empty($proxyHost)) {
+                            $param = array('trace' => false,
+                                'soap_version' => SOAP_1_2,
+                                'exceptions' => true,
+                                'proxy_host' => '"' . $proxyHost . '"',
+                                'proxy_port' => $proxyPort);
+                        } else {
+                            $param = array('trace' => false,
+                                'soap_version' => SOAP_1_2,
+                                'exceptions' => true);
+                        }
+                        $this->soapcli = new SoapClient($urlsoap,$param);
+                    }else{
+                        $this->soapcli = 'WS urlsoap not available' ;
+                    }
+                }else{
+                    $this->soapcli ='API key not available';
+                }
+            }
+            catch (SoapFault $fault)
+            {
+                $this->soapcli = "Error constructor compilatio " . $fault->faultcode ." " .$fault->faultstring ;
+            }
+            catch (Exception $e) {
+                $this->soapcli = "Error constructor compilatio with urlsoap" . $urlsoap;
+            }
+        }
 
 
-		/**
-		 * Method for the file load
-		 *
-		 * @param $title
-		 * @param $description
-		 * @param $filename
+        /**
+         * Method for the file load
+         *
+         * @param $title
+         * @param $description
+         * @param $filename
          * @param $mimeType
-		 * @param $content
-		 * @return string
-		 */
+         * @param $content
+         * @return string
+         */
         function sendDoc($title, $description, $filename, $mimeType, $content)
-		{
-			try
-			{
-				if (!is_object($this->soapcli))
-					return("Error in constructor compilatio() " . $this->soapcli);
-				$idDocument = $this->soapcli->__call('addDocumentBase64',
-					array(
-						$this->key,
-						utf8_encode(urlencode($title)),
-						utf8_encode(urlencode($description)),
-						utf8_encode(urlencode($filename)),
+        {
+            try
+            {
+                if (!is_object($this->soapcli))
+                    return("Error in constructor compilatio() " . $this->soapcli);
+                $idDocument = $this->soapcli->__call('addDocumentBase64',
+                    array(
+                        $this->key,
+                        utf8_encode(urlencode($title)),
+                        utf8_encode(urlencode($description)),
+                        utf8_encode(urlencode($filename)),
                         utf8_encode($mimeType),
-						base64_encode($content)
-					)
-				);
-				return $idDocument;
-			}
-			catch (SoapFault $fault)
-			{
+                        base64_encode($content)
+                    )
+                );
+                return $idDocument;
+            }
+            catch (SoapFault $fault)
+            {
                 return("Erreur sendDoc()" . $fault->faultcode ." " .$fault->faultstring);
-			}
-		}
+            }
+        }
 
 
-		/**
-		 * Method for recover a document's information
-		 * @param $compiHash
-		 * @return string
-		 */
+        /**
+         * Method for recover a document's information
+         * @param $compiHash
+         * @return string
+         */
         function getDoc($compiHash)
-		{
-			try
-			{
-				if (!is_object($this->soapcli)){
-					return("Error in constructor compilatio() " . $this->soapcli);
-				}
-				$param = array($this->key, $compiHash);
-				$idDocument = $this->soapcli->__call('getDocument', $param);
-				return $idDocument;
-			}
-			catch (SoapFault $fault)
-			{
+        {
+            try
+            {
+                if (!is_object($this->soapcli)){
+                    return("Error in constructor compilatio() " . $this->soapcli);
+                }
+                $param = array($this->key, $compiHash);
+                $idDocument = $this->soapcli->__call('getDocument', $param);
+                return $idDocument;
+            }
+            catch (SoapFault $fault)
+            {
                 return("Erreur getDoc()" . $fault->faultcode . " " .$fault->faultstring);
-			}
-		}
+            }
+        }
 
-		/**
-		 * method for recover an url document's report
-		 * @param $compiHash
-		 * @return string
-		 */
+        /**
+         * method for recover an url document's report
+         * @param $compiHash
+         * @return string
+         */
         function getReportUrl($compiHash)
-		{
-			try
-			{
-				if (!is_object($this->soapcli)){
-					return("Error in constructor compilatio() " . $this->soapcli);
-				}
-				$param = array($this->key,$compiHash);
-				$idDocument = $this->soapcli->__call('getDocumentReportUrl', $param);
-				return $idDocument;
-			}
-			catch (SoapFault $fault)
-			{
+        {
+            try
+            {
+                if (!is_object($this->soapcli)){
+                    return("Error in constructor compilatio() " . $this->soapcli);
+                }
+                $param = array($this->key,$compiHash);
+                $idDocument = $this->soapcli->__call('getDocumentReportUrl', $param);
+                return $idDocument;
+            }
+            catch (SoapFault $fault)
+            {
                 return("Erreur  getReportUrl()" . $fault->faultcode ." " .$fault->faultstring);
-			}
-		}
+            }
+        }
 
-		/**
-		 *  Method for deleting a Compialtio's account document
-		 * @param $compiHash
-		 * @return string
-		 */
+        /**
+         *  Method for deleting a Compialtio's account document
+         * @param $compiHash
+         * @return string
+         */
         function deldoc($compiHash)
-		{
-			try
-			{
-				if (!is_object($this->soapcli)){
-					return("Error in constructor compilatio() " . $this->soapcli);
-				}
-				$param = array($this->key,$compiHash);
-				$this->soapcli->__call('deleteDocument', $param);
-			}
-			catch (SoapFault $fault)
-			{
+        {
+            try
+            {
+                if (!is_object($this->soapcli)){
+                    return("Error in constructor compilatio() " . $this->soapcli);
+                }
+                $param = array($this->key,$compiHash);
+                $this->soapcli->__call('deleteDocument', $param);
+            }
+            catch (SoapFault $fault)
+            {
                 return("Erreur  deldoc()" . $fault->faultcode . " " .$fault->faultstring);
-			}
-		}
+            }
+        }
 
-		/**
-		 * Method for start the analysis for a document
-		 * @param $compiHash
-		 * @return string
-		 */
+        /**
+         * Method for start the analysis for a document
+         * @param $compiHash
+         * @return string
+         */
         function startAnalyse($compiHash)
-		{
-			try
-			{
-				if (!is_object($this->soapcli)){
-					return("Error in constructor compilatio() " . $this->soapcli);
-				}
-				$param = array($this->key,$compiHash);
+        {
+            try
+            {
+                if (!is_object($this->soapcli)){
+                    return("Error in constructor compilatio() " . $this->soapcli);
+                }
+                $param = array($this->key,$compiHash);
                 $this->soapcli->__call('startDocumentAnalyse', $param);
-			}
-			catch (SoapFault $fault)
-			{
+            }
+            catch (SoapFault $fault)
+            {
                 return("Erreur  startAnalyse()" . $fault->faultcode ." " .$fault->faultstring);
-			}
-		}
+            }
+        }
 
-		/**
-		 * Method for recover the account's quota
-		 * @return string
-		 */
+        /**
+         * Method for recover the account's quota
+         * @return string
+         */
         function getQuotas()
-		{
-			try
-			{
-				if (!is_object($this->soapcli)){
-					return("Error in constructor compilatio() " . $this->soapcli);
-				}
-				$param=array($this->key);
+        {
+            try
+            {
+                if (!is_object($this->soapcli)){
+                    return("Error in constructor compilatio() " . $this->soapcli);
+                }
+                $param=array($this->key);
                 $resultat=$this->soapcli->__call('getAccountQuotas', $param);
-				return $resultat;
-			}
-			catch (SoapFault $fault)
-			{
+                return $resultat;
+            }
+            catch (SoapFault $fault)
+            {
                 return("Erreur  getQuotas()" . $fault->faultcode ." " .$fault->faultstring);
-			}
-		}
+            }
+        }
 
-	}
+    }
 
-	/**
-	 * Method for identify a file extension and the possibility that the document can be managed by Compilatio
-	 * @param $filename
-	 * @return bool
-	 */
+    /**
+     * Method for identify a file extension and the possibility that the document can be managed by Compilatio
+     * @param $filename
+     * @return bool
+     */
     function verifiFileType($filename)
-	{
-		$types = array("doc","docx", "rtf", "xls", "xlsx", "ppt", "pptx", "odt", "pdf", "txt", "htm", "html");
-		$extension = substr($filename, strrpos($filename, ".")+1);
-		$extension = strtolower($extension);
-		if (in_array($extension, $types)) {
-			return true;
-		} else {
-			return false;
-		}
-	}
+    {
+        $types = array("doc","docx", "rtf", "xls", "xlsx", "ppt", "pptx", "odt", "pdf", "txt", "htm", "html");
+        $extension = substr($filename, strrpos($filename, ".")+1);
+        $extension = strtolower($extension);
+        if (in_array($extension, $types)) {
+            return true;
+        } else {
+            return false;
+        }
+    }
 
-	/**
-	 * Fonction  affichage de la barre de progression d'analyse version 3.1
-	 *
-	 * @param $statut From the document
-	 * @param $pour
-	 * @param $imagesPath
-	 * @param $text  Array includes the extract from the text
-	 * @return unknown_type
-	 */
-	function getProgressionAnalyseDocv31($status, $pour = 0, $imagesPath = '', $text = '')
-	{
+    /**
+     * Fonction  affichage de la barre de progression d'analyse version 3.1
+     *
+     * @param $statut From the document
+     * @param $pour
+     * @param $imagesPath
+     * @param $text  Array includes the extract from the text
+     * @return unknown_type
+     */
+    function getProgressionAnalyseDocv31($status, $pour = 0, $imagesPath = '', $text = '')
+    {
 
-		$refreshReturn = "<a href='javascript:window.location.reload(false);'><img src='"
-			.$imagesPath
-			."ajax-loader_green.gif' title='"
-			. $text['refresh']
-			. "' alt='"
-			. $text['refresh']
-			. "'/></a> ";
+        $refreshReturn = "<a href='javascript:window.location.reload(false);'><img src='"
+            .$imagesPath
+            ."ajax-loader_green.gif' title='"
+            . $text['refresh']
+            . "' alt='"
+            . $text['refresh']
+            . "'/></a> ";
         $startReturn = "<table cellpadding='0' cellspacing='0' style='border:0;margin:0;padding:0;'><tr>";
         $startReturn .= "<td width='25' style='border:0;margin:0;padding:0;'>&nbsp;</td>";
         $startReturn .= "<td valign='middle' align='right' style='border:0;margin:0;padding-right:10px;'>"
-			.$refreshReturn
-			."</td>";
+            .$refreshReturn
+            ."</td>";
         $startReturn .= "<td style='border:0;margin:0;padding:0;'>";
         $startReturnLittleWidth = "<table cellpadding='0' cellspacing='0' style='border:0;margin:0;padding:0;'><tr>";
         $startReturnLittleWidth .= "<td width='25' valign='middle' align='right' style='border:0;margin:0;padding:0;'>"
-			.$refreshReturn
-			."</td>";
-		$finretour = "</td></tr></table>";
-		if($status == "ANALYSE_IN_QUEUE" ){
-			return $startReturn . "<span style='font-size:11px'>" . $text['analysisinqueue'] . "</span>" . $finretour;
-		}
-		if($status == "ANALYSE_PROCESSING" ){
-			if($pour == 100){
-				return $startReturn
-				. "<span style='font-size:11px'>"
-				. $text['analysisinfinalization']
-				. "</span>"
-				. $finretour;
-			} else {
-				return $startReturnLittleWidth
+            .$refreshReturn
+            ."</td>";
+        $finretour = "</td></tr></table>";
+        if($status == "ANALYSE_IN_QUEUE" ){
+            return $startReturn . "<span style='font-size:11px'>" . $text['analysisinqueue'] . "</span>" . $finretour;
+        }
+        if($status == "ANALYSE_PROCESSING" ){
+            if($pour == 100){
+                return $startReturn
+                . "<span style='font-size:11px'>"
+                . $text['analysisinfinalization']
+                . "</span>"
+                . $finretour;
+            } else {
+                return $startReturnLittleWidth
                     . "<td align=\"right\" style=\"border:0;margin:0;padding-right:10px;\">"
-				. $pour
+                . $pour
                     . "%</td><td style=\"border:0;margin:0;padding:0;\"><div style=\"background"
-				. ":transparent url("
-				. $imagesPath
-				. "mini-jauge_fond.png) no-repeat scroll 0;height:12px;padding:0 0 0 2px;width:55px;\"><div style=\""
-				. "background:transparent url("
-				. $imagesPath
-				. "mini-jauge_gris.png) no-repeat scroll 0;height:12px;width:"
-				. $pour/2
-				. "px;\"></div></div>"
-				. $finretour;
-			}
-		}
-	}
+                . ":transparent url("
+                . $imagesPath
+                . "mini-jauge_fond.png) no-repeat scroll 0;height:12px;padding:0 0 0 2px;width:55px;\"><div style=\""
+                . "background:transparent url("
+                . $imagesPath
+                . "mini-jauge_gris.png) no-repeat scroll 0;height:12px;width:"
+                . $pour/2
+                . "px;\"></div></div>"
+                . $finretour;
+            }
+        }
+    }
 
-	/**
-	 * Method for display the PomprseuilmankBar (% de plagiat)
-	 *
-	 * @param $percentagePumping
-	 * @param $weakThreshold
-	 * @param $highThreshold
-	 * @param $imagesPath
-	 * @param $text : array  includes the extract from the text
-	 * @return unknown_type
-	 */
-	function getPomprankBarv31($pourcentagePompage, $weakThreshold, $highThreshold, $chemin_images='',$texte='') {
+    /**
+     * Method for display the PomprseuilmankBar (% de plagiat)
+     *
+     * @param $percentagePumping
+     * @param $weakThreshold
+     * @param $highThreshold
+     * @param $imagesPath
+     * @param $text : array  includes the extract from the text
+     * @return unknown_type
+     */
+    function getPomprankBarv31($pourcentagePompage, $weakThreshold, $highThreshold, $chemin_images='',$texte='') {
 
-		$pourcentagePompage = round($pourcentagePompage);
-		$pour = round((50*$pourcentagePompage)/100);
-		$return = "";
-		$couleur = "";
-		if($pourcentagePompage < $weakThreshold) {
-			$couleur = "vert";
-		} else if($pourcentagePompage >= $weakThreshold && $pourcentagePompage < $highThreshold) {
-			$couleur = "orange";
-		} else {
-			$couleur = "rouge";
-		}
-		$return .= "<div style='float:left;margin-right:2px;'><img src='"
-			. $chemin_images."mini-drapeau_$couleur.png' title='"
-			. $texte['result']
-			. "' alt='faible' width='15' height='15' /></div>";
-		$return .= "<div style='float:left; margin-right:5px;width:45px;text-align:right;'>"
-			. $pourcentagePompage
-			. " %</div>";
-		$return .= "<div style='float:left;background:transparent url("
-			. $chemin_images
-			. "mini-jauge_fond.png) no-repeat scroll 0;height:12px;margin-top:5px;padding:0 0 0 2px;width:55px;'>";
-		$return .= "<div style='float:left;background:transparent url("
-			. $chemin_images
-			. "mini-jauge_"
-			. $couleur
-			. ".png) no-repeat scroll 0;height:12px;width:"
-			. $pour
-			. "px'></div></div>";
+        $pourcentagePompage = round($pourcentagePompage);
+        $pour = round((50*$pourcentagePompage)/100);
+        $return = "";
+        $couleur = "";
+        if($pourcentagePompage < $weakThreshold) {
+            $couleur = "vert";
+        } else if($pourcentagePompage >= $weakThreshold && $pourcentagePompage < $highThreshold) {
+            $couleur = "orange";
+        } else {
+            $couleur = "rouge";
+        }
+        $return .= "<div style='float:left;margin-right:2px;'><img src='"
+            . $chemin_images."mini-drapeau_$couleur.png' title='"
+            . $texte['result']
+            . "' alt='faible' width='15' height='15' /></div>";
+        $return .= "<div style='float:left; margin-right:5px;width:45px;text-align:right;'>"
+            . $pourcentagePompage
+            . " %</div>";
+        $return .= "<div style='float:left;background:transparent url("
+            . $chemin_images
+            . "mini-jauge_fond.png) no-repeat scroll 0;height:12px;margin-top:5px;padding:0 0 0 2px;width:55px;'>";
+        $return .= "<div style='float:left;background:transparent url("
+            . $chemin_images
+            . "mini-jauge_"
+            . $couleur
+            . ".png) no-repeat scroll 0;height:12px;width:"
+            . $pour
+            . "px'></div></div>";
 
-		return $return;
-	}
-	/*
+        return $return;
+    }
+    /*
      * Method for validation of hash
      * @param $hash
      * @return bool
      *
      */
-	function isMd5($hash)
-	{
-		if(preg_match('`^[a-f0-9]{32}$`', $hash)) {
-			return true;
-		} else {
-			return false;
-		}
-	}
-	/*
+    function isMd5($hash)
+    {
+        if(preg_match('`^[a-f0-9]{32}$`', $hash)) {
+            return true;
+        } else {
+            return false;
+        }
+    }
+    /*
      * Method for identify Internet media type
      * @param $filename
      */
-	function typeMime($filename)
-	{
-		if (preg_match("@Opera(/| )([0-9].[0-9]{1,2})@", $_SERVER['HTTP_USER_AGENT'], $resultats)){
-			$navigateur = "Opera";
-		} elseif (preg_match("@MSIE ([0-9].[0-9]{1,2})@", $_SERVER['HTTP_USER_AGENT'], $resultats)){
-			$navigateur = "Internet Explorer";
-		} else {
-			$navigateur = "Mozilla";
-			$mime = parse_ini_file("mime.ini");
-			$extension = substr($filename, strrpos($filename, ".")+1);
-		}
-		if(array_key_exists($extension, $mime)) {
-			$type = $mime[$extension];
-		} else {
-			$type = ($navigateur!="Mozilla") ? 'application/octetstream' : 'application/octet-stream';
-		}
-		return $type;
-	}
+    function typeMime($filename)
+    {
+        if (preg_match("@Opera(/| )([0-9].[0-9]{1,2})@", $_SERVER['HTTP_USER_AGENT'], $resultats)){
+            $navigateur = "Opera";
+        } elseif (preg_match("@MSIE ([0-9].[0-9]{1,2})@", $_SERVER['HTTP_USER_AGENT'], $resultats)){
+            $navigateur = "Internet Explorer";
+        } else {
+            $navigateur = "Mozilla";
+            $mime = parse_ini_file("mime.ini");
+            $extension = substr($filename, strrpos($filename, ".")+1);
+        }
+        if(array_key_exists($extension, $mime)) {
+            $type = $mime[$extension];
+        } else {
+            $type = ($navigateur!="Mozilla") ? 'application/octetstream' : 'application/octet-stream';
+        }
+        return $type;
+    }
 ?>
\ No newline at end of file

Spacing +51 added lines, -51 removed lines

@@ -11,7 +11,7 @@ 
 	class compilatio
 	{
 		/*Identification key for the Compilatio account*/
-		var $key  = null;
+		var $key = null;
 		/*Webservice connection*/
 		var $soapcli;
 
@@ -26,34 +26,34 @@ 
 		{
 			try
 			{
-				if(!empty($key)){
+				if (!empty($key)) {
 					$this->key = $key;
-					if(!empty($urlsoap)){
-						if(!empty($proxyHost)) {
+					if (!empty($urlsoap)) {
+						if (!empty($proxyHost)) {
 							$param = array('trace' => false,
 								'soap_version' => SOAP_1_2,
 								'exceptions' => true,
-								'proxy_host' => '"' . $proxyHost . '"',
+								'proxy_host' => '"'.$proxyHost.'"',
 								'proxy_port' => $proxyPort);
 						} else {
 							$param = array('trace' => false,
 								'soap_version' => SOAP_1_2,
 								'exceptions' => true);
 						}
-						$this->soapcli = new SoapClient($urlsoap,$param);
-					}else{
-						$this->soapcli = 'WS urlsoap not available' ;
+						$this->soapcli = new SoapClient($urlsoap, $param);
+					} else {
+						$this->soapcli = 'WS urlsoap not available';
 					}
-				}else{
-					$this->soapcli ='API key not available';
+				} else {
+					$this->soapcli = 'API key not available';
 				}
 			}
 			catch (SoapFault $fault)
 			{
-				$this->soapcli = "Error constructor compilatio " . $fault->faultcode ." " .$fault->faultstring ;
+				$this->soapcli = "Error constructor compilatio ".$fault->faultcode." ".$fault->faultstring;
 			}
 			catch (Exception $e) {
-				$this->soapcli = "Error constructor compilatio with urlsoap" . $urlsoap;
+				$this->soapcli = "Error constructor compilatio with urlsoap".$urlsoap;
 			}
 		}
 
@@ -73,7 +73,7 @@ 
 			try
 			{
 				if (!is_object($this->soapcli))
-					return("Error in constructor compilatio() " . $this->soapcli);
+					return("Error in constructor compilatio() ".$this->soapcli);
 				$idDocument = $this->soapcli->__call('addDocumentBase64',
 					array(
 						$this->key,
@@ -88,7 +88,7 @@ 
 			}
 			catch (SoapFault $fault)
 			{
-                return("Erreur sendDoc()" . $fault->faultcode ." " .$fault->faultstring);
+                return("Erreur sendDoc()".$fault->faultcode." ".$fault->faultstring);
 			}
 		}
 
@@ -102,8 +102,8 @@ 
 		{
 			try
 			{
-				if (!is_object($this->soapcli)){
-					return("Error in constructor compilatio() " . $this->soapcli);
+				if (!is_object($this->soapcli)) {
+					return("Error in constructor compilatio() ".$this->soapcli);
 				}
 				$param = array($this->key, $compiHash);
 				$idDocument = $this->soapcli->__call('getDocument', $param);
@@ -111,7 +111,7 @@ 
 			}
 			catch (SoapFault $fault)
 			{
-                return("Erreur getDoc()" . $fault->faultcode . " " .$fault->faultstring);
+                return("Erreur getDoc()".$fault->faultcode." ".$fault->faultstring);
 			}
 		}
 
@@ -124,16 +124,16 @@ 
 		{
 			try
 			{
-				if (!is_object($this->soapcli)){
-					return("Error in constructor compilatio() " . $this->soapcli);
+				if (!is_object($this->soapcli)) {
+					return("Error in constructor compilatio() ".$this->soapcli);
 				}
-				$param = array($this->key,$compiHash);
+				$param = array($this->key, $compiHash);
 				$idDocument = $this->soapcli->__call('getDocumentReportUrl', $param);
 				return $idDocument;
 			}
 			catch (SoapFault $fault)
 			{
-                return("Erreur  getReportUrl()" . $fault->faultcode ." " .$fault->faultstring);
+                return("Erreur  getReportUrl()".$fault->faultcode." ".$fault->faultstring);
 			}
 		}
 
@@ -146,15 +146,15 @@ 
 		{
 			try
 			{
-				if (!is_object($this->soapcli)){
-					return("Error in constructor compilatio() " . $this->soapcli);
+				if (!is_object($this->soapcli)) {
+					return("Error in constructor compilatio() ".$this->soapcli);
 				}
-				$param = array($this->key,$compiHash);
+				$param = array($this->key, $compiHash);
 				$this->soapcli->__call('deleteDocument', $param);
 			}
 			catch (SoapFault $fault)
 			{
-                return("Erreur  deldoc()" . $fault->faultcode . " " .$fault->faultstring);
+                return("Erreur  deldoc()".$fault->faultcode." ".$fault->faultstring);
 			}
 		}
 
@@ -167,15 +167,15 @@ 
 		{
 			try
 			{
-				if (!is_object($this->soapcli)){
-					return("Error in constructor compilatio() " . $this->soapcli);
+				if (!is_object($this->soapcli)) {
+					return("Error in constructor compilatio() ".$this->soapcli);
 				}
-				$param = array($this->key,$compiHash);
+				$param = array($this->key, $compiHash);
                 $this->soapcli->__call('startDocumentAnalyse', $param);
 			}
 			catch (SoapFault $fault)
 			{
-                return("Erreur  startAnalyse()" . $fault->faultcode ." " .$fault->faultstring);
+                return("Erreur  startAnalyse()".$fault->faultcode." ".$fault->faultstring);
 			}
 		}
 
@@ -187,16 +187,16 @@ 
 		{
 			try
 			{
-				if (!is_object($this->soapcli)){
-					return("Error in constructor compilatio() " . $this->soapcli);
+				if (!is_object($this->soapcli)) {
+					return("Error in constructor compilatio() ".$this->soapcli);
 				}
-				$param=array($this->key);
-                $resultat=$this->soapcli->__call('getAccountQuotas', $param);
+				$param = array($this->key);
+                $resultat = $this->soapcli->__call('getAccountQuotas', $param);
 				return $resultat;
 			}
 			catch (SoapFault $fault)
 			{
-                return("Erreur  getQuotas()" . $fault->faultcode ." " .$fault->faultstring);
+                return("Erreur  getQuotas()".$fault->faultcode." ".$fault->faultstring);
 			}
 		}
 
@@ -209,8 +209,8 @@ 
 	 */
     function verifiFileType($filename)
 	{
-		$types = array("doc","docx", "rtf", "xls", "xlsx", "ppt", "pptx", "odt", "pdf", "txt", "htm", "html");
-		$extension = substr($filename, strrpos($filename, ".")+1);
+		$types = array("doc", "docx", "rtf", "xls", "xlsx", "ppt", "pptx", "odt", "pdf", "txt", "htm", "html");
+		$extension = substr($filename, strrpos($filename, ".") + 1);
 		$extension = strtolower($extension);
 		if (in_array($extension, $types)) {
 			return true;
@@ -249,11 +249,11 @@ 
 			.$refreshReturn
 			."</td>";
 		$finretour = "</td></tr></table>";
-		if($status == "ANALYSE_IN_QUEUE" ){
-			return $startReturn . "<span style='font-size:11px'>" . $text['analysisinqueue'] . "</span>" . $finretour;
+		if ($status == "ANALYSE_IN_QUEUE") {
+			return $startReturn."<span style='font-size:11px'>".$text['analysisinqueue']."</span>".$finretour;
 		}
-		if($status == "ANALYSE_PROCESSING" ){
-			if($pour == 100){
+		if ($status == "ANALYSE_PROCESSING") {
+			if ($pour == 100) {
 				return $startReturn
 				. "<span style='font-size:11px'>"
 				. $text['analysisinfinalization']
@@ -270,7 +270,7 @@ 
 				. "background:transparent url("
 				. $imagesPath
 				. "mini-jauge_gris.png) no-repeat scroll 0;height:12px;width:"
-				. $pour/2
+				. $pour / 2
 				. "px;\"></div></div>"
 				. $finretour;
 			}
@@ -287,15 +287,15 @@ 
 	 * @param $text : array  includes the extract from the text
 	 * @return unknown_type
 	 */
-	function getPomprankBarv31($pourcentagePompage, $weakThreshold, $highThreshold, $chemin_images='',$texte='') {
+	function getPomprankBarv31($pourcentagePompage, $weakThreshold, $highThreshold, $chemin_images = '', $texte = '') {
 
 		$pourcentagePompage = round($pourcentagePompage);
-		$pour = round((50*$pourcentagePompage)/100);
+		$pour = round((50 * $pourcentagePompage) / 100);
 		$return = "";
 		$couleur = "";
-		if($pourcentagePompage < $weakThreshold) {
+		if ($pourcentagePompage < $weakThreshold) {
 			$couleur = "vert";
-		} else if($pourcentagePompage >= $weakThreshold && $pourcentagePompage < $highThreshold) {
+		} else if ($pourcentagePompage >= $weakThreshold && $pourcentagePompage < $highThreshold) {
 			$couleur = "orange";
 		} else {
 			$couleur = "rouge";
@@ -328,7 +328,7 @@ 
      */
 	function isMd5($hash)
 	{
-		if(preg_match('`^[a-f0-9]{32}$`', $hash)) {
+		if (preg_match('`^[a-f0-9]{32}$`', $hash)) {
 			return true;
 		} else {
 			return false;
@@ -340,19 +340,19 @@ 
      */
 	function typeMime($filename)
 	{
-		if (preg_match("@Opera(/| )([0-9].[0-9]{1,2})@", $_SERVER['HTTP_USER_AGENT'], $resultats)){
+		if (preg_match("@Opera(/| )([0-9].[0-9]{1,2})@", $_SERVER['HTTP_USER_AGENT'], $resultats)) {
 			$navigateur = "Opera";
-		} elseif (preg_match("@MSIE ([0-9].[0-9]{1,2})@", $_SERVER['HTTP_USER_AGENT'], $resultats)){
+		} elseif (preg_match("@MSIE ([0-9].[0-9]{1,2})@", $_SERVER['HTTP_USER_AGENT'], $resultats)) {
 			$navigateur = "Internet Explorer";
 		} else {
 			$navigateur = "Mozilla";
 			$mime = parse_ini_file("mime.ini");
-			$extension = substr($filename, strrpos($filename, ".")+1);
+			$extension = substr($filename, strrpos($filename, ".") + 1);
 		}
-		if(array_key_exists($extension, $mime)) {
+		if (array_key_exists($extension, $mime)) {
 			$type = $mime[$extension];
 		} else {
-			$type = ($navigateur!="Mozilla") ? 'application/octetstream' : 'application/octet-stream';
+			$type = ($navigateur != "Mozilla") ? 'application/octetstream' : 'application/octet-stream';
 		}
 		return $type;
 	}

Braces +13 added lines, -20 removed lines

@@ -41,18 +41,16 @@ 
 								'exceptions' => true);
 						}
 						$this->soapcli = new SoapClient($urlsoap,$param);
-					}else{
+					} else{
 						$this->soapcli = 'WS urlsoap not available' ;
 					}
-				}else{
+				} else{
 					$this->soapcli ='API key not available';
 				}
-			}
-			catch (SoapFault $fault)
+			} catch (SoapFault $fault)
 			{
 				$this->soapcli = "Error constructor compilatio " . $fault->faultcode ." " .$fault->faultstring ;
-			}
-			catch (Exception $e) {
+			} catch (Exception $e) {
 				$this->soapcli = "Error constructor compilatio with urlsoap" . $urlsoap;
 			}
 		}
@@ -72,8 +70,9 @@ 
 		{
 			try
 			{
-				if (!is_object($this->soapcli))
-					return("Error in constructor compilatio() " . $this->soapcli);
+				if (!is_object($this->soapcli)) {
+									return("Error in constructor compilatio() " . $this->soapcli);
+				}
 				$idDocument = $this->soapcli->__call('addDocumentBase64',
 					array(
 						$this->key,
@@ -85,8 +84,7 @@ 
 					)
 				);
 				return $idDocument;
-			}
-			catch (SoapFault $fault)
+			} catch (SoapFault $fault)
 			{
                 return("Erreur sendDoc()" . $fault->faultcode ." " .$fault->faultstring);
 			}
@@ -108,8 +106,7 @@ 
 				$param = array($this->key, $compiHash);
 				$idDocument = $this->soapcli->__call('getDocument', $param);
 				return $idDocument;
-			}
-			catch (SoapFault $fault)
+			} catch (SoapFault $fault)
 			{
                 return("Erreur getDoc()" . $fault->faultcode . " " .$fault->faultstring);
 			}
@@ -130,8 +127,7 @@ 
 				$param = array($this->key,$compiHash);
 				$idDocument = $this->soapcli->__call('getDocumentReportUrl', $param);
 				return $idDocument;
-			}
-			catch (SoapFault $fault)
+			} catch (SoapFault $fault)
 			{
                 return("Erreur  getReportUrl()" . $fault->faultcode ." " .$fault->faultstring);
 			}
@@ -151,8 +147,7 @@ 
 				}
 				$param = array($this->key,$compiHash);
 				$this->soapcli->__call('deleteDocument', $param);
-			}
-			catch (SoapFault $fault)
+			} catch (SoapFault $fault)
 			{
                 return("Erreur  deldoc()" . $fault->faultcode . " " .$fault->faultstring);
 			}
@@ -172,8 +167,7 @@ 
 				}
 				$param = array($this->key,$compiHash);
                 $this->soapcli->__call('startDocumentAnalyse', $param);
-			}
-			catch (SoapFault $fault)
+			} catch (SoapFault $fault)
 			{
                 return("Erreur  startAnalyse()" . $fault->faultcode ." " .$fault->faultstring);
 			}
@@ -193,8 +187,7 @@ 
 				$param=array($this->key);
                 $resultat=$this->soapcli->__call('getAccountQuotas', $param);
 				return $resultat;
-			}
-			catch (SoapFault $fault)
+			} catch (SoapFault $fault)
 			{
                 return("Erreur  getQuotas()" . $fault->faultcode ." " .$fault->faultstring);
 			}

main/plagiarism/compilatio/upload.php

Doc Comments +6 added lines

@@ -278,6 +278,9 @@ 
 }
 
 
+/**
+ * @param string $txt
+ */
 function getFilename($txt) {
     $res = $txt;
     preg_match("|.*/([^/]+)|", $txt, $urlList);
@@ -287,6 +290,9 @@ 
     return $res;
 }
 
+/**
+ * @param string $txt
+ */
 function getWorkFolder($txt) {
     $res = "";
     preg_match("|(.*/)[^/]+|", $txt, $urlList);

Indentation +24 added lines, -24 removed lines

@@ -151,28 +151,28 @@ 
 } else {
     //non multiple
 
-	$documentId = $_GET['doc'];
-	compilatioUpdateWorkDocument($documentId, $coursId);
-	$workTable = Database::get_course_table(TABLE_STUDENT_PUBLICATION);
-	$query = "SELECT * FROM "
+    $documentId = $_GET['doc'];
+    compilatioUpdateWorkDocument($documentId, $coursId);
+    $workTable = Database::get_course_table(TABLE_STUDENT_PUBLICATION);
+    $query = "SELECT * FROM "
         . $workTable
         . " WHERE id='"
         . $documentId
         . "' AND c_id="
         . $coursId;
-	$sqlResult = Database::query($query);
-	$doc = Database::fetch_object($sqlResult);
-	$currentCourseRepositoryWeb = api_get_path(WEB_COURSE_PATH) . $_course["path"] . "/";
-	$WrkUrl = $currentCourseRepositoryWeb . $doc->url;
-	$WrkTitle = $doc->title;
-	$LocalWrkUrl = api_get_path(SYS_COURSE_PATH).$_course["path"] . "/" . $doc->url;
-	$mime = typeMime($WrkTitle);
-	$compilatio = new compilatio(
+    $sqlResult = Database::query($query);
+    $doc = Database::fetch_object($sqlResult);
+    $currentCourseRepositoryWeb = api_get_path(WEB_COURSE_PATH) . $_course["path"] . "/";
+    $WrkUrl = $currentCourseRepositoryWeb . $doc->url;
+    $WrkTitle = $doc->title;
+    $LocalWrkUrl = api_get_path(SYS_COURSE_PATH).$_course["path"] . "/" . $doc->url;
+    $mime = typeMime($WrkTitle);
+    $compilatio = new compilatio(
         $compilatioParameter['key'],
         $compilatioParameter['$urlsoap'],
         $compilatioParameter['proxy_host'],
         $compilatioParameter['proxy_port']);
-	if ($compilatioParameter['mode_transport'] == 'wget') {
+    if ($compilatioParameter['mode_transport'] == 'wget') {
 
         if (strlen($compilatioParameter['wget_uri']) > 2) {
             $filename = ereg_replace("/$",
@@ -195,14 +195,14 @@ 
         $mime = "text/plain";
         $compilatioId = $compilatio->sendDoc($doc->title, '', $filename, $mime, 'get_url');
     } else {
-		$pieces = explode("/", $doc->url);
-		$nbPieces = count($pieces);
-		$filename = $pieces[$nbPieces-1];
+        $pieces = explode("/", $doc->url);
+        $nbPieces = count($pieces);
+        $filename = $pieces[$nbPieces-1];
         $compilatioId = $compilatio->sendDoc($doc->title, '', $filename, $mime, file_get_contents($LocalWrkUrl));
     }
-	$compTable = Database::get_course_table("plagiarism_compilatio_docs");
+    $compTable = Database::get_course_table("plagiarism_compilatio_docs");
 
-	$sql4 = "INSERT INTO "
+    $sql4 = "INSERT INTO "
         . $compTable
         . "(c_id, id_doc, compilatio_id) VALUES ("
         . $coursId
@@ -211,10 +211,10 @@ 
         . "','"
         . $compilatioId
         . "')";
-	if (isMd5($compilatioId)) {
+    if (isMd5($compilatioId)) {
         Database::query($sql4);
-	} else {
-		$sql5 = "INSERT INTO "
+    } else {
+        $sql5 = "INSERT INTO "
             . $compTable
             . " (c_id, id_doc,compilatio_id) VALUES ("
             . $coursId
@@ -223,8 +223,8 @@ 
             ."','error')";
         Database::query($sql5);
 
-	}
-	if (isMd5($compilatioId)) {
+    }
+    if (isMd5($compilatioId)) {
         $soapRes = $compilatio->startAnalyse($compilatioId);
     } else {
         $typeMessage = 2;
@@ -239,7 +239,7 @@ 
     }else{
         $message = $msgWait;
         Display::display_confirmation_message($message);
-	}
+    }
 
 }
 

Spacing +20 added lines, -20 removed lines

@@ -15,16 +15,16 @@ 
 
 /* Variables */ 
 $typeMessage = 0;
-$errorCodeNotValid = get_lang('compilatioError') . get_lang('documentCodeNotValid');
-$errorLoadError = get_lang('compilatioLoadError') . get_lang('compilatioContactAdmin');
+$errorCodeNotValid = get_lang('compilatioError').get_lang('documentCodeNotValid');
+$errorLoadError = get_lang('compilatioLoadError').get_lang('compilatioContactAdmin');
 
 /*message to the user for be patient*/
 $msgWait = get_lang('PleaseWaitThisCouldTakeAWhile');
 
 /* if we have to upload severals documents*/
-if(isset($_REQUEST['type']) && $_REQUEST['type']=="multi"){
+if (isset($_REQUEST['type']) && $_REQUEST['type'] == "multi") {
     $docs = preg_split("/a/", $_REQUEST['doc']);
-    for ($k=0 ; $k < sizeof($docs)-1 ; $k++)	{
+    for ($k = 0; $k < sizeof($docs) - 1; $k++) {
         /* We have to modify the timeout server for send the heavy files */
         set_time_limit(600);
         $documentId = 0;
@@ -62,9 +62,9 @@ 
             $doc = Database::fetch_object($sqlResult);
             if ($doc) {
                 /*We load the document in compilatio through the webservice */
-                $currentCourseRepositoryWeb = api_get_path(WEB_COURSE_PATH) . $_course["path"] . "/";
-                $WrkUrl = $currentCourseRepositoryWeb . $doc->url;
-                $LocalWrkUrl = api_get_path(SYS_COURSE_PATH) . $_course["path"] . "/" . $doc->url;
+                $currentCourseRepositoryWeb = api_get_path(WEB_COURSE_PATH).$_course["path"]."/";
+                $WrkUrl = $currentCourseRepositoryWeb.$doc->url;
+                $LocalWrkUrl = api_get_path(SYS_COURSE_PATH).$_course["path"]."/".$doc->url;
                 $LocalWrkTitle = $doc->title;
                 $mime = typeMime($LocalWrkTitle);
                 $compilatio = new compilatio(
@@ -76,7 +76,7 @@ 
                 if ($compilatioParameter['mode_transport'] == 'wget') {
                     /*Compilatio's server recover tjre file throught wget like this:
                     username:password@http://somedomain.com/reg/remotefilename.tar.gz */
-                    if (strlen($compilatioParameter['wget_uri']) > 2)  {
+                    if (strlen($compilatioParameter['wget_uri']) > 2) {
                         $filename = ereg_replace("/$",
                                 "",
                                 $compilatioParameter['wget_uri'])
@@ -140,10 +140,10 @@ 
     if ($typeMessage == 1) {
         $message = $errorCodeNotValid;
         Display::display_error_message($message);
-    }elseif($typeMessage == 2){
+    }elseif ($typeMessage == 2) {
         $message = $errorLoadError;
         Display::display_error_message($message);
-    }else{
+    } else {
         $message = $msgWait;
         Display::display_confirmation_message($message);
     }
@@ -162,10 +162,10 @@ 
         . $coursId;
 	$sqlResult = Database::query($query);
 	$doc = Database::fetch_object($sqlResult);
-	$currentCourseRepositoryWeb = api_get_path(WEB_COURSE_PATH) . $_course["path"] . "/";
-	$WrkUrl = $currentCourseRepositoryWeb . $doc->url;
+	$currentCourseRepositoryWeb = api_get_path(WEB_COURSE_PATH).$_course["path"]."/";
+	$WrkUrl = $currentCourseRepositoryWeb.$doc->url;
 	$WrkTitle = $doc->title;
-	$LocalWrkUrl = api_get_path(SYS_COURSE_PATH).$_course["path"] . "/" . $doc->url;
+	$LocalWrkUrl = api_get_path(SYS_COURSE_PATH).$_course["path"]."/".$doc->url;
 	$mime = typeMime($WrkTitle);
 	$compilatio = new compilatio(
         $compilatioParameter['key'],
@@ -197,7 +197,7 @@ 
     } else {
 		$pieces = explode("/", $doc->url);
 		$nbPieces = count($pieces);
-		$filename = $pieces[$nbPieces-1];
+		$filename = $pieces[$nbPieces - 1];
         $compilatioId = $compilatio->sendDoc($doc->title, '', $filename, $mime, file_get_contents($LocalWrkUrl));
     }
 	$compTable = Database::get_course_table("plagiarism_compilatio_docs");
@@ -233,10 +233,10 @@ 
     if ($typeMessage == 1) {
         $message = $errorCodeNotValid;
         Display::display_error_message($message);
-    }elseif($typeMessage == 2){
+    }elseif ($typeMessage == 2) {
         $message = $errorLoadError;
         Display::display_error_message($message);
-    }else{
+    } else {
         $message = $msgWait;
         Display::display_confirmation_message($message);
 	}
@@ -268,7 +268,7 @@ 
 function getWorkTitle($docId, $coursId) {
     $res = "";
     $workTable = Database::get_course_table(TABLE_STUDENT_PUBLICATION);    
-    $sql = "SELECT title FROM " . $workTable . " WHERE c_id=" . $coursId . " AND id=" . $docId;
+    $sql = "SELECT title FROM ".$workTable." WHERE c_id=".$coursId." AND id=".$docId;
     $res = Database::query($sql);
     if (Database::num_rows($res) > 0) {
         $data = Database::fetch_array($res);
@@ -312,15 +312,15 @@ 
     $filename = getFilename($urlFile);
     $work_folder = getWorkFolder($urlFile);
     $urlFile_ext = getFileExtension($urlFile);
-    $coursePath = api_get_path(SYS_COURSE_PATH) . $_course["path"] . "/";
+    $coursePath = api_get_path(SYS_COURSE_PATH).$_course["path"]."/";
     $workTitle = getWorkTitle($docId, $coursId);
 
     if ($extensionFile != "" && $urlFile_ext == "") {
         /*rename the files in the FS whit the extension*/
         $shortFilename = $filename;                                        
         $cleanWorkTitle = api_replace_dangerous_char($workTitle); 
-        $newestFilename = $shortFilename . "_" . $cleanWorkTitle;
-        rename($coursePath . $urlFile, $coursePath . $work_folder . $newestFilename);
+        $newestFilename = $shortFilename."_".$cleanWorkTitle;
+        rename($coursePath.$urlFile, $coursePath.$work_folder.$newestFilename);
         /*rename the db's input with the extension*/
         $sql = "UPDATE "
             . $workTable

Braces +4 added lines, -4 removed lines

@@ -140,10 +140,10 @@ 
     if ($typeMessage == 1) {
         $message = $errorCodeNotValid;
         Display::display_error_message($message);
-    }elseif($typeMessage == 2){
+    } elseif($typeMessage == 2){
         $message = $errorLoadError;
         Display::display_error_message($message);
-    }else{
+    } else{
         $message = $msgWait;
         Display::display_confirmation_message($message);
     }
@@ -233,10 +233,10 @@ 
     if ($typeMessage == 1) {
         $message = $errorCodeNotValid;
         Display::display_error_message($message);
-    }elseif($typeMessage == 2){
+    } elseif($typeMessage == 2){
         $message = $errorLoadError;
         Display::display_error_message($message);
-    }else{
+    } else{
         $message = $msgWait;
         Display::display_confirmation_message($message);
 	}

main/work/work.lib.php

Doc Comments +21 added lines, -17 removed lines

@@ -158,7 +158,7 @@ 
  * Create a group of select from a date
  * @param FormValidator $form
  * @param string $prefix
- * @return array
+ * @return HTML_QuickForm_element[]
  */
 function create_group_date_select($form, $prefix = '')
 {
@@ -752,7 +752,7 @@ 
  * @author Bert Vanderkimpen
  * @author Yannick Warnier <ywarnier@beeznest.org> Adaptation for work tool
  * @param   string $base_work_dir Base work dir (.../work)
- * @param   string $desiredDirName complete path of the desired name
+ * @param   string $desired_dir_name complete path of the desired name
  *
  * @return  string actual directory name if it succeeds, boolean false otherwise
  */
@@ -774,7 +774,7 @@ 
 /**
  * Delete a work-tool directory
  * @param   int  $id work directory id to delete
- * @return  integer -1 on error
+ * @return  boolean|null -1 on error
  */
 function deleteDirWork($id)
 {
@@ -917,7 +917,7 @@ 
  * Update the url of a dir in the student_publication table
  * @param  array $work_data work original data
  * @param  string $newPath Example: "folder1"
- * @return bool
+ * @return boolean|null
  */
 function updateDirName($work_data, $newPath)
 {
@@ -974,6 +974,7 @@ 
 /**
  * Transform an all directory structure (only directories) in an array
  * @param   string path of the directory
+ * @param string $directory
  * @return  array the directory structure into an array
  * @author  Julio Montoya Dokeos
  * @version April 2008
@@ -1043,7 +1044,7 @@ 
  * @param   string the path of the directory
  * @param   boolean true if we want the total quantity of files
  * include in others child directories, false only  files in the directory
- * @return  array the first element is an integer with the number of files
+ * @return  integer[] the first element is an integer with the number of files
  * in the folder, the second element is the number of directories
  * @author  Julio Montoya
  * @version April 2008
@@ -2387,7 +2388,7 @@ 
 }
 
 /**
- * @param $name
+ * @param string $name
  * @param $values
  * @param string $checked
  * @return string
@@ -2913,7 +2914,7 @@ 
  * @param int $userId
  * @param int $workId
  * @param int $courseId
- * @return bool
+ * @return boolean|null
  */
 function allowOnlySubscribedUser($userId, $workId, $courseId)
 {
@@ -3031,7 +3032,7 @@ 
 /**
  * Get total score from a work list
  * @param $workList
- * @return int|null
+ * @return integer
  */
 function getTotalWorkScore($workList)
 {
@@ -3047,7 +3048,7 @@ 
  * Get comment count from a work list (docs sent by students)
  * @param array $workList
  * @param array $courseInfo
- * @return int|null
+ * @return integer
  */
 function getTotalWorkComment($workList, $courseInfo = array())
 {
@@ -3190,6 +3191,7 @@ 
  * @param int $parentId
  * @param array $courseInfo
  * @param int $sessionId
+ * @param integer $userId
  * @return int
  */
 function getLastWorkStudentFromParentByUser(
@@ -3826,12 +3828,11 @@ 
 
 /**
  * Creates a new task (directory) in the assignment tool
- * @param array $params
  * @param int $user_id
  * @param array $courseInfo
  * @param int $group_id
  * @param int $session_id
- * @return bool|int
+ * @return string|false
  * @note $params can have the following elements, but should at least have the 2 first ones: (
  *       'new_dir' => 'some-name',
  *       'description' => 'some-desc',
@@ -4327,7 +4328,7 @@ 
 }
 
 /**
- * @return array
+ * @return string[]
  */
 function getUploadDocumentType()
 {
@@ -4699,7 +4700,7 @@ 
  * @param int Session ID
  * @param $correction
  *
- * @return array|bool
+ * @return boolean
  */
 function getFileContents($id, $course_info, $sessionId = 0, $correction = false)
 {
@@ -4817,7 +4818,7 @@ 
  * @param int $userId
  * @param array $courseInfo
  * @param string $format
- * @return bool
+ * @return false|null
  */
 function exportAllWork($userId, $courseInfo, $format = 'pdf')
 {
@@ -4865,7 +4866,7 @@ 
  * @param array $courseInfo
  * @param int $sessionId
  * @param string $format
- * @return bool
+ * @return false|null
  */
 function exportAllStudentWorkFromPublication(
     $workId,
@@ -5003,7 +5004,7 @@ 
  * Downloads all user files per user
  * @param int $userId
  * @param array $courseInfo
- * @return bool
+ * @return false|null
  */
 function downloadAllFilesPerUser($userId, $courseInfo)
 {
@@ -5117,7 +5118,7 @@ 
 /**
  * @param array $courseInfo
  * @param int $workId
- * @return bool
+ * @return boolean|null
  */
 function protectWork($courseInfo, $workId)
 {
@@ -5192,6 +5193,9 @@ 
 /*
  * function for delete a document of the compilatio table if plagiarismTool is Compilatio
  */
+/**
+ * @param integer $itemId
+ */
 function plagiarismDeleteDoc($courseId, $itemId)
 {
     if (plagiarismActive()) {

Spacing +91 added lines, -91 removed lines

@@ -35,27 +35,27 @@ 
 
     if (!empty($id)) {
         $display_output .= '<a href="'.api_get_self().'?'.api_get_cidreq().'&origin='.$origin.'&gradebook='.$gradebook.'&id='.$my_back_id.'">'.
-            Display::return_icon('back.png', get_lang('BackToWorksList'),'',ICON_SIZE_MEDIUM).'</a>';
+            Display::return_icon('back.png', get_lang('BackToWorksList'), '', ICON_SIZE_MEDIUM).'</a>';
     }
 
     if (api_is_allowed_to_edit(null, true) && $origin != 'learnpath') {
         // Create dir
         if (empty($id)) {
             $display_output .= '<a href="'.api_get_self().'?'.api_get_cidreq().'&action=create_dir&origin='.$origin.'&gradebook='.$gradebook.'">';
-            $display_output .= Display::return_icon('new_work.png', get_lang('CreateAssignment'),'',ICON_SIZE_MEDIUM).'</a>';
+            $display_output .= Display::return_icon('new_work.png', get_lang('CreateAssignment'), '', ICON_SIZE_MEDIUM).'</a>';
         }
         if (empty($id)) {
             // Options
             $display_output .= '<a href="'.api_get_self().'?'.api_get_cidreq().'&action=settings&origin='.$origin.'&gradebook='.$gradebook.'">';
-            $display_output .= Display::return_icon('settings.png', get_lang('EditToolOptions'),'',ICON_SIZE_MEDIUM).'</a>';
+            $display_output .= Display::return_icon('settings.png', get_lang('EditToolOptions'), '', ICON_SIZE_MEDIUM).'</a>';
         }
-        $display_output .= '<a id="open-view-list" href="#">' . Display::return_icon('listwork.png', get_lang('ViewStudents'),'',ICON_SIZE_MEDIUM) . '</a>';
+        $display_output .= '<a id="open-view-list" href="#">'.Display::return_icon('listwork.png', get_lang('ViewStudents'), '', ICON_SIZE_MEDIUM).'</a>';
 
     }
 
     if (api_is_allowed_to_edit(null, true) && $origin != 'learnpath' && api_is_allowed_to_session_edit(false, true)) {
         // Delete all files
-        if (api_get_setting('permanently_remove_deleted_files') == 'true'){
+        if (api_get_setting('permanently_remove_deleted_files') == 'true') {
             $message = get_lang('ConfirmYourChoiceDeleteAllfiles');
         } else {
             $message = get_lang('ConfirmYourChoice');
@@ -112,7 +112,7 @@ 
  */
 function two_digits($number)
 {
-    $number = (int)$number;
+    $number = (int) $number;
     return ($number < 10) ? '0'.$number : $number;
 }
 
@@ -547,11 +547,11 @@ 
     );
 
     $columnModel = array(
-        array('name'=>'type', 'index'=>'type', 'width'=>'30',   'align'=>'center', 'sortable' => 'false'),
-        array('name'=>'title', 'index'=>'title', 'width'=>'250',   'align'=>'left'),
-        array('name'=>'expires_on', 'index'=>'expires_on', 'width'=>'80',  'align'=>'center', 'sortable'=>'false'),
-        array('name'=>'feedback', 'index'=>'feedback', 'width'=>'80',  'align'=>'center'),
-        array('name'=>'last_upload', 'index'=>'feedback', 'width'=>'125',  'align'=>'center'),
+        array('name'=>'type', 'index'=>'type', 'width'=>'30', 'align'=>'center', 'sortable' => 'false'),
+        array('name'=>'title', 'index'=>'title', 'width'=>'250', 'align'=>'left'),
+        array('name'=>'expires_on', 'index'=>'expires_on', 'width'=>'80', 'align'=>'center', 'sortable'=>'false'),
+        array('name'=>'feedback', 'index'=>'feedback', 'width'=>'80', 'align'=>'center'),
+        array('name'=>'last_upload', 'index'=>'feedback', 'width'=>'125', 'align'=>'center'),
     );
 
     if ($courseInfo['show_score'] == 0) {
@@ -588,10 +588,10 @@ 
 {
     $columnModel = array(
         array('name'=>'type', 'index'=>'type', 'width'=>'35', 'align'=>'center', 'sortable' => 'false'),
-        array('name'=>'title', 'index'=>'title',  'width'=>'300',   'align'=>'left', 'wrap_cell' => "true"),
-        array('name'=>'sent_date', 'index'=>'sent_date', 'width'=>'125',  'align'=>'center'),
-        array('name'=>'expires_on', 'index'=>'expires_on', 'width'=>'125',  'align'=>'center'),
-        array('name'=>'amount', 'index'=>'end_on', 'width'=>'110',  'align'=>'center'),
+        array('name'=>'title', 'index'=>'title', 'width'=>'300', 'align'=>'left', 'wrap_cell' => "true"),
+        array('name'=>'sent_date', 'index'=>'sent_date', 'width'=>'125', 'align'=>'center'),
+        array('name'=>'expires_on', 'index'=>'expires_on', 'width'=>'125', 'align'=>'center'),
+        array('name'=>'amount', 'index'=>'end_on', 'width'=>'110', 'align'=>'center'),
         array('name'=>'actions', 'index'=>'actions', 'width'=>'110', 'align'=>'left', 'sortable'=>'false')
     );
 
@@ -793,7 +793,7 @@ 
         return false;
     }
 
-    $base_work_dir = api_get_path(SYS_COURSE_PATH) .$_course['path'].'/work';
+    $base_work_dir = api_get_path(SYS_COURSE_PATH).$_course['path'].'/work';
     $work_data_url = $base_work_dir.$work_data['url'];
     $check = Security::check_abs_path($work_data_url.'/', $base_work_dir.'/');
 
@@ -863,7 +863,7 @@ 
 function get_work_path($id)
 {
     $table = Database::get_course_table(TABLE_STUDENT_PUBLICATION);
-    $course_id  = api_get_course_int_id();
+    $course_id = api_get_course_int_id();
     $sql = 'SELECT url FROM '.$table.'
             WHERE c_id = '.$course_id.' AND id='.intval($id);
     $res = Database::query($sql);
@@ -984,9 +984,9 @@ 
     if ($handle = @opendir($directory)) {
         while (false !== ($file = readdir($handle))) {
             if ($file != '.' && $file != '..') {
-                if (is_dir($directory. '/' . $file)) {
-                    $array_items = array_merge($array_items, directory_to_array($directory. '/' . $file));
-                    $file = $directory . '/' . $file;
+                if (is_dir($directory.'/'.$file)) {
+                    $array_items = array_merge($array_items, directory_to_array($directory.'/'.$file));
+                    $file = $directory.'/'.$file;
                     $array_items[] = preg_replace("/\/\//si", '/', $file);
                 }
             }
@@ -1018,7 +1018,7 @@ 
 
     $work_table = Database :: get_course_table(TABLE_STUDENT_PUBLICATION);
 
-    for($i = 0; $i < count($only_dir); $i++) {
+    for ($i = 0; $i < count($only_dir); $i++) {
         $url = $only_dir[$i];
 
         $params = [
@@ -1058,7 +1058,7 @@ 
             if (is_dir($path_dir.'/'.$entry)) {
                 $count_dir++;
                 if ($recurse) {
-                    $count += count_dir($path_dir . '/' . $entry, $recurse);
+                    $count += count_dir($path_dir.'/'.$entry, $recurse);
                 }
             } else {
                 $count++;
@@ -1323,11 +1323,11 @@ 
         $courseInfo
     );
 
-    if (!in_array($direction, array('asc','desc'))) {
+    if (!in_array($direction, array('asc', 'desc'))) {
         $direction = 'desc';
     }
     if (!empty($where_condition)) {
-        $where_condition = ' AND ' . $where_condition;
+        $where_condition = ' AND '.$where_condition;
     }
 
     $column = !empty($column) ? Database::escape_string($column) : 'sent_date';
@@ -1470,7 +1470,7 @@ 
         $direction = 'desc';
     }
     if (!empty($where_condition)) {
-        $where_condition = ' AND ' . $where_condition;
+        $where_condition = ' AND '.$where_condition;
     }
 
     $column = !empty($column) ? Database::escape_string($column) : 'sent_date';
@@ -1528,7 +1528,7 @@ 
             );
 
             $work['amount'] = Display::label(
-                $countUniqueAttempts . '/' .
+                $countUniqueAttempts.'/'.
                 $totalUsers,
                 'success'
             );
@@ -1577,7 +1577,7 @@ 
                         array(),
                         ICON_SIZE_SMALL
                     ),
-                    api_get_path(WEB_CODE_PATH) . 'work/downloadfolder.inc.php?id=' . $workId . '&' . api_get_cidreq()
+                    api_get_path(WEB_CODE_PATH).'work/downloadfolder.inc.php?id='.$workId.'&'.api_get_cidreq()
                 );
             } else {
                 $downloadLink = Display::url(
@@ -1661,7 +1661,7 @@ 
     $userCondition = " AND u.user_id = $studentId ";
     $sessionCondition = " AND w.session_id = $sessionId ";
     $workCondition = " AND w_rel.work_id = $workId";
-    $workParentCondition  = " AND w.parent_id = $workId";
+    $workParentCondition = " AND w.parent_id = $workId";
 
     $sql = "(
                 $select1 FROM $userTable u
@@ -1769,7 +1769,7 @@ 
         }
 
         if ($allowEdition && !empty($itemId)) {
-            $deleteLink  = Display::url($deleteIcon, $urlDelete.'&item_id='.$itemId.'&id='.$workId);
+            $deleteLink = Display::url($deleteIcon, $urlDelete.'&item_id='.$itemId.'&id='.$workId);
         } else {
             $deleteLink = null;
         }
@@ -1851,8 +1851,8 @@ 
     $limit = intval($limit);
 
     /*Call Compilatio's API*/
-    require_once(api_get_path(SYS_CODE_PATH) . 'plagiarism/compilatio/config.php');
-    require_once(api_get_path(SYS_CODE_PATH) . 'plagiarism/compilatio/compilatio.class.php');
+    require_once(api_get_path(SYS_CODE_PATH).'plagiarism/compilatio/config.php');
+    require_once(api_get_path(SYS_CODE_PATH).'plagiarism/compilatio/compilatio.class.php');
 
     $compilatio_web_folder = api_get_path(WEB_CODE_PATH).'plagiarism/compilatio/';
     $compilatio = new compilatio(
@@ -1863,7 +1863,7 @@ 
     );
 
 
-    if (!in_array($direction, array('asc','desc'))) {
+    if (!in_array($direction, array('asc', 'desc'))) {
         $direction = 'desc';
     }
 
@@ -1931,13 +1931,13 @@ 
         $work_assignment = get_work_assignment_by_id($work_id);
 
         if (!empty($studentId)) {
-            $where_condition.= " AND u.user_id = ".intval($studentId);
+            $where_condition .= " AND u.user_id = ".intval($studentId);
         }
 
         $sql = " $select
                 FROM $work_condition  $user_condition
                 WHERE $extra_conditions $where_condition $condition_session
-                    AND u.status != " . INVITEE . "
+                    AND u.status != ".INVITEE."
                 ORDER BY $column $direction";
 
         if (!empty($start) && !empty($limit)) {
@@ -1995,7 +1995,7 @@ 
                     $qualification_string = Display::label('-');
                 } else {
                     $label = 'info';
-                    $relativeScore = $work['qualification']/$work_data['qualification'];
+                    $relativeScore = $work['qualification'] / $work_data['qualification'];
                     if ($relativeScore < 0.5) {
                         $label = 'important';
                     } elseif ($relativeScore < 0.75) {
@@ -2051,10 +2051,10 @@ 
                 // If URL is present then there's a file to download keep BC.
                 if ($work['contains_file'] || !empty($work['url'])) {
                     $link_to_download = '<a href="'.$url.'download.php?id='.$item_id.'&'.api_get_cidreq().'">'.
-                        Display::return_icon('save.png', get_lang('Save'),array(), ICON_SIZE_SMALL).'</a> ';
+                        Display::return_icon('save.png', get_lang('Save'), array(), ICON_SIZE_SMALL).'</a> ';
                 }
 
-                $send_to = Portfolio::share('work', $work['id'],  array('style' => 'white-space:nowrap;'));
+                $send_to = Portfolio::share('work', $work['id'], array('style' => 'white-space:nowrap;'));
 
                 $feedback = null;
                 $count = getWorkCommentCount($item_id, $course_info);
@@ -2073,7 +2073,7 @@ 
                 $work_date = api_convert_and_format_date($work['sent_date']);
 
                 $work['sent_date_from_db'] = $work['sent_date'];
-                $work['sent_date'] = '<div class="date-time">' . date_to_str_ago(api_get_local_time($work['sent_date'])) . ' ' . $add_string . ' ' . $work_date . '</div>';
+                $work['sent_date'] = '<div class="date-time">'.date_to_str_ago(api_get_local_time($work['sent_date'])).' '.$add_string.' '.$work_date.'</div>';
 
                 // Actions.
                 $correction = '';
@@ -2091,8 +2091,8 @@ 
                         Display::return_icon('default.png', get_lang('View'), array(), ICON_SIZE_SMALL).'</a> ';
 
                     if ($unoconv && empty($work['contains_file'])) {
-                        $action .=  '<a href="'.$url.'work_list_all.php?'.api_get_cidreq().'&id='.$work_id.'&action=export_to_doc&item_id='.$item_id.'" title="'.get_lang('ExportToDoc').'" >'.
-                            Display::return_icon('export_doc.png', get_lang('ExportToDoc'),array(), ICON_SIZE_SMALL).'</a> ';
+                        $action .= '<a href="'.$url.'work_list_all.php?'.api_get_cidreq().'&id='.$work_id.'&action=export_to_doc&item_id='.$item_id.'" title="'.get_lang('ExportToDoc').'" >'.
+                            Display::return_icon('export_doc.png', get_lang('ExportToDoc'), array(), ICON_SIZE_SMALL).'</a> ';
                     }
 
                     $correction = '
@@ -2144,9 +2144,9 @@ 
 
                     if ($locked) {
                         if ($qualification_exists) {
-                            $action .= Display::return_icon('rate_work_na.png', get_lang('CorrectAndRate'),array(), ICON_SIZE_SMALL);
+                            $action .= Display::return_icon('rate_work_na.png', get_lang('CorrectAndRate'), array(), ICON_SIZE_SMALL);
                         } else {
-                            $action .= Display::return_icon('edit_na.png', get_lang('Comment'),array(), ICON_SIZE_SMALL);
+                            $action .= Display::return_icon('edit_na.png', get_lang('Comment'), array(), ICON_SIZE_SMALL);
                         }
                     } else {
                         if ($qualification_exists) {
@@ -2160,45 +2160,45 @@ 
 
                     if ($work['contains_file']) {
                         if ($locked) {
-                            $action .= Display::return_icon('move_na.png', get_lang('Move'),array(), ICON_SIZE_SMALL);
+                            $action .= Display::return_icon('move_na.png', get_lang('Move'), array(), ICON_SIZE_SMALL);
                         } else {
                             $action .= '<a href="'.$url.'work.php?'.api_get_cidreq().'&action=move&item_id='.$item_id.'&id='.$work['parent_id'].'" title="'.get_lang('Move').'">'.
-                                Display::return_icon('move.png', get_lang('Move'),array(), ICON_SIZE_SMALL).'</a>';
+                                Display::return_icon('move.png', get_lang('Move'), array(), ICON_SIZE_SMALL).'</a>';
                         }
                     }
 
                     if ($work['accepted'] == '1') {
                         $action .= '<a href="'.$url.'work_list_all.php?'.api_get_cidreq().'&id='.$work_id.'&action=make_invisible&item_id='.$item_id.'" title="'.get_lang('Invisible').'" >'.
-                            Display::return_icon('visible.png', get_lang('Invisible'),array(), ICON_SIZE_SMALL).'</a>';
+                            Display::return_icon('visible.png', get_lang('Invisible'), array(), ICON_SIZE_SMALL).'</a>';
                     } else {
                         $action .= '<a href="'.$url.'work_list_all.php?'.api_get_cidreq().'&id='.$work_id.'&action=make_visible&item_id='.$item_id.'" title="'.get_lang('Visible').'" >'.
-                            Display::return_icon('invisible.png', get_lang('Visible'),array(), ICON_SIZE_SMALL).'</a> ';
+                            Display::return_icon('invisible.png', get_lang('Visible'), array(), ICON_SIZE_SMALL).'</a> ';
                     }
 
                     if ($locked) {
                         $action .= Display::return_icon('delete_na.png', get_lang('Delete'), '', ICON_SIZE_SMALL);
                     } else {
-                        $action .= '<a href="'.$url.'work_list_all.php?'.api_get_cidreq().'&id='.$work_id.'&action=delete&item_id='.$item_id.'" onclick="javascript:if(!confirm('."'".addslashes(api_htmlentities(get_lang('ConfirmYourChoice'),ENT_QUOTES))."'".')) return false;" title="'.get_lang('Delete').'" >'.
-                            Display::return_icon('delete.png', get_lang('Delete'),'',ICON_SIZE_SMALL).'</a>';
+                        $action .= '<a href="'.$url.'work_list_all.php?'.api_get_cidreq().'&id='.$work_id.'&action=delete&item_id='.$item_id.'" onclick="javascript:if(!confirm('."'".addslashes(api_htmlentities(get_lang('ConfirmYourChoice'), ENT_QUOTES))."'".')) return false;" title="'.get_lang('Delete').'" >'.
+                            Display::return_icon('delete.png', get_lang('Delete'), '', ICON_SIZE_SMALL).'</a>';
                     }
                 } elseif ($is_author && (empty($work['qualificator_id']) || $work['qualificator_id'] == 0)) {
                     $action .= '<a href="'.$url.'view.php?'.api_get_cidreq().'&id='.$item_id.'" title="'.get_lang('View').'">'.
-                        Display::return_icon('default.png', get_lang('View'),array(), ICON_SIZE_SMALL).'</a>';
+                        Display::return_icon('default.png', get_lang('View'), array(), ICON_SIZE_SMALL).'</a>';
 
                     if (api_get_course_setting('student_delete_own_publication') == 1) {
                         if (api_is_allowed_to_session_edit(false, true)) {
                             $action .= '<a href="'.$url.'edit.php?'.api_get_cidreq().'&item_id='.$item_id.'&id='.$work['parent_id'].'" title="'.get_lang('Modify').'">'.
-                                Display::return_icon('edit.png', get_lang('Comment'),array(), ICON_SIZE_SMALL).'</a>';
+                                Display::return_icon('edit.png', get_lang('Comment'), array(), ICON_SIZE_SMALL).'</a>';
                         }
-                        $action .= ' <a href="'.$url.'work_list.php?'.api_get_cidreq().'&action=delete&item_id='.$item_id.'&id='.$work['parent_id'].'" onclick="javascript:if(!confirm('."'".addslashes(api_htmlentities(get_lang('ConfirmYourChoice'),ENT_QUOTES))."'".')) return false;" title="'.get_lang('Delete').'"  >'.
-                            Display::return_icon('delete.png',get_lang('Delete'),'',ICON_SIZE_SMALL).'</a>';
+                        $action .= ' <a href="'.$url.'work_list.php?'.api_get_cidreq().'&action=delete&item_id='.$item_id.'&id='.$work['parent_id'].'" onclick="javascript:if(!confirm('."'".addslashes(api_htmlentities(get_lang('ConfirmYourChoice'), ENT_QUOTES))."'".')) return false;" title="'.get_lang('Delete').'"  >'.
+                            Display::return_icon('delete.png', get_lang('Delete'), '', ICON_SIZE_SMALL).'</a>';
                     } else {
-                        $action .= Display::return_icon('edit_na.png', get_lang('Modify'),array(), ICON_SIZE_SMALL);
+                        $action .= Display::return_icon('edit_na.png', get_lang('Modify'), array(), ICON_SIZE_SMALL);
                     }
                 } else {
                     $action .= '<a href="'.$url.'view.php?'.api_get_cidreq().'&id='.$item_id.'" title="'.get_lang('View').'">'.
-                        Display::return_icon('default.png', get_lang('View'),array(), ICON_SIZE_SMALL).'</a>';
-                    $action .= Display::return_icon('edit_na.png', get_lang('Modify'),array(), ICON_SIZE_SMALL);
+                        Display::return_icon('default.png', get_lang('View'), array(), ICON_SIZE_SMALL).'</a>';
+                    $action .= Display::return_icon('edit_na.png', get_lang('Modify'), array(), ICON_SIZE_SMALL);
                 }
 
                 // Status.
@@ -2220,7 +2220,7 @@ 
                     . $course_id;
                 $compiSqlResult = Database::query($compilatioQuery);
                 $compi = Database::fetch_object($compiSqlResult);
-                $workDirectory = api_get_path(SYS_COURSE_PATH) . $course_info['directory'];
+                $workDirectory = api_get_path(SYS_COURSE_PATH).$course_info['directory'];
                 if (isset($compi->compilatio_id)) {
                     $actionCompilatio = "<div id='id_avancement"
                         . $item_id
@@ -2235,8 +2235,8 @@ 
                         $actionCompilatio = "<div style='font-style:italic'>"
                             . get_lang('compilatioFileisnotsupported')
                             ."</div>";
-                    } elseif (filesize($workDirectory . "/" . $work['url']) > $compilatioParameter['max_filesize']) {
-                        $sizeFile = round(filesize($workDirectory . "/" . $work['url']) / 1000000);
+                    } elseif (filesize($workDirectory."/".$work['url']) > $compilatioParameter['max_filesize']) {
+                        $sizeFile = round(filesize($workDirectory."/".$work['url']) / 1000000);
                         $status = "BAD_FILESIZE";
                         $actionCompilatio = "<div style='font-style:italic'>"
                             .get_lang('compilatioTooHeavyDocument')
@@ -2277,7 +2277,7 @@ 
     $_course = api_get_course_info();
     $task_id = $task_data['id'];
     $task_title = !empty($task_data['title']) ? $task_data['title'] : basename($task_data['url']);
-    $subject = '[' . api_get_setting('siteName') . '] ';
+    $subject = '['.api_get_setting('siteName').'] ';
 
     // The body can be as long as you wish, and any combination of text and variables
     $content = get_lang('ReminderToSubmitPendingTask')."\n".get_lang('CourseName').' : '.$_course['name']."\n";
@@ -2288,7 +2288,7 @@ 
     $mails_sent_to = array();
     foreach ($list_users as $user) {
         $name_user = api_get_person_name($user[1], $user[0], null, PERSON_NAME_EMAIL_ADDRESS);
-        $dear_line = get_lang('Dear')." ".api_get_person_name($user[1], $user[0]) .", \n\n";
+        $dear_line = get_lang('Dear')." ".api_get_person_name($user[1], $user[0]).", \n\n";
         $body      = $dear_line.$content;
         MessageManager::send_message($user[3], $subject, $body);
         $mails_sent_to[] = $name_user;
@@ -2317,22 +2317,22 @@ 
     } else {
         $students = CourseManager::get_student_list_from_course_code($courseCode, true, $sessionId);
     }
-    $emailsubject = '[' . api_get_setting('siteName') . '] '.get_lang('HomeworkCreated');
+    $emailsubject = '['.api_get_setting('siteName').'] '.get_lang('HomeworkCreated');
     $currentUser = api_get_user_info(api_get_user_id());
     if (!empty($students)) {
-        foreach($students as $student) {
+        foreach ($students as $student) {
             $user_info = api_get_user_info($student["user_id"]);
-            if(!empty($user_info["mail"])) {
+            if (!empty($user_info["mail"])) {
                 $name_user = api_get_person_name(
                     $user_info["firstname"],
                     $user_info["lastname"],
                     null,
                     PERSON_NAME_EMAIL_ADDRESS
                 );
-                $link = api_get_path(WEB_CODE_PATH) . 'work/work_list_all.php?' . api_get_cidreq() . '&id=' . $workId;
+                $link = api_get_path(WEB_CODE_PATH).'work/work_list_all.php?'.api_get_cidreq().'&id='.$workId;
                 $emailbody = get_lang('Dear')." ".$name_user.",\n\n";
                 $emailbody .= get_lang('HomeworkHasBeenCreatedForTheCourse')." ".$courseCode.". "."\n\n".
-                    '<a href="'. $link . '">' . get_lang('PleaseCheckHomeworkPage') . '</a>';
+                    '<a href="'.$link.'">'.get_lang('PleaseCheckHomeworkPage').'</a>';
                 $emailbody .= "\n\n".api_get_person_name($currentUser["firstname"], $currentUser["lastname"]);
 
                 $additionalParameters = array(
@@ -2374,7 +2374,7 @@ 
     $url = Database::escape_string($url);
     $sql = "SELECT id FROM $work_table WHERE url='$url'";
     $result = Database::query($sql);
-    if (Database::num_rows($result)> 0) {
+    if (Database::num_rows($result) > 0) {
         $row = Database::fetch_row($result);
         if (empty($row)) {
             return false;
@@ -2396,7 +2396,7 @@ 
 {
     $output = '<select name="'.$name.'" id="'.$name.'">';
     foreach ($values as $key => $value) {
-        $output .= '<option value="'.$key.'" '.(($checked==$key) ? 'selected="selected"' : '').'>'.$value.'</option>';
+        $output .= '<option value="'.$key.'" '.(($checked == $key) ? 'selected="selected"' : '').'>'.$value.'</option>';
     }
     $output .= '</select>';
     return $output;
@@ -2410,9 +2410,9 @@ 
  */
 function make_checkbox($name, $checked = '', $label = null)
 {
-    $check = '<input id ="'.$name.'" type="checkbox" value="1" name="'.$name.'" '.((!empty($checked))?'checked="checked"':'').'/>';
+    $check = '<input id ="'.$name.'" type="checkbox" value="1" name="'.$name.'" '.((!empty($checked)) ? 'checked="checked"' : '').'/>';
     if (!empty($label)) {
-        $check .="<label for ='$name'>$label</label>";
+        $check .= "<label for ='$name'>$label</label>";
     }
     return $check;
 }
@@ -2566,7 +2566,7 @@ 
     }
 
     if (!empty($studentId)) {
-        $sql_users.= " AND u.user_id = ".intval($studentId);
+        $sql_users .= " AND u.user_id = ".intval($studentId);
     }
 
     $group_id = api_get_group_id();
@@ -2576,7 +2576,7 @@ 
     if ($group_id) {
         $group_user_list = GroupManager::get_subscribed_users($group_id);
         if (!empty($group_user_list)) {
-            foreach($group_user_list as $group_user) {
+            foreach ($group_user_list as $group_user) {
                 $new_group_user_list[] = $group_user['user_id'];
             }
         }
@@ -2997,7 +2997,7 @@ 
 function getWorkComments($work)
 {
     $commentTable = Database::get_course_table(TABLE_STUDENT_PUBLICATION_ASSIGNMENT_COMMENT);
-    $userTable= Database::get_main_table(TABLE_MAIN_USER);
+    $userTable = Database::get_main_table(TABLE_MAIN_USER);
 
     $courseId = intval($work['c_id']);
     $workId = intval($work['id']);
@@ -3680,7 +3680,7 @@ 
             null,
             PERSON_NAME_EMAIL_ADDRESS
         );
-        $subject = "[" . api_get_setting('siteName') . "] ".get_lang('SendMailBody')."\n ".get_lang('CourseName').": ".$courseInfo['name']."  ";
+        $subject = "[".api_get_setting('siteName')."] ".get_lang('SendMailBody')."\n ".get_lang('CourseName').": ".$courseInfo['name']."  ";
         foreach ($user_list as $user_data) {
             $to_user_id = $user_data['user_id'];
             $user_info = api_get_user_info($to_user_id);
@@ -3729,7 +3729,7 @@ 
 
     $title = $values['title'];
     $description = $values['description'];
-    $contains_file = isset($values['contains_file']) && !empty($values['contains_file']) ? intval($values['contains_file']): 0;
+    $contains_file = isset($values['contains_file']) && !empty($values['contains_file']) ? intval($values['contains_file']) : 0;
 
     $saveWork = true;
     $message = null;
@@ -4103,8 +4103,8 @@ 
                     LINK_STUDENTPUBLICATION,
                     $workId,
                     $params['new_dir'],
-                    (float)$params['weight'],
-                    (float)$params['qualification'],
+                    (float) $params['weight'],
+                    (float) $params['qualification'],
                     $params['description'],
                     1,
                     api_get_session_id()
@@ -4364,15 +4364,15 @@ 
     // changing the tool setting: is a student allowed to delete his/her own document
 
     // counting the number of occurrences of this setting (if 0 => add, if 1 => update)
-    $query = "SELECT * FROM " . $table_course_setting . "
+    $query = "SELECT * FROM ".$table_course_setting."
               WHERE c_id = $courseId AND variable = 'student_delete_own_publication'";
 
     $result = Database::query($query);
     $number_of_setting = Database::num_rows($result);
 
     if ($number_of_setting == 1) {
-        $query = "UPDATE " . $table_course_setting . " SET
-                  value='" . Database::escape_string($studentDeleteOwnPublication) . "'
+        $query = "UPDATE ".$table_course_setting." SET
+                  value='" . Database::escape_string($studentDeleteOwnPublication)."'
                   WHERE variable = 'student_delete_own_publication' AND c_id = $courseId";
         Database::query($query);
     } else {
@@ -4411,9 +4411,9 @@ 
     $work_table = Database :: get_course_table(TABLE_STUDENT_PUBLICATION);
     $item_id = intval($item_id);
     $course_id = $course_info['real_id'];
-    $sql = "UPDATE  " . $work_table . "
+    $sql = "UPDATE  ".$work_table."
             SET accepted = 0
-            WHERE c_id = $course_id AND id = '" . $item_id . "'";
+            WHERE c_id = $course_id AND id = '".$item_id."'";
     Database::query($sql);
     api_item_property_update(
         $course_info,
@@ -4557,7 +4557,7 @@ 
                 $courseCode,
                 $sessionId,
                 $limitString,
-                $orderBy ,
+                $orderBy,
                 STUDENT,
                 $getCount
             );
@@ -4894,8 +4894,8 @@ 
     if (!empty($sessionId)) {
         $sessionInfo = api_get_session_info($sessionId);
         if (!empty($sessionInfo)) {
-            $header .= ' - ' . $sessionInfo['name'];
-            $header .= '<br />' . $sessionInfo['description'];
+            $header .= ' - '.$sessionInfo['name'];
+            $header .= '<br />'.$sessionInfo['description'];
             $teachers = SessionManager::getCoachesByCourseSessionToString(
                 $sessionId,
                 $courseInfo['real_id']
@@ -4911,12 +4911,12 @@ 
     $expiresOn = null;
 
     if (!empty($assignment) && isset($assignment['expires_on'])) {
-        $content .= '<br /><strong>' . get_lang('ExpirationDate') . '</strong>: ' . api_get_local_time($assignment['expires_on']);
+        $content .= '<br /><strong>'.get_lang('ExpirationDate').'</strong>: '.api_get_local_time($assignment['expires_on']);
         $expiresOn = api_get_local_time($assignment['expires_on']);
     }
 
     if (!empty($workData['description'])) {
-        $content .= '<br /><strong>' . get_lang('Description') . '</strong>: ' . $workData['description'];
+        $content .= '<br /><strong>'.get_lang('Description').'</strong>: '.$workData['description'];
     }
 
     $workList = get_work_user_list(null, null, null, null, $workId);
@@ -4937,7 +4937,7 @@ 
                 );
 
                 $column = 0;
-                foreach($headers as $header) {
+                foreach ($headers as $header) {
                     $table->setHeaderContents(0, $column, $header);
                     $column++;
                 }
@@ -4985,7 +4985,7 @@ 
 
                 if (!empty($content)) {
                     $params = array(
-                        'filename' => $workData['title'] . '_' . api_get_local_time(),
+                        'filename' => $workData['title'].'_'.api_get_local_time(),
                         'pdf_title' => api_replace_dangerous_char($workData['title']),
                         'course_code' => $courseInfo['code'],
                         'add_signatures' => false
@@ -5016,7 +5016,7 @@ 
     $tempZipFile = api_get_path(SYS_ARCHIVE_PATH).api_get_unique_id().".zip";
     $coursePath = api_get_path(SYS_COURSE_PATH).$courseInfo['path'].'/work/';
 
-    $zip  = new PclZip($tempZipFile);
+    $zip = new PclZip($tempZipFile);
 
     $workPerUser = getWorkPerUser($userId);
 
@@ -5180,8 +5180,8 @@ 
  */
 function compilatioDeleteDoc($courseId, $itemId)
 {
-    $plagiarism_table   = Database :: get_course_table(TABLE_COMPILATIO);
-    $queryString4   = "DELETE FROM "
+    $plagiarism_table = Database :: get_course_table(TABLE_COMPILATIO);
+    $queryString4 = "DELETE FROM "
         . $plagiarism_table
         . " WHERE c_id = "
         . $courseId

main/inc/lib/display.lib.php

Spacing +77 added lines, -77 removed lines

@@ -55,7 +55,7 @@ 
      * @param string Optional help file name
      * @param string $page_header
      */
-    public static function display_header($tool_name ='', $help = null, $page_header = null)
+    public static function display_header($tool_name = '', $help = null, $page_header = null)
     {
         $origin = api_get_origin();
         $showHeader = true;
@@ -351,7 +351,7 @@ 
         $grid_class = array(),
         $elementCount = 0
     ) {
-        $column =  0;
+        $column = 0;
         $default_items_per_page = isset($paging_options['per_page']) ? $paging_options['per_page'] : 20;
 
         $table = new SortableTableFromArray($content, $column, $default_items_per_page, $name);
@@ -476,7 +476,7 @@ 
      * @param bool	Filter (true) or not (false)
      * @return void
      */
-    public static function display_confirmation_message ($message, $filter = true, $returnValue = false)
+    public static function display_confirmation_message($message, $filter = true, $returnValue = false)
     {
         $message = self::return_message($message, 'confirm', $filter);
         if ($returnValue) {
@@ -493,7 +493,7 @@ 
      * @param bool	Filter (true) or not (false)
      * @return void
      */
-    public static function display_error_message ($message, $filter = true, $returnValue = false)
+    public static function display_error_message($message, $filter = true, $returnValue = false)
     {
         $message = self::return_message($message, 'error', $filter);
         if ($returnValue) {
@@ -508,7 +508,7 @@ 
      * @param string $type
      * @param bool $filter
      */
-    public static function return_message_and_translate($message, $type='normal', $filter = true)
+    public static function return_message_and_translate($message, $type = 'normal', $filter = true)
     {
         $message = get_lang($message);
         echo self::return_message($message, $type, $filter);
@@ -532,7 +532,7 @@ 
         }
 
         $class = "";
-        switch($type) {
+        switch ($type) {
             case 'warning':
                $class .= 'alert alert-warning';
                break;
@@ -560,7 +560,7 @@ 
      * @param string  optional, class from stylesheet
      * @return string encrypted mailto hyperlink
      */
-    public static function encrypted_mailto_link ($email, $clickable_text = null, $style_class = '')
+    public static function encrypted_mailto_link($email, $clickable_text = null, $style_class = '')
     {
         if (is_null($clickable_text)) {
             $clickable_text = $email;
@@ -575,14 +575,14 @@ 
         }
         // Encrypt email
         $hmail = '';
-        for ($i = 0; $i < strlen($email); $i ++) {
+        for ($i = 0; $i < strlen($email); $i++) {
             $hmail .= '&#'.ord($email {
             $i }).';';
         }
         $hclickable_text = null;
         // Encrypt clickable text if @ is present
         if (strpos($clickable_text, '@')) {
-            for ($i = 0; $i < strlen($clickable_text); $i ++) {
+            for ($i = 0; $i < strlen($clickable_text); $i++) {
                 $hclickable_text .= '&#'.ord($clickable_text {
                 $i }).';';
             }
@@ -614,7 +614,7 @@ 
         }
         // Encrypt email
         $hmail = '';
-        for ($i = 0; $i < strlen($email); $i ++) {
+        for ($i = 0; $i < strlen($email); $i++) {
             $hmail .= '&#'.ord($email {
             $i }).';';
         }
@@ -647,7 +647,7 @@ 
     public static function get_alphabet_options($selected_letter = '')
     {
         $result = '';
-        for ($i = 65; $i <= 90; $i ++) {
+        for ($i = 65; $i <= 90; $i++) {
             $letter = chr($i);
             $result .= '<option value="'.$letter.'"';
             if ($selected_letter == $letter) {
@@ -668,7 +668,7 @@ 
     public static function get_numeric_options($min, $max, $selected_num = 0)
     {
         $result = '';
-        for ($i = $min; $i <= $max; $i ++) {
+        for ($i = $min; $i <= $max; $i++) {
             $result .= '<option value="'.$i.'"';
             if (is_int($selected_num))
                 if ($selected_num == $i) {
@@ -745,7 +745,7 @@ 
             $size = ICON_SIZE_SMALL;
         }
 
-        $size_extra = $size . '/';
+        $size_extra = $size.'/';
 
         // Checking the img/ folder
         $icon = $w_code_path.'img/'.$image;
@@ -753,7 +753,7 @@ 
         $theme = 'themes/chamilo/icons/';
 
         if ($loadThemeIcon) {
-            $theme = 'themes/' . api_get_visual_theme() . '/icons/';
+            $theme = 'themes/'.api_get_visual_theme().'/icons/';
             // Checking the theme icons folder example: app/Resources/public/css/themes/chamilo/icons/XXX
             if (is_file($alternateCssPath.$theme.$size_extra.$image)) {
                 $icon = $alternateWebCssPath.$theme.$size_extra.$image;
@@ -775,11 +775,11 @@ 
         // ask for the SVG version directly
         $testServer = api_get_setting('server_type');
         if ($testServer == 'test' && $return_only_path == false) {
-            $svgImage = substr($image, 0, -3) . 'svg';
-            if (is_file($code_path . $theme . 'svg/' . $svgImage)) {
-                $icon = $w_code_path . $theme . 'svg/' . $svgImage;
-            } elseif (is_file($code_path . 'img/icons/svg/' . $svgImage)) {
-                $icon = $w_code_path . 'img/icons/svg/' . $svgImage;
+            $svgImage = substr($image, 0, -3).'svg';
+            if (is_file($code_path.$theme.'svg/'.$svgImage)) {
+                $icon = $w_code_path.$theme.'svg/'.$svgImage;
+            } elseif (is_file($code_path.'img/icons/svg/'.$svgImage)) {
+                $icon = $w_code_path.'img/icons/svg/'.$svgImage;
             }
 
             if (empty($additional_attributes['height'])) {
@@ -859,7 +859,7 @@ 
             }
         }
         //some tags don't have this </XXX>
-        if (in_array($tag, array('img','input','br'))) {
+        if (in_array($tag, array('img', 'input', 'br'))) {
             $return_value = '<'.$tag.' '.$attribute_list.' />';
         } else {
             $return_value = '<'.$tag.' '.$attribute_list.' > '.$content.'</'.$tag.'>';
@@ -912,13 +912,13 @@ 
     public static function input($type, $name, $value, $attributes = array())
     {
          if (isset($type)) {
-             $attributes['type']= $type;
+             $attributes['type'] = $type;
          }
          if (isset($name)) {
-             $attributes['name']= $name;
+             $attributes['name'] = $name;
          }
          if (isset($value)) {
-             $attributes['value']= $value;
+             $attributes['value'] = $value;
         }
         return self::tag('input', '', $attributes);
     }
@@ -951,14 +951,14 @@ 
     ) {
         $html = '';
         $extra = '';
-        $default_id = 'id="' . $name . '" ';
+        $default_id = 'id="'.$name.'" ';
         foreach ($extra_attributes as $key => $parameter) {
             if ($key == 'id') {
                 $default_id = '';
             }
-            $extra .= $key . '="' . $parameter . '" ';
+            $extra .= $key.'="'.$parameter.'" ';
         }
-        $html .= '<select name="' . $name . '" ' . $default_id . ' ' . $extra . '>';
+        $html .= '<select name="'.$name.'" '.$default_id.' '.$extra.'>';
 
         if ($show_blank_item) {
             if (empty($blank_item_text)) {
@@ -966,14 +966,14 @@ 
             } else {
                 $blank_item_text = Security::remove_XSS($blank_item_text);
             }
-            $html .= self::tag('option', '-- ' . $blank_item_text . ' --', array('value' => '-1'));
+            $html .= self::tag('option', '-- '.$blank_item_text.' --', array('value' => '-1'));
         }
         if ($values) {
             foreach ($values as $key => $value) {
                 if (is_array($value) && isset($value['name'])) {
                     $value = $value['name'];
                 }
-                $html .= '<option value="' . $key . '"';
+                $html .= '<option value="'.$key.'"';
 
                 if (is_array($default)) {
                     foreach ($default as $item) {
@@ -988,7 +988,7 @@ 
                     }
                 }
 
-                $html .= '>' . $value . '</option>';
+                $html .= '>'.$value.'</option>';
             }
         }
         $html .= '</select>';
@@ -1014,7 +1014,7 @@ 
      */
     public static function tabs($header_list, $content_list, $id = 'tabs', $attributes = array(), $ul_attributes = array())
     {
-        if (empty($header_list) || count($header_list) == 0 ) {
+        if (empty($header_list) || count($header_list) == 0) {
             return '';
         }
 
@@ -1051,7 +1051,7 @@ 
 
         $main_div = self::tag('div', $ul.self::tag('div', $divs, ['class' => 'tab-content']), $attributes);
 
-        return $main_div ;
+        return $main_div;
     }
 
     /**
@@ -1099,8 +1099,8 @@ 
      */
     public static function grid_html($div_id)
     {
-        $table  = self::tag('table','', array('id' => $div_id));
-        $table .= self::tag('div','', array('id' => $div_id.'_pager'));
+        $table  = self::tag('table', '', array('id' => $div_id));
+        $table .= self::tag('div', '', array('id' => $div_id.'_pager'));
         return $table;
     }
 
@@ -1161,7 +1161,7 @@ 
         $obj->colNames      = $column_names;
         $obj->colModel      = $column_model;
         $obj->pager         = '#'.$div_id.'_pager';
-        $obj->datatype  = 'json';
+        $obj->datatype = 'json';
         $obj->viewrecords = 'true';
 
         $all_value = 10000000;
@@ -1224,10 +1224,10 @@ 
         // Adding static data.
         if (!empty($data)) {
             $data_var = $div_id.'_data';
-            $json.=' var '.$data_var.' = '.json_encode($data).';';
+            $json .= ' var '.$data_var.' = '.json_encode($data).';';
             $obj->data = $data_var;
             $obj->datatype = 'local';
-            $json.="\n";
+            $json .= "\n";
         }
 
         $obj->end = 'end';
@@ -1241,20 +1241,20 @@ 
         }
 
         // Fixing true/false js values that doesn't need the ""
-        $json_encode = str_replace(':"true"',':true',$json_encode);
+        $json_encode = str_replace(':"true"', ':true', $json_encode);
         // wrap_cell is not a valid jqgrid attributes is a hack to wrap a text
         $json_encode = str_replace('"wrap_cell":true', 'cellattr : function(rowId, value, rowObject, colModel, arrData) { return \'class = "jqgrid_whitespace"\'; }', $json_encode);
-        $json_encode = str_replace(':"false"',':false',$json_encode);
+        $json_encode = str_replace(':"false"', ':false', $json_encode);
         $json_encode = str_replace('"formatter":"action_formatter"', 'formatter:action_formatter', $json_encode);
-        $json_encode = str_replace(array('{"first":"first",','"end":"end"}'), '', $json_encode);
+        $json_encode = str_replace(array('{"first":"first",', '"end":"end"}'), '', $json_encode);
         if (
             plagiarismActive()
             && plagiarismTool()
             == TOOL_COMPILATIO
-            && ( strpos($_SERVER['REQUEST_URI'], 'work/work.php')
+            && (strpos($_SERVER['REQUEST_URI'], 'work/work.php')
                 !== false
                 || strpos($_SERVER['REQUEST_URI'], 'work/work_list_all.php')
-                != false )
+                != false)
         ) {
             $json_encode = str_replace('"function () { compilatioInit() }"',
                 'function () { compilatioInit() }',
@@ -1263,7 +1263,7 @@ 
         }
 
         // Creating the jqgrid element.
-        $json .= '$("#' . $div_id . '").jqGrid({';
+        $json .= '$("#'.$div_id.'").jqGrid({';
         //$json .= $beforeSelectRow;
 
         $json .= $json_encode;
@@ -1275,22 +1275,22 @@ 
             $groups = '';
             foreach ($extra_params['groupHeaders'] as $group) {
                 //{ "startColumnName" : "courses", "numberOfColumns" : 1, "titleText" : "Order Info" },
-                $groups .= '{ "startColumnName" : "' . $group['startColumnName'] . '", "numberOfColumns" : ' . $group['numberOfColumns'] . ', "titleText" : "' . $group['titleText']  . '" },';
+                $groups .= '{ "startColumnName" : "'.$group['startColumnName'].'", "numberOfColumns" : '.$group['numberOfColumns'].', "titleText" : "'.$group['titleText'].'" },';
 
             }
             $json .= '$("#'.$div_id.'").jqGrid("setGroupHeaders", {
                 "useColSpanStyle" : false,
                 "groupHeaders"    : [
-                    ' . $groups . '
+                    ' . $groups.'
                 ]
             });';
         }
 
         $all_text = addslashes(get_lang('All'));
         $json .= '$("'.$obj->pager.' option[value='.$all_value.']").text("'.$all_text.'");';
-        $json.= "\n";
+        $json .= "\n";
         // Adding edit/delete icons.
-        $json.= $formatter;
+        $json .= $formatter;
 
         return $json;
     }
@@ -1321,7 +1321,7 @@ 
         }
 
         if (!empty($rows)) {
-	        foreach($rows as $content) {
+	        foreach ($rows as $content) {
 	            $table->setCellContents($row, $column, $content);
                 $row++;
             }
@@ -1483,7 +1483,7 @@ 
 
         // Show all tool icons where there is something new.
         $return = '&nbsp;';
-        foreach($notifications as $notification) {
+        foreach ($notifications as $notification) {
             $lastDate = date('d/m/Y H:i', convert_sql_date($notification['lastedit_date']));
             $type = $notification['lastedit_type'];
             $label = get_lang('TitleNotification').": ".get_lang($type)." ($lastDate)";
@@ -1526,9 +1526,9 @@ 
             $tbl_session            = Database :: get_main_table(TABLE_MAIN_SESSION);
             $active = false;
             // Request for the name of the general coach
-            $sql ='SELECT tu.lastname, tu.firstname, ts.*
+            $sql = 'SELECT tu.lastname, tu.firstname, ts.*
                     FROM '.$tbl_session.' ts
-                    LEFT JOIN '.$main_user_table .' tu
+                    LEFT JOIN '.$main_user_table.' tu
                     ON ts.id_coach = tu.user_id
                     WHERE ts.id = '.intval($session_id);
             $rs = Database::query($sql);
@@ -1538,7 +1538,7 @@ 
             $session = array();
             $session['title'] = $session_info['name'];
             $session['coach'] = '';
-            $session['dates'] =  '';
+            $session['dates'] = '';
 
             if (
                 (
@@ -1620,8 +1620,8 @@ 
                     );
                 }
 
-                if ( api_get_setting('show_session_coach') === 'true' ) {
-                    $session['coach'] = get_lang('GeneralCoach') . ': ' . api_get_person_name(
+                if (api_get_setting('show_session_coach') === 'true') {
+                    $session['coach'] = get_lang('GeneralCoach').': '.api_get_person_name(
                         $session_info['firstname'],
                         $session_info['lastname']
                     );
@@ -1673,9 +1673,9 @@ 
 		$percentage = isset($point_info['point_average']) ? $point_info['point_average'] : 0;
 
 		if (!empty($percentage)) {
-            $percentage = $percentage*125/100;
+            $percentage = $percentage * 125 / 100;
         }
-		$accesses =  isset($point_info['accesses']) ? $point_info['accesses'] : 0;
+		$accesses = isset($point_info['accesses']) ? $point_info['accesses'] : 0;
 
 		$star_label = sprintf(get_lang('XStarsOutOf5'), $point_info['point_average_star']);
 
@@ -1690,19 +1690,19 @@ 
 
 		$labels = array();
 
-		$labels[]= $number_of_users_who_voted == 1 ? $number_of_users_who_voted.' '.get_lang('Vote') : $number_of_users_who_voted.' '.get_lang('Votes');
-		$labels[]= $accesses == 1 ? $accesses.' '.get_lang('Visit') : $accesses.' '.get_lang('Visits');
+		$labels[] = $number_of_users_who_voted == 1 ? $number_of_users_who_voted.' '.get_lang('Vote') : $number_of_users_who_voted.' '.get_lang('Votes');
+		$labels[] = $accesses == 1 ? $accesses.' '.get_lang('Visit') : $accesses.' '.get_lang('Visits');
 		if (!empty($number_of_users_who_voted)) {
-			$labels[]= get_lang('Average').' '.$point_info['point_average_star'].'/5';
+			$labels[] = get_lang('Average').' '.$point_info['point_average_star'].'/5';
 		}
 
-		$labels[]= $point_info['user_vote']  ? get_lang('YourVote').' ['.$point_info['user_vote'].']' : get_lang('YourVote'). ' [?] ';
+		$labels[] = $point_info['user_vote'] ? get_lang('YourVote').' ['.$point_info['user_vote'].']' : get_lang('YourVote').' [?] ';
 
 		if (!$add_div_wrapper && api_is_anonymous()) {
-			$labels[]= Display::tag('span', get_lang('LoginToVote'), array('class' => 'error'));
+			$labels[] = Display::tag('span', get_lang('LoginToVote'), array('class' => 'error'));
 		}
 
-        $html .= Display::div(implode(' | ', $labels) , array('id' =>  'vote_label_'.$id, 'class' => 'vote_label_info'));
+        $html .= Display::div(implode(' | ', $labels), array('id' =>  'vote_label_'.$id, 'class' => 'vote_label_info'));
         $html .= ' '.Display::span(' ', array('id' =>  'vote_label2_'.$id));
 
         if ($add_div_wrapper) {
@@ -1780,8 +1780,8 @@ 
         if (!empty($list)) {
             $html = '<dl class="dl-horizontal">';
             foreach ($list as $item) {
-                $html .= '<dt>' . $item['title'] . '</dt>';
-                $html .= '<dd>' . $item['content'] . '</dd>';
+                $html .= '<dt>'.$item['title'].'</dt>';
+                $html .= '<dd>'.$item['content'].'</dd>';
             }
             $html .= '</dl>';
         }
@@ -1823,7 +1823,7 @@ 
      * @param string $type
      * @return null|string
      */
-    public static function badge($count, $type ="warning")
+    public static function badge($count, $type = "warning")
     {
         $class = '';
 
@@ -1899,7 +1899,7 @@ 
         if (!empty($content)) {
             $html = '<span class="label '.$class.'">';
             $html .= $content;
-            $html .='</span>';
+            $html .= '</span>';
         }
 
         return $html;
@@ -1920,7 +1920,7 @@ 
                     $class = 'class ="active"';
                 }
 
-                if (basename($_SERVER['REQUEST_URI']) == basename($value['url']) ) {
+                if (basename($_SERVER['REQUEST_URI']) == basename($value['url'])) {
                     $class = 'class ="active"';
                 }
                 $html .= "<li $class >";
@@ -2171,7 +2171,7 @@ 
         }
         $label = $nextValue + $localCounter + 1;
         if ($isMedia) {
-            $label = ($fixedValue + 1) .' '.chr(97 + $localCounter);
+            $label = ($fixedValue + 1).' '.chr(97 + $localCounter);
             $link_to_show = $link.$fixedValue.'#questionanchor'.$itemId;
         }
         $link = Display::url($label.' ', $link_to_show, $linkAttributes);
@@ -2283,7 +2283,7 @@ 
      */
     public static function getVCardUserLink($userId)
     {
-        $vCardUrl = api_get_path(WEB_PATH).'main/social/vcard_export.php?userId='.intval($userId);;
+        $vCardUrl = api_get_path(WEB_PATH).'main/social/vcard_export.php?userId='.intval($userId); ;
 
         return $vCardUrl;
     }
@@ -2301,7 +2301,7 @@ 
     {
         $title = !empty($title) ? '<div class="panel-heading"><h3 class="panel-title">'.$title.'</h3>'.$extra.'</div>' : '';
         $footer = !empty($footer) ? '<div class="panel-footer ">'.$footer.'</div>' : '';
-        $styles = ['primary','success','info','warning','danger'];
+        $styles = ['primary', 'success', 'info', 'warning', 'danger'];
         $style = !in_array($style, $styles) ? 'default' : $style;
 
         return '
@@ -2346,7 +2346,7 @@ 
         $attributes['class'] = isset($attributes['class']) ? "$buttonClass {$attributes['class']}" : $buttonClass;
 
         if (!$includeText) {
-            $text = '<span class="sr-only">' . $text . '</span>';
+            $text = '<span class="sr-only">'.$text.'</span>';
         }
 
         return self::url("$icon $text", $url, $attributes);
@@ -2361,16 +2361,16 @@ 
      */
     public static function toolbarAction($id, $content = array(), $col = 2, $right = true)
     {
-        $columns = 12/$col;
+        $columns = 12 / $col;
         $html = '';
-        $html .= '<div id="' . $id . '" class="actions">';
+        $html .= '<div id="'.$id.'" class="actions">';
         $html .= '<div class="row">';
         if ($col > 4) {
             $html = '<div class="alert alert-warning" role="alert">Action toolbar design does not work when exceeding four columns - check Display::toolbarAction()</div>';
         } else {
-            for ( $i = 0; $i < $col; $i++ ) {
-                $html .= '<div class="col-md-' . $columns . '">';
-                if ( $col == 2 && $i == 1 ) {
+            for ($i = 0; $i < $col; $i++) {
+                $html .= '<div class="col-md-'.$columns.'">';
+                if ($col == 2 && $i == 1) {
                     if ($right === true) {
                         $html .= '<div class="pull-right">';
                         $html .= (isset($content[$i]) ? $content[$i] : '');
@@ -2485,9 +2485,9 @@ 
             $params['class'] = 'panel panel-default';
             $html = null;
             if (!empty($title)) {
-                $html .= '<div class="panel-heading">'.$title.'</div>' . PHP_EOL;
+                $html .= '<div class="panel-heading">'.$title.'</div>'.PHP_EOL;
             }
-            $html.= '<div class="panel-body">'.$content.'</div>' . PHP_EOL;
+            $html .= '<div class="panel-body">'.$content.'</div>'.PHP_EOL;
             $html = Display::div($html, $params);
         }
         return $html;

main/work/work_list_all.php

Spacing +12 added lines, -12 removed lines

@@ -4,7 +4,7 @@ 
 use ChamiloSession as Session;
 
 require_once '../inc/global.inc.php';
-$current_course_tool  = TOOL_STUDENTPUBLICATION;
+$current_course_tool = TOOL_STUDENTPUBLICATION;
 
 api_protect_course_script(true);
 
@@ -44,7 +44,7 @@ 
 $user_id = api_get_user_id();
 
 if (!empty($group_id)) {
-    $group_properties  = GroupManager :: get_group_properties($group_id);
+    $group_properties = GroupManager :: get_group_properties($group_id);
     $show_work = false;
 
     if (api_is_allowed_to_edit(false, true)) {
@@ -161,7 +161,7 @@ 
     $actionsLeft .= '<a href="'.api_get_path(WEB_CODE_PATH).'work/add_document.php?'.api_get_cidreq().'&id='.$workId.'">';
     $actionsLeft .= Display::return_icon('new_document.png', get_lang('AddDocument'), '', ICON_SIZE_MEDIUM).'</a>';
 
-    $actionsLeft .=  '<a href="'.api_get_path(WEB_CODE_PATH).'work/add_user.php?'.api_get_cidreq().'&id='.$workId.'">';
+    $actionsLeft .= '<a href="'.api_get_path(WEB_CODE_PATH).'work/add_user.php?'.api_get_cidreq().'&id='.$workId.'">';
     $actionsLeft .= Display::return_icon('addworkuser.png', get_lang('AddUsers'), '', ICON_SIZE_MEDIUM).'</a>';
 
     $actionsLeft .= '<a href="'.api_get_path(WEB_CODE_PATH).'work/work_list_all.php?'.api_get_cidreq().'&id='.$workId.'&action=export_pdf">';
@@ -190,7 +190,7 @@ 
 $plagiarismListJqgridLine = array();
 if (plagiarismActive()) {
     if (plagiarismTool() == TOOL_COMPILATIO) {
-        require_once(api_get_path(SYS_CODE_PATH) . "/plagiarism/compilatio/config.php");
+        require_once(api_get_path(SYS_CODE_PATH)."/plagiarism/compilatio/config.php");
         $plagiarismTable = Database :: get_course_table(TABLE_COMPILATIO);
         $plagiarismListJqgridColumn = array('Compilatio');
         $plagiarismListJqgridLine = array(
@@ -449,7 +449,7 @@ 
             . "gbox_results"
             .  '&amp;'
             . 'selectall=1" onclick="javascript: setCheckbox(true, \''
-            . "gbox_results". '\'); return false;">'
+            . "gbox_results".'\'); return false;">'
             . get_lang('SelectAll')
             . '</a>';
         $html .= '<a class="btn btn-default" href="?'
@@ -472,7 +472,7 @@ 
 //            $html .= '<li><a data-action ="' . $action . '" href="#" onclick="javascript:action_click(this, \'' . "gbox_results" . '\');">' . $label . '</a></li>';
 //        }
         /*fin original*/
-        foreach ($form_action as $action ) {
+        foreach ($form_action as $action) {
             $html .= '<li><a data-action ="'
                 . $action["data-action"]
                 . '" href="#" '
@@ -482,7 +482,7 @@ 
                 . '</a></li>';
         }
         $html .= '</ul>';
-        $html .= '</div>';//btn-group
+        $html .= '</div>'; //btn-group
         $html .= '</div>'; //toolbar
     }
     echo $html;
@@ -491,19 +491,19 @@ 
     $compTable = Database::get_course_table("plagiarism_compilatio_docs");
 
     $listCompilatioDocId = array();
-    $compilatioQuery = "SELECT * FROM " . $compTable . " WHERE c_id= " . $courseId;
+    $compilatioQuery = "SELECT * FROM ".$compTable." WHERE c_id= ".$courseId;
     $compilatioResult = Database::query($compilatioQuery);
     while ($compilatioData = Database::fetch_array($compilatioResult)) {
         array_push($listCompilatioDocId, $compilatioData['id_doc']);
     }
     $javascriptWorkId = "";
-    $sql = "SELECT * FROM " . $workTable . " WHERE c_id= " . $courseId . " AND parent_id= ". $workId. " AND active=1";
+    $sql = "SELECT * FROM ".$workTable." WHERE c_id= ".$courseId." AND parent_id= ".$workId." AND active=1";
     file_put_contents("/tmp/aaaSql", print_r($sql, 1)); // IDZ
     $res = Database::query($sql);
 
     while ($data = Database::fetch_array($res)) {
         if (in_array($data['id'], $listCompilatioDocId)) {
-            $javascriptWorkId .= $data['id'] . "a";
+            $javascriptWorkId .= $data['id']."a";
         }
     }
 }
@@ -527,7 +527,7 @@ 
     var refreshDelaisAfter = 30000;
     var allWorkId = "<?php echo $javascriptWorkId?>";
     var workFolderId = "<?php echo $workId; ?>";
-    var compilationWebUrl = "<?php echo api_get_path(WEB_CODE_PATH) . 'plagiarism/compilatio/' ?>";
+    var compilationWebUrl = "<?php echo api_get_path(WEB_CODE_PATH).'plagiarism/compilatio/' ?>";
     var divExisteTimer = null;
     var msgWaitJS = '<?php echo Display::display_confirmation_message(get_lang('PleaseWaitThisCouldTakeAWhile')); ?>';
     var div = document.createElement('div');
@@ -656,7 +656,7 @@ 
             type: "get",
             dataType: "html",
             error: function(){
-                showData("<?php echo get_lang('compilatioComunicationAjaxImpossible');?>");
+                showData("<?php echo get_lang('compilatioComunicationAjaxImpossible'); ?>");
             },
             success: function( strData ){
                 showData( strData );

main/work/work.php

Spacing +12 added lines, -12 removed lines

@@ -8,7 +8,7 @@ 
 use ChamiloSession as Session;
 
 require_once '../inc/global.inc.php';
-$current_course_tool  = TOOL_STUDENTPUBLICATION;
+$current_course_tool = TOOL_STUDENTPUBLICATION;
 
 api_protect_course_script(true);
 
@@ -39,7 +39,7 @@ 
 $item_id = isset($_REQUEST['item_id']) ? intval($_REQUEST['item_id']) : null;
 $origin = isset($_REQUEST['origin']) ? Security::remove_XSS($_REQUEST['origin']) : '';
 $course_dir = api_get_path(SYS_COURSE_PATH).$_course['path'];
-$base_work_dir = $course_dir . '/work';
+$base_work_dir = $course_dir.'/work';
 $action = isset($_REQUEST['action']) ? $_REQUEST['action'] : 'list';
 
 //Download folder
@@ -76,7 +76,7 @@ 
         'url' => api_get_path(WEB_CODE_PATH).'work/work.php?'.api_get_cidreq(),
         'name' => get_lang('StudentPublications'),
     );
-    $url_dir = api_get_path(WEB_CODE_PATH).'work/work.php?&id=' . $work_id.'&'.api_get_cidreq();
+    $url_dir = api_get_path(WEB_CODE_PATH).'work/work.php?&id='.$work_id.'&'.api_get_cidreq();
     if (!empty($my_folder_data)) {
         $interbreadcrumb[] = array('url' => $url_dir, 'name' =>  $my_folder_data['title']);
     }
@@ -119,7 +119,7 @@ 
             $interbreadcrumb[] = array('url' => '#', 'name' => get_lang('EditToolOptions'));
         }
         if ($action == 'create_dir') {
-            $interbreadcrumb[] = array('url' => '#','name' => get_lang('CreateAssignment'));
+            $interbreadcrumb[] = array('url' => '#', 'name' => get_lang('CreateAssignment'));
         }
     }
 }
@@ -170,7 +170,7 @@ 
         $form = new FormValidator(
             'form1',
             'post',
-            api_get_path(WEB_CODE_PATH) . 'work/work.php?action=create_dir&' . api_get_cidreq()
+            api_get_path(WEB_CODE_PATH).'work/work.php?action=create_dir&'.api_get_cidreq()
         );
         $form->addElement('header', get_lang('CreateAssignment'));
         $form->addElement('hidden', 'action', 'add');
@@ -207,7 +207,7 @@ 
             $result = deleteDirWork($_REQUEST['id']);
             if ($result) {
                 $message = Display::return_message(
-                    get_lang('DirDeleted') . ': ' . $work_to_delete['title'],
+                    get_lang('DirDeleted').': '.$work_to_delete['title'],
                     'success'
                 );
                 Session::write('message', $message);
@@ -235,19 +235,19 @@ 
         if ($is_allowed_to_edit) {
             $move_to_path = get_work_path($_REQUEST['move_to_id']);
 
-            if ($move_to_path==-1) {
+            if ($move_to_path == -1) {
                 $move_to_path = '/';
             } elseif (substr($move_to_path, -1, 1) != '/') {
-                $move_to_path = $move_to_path .'/';
+                $move_to_path = $move_to_path.'/';
             }
 
             // Security fix: make sure they can't move files that are not in the document table
             if ($path = get_work_path($item_id)) {
-                if (move($course_dir.'/'.$path, $base_work_dir . $move_to_path)) {
+                if (move($course_dir.'/'.$path, $base_work_dir.$move_to_path)) {
                     // Update db
                     updateWorkUrl(
                         $item_id,
-                        'work' . $move_to_path,
+                        'work'.$move_to_path,
                         $_REQUEST['move_to_id']
                     );
 
@@ -345,7 +345,7 @@ 
             $content .= '</div>';
             $content .= '</div>';
             $content .= '<div id="student-list-work" style="display: none" class="table-responsive">';
-            $content .= '<div class="toolbar"><a id="closed-view-list" href="#"><em class="fa fa-times-circle"></em> ' .get_lang('Close'). '</a></div>';
+            $content .= '<div class="toolbar"><a id="closed-view-list" href="#"><em class="fa fa-times-circle"></em> '.get_lang('Close').'</a></div>';
             $content .= showStudentList($work_id);
             $content .= '</div>';
         } else {
@@ -390,7 +390,7 @@ 
     var refreshDelaisAfter = 30000; // en ms pour chercher le % avancement de la detection
     var allWorkId = "<?php echo $javascriptWorkId?>";	// list of workid to check 1a5a6a7a8a9a10a11
     var workFolderId = "<?php echo $workId; ?>";
-    var compilationWebUrl = "<?php echo api_get_path(WEB_CODE_PATH) . 'plagiarism/compilatio/' ?>";
+    var compilationWebUrl = "<?php echo api_get_path(WEB_CODE_PATH).'plagiarism/compilatio/' ?>";
     var divExisteTimer = null;
 
     function compilatioInit() {

main/admin/configure_extensions.php

Spacing +18 added lines, -18 removed lines

@@ -4,11 +4,11 @@ 
  * Edition of extensions configuration
  * @package chamilo.admin
  */
-$cidReset=true;
+$cidReset = true;
 require_once '../inc/global.inc.php';
-$this_section=SECTION_PLATFORM_ADMIN;
+$this_section = SECTION_PLATFORM_ADMIN;
 api_protect_admin_script();
-$interbreadcrumb[] = array ('url' => 'index.php', 'name' => get_lang('PlatformAdmin'));
+$interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('PlatformAdmin'));
 // Database Table Definitions
 $tbl_settings_current = Database::get_main_table(TABLE_MAIN_SETTINGS_CURRENT);
 $message = '';
@@ -21,16 +21,16 @@ 
 					WHERE variable="service_visio"
 					AND subkey="active"';
 			$rs = Database::query($sql);
-			if (Database::affected_rows($rs)>0) {
+			if (Database::affected_rows($rs) > 0) {
 				// select all the courses and insert the tool inside
 				$sql = 'SELECT id FROM '.Database::get_main_table(TABLE_MAIN_COURSE);
 				$rs = Database::query($sql);
-				while($row = Database::fetch_array($rs)){
-					if(!empty($_POST['visio_host'])) {
+				while ($row = Database::fetch_array($rs)) {
+					if (!empty($_POST['visio_host'])) {
 						$tool_table = Database::get_course_table(TABLE_TOOL_LIST);
 						$select = "SELECT id FROM $tool_table WHERE c_id =".$row['id']." AND name='".TOOL_VISIO_CONFERENCE."'";
 						$selectres = Database::query($select);
-						if (Database::num_rows($selectres)<1) {
+						if (Database::num_rows($selectres) < 1) {
 							$sql = 'INSERT INTO '.$tool_table.' SET
 							        c_id =  '.$row['id'].',
 									name="'.TOOL_VISIO_CONFERENCE.'",
@@ -45,7 +45,7 @@ 
 						}
 						$select = "SELECT id FROM $tool_table WHERE c_id =".$row['id']." AND name='".TOOL_VISIO_CLASSROOM."'";
 						$selectres = Database::query($select);
-						if(Database::num_rows($selectres)<1) {
+						if (Database::num_rows($selectres) < 1) {
 							$sql = 'INSERT INTO '.$tool_table.' SET
 							        c_id =  '.$row['id'].',
 									name="'.TOOL_VISIO_CLASSROOM.'",
@@ -82,12 +82,12 @@ 
 			$rs = Database::query($sql);
 
 			$sql = 'UPDATE '.$tbl_settings_current.' SET
-					selected_value="'.($_POST['visio_use_rtmpt']=='true'?'true':'false').'"
+					selected_value="'.($_POST['visio_use_rtmpt'] == 'true' ? 'true' : 'false').'"
 					WHERE variable="service_visio"
 					AND subkey="visio_use_rtmpt"';
 			$rs = Database::query($sql);
 
-			if(empty($message)) {
+			if (empty($message)) {
 				$message = get_lang('ServiceReconfigured');
 			}
 			break;
@@ -100,7 +100,7 @@ 
 
 			$rs = Database::query($sql);
 
-			if (Database::affected_rows($rs)>0){
+			if (Database::affected_rows($rs) > 0) {
 				$message = get_lang('ServiceActivated');
 			}
 
@@ -153,20 +153,20 @@ 
 		WHERE variable LIKE "service_%" AND subkey="active" and selected_value="true"';
 
 $rs = Database::query($sql);
-while($row = Database::fetch_array($rs)){
+while ($row = Database::fetch_array($rs)) {
 	$listActiveServices[] = $row['variable'];
 }
 
 // javascript to handle accordion behaviour
 $javascript_message = '';
-if(!empty($message)){
+if (!empty($message)) {
 	$javascript_message =
 	'
 	document.getElementById("message").style.display = "block";
 	var timer = setTimeout(hideMessage,5000);
 	';
 }
-$htmlHeadXtra[]= '
+$htmlHeadXtra[] = '
 <script type="text/javascript">
 var listeDiv;
 var extensionsHeader = new Array();
@@ -215,7 +215,7 @@ 
 ?>
 <div id="message" style="display: none">
 	<?php
-	if(!empty($message))
+	if (!empty($message))
 		Display::display_normal_message($message)
 	?>
 </div>
@@ -330,13 +330,13 @@ 
 						$renderer = $form -> defaultRenderer();
 						$renderer -> setElementTemplate('<div style="text-align:left">{label}</div><div style="text-align:left">{element}</div>');
 						//$form -> addElement('html','<br /><br />');
-						if(in_array('service_ppt2lp',$listActiveServices))
+						if (in_array('service_ppt2lp', $listActiveServices))
 						{
 							$sql = 'SELECT subkey, selected_value FROM '.$tbl_settings_current.'
 									WHERE variable = "service_ppt2lp"
 									AND subkey <> "active"';
 							$rs = Database::query($sql);
-							while($row = Database::fetch_array($rs,'ASSOC'))
+							while ($row = Database::fetch_array($rs, 'ASSOC'))
 							{
 								$defaults[$row['subkey']] = $row['selected_value'];
 							}
@@ -502,7 +502,7 @@ 
 <!-- COMPILATIO -->
 <div id="main_compilatio">
     <div id="extension_header_compilatio" class="accordion_header">
-        <a href="#"><?php echo get_lang('compilatioPlagiarismDetector');?></a>
+        <a href="#"><?php echo get_lang('compilatioPlagiarismDetector'); ?></a>
     </div>
     <div id="extension_content_compilatio" style="display:none;padding:0;width:780px;" class="accordion_content">
         <table width="100%" cellpadding="0" cellspacing="0">

main/plagiarism/compilatio/download.php

Indentation +12 added lines, -12 removed lines

@@ -29,31 +29,31 @@
 $course_info = api_get_course_info();

 

 if (empty($course_info)) {

-	api_not_allowed(true);

+    api_not_allowed(true);

 }

 

 $tbl_student_publication = Database::get_course_table(TABLE_STUDENT_PUBLICATION);

 

 if (!empty($course_info['real_id'])) {    

-	$sql = 'SELECT * FROM '.$tbl_student_publication.' WHERE c_id = '.$course_info['real_id'].' AND id = "'.$id.'"';

-	$result = Database::query($sql);

-	if ($result && Database::num_rows($result)) {

-	    $row = Database::fetch_array($result, 'ASSOC');        

+    $sql = 'SELECT * FROM '.$tbl_student_publication.' WHERE c_id = '.$course_info['real_id'].' AND id = "'.$id.'"';

+    $result = Database::query($sql);

+    if ($result && Database::num_rows($result)) {

+        $row = Database::fetch_array($result, 'ASSOC');        

         $full_file_name = api_get_path(SYS_COURSE_PATH).api_get_course_path().'/'.$row['url'];

            

         $item_info = api_get_item_property_info(api_get_course_int_id(), 'work', $row['id']);           

         if (empty($item_info)) {

             exit;

         }          	    

-	    if ($course_info['show_score'] == 0 || $item_info['visibility'] == 1 && $row['accepted'] == 1 && ($row['user_id'] == api_get_user_id() || api_is_allowed_to_edit())) {

-		    $title = str_replace(' ', '_', $row['title']);

+        if ($course_info['show_score'] == 0 || $item_info['visibility'] == 1 && $row['accepted'] == 1 && ($row['user_id'] == api_get_user_id() || api_is_allowed_to_edit())) {

+            $title = str_replace(' ', '_', $row['title']);

             event_download($title);

-		    if (Security::check_abs_path($full_file_name, api_get_path(SYS_COURSE_PATH).api_get_course_path().'/')) {

-		        DocumentManager::file_send_for_download($full_file_name, true, $title);

-		    }

-	    } else {

+            if (Security::check_abs_path($full_file_name, api_get_path(SYS_COURSE_PATH).api_get_course_path().'/')) {

+                DocumentManager::file_send_for_download($full_file_name, true, $title);

+            }

+        } else {

             api_not_allowed();

         }	    

-	}

+    }

 }

 exit;
\ No newline at end of file

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
 

 session_cache_limiter('public');

 require_once '../inc/global.inc.php';

-$current_course_tool  = TOOL_STUDENTPUBLICATION;

+$current_course_tool = TOOL_STUDENTPUBLICATION;

 

 $this_section = SECTION_COURSES;

 

main/plagiarism/compilatio/compiladmin.php

Indentation +42 added lines, -42 removed lines

@@ -1,62 +1,62 @@ 
 <?php

-  /*

+    /*

   Module Compilatio v0.9 pour Dokeos

   */

-  /*

+    /*

    * admin's section:  allow to see the credit's quota and do a SOAP's connection test

    * Partie admin: permet de voir le quotas de cr?dit et de faire un test de connexion SOAP

   */

-  ini_set('soap.wsdl_cache_enabled', 0);

-  ini_set('default_socket_timeout', '1000');

+    ini_set('soap.wsdl_cache_enabled', 0);

+    ini_set('default_socket_timeout', '1000');

 

-  require_once '../../inc/lib/api.lib.php';

-  require_once '../compilatio/compilatio.class.php';

-  require_once '../compilatio/config.php';

+    require_once '../../inc/lib/api.lib.php';

+    require_once '../compilatio/compilatio.class.php';

+    require_once '../compilatio/config.php';

 

-  $compilatio = new compilatio(

-      $compilatioParameter['key'],

-      $compilatioParameter['$urlsoap'],

-      $compilatioParameter['proxy_host'],

-      $compilatioParameter['proxy_port']

-  );

-  $use_space = number_format($quotas->usedSpace/1000000, 2);

-  $total_space=$quotas->space/1000000;

+    $compilatio = new compilatio(

+        $compilatioParameter['key'],

+        $compilatioParameter['$urlsoap'],

+        $compilatioParameter['proxy_host'],

+        $compilatioParameter['proxy_port']

+    );

+    $use_space = number_format($quotas->usedSpace/1000000, 2);

+    $total_space=$quotas->space/1000000;

 

-  echo "<h3>" . get_lang('compilatioDescription') . "</h3>";

+    echo "<h3>" . get_lang('compilatioDescription') . "</h3>";

 

-  echo "<b>"

-      . get_lang('compilatioQuota')

-      . ":"

-      . " </b><br>"

-      . get_lang('compilatioCredit')

-      . ": "

-      . $quotas->usedCredits

-      . get_lang('compilatioOn')

-      . $quotas->credits;

+    echo "<b>"

+        . get_lang('compilatioQuota')

+        . ":"

+        . " </b><br>"

+        . get_lang('compilatioCredit')

+        . ": "

+        . $quotas->usedCredits

+        . get_lang('compilatioOn')

+        . $quotas->credits;

 

 ?>

 <br><br>

 <?

-  if(!isset($_GET['action']))

-  {

+    if(!isset($_GET['action']))

+    {

 ?>

 <body style="margin:0px;padding:0px">

 <form style="margin:0px;" method="GET">

   <input type="submit" name="action" value="Test de Connexion SOAP">

 </form>

 <?

-  } else {

-  echo get_lang('compilatioConnectionTestSoap')."<br>";

-  echo "1) ".get_lang('compilatioServerConnection')."<br>";

-  $compilatio = new compilatio(

-      $compilatioParameter['key'],

-      $compilatioParameter['$urlsoap'],

-      $compilatioParameter['proxy_host'],

-      $compilatioParameter['proxy_port']

-  );

+    } else {

+    echo get_lang('compilatioConnectionTestSoap')."<br>";

+    echo "1) ".get_lang('compilatioServerConnection')."<br>";

+    $compilatio = new compilatio(

+        $compilatioParameter['key'],

+        $compilatioParameter['$urlsoap'],

+        $compilatioParameter['proxy_host'],

+        $compilatioParameter['proxy_port']

+    );

 

-  if ($compilatio)

-  {

+    if ($compilatio)

+    {

     echo get_lang('compilatioConnectionAccomplished')."<br>";

     echo "2) ".get_lang('compilatioSendTextToServer')."<br>";

     $text = get_lang('compilatioTestSendText'). $compilatioParameter['key'];

@@ -68,14 +68,14 @@ 
         $text

     );

     if (isMd5($id_compi)) {

-      echo get_lang('compilatioSuccessfulTransfer')."<br>";

+        echo get_lang('compilatioSuccessfulTransfer')."<br>";

     } else {

-      echo get_lang('compilatioFailedTransfer')."<br>";

+        echo get_lang('compilatioFailedTransfer')."<br>";

     }

-  } else {

+    } else {

     echo get_lang('compilatioNotConnection')."<br>";

     echo get_lang('compilatioParamVerification')."<br>";

-  }

+    }

 }

 ?>

 </body>

Spacing +5 added lines, -5 removed lines

@@ -19,10 +19,10 @@ 
       $compilatioParameter['proxy_host'],

       $compilatioParameter['proxy_port']

   );

-  $use_space = number_format($quotas->usedSpace/1000000, 2);

-  $total_space=$quotas->space/1000000;

+  $use_space = number_format($quotas->usedSpace / 1000000, 2);

+  $total_space = $quotas->space / 1000000;

 

-  echo "<h3>" . get_lang('compilatioDescription') . "</h3>";

+  echo "<h3>".get_lang('compilatioDescription')."</h3>";

 

   echo "<b>"

       . get_lang('compilatioQuota')

@@ -37,7 +37,7 @@ 
 ?>

 <br><br>

 <?

-  if(!isset($_GET['action']))

+  if (!isset($_GET['action']))

   {

 ?>

 <body style="margin:0px;padding:0px">

@@ -59,7 +59,7 @@ 
   {

     echo get_lang('compilatioConnectionAccomplished')."<br>";

     echo "2) ".get_lang('compilatioSendTextToServer')."<br>";

-    $text = get_lang('compilatioTestSendText'). $compilatioParameter['key'];

+    $text = get_lang('compilatioTestSendText').$compilatioParameter['key'];

     $id_compi = $compilatio->SendDoc(

         'Doc de test',

         'test',