adamjakab / SuiteCRM

include/SugarFields/Fields/Image/SugarFieldImage.php

Doc Comments +3 added lines

@@ -112,6 +112,9 @@
 
     }
 
+    /**
+     * @param UploadFile $upload_file
+     */
     public function verify_image($upload_file){
         global $sugar_config;
 

include/SugarFields/Parsers/DetailViewMetaParser.php

Doc Comments +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
 /**
  * parse
  * 
- * @param $filePath The file path of the HTML file to parse
+ * @param string $filePath The file path of the HTML file to parse
  * @param $vardefs The module's vardefs
  * @param $moduleDir The module's directory
  * @param $merge boolean value indicating whether or not to merge the parsed contents

include/SugarFields/Parsers/MetaParser.php

Doc Comments +23 added lines, -7 removed lines

@@ -96,7 +96,7 @@ 
 /**
  * getFormElementsNames
  * Parses for the name values of input, select, textarea types from string content
- * @param $contents The String contents to parse
+ * @param string $contents The String contents to parse
  * @return $matches Array of name/value pairs
  */
 function getFormElementsNames($contents) {
@@ -108,7 +108,7 @@ 
 /**
  * getTagAttribute
  * Returns the name/value of a tag attribute where name is set to $name
- * @param $name The name of the attribute
+ * @param string $name The name of the attribute
  * @param $contents The contents to parse
  * @param $filter Option regular expression to filter value
  * @return Array of name/value for matching attribute
@@ -163,7 +163,7 @@ 
  * and method call getElementsByType("<td>", $contents) returns
  * "<td>Text <table><tr><td>a</td></tr></table></td>"
  *
- * @param $type The type of element to parse out and return
+ * @param string $type The type of element to parse out and return
  * @return a tag element format Array
  */
 function getElementsByType($type, $contents) {
@@ -208,6 +208,7 @@ 
 /**
  * getElementValue
  *
+ * @param string $type
  */
 function getElementValue($type, $contents, $filter = "(.*?)") {
    $exp = "'<".$type."[^>]*?>".$filter."</".$type."[^>]*?>'si";
@@ -244,7 +245,7 @@ 
  * Returns the highest number of <td>...</td> blocks within a <tr>...</tr> block.
  * @param $contents The table contents to parse
  * @param $filter Optional filter to parse for an attribute within the td block.
- * @return The maximum column count
+ * @return integer maximum column count
  */
 function getMaxColumns($contents, $filter) {
    preg_match_all("'(<tr[^>]*?>)(.*?)(</tr[^>]*?>)'si", $contents, $matches, PREG_SET_ORDER);
@@ -281,6 +282,9 @@ 
  * trimHTML
  * This function removes the \r (return), \n (newline) and \t (tab) markup from string
  */
+/**
+ * @param string $contents
+ */
 function trimHTML($contents) {
    $contents = str_replace(array("\r"), array(""), $contents);
    $contents = str_replace(array("\n"), array(""), $contents);
@@ -342,6 +346,9 @@ 
 
 }
 
+/**
+ * @param string $javascript
+ */
 static function parseDelimiters($javascript) {
 	$newJavascript = '';
 	$scriptLength = strlen($javascript);
@@ -441,7 +448,7 @@ 
  * dirList
  * Utility method to list all the files in a given directory.
  *
- * @param $directory The directory to scan
+ * @param string $directory The directory to scan
  * @return $results The files in the directory that were found
  */
 function dirList ($directory) {
@@ -496,6 +503,9 @@ 
    return null;
 }
 
+/**
+ * @param string $moduleDir
+ */
 function applyPreRules($moduleDir, $panels) {
    if(file_exists("include/SugarFields/Parsers/Rules/".$moduleDir."ParseRule.php")) {
 	  require_once("include/SugarFields/Parsers/Rules/".$moduleDir."ParseRule.php");
@@ -506,6 +516,9 @@ 
    return $panels;
 }
 
+/**
+ * @param string $moduleDir
+ */
 function applyRules($moduleDir, $panels) {
    return $this->applyPostRules($moduleDir, $panels);
 }
@@ -537,6 +550,9 @@ 
    return $panels;
 }
 
+/**
+ * @param string $moduleDir
+ */
 function createFileContents($moduleDir, $panels, $templateMeta=array(), $htmlFilePath) {
 
 $header = "<?php\n\n";
@@ -587,7 +603,7 @@ 
  * mergePanels
  * This function merges the $panels Array against the $masterCopy's meta data definition
  * @param $panels meta data Array to merge
- * @param $moduleDir Directory name of the module
+ * @param string $moduleDir Directory name of the module
  * @param $masterCopy file path to the meta data master copy
  * @return Array of merged $panel definition
  */
@@ -680,7 +696,7 @@ 
  * mergeTemplateMeta
  * This function merges the $templateMeta Array against the $masterCopy's meta data definition
  * @param $templateMeta meta data Array to merge
- * @param $moduleDir Directory name of the module
+ * @param string $moduleDir Directory name of the module
  * @param $masterCopy file path to the meta data master copy
  * @return Array of merged $templateMeta definition
  */

include/SugarFields/Parsers/Rules/AddressRule.php

Doc Comments +2 added lines, -2 removed lines

@@ -113,7 +113,7 @@ 
  * have not been moved from the address panel
  * 
  * @param $addressPanel Array of address panel contents
- * @param $suffix The address suffix (billing, shipping, primary, alternate) to check for
+ * @param string $suffix The address suffix (billing, shipping, primary, alternate) to check for
  * @return boolean
  */
 function hasAddressFieldsIntact($addressPanel, $suffix) {
@@ -135,7 +135,7 @@ 
  * This function scans the panels and locates the street address field for the given key
  * and replaces the Array definition (from the merging process) with a String value.
  * @param $panels Array of the view's panels
- * @param $street String key value of the street to search for
+ * @param string $street String key value of the street to search for
  * @returns $panels Array of view's panels with street value substituted
  */
 function removeStreetFieldOverride($panels, $street) {

include/SugarFields/Parsers/Rules/BaseRule.php

Doc Comments +6 added lines

@@ -75,6 +75,9 @@ 
    return false;
 }
 
+/**
+ * @param string $regExp
+ */
 function matches($mixed, $regExp) {
    if(is_array($mixed) && isset($mixed['name']) && preg_match($regExp, $mixed['name'])) {
    	  return true;
@@ -84,6 +87,9 @@ 
    return false;	
 }
 
+/**
+ * @param string $regExp
+ */
 function getMatch($mixed, $regExp) {
    if(is_array($mixed) && isset($mixed['name']) && preg_match($regExp, $mixed['name'], $matches)) {
    	  return $matches;

include/SugarFields/Parsers/SearchFormMetaParser.php

Doc Comments +6 added lines

@@ -137,6 +137,9 @@ 
   return $section;
 }
 
+/**
+ * @param string $section
+ */
 function processSection($section, $table, $filePath, $vardefs=array()) {
    
    $toptr = $this->getElementsByType("tr", $table);
@@ -251,6 +254,9 @@ 
    return $metarow;
 }
 
+/**
+ * @param string $moduleDir
+ */
 function applyRules($moduleDir, $section=array()) {
    require_once('include/SugarFields/Parsers/Rules/BaseRule.php');
    $baseRule = new BaseRule();

include/SugarFields/SugarFieldHandler.php

Doc Comments +3 added lines

@@ -47,6 +47,9 @@
     function SugarFieldHandler() {
     }
 
+    /**
+     * @param string $field
+     */
     static function fixupFieldType($field) {
             switch($field) {
                case 'double':

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,