adamjakab / SuiteCRM

data/Link2.php

Doc Comments +3 added lines, -2 removed lines

@@ -74,7 +74,7 @@ 
      * @param  $linkName String name of a link field in the module's vardefs
      * @param  $bean SugarBean focus bean for this link (one half of a relationship)
      * @param  $linkDef Array Optional vardef for the link in case it can't be found in the passed in bean for the global dictionary
-     * @return void
+     * @return false|null
      *
      */
     function __construct($linkName, $bean, $linkDef = false){
@@ -356,7 +356,7 @@ 
     /**
      * @param array $params optional parameters. Possible Values;
      * 'return_as_array': returns the query broken into
-     * @return String/Array query to grab just ids for this relationship
+     * @return string query to grab just ids for this relationship
      */
     function getQuery($params = array())
     {
@@ -686,6 +686,7 @@ 
     /**
      * @deprecated
      * Gets the vardef for the relationship of this link.
+     * @param string $def_name
      */
     public function _get_link_table_definition($table_name, $def_name) {
 

data/Relationships/M2MRelationship.php

Doc Comments +4 added lines, -4 removed lines

@@ -365,8 +365,8 @@ 
 
     /**
      * Removes the reversed version of this relationship
-     * @param $lhs
-     * @param $rhs
+     * @param SugarBean $lhs
+     * @param SugarBean $rhs
      * @param array $additionalFields
      * @return void
      */
@@ -383,7 +383,7 @@ 
     }
 
     /**
-     * @param  $link Link2 loads the relationship for this link.
+     * @param  Link2 $link Link2 loads the relationship for this link.
      * @return void
      */
     public function load($link, $params = array())
@@ -532,7 +532,7 @@ 
     /**
      * Similar to getQuery or Get join, except this time we are starting from the related table and
      * searching for items with id's matching the $link->focus->id
-     * @param  $link
+     * @param  Link2 $link
      * @param array $params
      * @param bool $return_array
      * @return String|Array

data/Relationships/One2MRelationship.php

Doc Comments +1 added lines, -1 removed lines

@@ -119,7 +119,7 @@
      * @param  $lhs SugarBean left side bean to add to the relationship.
      * @param  $rhs SugarBean right side bean to add to the relationship.
      * @param  $additionalFields key=>value pairs of fields to save on the relationship
-     * @return boolean true if successful
+     * @return boolean|null true if successful
      */
     public function add($lhs, $rhs, $additionalFields = array())
     {

data/Relationships/SugarRelationship.php

Doc Comments +1 added lines, -3 removed lines

@@ -197,7 +197,7 @@ 
 
     /**
      * @param array $row values to be inserted into the relationship
-     * @return bool|void null if new row was inserted and true if an existing row was updated
+     * @return resource|null null if new row was inserted and true if an existing row was updated
      */
     protected function addRow($row)
     {
@@ -399,8 +399,6 @@ 
 
     /**
      * @param $optional_array clause to add to the where query when populating this relationship. It should be in the
-     * @param string $add_and
-     * @param string $prefix
      * @return string
      */
     protected function getOptionalWhereClause($optional_array) {

include/connectors/component.php

Doc Comments +3 added lines

@@ -342,6 +342,9 @@
  		return $this->_source;
  	}
 
+ 	/**
+ 	 * @param source $source
+ 	 */
  	public function setSource($source){
  		$this->_source = $source;
  	}

include/connectors/ConnectorFactory.php

Doc Comments +1 added lines

@@ -63,6 +63,7 @@
 	 * Split the class name by _ and go through the class name
 	 * which represents the inheritance structure to load up all required parents.
 	 * @param string $class the root class we want to load.
+	 * @param string $type
 	 */
 	public static function load($class, $type){
 		self::loadClass($class, $type);

include/connectors/filters/FilterFactory.php

Doc Comments +3 added lines

@@ -47,6 +47,9 @@
 
 	static $filter_map = array();
 
+	/**
+	 * @param string $source_name
+	 */
 	public static function getInstance($source_name, $filter_name=''){
 		require_once('include/connectors/filters/default/filter.php');
 		$key = $source_name . $filter_name;

include/connectors/sources/default/source.php

Doc Comments +5 added lines, -2 removed lines

@@ -318,6 +318,9 @@ 
  	    return !empty($this->_config['properties'][$name]);
  	}
 
+ 	/**
+ 	 * @param string $name
+ 	 */
  	public function getProperty($name)
  	{
  	    if(!empty($this->_config) && isset($this->_config['properties'][$name])) {
@@ -336,7 +339,7 @@ 
  	 * This method is used to indicate whether or not a data source has testing enabled so that
  	 * the administration interface may call the test method on the data source instance
  	 *
- 	 * @return enabled boolean value indicating whether or not testing is enabled
+ 	 * @return boolean boolean value indicating whether or not testing is enabled
  	 */
  	public function hasTestingEnabled() {
  		return $this->_has_testing_enabled;
@@ -348,7 +351,7 @@ 
  	 * It is up to subclasses to implement a test and set _has_testing_enabled to true so that
  	 * a test button is rendered in the administration interface
  	 *
- 	 * @return result boolean result of the test function
+ 	 * @return boolean boolean result of the test function
  	 */
     public function test() {
     	return false;

include/connectors/sources/SourceFactory.php

Doc Comments -1 removed lines

@@ -48,7 +48,6 @@
 
 	/**
 	 * Given a source param, load the correct source and return the object
-	 * @param string $source string representing the source to load
 	 * @return source
 	 */
 	public static function getSource($class, $call_init = true) {