adamjakab / SuiteCRM

include/SugarFolders/SugarFolders.php

Doc Comments +7 added lines

@@ -126,6 +126,8 @@ 
 	 * @param string fromFolder GUID of source folder
 	 * @param string toFolder GUID of destination folder
 	 * @param string beanId GUID of SugarBean being moved
+	 * @param string $fromFolder
+	 * @param string $toFolder
 	 */
 	function move($fromFolder, $toFolder, $beanId) {
 		$q = "UPDATE folders_rel SET folder_id = '{$toFolder}' WHERE folder_id = '{$fromFolder}' AND polymorphic_id = '{$beanId}' AND deleted = 0";
@@ -134,6 +136,9 @@ 
 
 	/**
 	 * Copies one bean from one folder to another
+	 * @param string $fromFolder
+	 * @param string $toFolder
+	 * @param string $module
 	 */
 	function copyBean($fromFolder, $toFolder, $beanId, $module) {
 		$q = "INSERT INTO folders_rel (id, folder_id, polymorphic_module, polymorphic_id, deleted)
@@ -448,6 +453,7 @@ 
 
 	/**
 	 * Convenience method, pass a SugarBean and User to this to add anything to a given folder
+	 * @param Email $bean
 	 */
 	function addBean($bean, $user=null) {
 		if(empty($bean->id) || empty($bean->module_dir)) {
@@ -742,6 +748,7 @@ 
 
 	/**
 	 * Builds children nodes for folders for TreeView
+	 * @param string $nodePath
 	 * @return $folderNode TreeView node
 	 */
 	function buildTreeNodeFolders($a, $nodePath, $folderStates, $subscriptions) {

include/SugarLogger/LoggerTemplate.php

Doc Comments +1 added lines, -1 removed lines

@@ -47,8 +47,8 @@
     /**
      * Main method for handling logging a message to the logger
      *
-     * @param string $level logging level for the message
      * @param string $message
+     * @return void
      */
     public function log(
         $method,

include/SugarLogger/SugarLogger.php

Doc Comments +1 added lines

@@ -97,6 +97,7 @@
 
     /**
      * Used by the diagnostic tools to get SugarLogger log file information
+     * @return string
      */
     public function getLogFileNameWithPath()
     {

include/SugarOAuthServer.php

Doc Comments -1 removed lines

@@ -158,7 +158,6 @@
      * Create OAuth provider
      *
      * Checks current request for OAuth valitidy
-     * @param bool $add_rest add REST endpoint as request path
      */
     public function __construct($req_path = '')
     {

include/SugarObjects/LanguageManager.php

Doc Comments +7 added lines

@@ -105,6 +105,10 @@ 
 		}
 	}
 
+	/**
+	 * @param string $module
+	 * @param string $lang
+	 */
 	static function saveCache($module,$lang, $loaded_mod_strings, $additonal_objects= array()){
 		if(empty($lang))
 			$lang = $GLOBALS['sugar_config']['default_language'];
@@ -226,6 +230,9 @@ 
 			LanguageManager::saveCache($module, $lang, $loaded_mod_strings);
 	}
 
+	/**
+	 * @param string $module
+	 */
 	static function loadModuleLanguage($module, $lang, $refresh=false){
 		//here check if the cache file exists, if it does then load it, if it doesn't
 		//then call refreshVardef

include/Sugarpdf/FontManager.php

Doc Comments +3 added lines, -3 removed lines

@@ -139,7 +139,7 @@ 
     /**
      * This method calculate the font size of $this->filename in KB
      * .php file + .z file + .ctg.z file
-     * @return Integer font Size in KB
+     * @return double font Size in KB
      */
     private function getFontSize(){
         $fileSize=filesize($this->fontPath.$this->filename);
@@ -326,8 +326,8 @@ 
     }
     /**
      * This method add a font to SugarCRM from a font file and a metric file using MakeFont()
-     * @param $font_file string
-     * @param $metric_file string
+     * @param string $font_file string
+     * @param string $metric_file string
      * @param $embedded boolean
      * @param $encoding_table string
      * @param $patch array

include/Sugarpdf/Sugarpdf.php

Doc Comments +7 added lines, -6 removed lines

@@ -263,7 +263,7 @@ 
     *
     * @param string $family Family font. It can be either a name defined by AddFont() or one of the standard Type1 families (case insensitive):<ul><li>times (Times-Roman)</li><li>timesb (Times-Bold)</li><li>timesi (Times-Italic)</li><li>timesbi (Times-BoldItalic)</li><li>helvetica (Helvetica)</li><li>helveticab (Helvetica-Bold)</li><li>helveticai (Helvetica-Oblique)</li><li>helveticabi (Helvetica-BoldOblique)</li><li>courier (Courier)</li><li>courierb (Courier-Bold)</li><li>courieri (Courier-Oblique)</li><li>courierbi (Courier-BoldOblique)</li><li>symbol (Symbol)</li><li>zapfdingbats (ZapfDingbats)</li></ul> It is also possible to pass an empty string. In that case, the current family is retained.
     * @param string $style Font style. Possible values are (case insensitive):<ul><li>empty string: regular</li><li>B: bold</li><li>I: italic</li><li>U: underline</li><li>D: line trough</li></ul> or any combination. The default value is regular. Bold and italic styles do not apply to Symbol and ZapfDingbats basic fonts or other fonts when not defined.
-    * @param float $size Font size in points. The default value is the current size. If no size has been specified since the beginning of the document, the value taken is 12
+    * @param integer $size Font size in points. The default value is the current size. If no size has been specified since the beginning of the document, the value taken is 12
     * @param string $fontfile The font definition file. By default, the name is built from the family and style, in lower case with no spaces.
     * @access public
     * @see include/tcpdf/TCPDF#SetFont()
@@ -299,6 +299,7 @@ 
      * the string to print in the PDF.
      * The cell method is used by all the methods which print text (Write, MultiCell).
      * @see include/tcpdf/TCPDF#Cell()
+     * @param integer $w
      */
     public function Cell($w, $h=0, $txt='', $border=0, $ln=0, $align='', $fill=0, $link='', $stretch=0) {
         parent::Cell($w, $h, prepare_string($txt), $border, $ln, $align, $fill, $link, $stretch);
@@ -425,7 +426,7 @@ 
      *              Array[line number (0 to x)][cell header]['options'] = Array[cell properties] = values
      * @param $returnHtml (bool) Return the table as HTML code instead of printing the HTML table
      * @param $options Array which can contain : table (array of "HTML proprty"=>"value"),td (array of "HTML proprty"=>"value"), tr (array of "HTML proprty"=>"value"), isheader(bool), header (array of "HTML proprty"=>"value"), width (array 'column name'=>'width value + unit OR nothing')
-     * @return the HTML code if $returnHtml set to true
+     * @return string|null HTML code if $returnHtml set to true
      */
     public function writeHTMLTable($item, $returnHtml=false, $options=NULL){
         //TODO ISSUE - width in % for the td have to be multiply by the number of column.
@@ -488,10 +489,10 @@ 
 
     /**
      * return the HTML code of the value wrap with the tag $tag. This method handle options (general and specific)
-     * @param $tag
+     * @param string $tag
      * @param $value
      * @param $options
-     * @return the HTML wrapped code
+     * @return string HTML wrapped code
      */
     private function wrap($tag, $value, $options){
         if(empty($options[$tag])){
@@ -515,7 +516,7 @@ 
      * Return the heigth of a line depending of the width, the font and the content
      * @param $line Array containing the data of all the cells of the line
      * @param $width Array containing the width of all the cells of the line
-     * @return The heigth of the line
+     * @return double heigth of the line
      */
     private function getLineHeightFromArray($line, $width){
         $h=0;
@@ -618,7 +619,7 @@ 
     * between words.
     * This method returns the estimated number of lines required to print the text.
     * @param string $txt text to print
-    * @param float $w width of cell. If 0, they extend up to the right margin of the page.
+    * @param integer $w width of cell. If 0, they extend up to the right margin of the page.
     * @return int Return the estimated number of lines.
     * @access public
     * @since 4.5.011

include/Sugarpdf/SugarpdfFactory.php

Doc Comments +6 added lines, -2 removed lines

@@ -45,7 +45,8 @@ 
     /**
      * load the correct Tcpdf
      * @param string $type Tcpdf Type
-     * @return valid Tcpdf
+     * @param string $module
+     * @return Sugarpdf Tcpdf
      */
     function loadSugarpdf($type = 'default', $module, $bean = null, $sugarpdf_object_map = array()){
         $type = strtolower(basename($type));
@@ -109,7 +110,9 @@ 
      * This is a private function which just helps the getSugarpdf function generate the
      * proper Tcpdf object
      * 
-     * @return a valid Sugarpdf
+     * @param string $file
+     * @param string $type
+     * @return Sugarpdf valid Sugarpdf
      */
     function _buildFromFile($file, &$bean, $sugarpdf_object_map, $type, $module){
         require_once($file);
@@ -132,6 +135,7 @@ 
      * @param object bean = the bean to pass to the Sugarpdf
      * @param array Sugarpdf_object_map - the array which holds obejcts to pass between the
      *                                controller and the tcpdf.
+     * @param string $class
      * 
      * @return Sugarpdf
      */

include/Sugarpdf/SugarpdfHelper.php

Doc Comments +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
 
     /**
      * This function prepare a string to be ready for the PDF printing.
-     * @param $string
+     * @param string $string
      * @return string
      */
     function prepare_string($string){