adamjakab / SuiteCRM

modules/Trackers/store/Store.php

Doc Comments +1 added lines

@@ -51,6 +51,7 @@
      * flush
      * This is the method implementations need to provide to store a monitor instance
      * @param $monitor An instance of Monitor
+     * @return void
      */
     public function flush($monitor);
 

modules/Trackers/Tracker.php

Doc Comments +3 added lines

@@ -114,6 +114,9 @@
         return $list;
     }
 
+    /**
+     * @param string $item_id
+     */
     function makeInvisibleForAll($item_id)
     {
         $query = "UPDATE $this->table_name SET visible = 0 WHERE item_id = '$item_id' AND visible = 1";

modules/UpgradeWizard/SugarMerge/SearchMerge.php

Doc Comments +4 added lines

@@ -127,6 +127,10 @@
 		$this->newData[$this->module] = $this->newData[$this->module][$this->viewDefs];
 		
 	}
+
+	/**
+	 * @param string $to
+	 */
 	public function save($to){
 		return write_array_to_file("$this->varName['$this->module']", $this->newData[$this->module], $to);
 	}

modules/UpgradeWizard/uw_ajax.php

Doc Comments +3 added lines

@@ -48,6 +48,9 @@
 
 ////	COMMON
 
+/**
+ * @param string $type
+ */
 function ajaxSqlProgress($persistence, $sql, $type) {
 	global $mod_strings;
 

modules/UpgradeWizard/uw_utils.php

Doc Comments +22 added lines, -3 removed lines

@@ -283,7 +283,7 @@ 
  *
  * @param string the directory name to remove
  * @param boolean whether to just empty the given directory, without deleting the given directory.
- * @return boolean True/False whether the directory was deleted.
+ * @return boolean|null True/False whether the directory was deleted.
  */
 
 function copyRecursiveBetweenDirectories($from,$to){
@@ -1176,6 +1176,7 @@ 
 
 /**
  * is a file that we blow away automagically
+ * @param string $file
  */
 function isAutoOverwriteFile($file) {
 	$overwriteDirs = array(
@@ -1811,6 +1812,9 @@ 
 	return $errors;
 }
 
+/**
+ * @param string $file_path
+ */
 function fileCopy($file_path){
 	if(file_exists(clean_path($_SESSION['unzip_dir'].'/'.$_SESSION['zip_from_dir'].'/'.$file_path))) {
 		$file = clean_path($_SESSION['unzip_dir'].'/'.$_SESSION['zip_from_dir'].'/'.$file_path);
@@ -1930,6 +1934,9 @@ 
 }
 
 if ( !function_exists('extractFile') ) {
+/**
+ * @return string
+ */
 function extractFile($zip_file, $file_in_zip) {
     global $base_tmp_upgrade_dir;
 
@@ -1947,6 +1954,9 @@ 
 }
 
 if ( !function_exists('extractManifest') ) {
+/**
+ * @return string
+ */
 function extractManifest($zip_file) {
 	logThis('extracting manifest.');
     return(extractFile($zip_file, "manifest.php"));
@@ -2027,7 +2037,7 @@ 
 /**
  * Verifies a manifest from a patch or module to be compatible with the current Sugar version and flavor
  * @param array manifest Standard manifest array
- * @return string Error message, blank on success
+ * @return null|boolean Error message, blank on success
  */
 function validate_manifest($manifest) {
 	logThis('validating manifest.php file');
@@ -2548,6 +2558,9 @@ 
     }
 }
 
+/**
+ * @param boolean $disable
+ */
 function handleExecuteSqlKeys($db, $tableName, $disable)
 {
     if(empty($tableName)) return true;
@@ -2659,6 +2672,9 @@ 
 }
 
 
+/**
+ * @param string $query
+ */
 function getAlterTable($query){
 	$query = strtolower($query);
 	if (preg_match('/^\s*alter\s+table\s+/', $query)) {
@@ -2877,6 +2893,9 @@ 
 	return $currState;
 }
 
+/**
+ * @param string $step
+ */
 function didThisStepRunBefore($step,$SubStep=''){
 	if($step == null) return;
 	$upgrade_progress_dir = sugar_cached('upgrades/temp');
@@ -3249,7 +3268,7 @@ 
  * to have the value of the $column_name parameter
  *
  * @param $bean SugarBean which we are adding team_set_id column to
- * @param $column_name The name of the column containing the default team_set_id value
+ * @param string $column_name The name of the column containing the default team_set_id value
  */
 function upgradeTeamColumn($bean, $column_name) {
 	//first let's check to ensure that the team_set_id field is defined, if not it could be the case that this is an older

modules/UserPreferences/UserPreference.php

Doc Comments +2 added lines, -2 removed lines

@@ -185,7 +185,7 @@ 
      * Loads preference by category from database. Saving will be done in utils.php -> sugar_cleanup
      *
      * @param string $category name of the category to retreive, defaults to global scope
-     * @return bool successful?
+     * @return null|boolean successful?
      */
     public function loadPreferences(
         $category = 'global'
@@ -291,7 +291,6 @@ 
      * sugar_cleanup if a setPreference has been called during one round trip.
      *
      * @global user will use current_user if no user specificed in $user param
-     * @param user $user User object to retrieve, otherwise user current_user
      * @param bool $all save all of the preferences? (Dangerous)
      *
      */
@@ -387,6 +386,7 @@ 
     /**
      * Updates every user pref with a new key value supports 2 levels deep, use append to
      * array if you want to append the value to an array
+     * @param string $key
      */
     public static function updateAllUserPrefs(
         $key,

modules/Users/authentication/AuthenticationController.php

Doc Comments +1 added lines, -1 removed lines

@@ -187,7 +187,7 @@
 	 * This is called on every page hit.
 	 * It returns true if the current session is authenticated or false otherwise
 	 *
-	 * @return booelan
+	 * @return boolean
 	 */
 	public function sessionAuthenticate()
 	{

modules/Users/authentication/EmailAuthenticate/EmailAuthenticateUser.php

Doc Comments +1 added lines, -1 removed lines

@@ -103,7 +103,7 @@
     /**
      * Sends the users password to the email address or sends
      *
-     * @param unknown_type $user_id
+     * @param string $user_id
      * @param unknown_type $password
      */
     function sendEmailPassword($user_id, $password){

modules/Users/authentication/LDAPAuthenticate/LDAPAuthenticateUser.php

Doc Comments +2 added lines, -1 removed lines

@@ -239,7 +239,7 @@ 
 
 	/**
 	 * takes in a name and creates the appropriate search filter for that user name including any additional filters specified in the system settings page
-	 * @param $name
+	 * @param string $name
 	 * @return String
 	 */
 	function getUserNameFilter($name){
@@ -339,6 +339,7 @@ 
 	 /**
     * @return string appropriate value for username when binding to directory server.
     * @param string $user_name the value provided in login form
+    * @param string $password
     * @desc Take the login username and return either said username for AD or lookup
      * distinguished name using anonymous credentials for OpenLDAP.
      * Contributions by Erik Mitchell erikm@logicpd.com