[hal_json] Backward-compatibility - returning a global REQUEST in callDialogMethod

bt5/erp5_hal_json_style/SkinTemplateItem/portal_skins/erp5_hal_json_style/Base_callDialogMethod.py

@@ -14,9 +14,10 @@ def isFieldType(field, type_name):
 from Products.Formulator.Errors import FormValidationError, ValidationError
 from ZTUtils import make_query
 
-request = REQUEST
-if REQUEST is None:
-  request = container.REQUEST
+# Kato: I do not understand why we throw away REQUEST from parameters (hidden in **kw)
+# and use container.REQUEST just to introduce yet another global state
+# because REUQEST from arguments is a different instance than container.REQUEST!
+request = container.REQUEST
 
 # request.form holds POST data thus containing 'field_' + field.id items
 # such as 'field_your_some_field'

bt5/erp5_hal_json_style/SkinTemplateItem/portal_skins/erp5_hal_json_style/Base_callDialogMethod.xml

@@ -50,7 +50,7 @@
         </item>
         <item>
             <key> <string>_params</string> </key>
-            <value> <string>dialog_method, dialog_id, dialog_category=\'\', update_method=None, REQUEST=None, silent_mode=0, **kw</string> </value>
+            <value> <string>dialog_method, dialog_id, dialog_category=\'\', update_method=None, **kw</string> </value>
         </item>
         <item>
             <key> <string>id</string> </key>