View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0002496 | Composr | core_addon_management | public | 2016-04-24 23:34 | 2016-10-19 15:06 |
| Reporter | Chris Graham | Assigned To | Chris Graham | ||
| Severity | Feature-request | ||||
| Status | closed | Resolution | no change required | ||
| Product Version | |||||
| Fixed in Version | |||||
| Summary | 0002496: Simplifications to addon system | ||||
| Description | I don't believe we need the addon.inf files inside the addon tars, the addon_install_code.php files, or most of the DB tables. The addons_dependencies and addons_files tables can be removed. The addons table can be stripped down so it is more similar the modules and blocks tables (i.e. just for versioning). These things exist from the older ocPortal designs, and have carried through to Composr while other parts of our addon system were indeed streamlined. Instead 100% use of addon_registry hooks would be used for all meta information. The addon management tools will generate these as required. | ||||
| Tags | No tags attached. | ||||
| Time estimation (hours) | 3 | ||||
| Sponsorship open | |||||
|
|
I was a bit confused when I wrote this up, trying to remember the state of play within both ocPortal and Composr, when both were divergent. Composr actually uses the addon.inf for good reasons. Having the registry hook there implies a trust to run PHP code. We can't have it so that every third party who generates a theme or language pack has to be trusted with a PHP file in their addon. addon_install_code.php is not strictly necessary but it is a nice little feature to have simple code execution without having to build a proper registry hook. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2016-04-24 23:34 | Chris Graham | New Issue | |
| 2016-10-19 15:06 | Chris Graham | Note Added: 0004438 | |
| 2016-10-19 15:06 | Chris Graham | Status | non-assigned => closed |
| 2016-10-19 15:06 | Chris Graham | Assigned To | => Chris Graham |
| 2016-10-19 15:06 | Chris Graham | Resolution | open => no change required |
