Fixed: Form fields are not properly populated with values from the context due to a regression introduced by commit 69697d1 (OFBiz-13331)#936
Conversation
…ntext due to a regression introduced by commit 69697d1 (OFBiz-13331) This commit restores the condition that determines whether request or context parameters are used to the behavior prior to commit 69697d1. It also implements the new useRequestParameters.<form_name> context attribute for ModelGrid forms, which was introduced in that commit.
|
|
I have verified this pull request: it looks good to me and it fixes a relevant regression. Considering the severity of the issue, I suggest to merge this pull request as soon as possible. I could do it but I'd like to wait for @nmalin to verify if the use case addressed by his commit 69697d1 and described in OFBIZ-13331 is properly supported by this pull request.
|
|
@nmalin I decided to merge the PR to restore the form functionality, which was currently broken on trunk and on our demo instance. You can now review your work on OFBIZ-13331 without any time pressure. |




This commit restores the condition that determines whether request or context parameters are used to the behavior prior to commit 69697d1. It also implements the new useRequestParameters.<form_name> context attribute for ModelGrid forms, which was introduced in that commit.