adamjakab / SuiteCRM

ModuleInstall/PackageManager/PackageManagerDisplay.php

Doc Comments +2 added lines

@@ -302,6 +302,7 @@ 
      * A Static method used to build the initial treeview when the page is first displayed
      *
      * @param String div_id - this div in which to display the tree
+     * @param string $div_id
      * @return Tree - the tree that is built
      */
     static function buildTreeView($div_id, $isAlive = true){
@@ -332,6 +333,7 @@ 
      * @param String type - module/patch....
      * @param String manifest - the path to the manifest file
      * @param String modify_field - the field to update when the radio button is changed
+     * @param string $form_action
      * @return String - a form used to display the license
      */
     function getLicenseDisplay($license_file, $form_action, $next_step, $zipFile, $type, $manifest, $modify_field){

ModuleInstall/PackageManager/PackageManagerDownloader.php

Doc Comments +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
 	 * @param save_dir			(optional) if specified it will direct where to save the file once downloaded
 	 * @param download_sever	(optional) if specified it will direct the url for the download
 	 *
-	 * @return the full path of the saved file
+	 * @return string full path of the saved file
 	 */
 	function download($session_id, $file_name, $save_dir = '', $download_server = ''){
 		if(empty($save_dir)){

modules/ACL/ACLController.php

Doc Comments +8 added lines, -1 removed lines

@@ -76,6 +76,9 @@ 
 	}
 	/* END - SECURITY GROUPS */	
 
+	/**
+	 * @param string $value
+	 */
 	static function requireOwner($category, $value, $type='module'){
 			global $current_user;
 			if(is_admin($current_user))return false;
@@ -83,6 +86,10 @@ 
 	}
 
 	/* BEGIN - SECURITY GROUPS */
+
+	/**
+	 * @param string $value
+	 */
 	static function requireSecurityGroup($category, $value, $type='module'){
 			global $current_user;
 			if(is_admin($current_user))return false;
@@ -142,7 +149,7 @@ 
 	 * Check to see if the module is available for this user.
 	 *
 	 * @param String $module_name
-	 * @return true if they are allowed.  false otherwise.
+	 * @return boolean if they are allowed.  false otherwise.
 	 */
 	static function checkModuleAllowed($module_name, $actions)
 	{

modules/ACLActions/ACLAction.php

Doc Comments +1 added lines, -1 removed lines

@@ -158,7 +158,7 @@
      * returns the label  associated with an access level
      * these label definitions  exist in the definitions in modules/ACLActions/actiondefs.php
      * @param INT $access - the access level you want the color for
-     * @return the access level label or false if the level does not exist
+     * @return string|false access level label or false if the level does not exist
      */
     protected static function AccessLabel($access){
         global $ACLActionAccessLevels;

modules/ACLRoles/ACLRole.php

Doc Comments +1 added lines, -1 removed lines

@@ -68,7 +68,7 @@
  *
  * Sets the relationship between a role and an action and sets the access level of that relationship
  *
- * @param GUID $role_id - the role id
+ * @param string $role_id - the role id
  * @param GUID $action_id - the ACL Action id
  * @param int $access - the access level ACL_ALLOW_ALL ACL_ALLOW_NONE ACL_ALLOW_OWNER...
  */

modules/Activities/EmailReminder.php

Doc Comments +1 added lines, -1 removed lines

@@ -123,7 +123,7 @@
      * @param SugarBean $bean
      * @param Administration $admin
      * @param array $recipients
-     * @return boolean
+     * @return null|boolean
      */
     public function sendReminders(SugarBean $bean, Administration $admin, $recipients)
     {

modules/Administration/Administration.php

Doc Comments +4 added lines

@@ -157,6 +157,10 @@
         $this->retrieveSettings(false, true);
     }
 
+    /**
+     * @param string $category
+     * @param string $key
+     */
     function saveSetting($category, $key, $value) {
         $result = $this->db->query("SELECT count(*) AS the_count FROM config WHERE category = '{$category}' AND name = '{$key}'");
         $row = $this->db->fetchByAssoc($result);

modules/Administration/Common.php

Doc Comments +8 added lines, -1 removed lines

@@ -74,6 +74,7 @@ 
 /**
  * @return string&
  * @param the_array array, language string, module string
+ * @param string $old_contents
  * @desc Returns the contents of the customized language pack.
  */
 function &create_field_lang_pak_contents($old_contents, $key, $value, $language, $module)
@@ -346,7 +347,7 @@ 
 
 /**
  * @return bool
- * @param dropdown_name string, language string
+ * @param dropdown_name string language string
  * @desc Creates a new dropdown type.
  */
 function create_dropdown_type($dropdown_name, $language)
@@ -549,6 +550,9 @@ 
    save_custom_app_list_strings_contents($new_contents, $language);
 }
 
+/**
+ * @param string $file_contents
+ */
 function replace_or_add_dropdown_type($dropdown_type, &$dropdown_array,
    &$file_contents)
 {
@@ -580,6 +584,9 @@ 
 	return $new_contents;
 }
 
+/**
+ * @param string $file_contents
+ */
 function replace_or_add_app_string($name, $value,
    &$file_contents)
 {

modules/Administration/controller.php

Doc Comments -1 removed lines

@@ -123,7 +123,6 @@
      * to be able to add additional parameters which are currently not supported
      * in the UI (i.e. additional curl settings for elastic search for auth)
      *
-     * @param array $config
      * @return array
      */
     protected function mergeFtsConfig($type, $newConfig)