View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0003636 | Composr | core_configuration | public | 2018-06-28 16:01 | 2018-10-21 01:27 |
| Reporter | Chris Graham | Assigned To | Chris Graham | ||
| Severity | Feature-request | ||||
| Status | resolved | Resolution | fixed | ||
| Product Version | |||||
| Fixed in Version | |||||
| Summary | 0003636: Configuration option setting checks | ||||
| Description | There are a number of cases where changing configuration options may be destructive. Currently some are hard-coded in admin_config.php: 1) "Make sure we haven't locked ourselves out due to URL Scheme support" - you can't set a URL Scheme if the rewrites aren't configured first 2) "Make sure we haven't just locked staff out" - if the site name is changed some forum layer renaming has to happen for MSNs with separate staff rosters 3) "Empty thumbnail cache if needed" - if the thumbnail size is changed, the thumbnail cache needs emptying 4) "Empty language cache if needed" - if US English is toggled, the language cache needs emptying And we should also have: 1) "Make sure setting e-mail address login [one_per_email_address=2] doesn't lock the current admin out because they have no e-mail address" We should implement this all in a cleaner way. Each config hook should have a check method, which checks the new value is valid / does extra cleanup that is needed. That's more extendible. | ||||
| Tags | No tags attached. | ||||
| Time estimation (hours) | 1 | ||||
| Sponsorship open | |||||
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2018-06-28 16:01 | Chris Graham | New Issue | |
| 2018-06-28 16:01 | Chris Graham | Sponsorship open | 0 => |
| 2018-06-28 16:01 | Chris Graham | Category | core => core_configuration |
| 2018-08-09 01:36 | Chris Graham | Note Added: 0005791 | |
| 2018-10-21 01:27 | Chris Graham | Status | non-assigned => resolved |
| 2018-10-21 01:27 | Chris Graham | Resolution | open => fixed |
| 2018-10-21 01:27 | Chris Graham | Assigned To | => Chris Graham |
