Openbravo Issue Tracking System - POS2 | |||||
| View Issue Details | |||||
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0055149 | POS2 | Core | public | 2024-04-10 08:16 | 2024-05-06 10:33 |
| Reporter | marvintm | ||||
| Assigned To | Triage Platform Base | ||||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | acknowledged | Resolution | open | ||
| Platform | OS | 5 | OS Version | ||
| Product Version | |||||
| Target Version | Fixed in Version | ||||
| Merge Request Status | |||||
| Review Assigned To | |||||
| OBNetwork customer | |||||
| Support ticket | |||||
| Regression level | |||||
| Regression date | |||||
| Regression introduced in release | |||||
| Regression introduced by commit | |||||
| Triggers an Emergency Pack | No | ||||
| Summary | 0055149: Automatically generated configuration names are not specific enough | ||||
| Description | The BaseFormInput component is automatically using a configuration if present, so that it is possible to inject additional buttons to it: https://gitlab.com/openbravo/product/pmods/org.openbravo.core2/-/blob/master/web-jspack/org.openbravo.core2/src/components/BaseFormInput/BaseFormInput.jsx?ref_type=heads#L114 [^] This is great because it allows for modules to add buttons to such inputs. However, the names of the expected configuration in practice, when these inputs are coming from a filter in a FilterableGrid, are just "fieldName_additionalButtons" This seems to be not specific enough, and we believe the probability that we will have collisions later on will be high. | ||||
| Steps To Reproduce | Create a FilterableGrid with some filter. Check the name of the expected configuration for the input of the filters. | ||||
| Proposed Solution | The simplest solution seems to be to concatenate the ids of the "parent" components on the name, so that different FilterableGrids have different configurations for their filters. | ||||
| Additional Information | |||||
| Tags | No tags attached. | ||||
| Relationships | |||||
| Attached Files | |||||
| Issue History | |||||
| Date Modified | Username | Field | Change | ||
| 2024-04-10 08:16 | marvintm | New Issue | |||
| 2024-04-10 08:16 | marvintm | Assigned To | => Triage Platform Base | ||
| 2024-04-10 08:16 | marvintm | Triggers an Emergency Pack | => No | ||
| 2024-04-10 08:54 | AugustoMauch | Status | new => acknowledged | ||
| 2024-05-06 10:33 | alostale | Severity | major => minor | ||
| There are no notes attached to this issue. |