Ascending order - Click to sort in descending order
Show
eileen Eileen McNaughton added a comment - https://github.com/civicrm/civicrm-core/pull/5577
Hide
dgg David Greenberg added a comment -

This change caused a regression. If a description is present for a recurring processor, the recurring payment fields never load on the back-office contribution form.

This is because line 442 fails since the array elements in $this->_processors have been modified w/ addition of the description.

$this->_recurPaymentProcessors = array_intersect_assoc($this->_processors, $recurring);

Show
dgg David Greenberg added a comment - This change caused a regression. If a description is present for a recurring processor, the recurring payment fields never load on the back-office contribution form. This is because line 442 fails since the array elements in $this->_processors have been modified w/ addition of the description. $this->_recurPaymentProcessors = array_intersect_assoc($this->_processors, $recurring);
Hide
dgg David Greenberg added a comment -

https://github.com/civicrm/civicrm-core/pull/5713 fixes the problem with recurring contribution fields not showing up when they should.

Show
dgg David Greenberg added a comment - https://github.com/civicrm/civicrm-core/pull/5713 fixes the problem with recurring contribution fields not showing up when they should.
Hide
eileen Eileen McNaughton added a comment - - edited

Another couple of places for this
https://github.com/civicrm/civicrm-core/pull/5826

Show
eileen Eileen McNaughton added a comment - - edited Another couple of places for this https://github.com/civicrm/civicrm-core/pull/5826
Hide
dgg David Greenberg added a comment -

Additional PR to be merged before closing again.

Show
dgg David Greenberg added a comment - Additional PR to be merged before closing again.