Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: None
-
Fix Version/s: 5.0.0-b2
-
Component/s: None
-
Labels:None
Description
AdminCompanyLookandfeel::customize() uses this code block which fails to properly consider the admin_url field
if (isset($this->post['admin_logo'])) {
$logo_admin = null;
$file_name = explode(DS, $this->post['admin_logo']);
$file_name = end($file_name);
if ($this->post['admin_type'] == 'logo') {
$logo_admin = WEBDIR . 'uploads/themes/asset/' . $file_name;
} elseif ($this->post['admin_type'] == 'url') {
$logo_admin = trim($this->post['admin_url']);
}
$this->Companies->setSetting($this->company_id, 'logo_admin', $logo_admin);
}
Similar for the client logo. Because of this you are unable to set a logo for either interface using the url fields.
Activity
| Field | Original Value | New Value |
|---|---|---|
| Rank | Ranked higher |
| Sprint | 5.0.0b2 Sprint 1 [ 124 ] |
| Rank | Ranked higher |
| Rank | Ranked higher |
| Issue Type | Improvement [ 4 ] | Bug [ 1 ] |
| Assignee | Abdy Franco [ abdy ] |
| Status | Open [ 1 ] | In Progress [ 3 ] |
| Remaining Estimate | 0 minutes [ 0 ] | |
| Time Spent | 41 minutes [ 2460 ] | |
| Worklog Id | 14646 [ 14646 ] |
| Status | In Progress [ 3 ] | In Review [ 5 ] |
| Resolution | Fixed [ 1 ] |
| Status | In Review [ 5 ] | Closed [ 6 ] |
