Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0055149
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[POS2] Coreminorhave not tried2024-04-10 08:162024-07-01 08:56
ReportermarvintmView Statuspublic 
Assigned ToTriage Platform Base 
PrioritynormalResolutionopenFixed in Version
StatusscheduledFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned To
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0055149: Automatically generated configuration names are not specific enough

DescriptionThe 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 ReproduceCreate a FilterableGrid with some filter.
Check the name of the expected configuration for the input of the filters.
Proposed SolutionThe 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.
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
There are no notes attached to this issue.

- 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
2024-06-17 08:38 AugustoMauch Status acknowledged => scheduled


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker