diff --git a/bt5/erp5_hal_json_style/SkinTemplateItem/portal_skins/erp5_hal_json_style/Base_callDialogMethod.py b/bt5/erp5_hal_json_style/SkinTemplateItem/portal_skins/erp5_hal_json_style/Base_callDialogMethod.py index d9e70f9df55dc1389b30088fc7adc1297eaf53ef..816061adc13a84340c5867a14513fb84fd52860c 100644 --- a/bt5/erp5_hal_json_style/SkinTemplateItem/portal_skins/erp5_hal_json_style/Base_callDialogMethod.py +++ b/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' diff --git a/bt5/erp5_hal_json_style/SkinTemplateItem/portal_skins/erp5_hal_json_style/Base_callDialogMethod.xml b/bt5/erp5_hal_json_style/SkinTemplateItem/portal_skins/erp5_hal_json_style/Base_callDialogMethod.xml index 36936eb7074bc73cefefd010b2c67ecdd0cef4b6..a2f51df0c43ec9ed4816310f8a40c9c81ee5a408 100644 --- a/bt5/erp5_hal_json_style/SkinTemplateItem/portal_skins/erp5_hal_json_style/Base_callDialogMethod.xml +++ b/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>