X-Git-Url: https://vcs.fsf.org/?a=blobdiff_plain;f=api%2Fv3%2FPledge.php;h=eb5949599c312304c2432dd14ad7a08a3bfa3e7f;hb=6295e9db88e6afc963d69b272ab0fbfbb654c24c;hp=00d21ca87a283618ffc146ab470c8c26cf19bf93;hpb=11df99209d13885aaf45a5c94612bdc65eb490ff;p=civicrm-core.git diff --git a/api/v3/Pledge.php b/api/v3/Pledge.php index 00d21ca87a..eb5949599c 100644 --- a/api/v3/Pledge.php +++ b/api/v3/Pledge.php @@ -1,5 +1,4 @@ $params['id'] - ), $params, 'pledge', 'delete'); + 'id' => $params['id'], + ), $params, 'Pledge', 'delete'); } else { return civicrm_api3_create_error('Could not delete pledge'); @@ -84,6 +68,8 @@ function civicrm_api3_pledge_delete($params) { } /** + * Adjust metadata for pledge delete action. + * * @param array $params */ function _civicrm_api3_pledge_delete_spec(&$params) { @@ -92,7 +78,9 @@ function _civicrm_api3_pledge_delete_spec(&$params) { } /** - * return field specification specific to get requests + * Adjust field specification specific to get requests. + * + * @param array $params */ function _civicrm_api3_pledge_get_spec(&$params) { $params['next_pay_date'] = array( @@ -108,7 +96,9 @@ function _civicrm_api3_pledge_get_spec(&$params) { } /** - * return field specification specific to get requests + * Adjust field specification specific to get requests. + * + * @param array $params */ function _civicrm_api3_pledge_create_spec(&$params) { @@ -122,49 +112,46 @@ function _civicrm_api3_pledge_create_spec(&$params) { } /** - * Retrieve a set of pledges, given a set of input params + * Retrieve a set of pledges, given a set of input params. * - * @param array $params input parameters. Use interrogate for possible fields + * @param array $params + * Input parameters. Use interrogate for possible fields. * - * @return array array of pledges, if error an array with an error id and error message - * {@getfields pledge_get} - * @example PledgeGet.php - * @access public + * @return array + * array of pledges, if error an array with an error id and error message */ function civicrm_api3_pledge_get($params) { $mode = CRM_Contact_BAO_Query::MODE_PLEDGE; - $entity = 'pledge'; - list($dao, $query) = _civicrm_api3_get_query_object($params, $mode, $entity); + list($dao, $query) = _civicrm_api3_get_query_object($params, $mode, 'Pledge'); $pledge = array(); while ($dao->fetch()) { $pledge[$dao->pledge_id] = $query->store($dao); } - return civicrm_api3_create_success($pledge, $params, 'pledge', 'get', $dao); + return civicrm_api3_create_success($pledge, $params, 'Pledge', 'get', $dao); } /** - * Set default to not return test params + * Set default to not return test params. */ function _civicrm_api3_pledge_get_defaults() { return array('pledge_test' => 0); } /** - * Legacy function - I removed a bunch of stuff no longer required from here but it still needs + * Legacy function to format pledge parameters. + * + * I removed a bunch of stuff no longer required from here but it still needs * more culling * take the input parameter list as specified in the data model and * convert it into the same format that we use in QF and BAO object * - * @param array $values The reformatted properties that we can use internally - * @param bool $create - * - * @return array|CRM_Error - * @access public + * @param array $values + * The reformatted properties that we can use internally. */ -function _civicrm_api3_pledge_format_params(&$values, $create = FALSE) { +function _civicrm_api3_pledge_format_params(&$values) { // probably most of the below can be removed.... just needs a little more review if (array_key_exists('original_installment_amount', $values)) { @@ -195,4 +182,3 @@ function _civicrm_api3_pledge_format_params(&$values, $create = FALSE) { $values['scheduled_date'] = $values['start_date']; } } -