//setting default join date and receive date
if ($this->_action == CRM_Core_Action::ADD) {
- $defaults['receive_date'] = date('Y-m-d H:i:s');
+ $defaults['receive_date'] = CRM_Utils_Time::date('Y-m-d H:i:s');
}
$defaults['num_terms'] = 1;
//setting default join date if there is no join date
if (empty($defaults['join_date'])) {
- $defaults['join_date'] = date('Y-m-d');
+ $defaults['join_date'] = CRM_Utils_Time::date('Y-m-d');
}
if (!empty($defaults['membership_end_date'])) {
CRM_Member_StatusOverrideTypes::getSelectOptions()
);
- $this->add('datepicker', 'status_override_end_date', ts('Status Override End Date'), '', FALSE, ['minDate' => date('Y-m-d'), 'time' => FALSE]);
+ $this->add('datepicker', 'status_override_end_date', ts('Status Override End Date'), '', FALSE, ['minDate' => CRM_Utils_Time::date('Y-m-d'), 'time' => FALSE]);
$this->addElement('checkbox', 'record_contribution', ts('Record Membership Payment?'));
* @param array $formValues
* @param object $membership
* Object.
- * @param array $customValues
*
* @return bool
* true if mail was sent successfully
* & needs rationalising.
*
*/
- public static function emailReceipt(&$form, &$formValues, &$membership, $customValues = NULL) {
+ public static function emailReceipt($form, &$formValues, $membership) {
// retrieve 'from email id' for acknowledgement
$receiptFrom = $formValues['from_email_address'] ?? NULL;
$formValues['paidBy'] = $paymentInstrument[$formValues['payment_instrument_id']];
}
- $form->assign('customValues', $customValues);
-
if ($form->_mode) {
// @todo move this outside shared code as Batch entry just doesn't
$form->assign('address', CRM_Utils_Address::getFormattedBillingAddressFieldsFromParameters(
$form->assign('receive_date', CRM_Utils_Array::value('receive_date', $formValues));
$form->assign('formValues', $formValues);
- if (empty($lineItem)) {
- $form->assign('mem_start_date', CRM_Utils_Date::formatDateOnlyLong($membership->start_date));
- if (!CRM_Utils_System::isNull($membership->end_date)) {
- $form->assign('mem_end_date', CRM_Utils_Date::formatDateOnlyLong($membership->end_date));
- }
- $form->assign('membership_name', CRM_Member_PseudoConstant::membershipType($membership->membership_type_id));
+ $form->assign('mem_start_date', CRM_Utils_Date::formatDateOnlyLong($membership->start_date));
+ if (!CRM_Utils_System::isNull($membership->end_date)) {
+ $form->assign('mem_end_date', CRM_Utils_Date::formatDateOnlyLong($membership->end_date));
}
+ $form->assign('membership_name', CRM_Member_PseudoConstant::membershipType($membership->membership_type_id));
// @todo - if we have to figure out if this is for batch processing it doesn't belong in the shared function.
$isBatchProcess = is_a($form, 'CRM_Batch_Form_Entry');
//get the payment processor id as per mode. Try removing in favour of beginPostProcess.
$params['payment_processor_id'] = $formValues['payment_processor_id'] = $this->_paymentProcessor['id'];
- $params['register_date'] = date('YmdHis');
+ $params['register_date'] = CRM_Utils_Time::date('YmdHis');
// add all the additional payment params we need
$formValues['amount'] = $params['total_amount'];
// we do need contribution and recurring records.
$result = NULL;
if (!empty($paymentParams['is_recur'])) {
- $financialType = new CRM_Financial_DAO_FinancialType();
- $financialType->id = $params['financial_type_id'];
- $financialType->find(TRUE);
$this->_params = $formValues;
$contribution = $this->processContribution(
$paymentParams,
- NULL,
[
'contact_id' => $this->_contributorContactID,
'line_item' => $lineItem,
'is_test' => $isTest,
'campaign_id' => $paymentParams['campaign_id'] ?? NULL,
- 'contribution_page_id' => $formValues['contribution_page_id'] ?? NULL,
'source' => CRM_Utils_Array::value('source', $paymentParams, CRM_Utils_Array::value('description', $paymentParams)),
- 'thankyou_date' => $paymentParams['thankyou_date'] ?? NULL,
'payment_instrument_id' => $paymentInstrumentID,
- ],
- $financialType
+ 'financial_type_id' => $params['financial_type_id'],
+ ]
);
//create new soft-credit record, CRM-13981
$paymentParams['contactID'] = $this->_contactID;
$paymentParams['contributionID'] = $contribution->id;
- $paymentParams['contributionPageID'] = $contribution->contribution_page_id;
$paymentParams['contributionRecurID'] = $contribution->contribution_recur_id;
$params['contribution_id'] = $paymentParams['contributionID'];
$params['contribution_recur_id'] = $paymentParams['contributionRecurID'];
}
$endDate = $startDate = NULL;
}
- $now = date('YmdHis');
- $params['receive_date'] = date('Y-m-d H:i:s');
+ $now = CRM_Utils_Time::date('YmdHis');
+ $params['receive_date'] = CRM_Utils_Time::date('Y-m-d H:i:s');
$params['invoice_id'] = $formValues['invoiceID'];
$params['contribution_source'] = ts('%1 Membership Signup: Credit card or direct debit (by %2)',
[1 => $this->getSelectedMembershipLabels(), 2 => $userName]
// @todo figure out why recieve_date isn't being set right here.
if (empty($params['receive_date'])) {
- $params['receive_date'] = date('Y-m-d H:i:s');
+ $params['receive_date'] = CRM_Utils_Time::date('Y-m-d H:i:s');
}
$membershipParams = array_merge($params, $membershipTypeValues[$lineItemValues['membership_type_id']]);
*/
protected function emailMembershipReceipt($formValues, $membership) {
$customValues = $this->getCustomValuesForReceipt($formValues, $membership);
+ $this->assign('customValues', $customValues);
- return self::emailReceipt($this, $formValues, $membership, $customValues);
+ return self::emailReceipt($this, $formValues, $membership);
}
/**
* being fixed later.
*
* @param array $params
- * @param array $result
* @param array $contributionParams
* Parameters to be passed to contribution create action.
* This differs from params in that we are currently adding params to it and 1) ensuring they are being
* - line_item
* - is_test
* - campaign_id
- * - contribution_page_id
* - source
* - payment_type_id
- * - thankyou_date (not all forms will set this)
- *
- * @param CRM_Financial_DAO_FinancialType $financialType
*
* @return \CRM_Contribute_DAO_Contribution
*
*/
protected function processContribution(
$params,
- $result,
- $contributionParams,
- $financialType
+ $contributionParams
) {
$form = $this;
$transaction = new CRM_Core_Transaction();
$isEmailReceipt = !empty($form->_values['is_email_receipt']);
// add these values for the recurringContrib function ,CRM-10188
- $params['financial_type_id'] = $financialType->id;
+ $params['financial_type_id'] = $contributionParams['financial_type_id'];
//@todo - this is being set from the form to resolve CRM-10188 - an
// eNotice caused by it not being set @ the front end
}
$params['is_recur'] = TRUE;
$params['payment_instrument_id'] = $contributionParams['payment_instrument_id'] ?? NULL;
- $recurringContributionID = $this->legacyProcessRecurringContribution($params, $contactID, $financialType);
+ $recurringContributionID = $this->legacyProcessRecurringContribution($params, $contactID);
- $now = date('YmdHis');
+ $now = CRM_Utils_Time::date('YmdHis');
$receiptDate = $params['receipt_date'] ?? NULL;
if ($isEmailReceipt) {
$receiptDate = $now;
if (isset($params['amount'])) {
$contributionParams = array_merge([
- 'financial_type_id' => $financialType->id,
- 'receive_date' => !empty($params['receive_date']) ? CRM_Utils_Date::processDate($params['receive_date']) : date('YmdHis'),
+ 'receive_date' => !empty($params['receive_date']) ? CRM_Utils_Date::processDate($params['receive_date']) : CRM_Utils_Time::date('YmdHis'),
'tax_amount' => $params['tax_amount'] ?? NULL,
- 'amount_level' => $params['amount_level'] ?? NULL,
'invoice_id' => $params['invoiceID'],
'currency' => $params['currencyID'],
'is_pay_later' => $params['is_pay_later'] ?? 0,
- //configure cancel reason, cancel date and thankyou date
- //from 'contribution' type profile if included
- 'cancel_reason' => $params['cancel_reason'] ?? 0,
- 'cancel_date' => isset($params['cancel_date']) ? CRM_Utils_Date::format($params['cancel_date']) : NULL,
- 'thankyou_date' => isset($params['thankyou_date']) ? CRM_Utils_Date::format($params['thankyou_date']) : NULL,
//setting to make available to hook - although seems wrong to set on form for BAO hook availability
'skipLineItem' => $params['skipLineItem'] ?? 0,
], $contributionParams);
// @todo this is the wrong place for this - it should be done as close to form submission
// as possible
$contributionParams['total_amount'] = $params['amount'];
-
$contribution = CRM_Contribute_BAO_Contribution::add($contributionParams);
// lets store it in the form variable so postProcess hook can get to this and use it
*
* @param array $params
* @param int $contactID
- * @param string $contributionType
*
- * @return int|null
+ * @return int
*/
- protected function legacyProcessRecurringContribution(&$params, $contactID, $contributionType) {
+ protected function legacyProcessRecurringContribution(array $params, $contactID): int {
$form = $this;
- if (empty($params['is_recur'])) {
- return NULL;
- }
$recurParams = ['contact_id' => $contactID];
$recurParams['amount'] = $params['amount'] ?? NULL;
$recurParams['frequency_unit'] = $params['frequency_unit'] ?? NULL;
$recurParams['frequency_interval'] = $params['frequency_interval'] ?? NULL;
$recurParams['installments'] = $params['installments'] ?? NULL;
- $recurParams['financial_type_id'] = $params['financial_type_id'] ?? NULL;
+ $recurParams['financial_type_id'] = $params['financial_type_id'];
$recurParams['currency'] = $params['currency'] ?? NULL;
$recurParams['payment_instrument_id'] = $params['payment_instrument_id'];
$recurParams['is_test'] = 1;
}
- $recurParams['start_date'] = $recurParams['create_date'] = $recurParams['modified_date'] = date('YmdHis');
+ $recurParams['start_date'] = $recurParams['create_date'] = $recurParams['modified_date'] = CRM_Utils_Time::date('YmdHis');
if (!empty($params['receive_date'])) {
- $recurParams['start_date'] = date('YmdHis', strtotime($params['receive_date']));
+ $recurParams['start_date'] = date('YmdHis', CRM_Utils_Time::strtotime($params['receive_date']));
}
$recurParams['invoice_id'] = $params['invoiceID'] ?? NULL;
$recurParams['contribution_status_id'] = CRM_Core_PseudoConstant::getKey('CRM_Contribute_BAO_Contribution', 'contribution_status_id', 'Pending');
$recurParams['payment_processor_id'] = $params['payment_processor_id'] ?? NULL;
- $recurParams['is_email_receipt'] = $params['is_email_receipt'] ?? NULL;
+ $recurParams['is_email_receipt'] = (bool) ($params['is_email_receipt'] ?? FALSE);
// we need to add a unique trxn_id to avoid a unique key error
// in paypal IPN we reset this when paypal sends us the real trxn id, CRM-2991
$recurParams['trxn_id'] = $params['trxn_id'] ?? $params['invoiceID'];
- $recurParams['financial_type_id'] = $contributionType->id;
$campaignId = $params['campaign_id'] ?? $form->_values['campaign_id'] ?? NULL;
$recurParams['campaign_id'] = $campaignId;
- $recurring = CRM_Contribute_BAO_ContributionRecur::add($recurParams);
- if (is_a($recurring, 'CRM_Core_Error')) {
- throw new CRM_Core_Exception(CRM_Core_Error::getMessages($recurring));
- }
-
- return $recurring->id;
+ return CRM_Contribute_BAO_ContributionRecur::add($recurParams)->id;
}
}