Skip to content

Conversation

@PierreSmits
Copy link
Member

According to the definition in widget-form.xsd the use of a combination of a form with type="list" is deprecated in favour of a grid.
Refactor various list forms into grids.
Refactor various list form references in screens.

modified in product component:
ConfigScreens.xml: from form ref to grid ref , additional cleanup
ConfigForms.xml: from form definition with list ref to grid definition with list ref, additional clean-up

According to the definition in widget-form.xsd the use of a combination of a form with type="list" is deprecated in favour of a grid.
Refactor various list forms into grids.
Refactor various list form references in screens.

modified in product component:
ConfigScreens.xml: from form ref to grid ref , additional cleanup
ConfigForms.xml: from form definition with list ref to grid definition with list ref, additional clean-up
@JacquesLeRoux
Copy link
Contributor

Hi Pierre,

Is it normal that ProductConfigOptionList and ListProductConfigItem are transformed to grid w/o calls to them?

@PierreSmits
Copy link
Member Author

PierreSmits commented Dec 4, 2021

Good morning Jacques,

You mean like there are no triggers (menu-items) and request- and view-maps to show screens and data for these two? or other code elements referencing the screen?

@JacquesLeRoux
Copy link
Contributor

It's about no calls for them in ConfigScreens.xml

According to the definition in widget-form.xsd the use of a combination of a form with type="list" is deprecated in favour of a grid.
Refactor various list forms into grids.
Refactor various list form references in screens.

Overlooked in ConfigScreens.xml references regarding ProductConfigOptionList and ListProductConfigItem

modified in product component:
ConfigScreens.xml: from form ref to grid ref , additional cleanup
ConfigForms.xml: from form definition with list ref to grid definition with list ref, additional clean-up
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 4, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@PierreSmits
Copy link
Member Author

Hi Jacques,

I have corrected the situation with an additional commit to this PR. I overlooked the references to those two definitions in ConfigScreens.xml.

Your 'Is it normal' threw me off. My answer to the more generic question can only, after more than 13 years of reviewing OFBiz code and practices, and contributing to this project, be that it is normal that things get overlooked by both non- and privileged contributors.

Thanks again for your feedback and collaboration.

@JacquesLeRoux JacquesLeRoux merged commit d60b234 into apache:trunk Dec 4, 2021
@PierreSmits PierreSmits deleted the OFBIZ-11345-List-and-Grid branch December 4, 2021 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants