Details
-
Type: Bug
-
Status: Closed
-
Priority: Major
-
Resolution: Fixed
-
Affects Version/s: None
-
Fix Version/s: 5.2.0-b1
-
Component/s: None
-
Labels:None
Description
To reproduce:
- Create a wizard list order form
- Place and order through the form
- Enter invalid payment credentials and check the box to save them
- Submit and receive an error "The payment account could not be found for that client."
In controllers/checkout.php change around line 704 from:if ($type == 'ach') { $account_id = $this->Accounts->addAch($this->post); } elseif ($type == 'cc') { if (isset($this->post['expiration_year']) || isset($this->post['expiration_month'])) { // Concatenate the expiration date to the form 'yyyymm' $this->post['expiration'] = ( isset($this->post['expiration_year']) ? $this->post['expiration_year'] : '' ) . (isset($this->post['expiration_month']) ? $this->post['expiration_month'] : ''); } // Remove type, it will be automatically determined unset($this->post['type']); $account_id = $this->Accounts->addCc($this->post); }
To
if ($type == 'ach') { $account_id = $this->Accounts->addAch($this->post); } elseif ($type == 'cc') { if (isset($this->post['expiration_year']) || isset($this->post['expiration_month'])) { // Concatenate the expiration date to the form 'yyyymm' $this->post['expiration'] = ( isset($this->post['expiration_year']) ? $this->post['expiration_year'] : '' ) . (isset($this->post['expiration_month']) ? $this->post['expiration_month'] : ''); } // Remove type, it will be automatically determined unset($this->post['type']); $account_id = $this->Accounts->addCc($this->post); } if (($errors = $this->Accounts->errors())) { $this->Input->setErrors($errors); return; }
Activity
Jonathan Reissmueller
created issue -
Jonathan Reissmueller
made changes -
Field | Original Value | New Value |
---|---|---|
Rank | Ranked higher |
Jonathan Reissmueller
made changes -
Sprint | 5.2.0 Wrapup [ 142 ] |
Jonathan Reissmueller
made changes -
Rank | Ranked higher |
Jonathan Reissmueller
made changes -
Rank | Ranked higher |
Jonathan Reissmueller
made changes -
Assignee | Jonathan Reissmueller [ jonathan ] |
Jonathan Reissmueller
made changes -
Status | Open [ 1 ] | In Progress [ 3 ] |
Jonathan Reissmueller
made changes -
Status | In Progress [ 3 ] | In Review [ 5 ] |
Resolution | Fixed [ 1 ] |
Jonathan Reissmueller
made changes -
Remaining Estimate | 0 minutes [ 0 ] | |
Time Spent | 8 minutes [ 480 ] | |
Worklog Id | 15250 [ 15250 ] |
Abdy Franco
made changes -
Time Spent | 8 minutes [ 480 ] | 19 minutes [ 1140 ] |
Worklog Id | 15254 [ 15254 ] |
Abdy Franco
made changes -
Status | In Review [ 5 ] | Closed [ 6 ] |