xhtml_style: fill request.form during Base_renderForm

Base_renderForm was using request.set - which only sets in request.other,
and this causes Base_renderForm to behaves differently from Base_redirect
with forms looking explicitly in request.form, like for example listbox
does before calling the list method.

See merge request !2032

bt5/erp5_configurator_standard/TestTemplateItem/portal_components/test.erp5.testStandardConfigurationWorkflow.py

@@ -33,15 +33,7 @@ from DateTime import DateTime
 from Products.ERP5Type.tests.Sequence import SequenceList
 from erp5.component.module.ConfiguratorTestMixin import \
     TestLiveConfiguratorWorkflowMixin
-
-
-class FakeFormBoxEditor(dict):
-  """Quick replacement for formbox editor.
-  """
-  __allow_access_to_unprotected_subobjects__ = 1
-
-  def as_dict(self):
-    return self
+from Products.ERP5Form.FormBox import FormBoxEditor
 
 
 class StandardConfigurationMixin(TestLiveConfiguratorWorkflowMixin):
@@ -1169,6 +1161,16 @@ class StandardConfigurationMixin(TestLiveConfiguratorWorkflowMixin):
         ],
     )
 
+    form_box_editor = FormBoxEditor(
+      result=(
+        {
+          # this is what GenericSolver_viewConfigurationFormBox/my_tested_property_list passes as default hidden
+          'tested_property_list': solver_decision.getCausalityValue().getTestedPropertyList(),
+        },
+        [],
+      ),
+      context=None,
+    )
     listbox = (
         {
             'listbox_key':
@@ -1177,9 +1179,7 @@ class StandardConfigurationMixin(TestLiveConfiguratorWorkflowMixin):
                 'portal_solvers/Accept Solver',
             # this is what GenericSolver_viewConfigurationFormBox/my_tested_property_list passes as default hidden
             'solver_configuration':
-                FakeFormBoxEditor(
-                    tested_property_list=solver_decision.getCausalityValue()
-                    .getTestedPropertyList()),
+                form_box_editor,
             'comment':
                 '',
         },)
@@ -1607,17 +1607,24 @@ class StandardConfigurationMixin(TestLiveConfiguratorWorkflowMixin):
         ],
     )
 
+    form_box_editor = FormBoxEditor(
+      result=(
+        {
+          # this is what GenericSolver_viewConfigurationFormBox/my_tested_property_list passes as default hidden
+          'tested_property_list': solver_decision.getCausalityValue().getTestedPropertyList(),
+        },
+        [],
+      ),
+      context=None,
+    )
     listbox = (
         {
             'listbox_key':
                 solver_decision.getPath(),
             'solver':
                 'portal_solvers/Accept Solver',
-            # this is what GenericSolver_viewConfigurationFormBox/my_tested_property_list passes as default hidden
             'solver_configuration':
-                FakeFormBoxEditor(
-                    tested_property_list=solver_decision.getCausalityValue()
-                    .getTestedPropertyList()),
+                form_box_editor,
             'comment':
                 '',
         },)

bt5/erp5_ui_test/PathTemplateItem/portal_tests/page_templates_zuite/testDialogActions.zpt

@@ -50,6 +50,16 @@
   <td>"Update" action is done with "A value from the user".</td>
   <td></td>
 </tr>
+<tr>
+  <td>assertValue</td>
+  <td>//input[@name="field_your_string_field"]</td>
+  <td>A value from the user</td>
+</tr>
+<tr>
+  <td>assertTextPresent</td>
+  <td>A value pushed by update action</td>
+  <td></td>
+</tr>
 <tr>
   <td>clickAndWait</td>
   <td>Base_callDialogMethod:method</td>
@@ -87,4 +97,4 @@
   <td></td>
 </tr>
 </body>
-</html>
+</html>
\ No newline at end of file

bt5/erp5_ui_test/SkinTemplateItem/portal_skins/erp5_ui_test/Foo_updateAction.py

 """Call the Foo_viewDummyDialog"""
-kw.update(context.REQUEST.form)
-return context.ERP5Site_redirect("%s/Foo_viewDummyDialog" % context.absolute_url(),
-        keep_items={'portal_status_message': '"Update" action is done with "%s".' % string_field}, **kw)
+return context.Base_renderForm(
+  'Foo_viewDummyDialog',
+  message='"Update" action is done with "%s".' % string_field,
+  keep_items={
+    'key_posted_during_on_update': 'A value pushed by update action'
+  }
+)

bt5/erp5_ui_test/SkinTemplateItem/portal_skins/erp5_ui_test/Foo_viewDummyDialog.xml

@@ -76,6 +76,7 @@
                     <value>
                       <list>
                         <string>your_string_field</string>
+                        <string>your_inspect_request</string>
                       </list>
                     </value>
                 </item>

product/ERP5/bootstrap/erp5_xhtml_style/SkinTemplateItem/portal_skins/erp5_xhtml_style/Base_renderForm.py

@@ -22,5 +22,6 @@ if REQUEST is None:
 
 for key, value in keep_items.items():
   REQUEST.set(key, value)
+  REQUEST.form[key] = value
 
 return getattr(context, form_id)()