View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0005375 | Composr alpha bug reports | [All Projects] General / Uncategorised | public | 2023-03-01 03:59 | 2024-09-13 03:01 |
| Reporter | Patrick Schmalstig | Assigned To | Chris Graham | ||
| Severity | Minor-bug | ||||
| Status | closed | Resolution | fixed | ||
| Summary | 0005375: Setup Wizard: install test content fails on catalogues hook when eCommerce is not installed due to missing tax code field | ||||
| Description | When running install_test_content() with the ecommerce addon uninstalled, the catalogues hook fails because the products catalogue still exists with a tax_code field (which belongs to the ecommerce addon, thus does not exist). | ||||
| Additional Information | Needs discussion on how to handle situations like this: * Should ecommerce remove all products-type catalogues on uninstall? * Should special fields like tax_code fall back to a core field type installed with Composr depending on the set database type for that field (in this case, probably a list or short text)? * When field hooks are uninstalled, should all fields of the same type be removed from the database? * Should tax_code be a core field, not an ecommerce field? | ||||
| Tags | No tags attached. | ||||
| Sponsorship open | |||||
| related to | 0005531 | resolved | Patrick Schmalstig | V11 Uninstalling Shopping Module. |
|
|
Chris, please let me know what you think and then re-assign back to me. Thank you! |
|
|
Field hook checks were not working to begin with. This was fixed a while ago. Bad or missing field names fall back to short_text. It will only error now if even short_text fails. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2023-03-01 03:59 | Patrick Schmalstig | New Issue | |
| 2023-03-01 03:59 | Patrick Schmalstig | Status | non-assigned => assigned |
| 2023-03-01 03:59 | Patrick Schmalstig | Assigned To | => Patrick Schmalstig |
| 2023-03-01 04:00 | Patrick Schmalstig | Additional Information Updated | View Revisions |
| 2023-03-01 04:01 | Patrick Schmalstig | Additional Information Updated | View Revisions |
| 2023-03-23 20:56 | Patrick Schmalstig | Assigned To | Patrick Schmalstig => Chris Graham |
| 2023-03-23 20:57 | Patrick Schmalstig | Note Added: 0007989 | |
| 2024-01-06 18:33 | Patrick Schmalstig | Relationship added | related to 0005531 |
| 2024-09-13 03:01 | Patrick Schmalstig | Status | assigned => closed |
| 2024-09-13 03:01 | Patrick Schmalstig | Resolution | open => fixed |
| 2024-09-13 03:01 | Patrick Schmalstig | Note Added: 0009363 |
