Bug #1437
required fields in billing not checked
| Status: | Closed | Start date: | 2008-09-01 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | 2009-05-15 | |
| Assignee: | Michael Staatz | % Done: | 0% |
|
| Category: | - | Spent time: | 0.51 hour | |
| Target version: | - | |||
| Votes: | 0 |
Description
Hi,
during the checkout process, the fields marked as required in the billing step are not checked, when you do not check that there is a other delivery address.
The following patch will insert the check just before the switch($this->currentStep), so the billing address is always checked, when it was submittet and the currentStep is set to "billing" if there was an issue with the check.
I appended a svn diff.
Kind regards ...
/Frank/Associated revisions
[Bug 1437] Add an option to always use the e-mail address from the EM,r=Oliver
[Bug 1437] Add an option to always use the e-mail address from the EM,r=Oliver
History
Updated by Michael Staatz about 4 years ago
- Due date set to 2009-05-15
- Status changed from New to Needs Feedback
- Assignee set to Michael Staatz
There is actually the following line in class.tx_commerce_pi3.php:
Line 295: if (!$this->validateAddress('billing')){ $this->currentStep='billing';}
this is because the function validateAddress is validating $this->piVars['check'] and in the first step this is 'billing' and i doublechecked $this->currentStep and it is also 'billing'.
So please explain better to reproduce the error.
If you don't think so, please give feedback.
best regards
Michael
Updated by Ingo Schmitt over 3 years ago
- Status changed from Needs Feedback to Closed
No Feedback, Issue cloesd