View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 2292 | Composr | core | public | 2016-03-14 21:43 | 2016-10-13 04:45 |
| Reporter | Chris Graham | Assigned To | Chris Graham | ||
| Priority | normal | Severity | feature | ||
| Status | resolved | Resolution | fixed | ||
| Summary | 2292: Refactor http_download_file function | ||||
| Description | This function is very long, and takes too many parameters. Break it down into a class, with a simple frontend function to call it. Turn some of the more exotic parameters that exist now into parameters via a map array parameter. Make sure remaining parameters are in a logical order. Stop using globals, make them properties that can be read by code that instantiates the class (i.e. not using the wrapper function). Rename the wrapper function to cms_http_request. | ||||
| Tags | Risk: Core rearchitecting | ||||
| Attach Tags | |||||
| Time estimation (hours) | 6 | ||||
| Sponsorship open | |||||
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2016-03-14 21:43 | Chris Graham | New Issue | |
| 2016-03-14 21:44 | Chris Graham | Tag Attached: Core rearchitecting | |
| 2016-03-14 22:33 | Chris Graham | Time estimation (hours) | 4 => 6 |
| 2016-03-14 22:33 | Chris Graham | Description Updated | |
| 2016-06-08 00:15 | Chris Graham | Tag Renamed | Core rearchitecting => Risk: Core rearchitecting |
| 2016-10-13 04:45 | Chris Graham | Status | Not Assigned => Resolved |
| 2016-10-13 04:45 | Chris Graham | Resolution | open => fixed |
| 2016-10-13 04:45 | Chris Graham | Assigned To | => Chris Graham |
