Details
Bug
Minor
Description
In cPanel, it's possible to receive an "Undefined property: stdClass::domain" error.
See https://www.blesta.com/forums/index.php?/topic/13624-error-adding-product-to-basket-cpanel-module/
It looks like cPanel::getClientAddFields should check whether all input $vars properties exist before attempting to use them. In this case, it appears there is an expression:
$this->Html->ifSet($vars->cpanel_domain, $vars->domain),
... that should also check whether $vars->domain is set, e.g.:
$this->Html->ifSet($vars->cpanel_domain, $this->Html->ifSet($vars->domain)),
Activity
Automated transition triggered when Tyson Phillips (Inactive) merged pull request #27 in Stash -
| Status | In Review [ 5 ] | Closed [ 6 ] |
| Time Spent | 10 minutes [ 600 ] | 15 minutes [ 900 ] |
| Worklog Id | 13277 [ 13277 ] |
Tyson Phillips (Inactive)
made changes -
| Remaining Estimate | 0 minutes [ 0 ] | |
| Time Spent | 10 minutes [ 600 ] | |
| Worklog Id | 13274 [ 13274 ] |
Automated transition triggered when Tyson Phillips (Inactive) created pull request #27 in Stash -
| Status | In Progress [ 3 ] | In Review [ 5 ] |
| Resolution | Fixed [ 1 ] |
Tyson Phillips (Inactive)
made changes -
| Summary | cPanel: Possible undefined property error | cPanel: Undefined property error when cpanel_domain not set |
Tyson Phillips (Inactive)
made changes -
| Status | Open [ 1 ] | In Progress [ 3 ] |
Tyson Phillips (Inactive)
made changes -
| Assignee | Tyson Phillips [ tyson ] |
Tyson Phillips (Inactive)
made changes -
| Sprint | 4.9.0 Sprint 6 [ 104 ] |
Tyson Phillips (Inactive)
made changes -
| Field | Original Value | New Value |
|---|---|---|
| Rank | Ranked higher |
Tyson Phillips (Inactive)
created issue -
