Commit c46fbec0 authored by Kazuhiko Shiozaki's avatar Kazuhiko Shiozaki

Skip an empty selection name that is now valid for a listbox where we don't...

Skip an empty selection name that is now valid for a listbox where we don't want to store a selection.

git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@39353 20353a03-c40f-0410-a6d1-a30d3c3de9de
parent 8158681a
...@@ -81,7 +81,8 @@ for field_path, field in skins_tool.ZopeFind(\n ...@@ -81,7 +81,8 @@ for field_path, field in skins_tool.ZopeFind(\n
if field.get_tales(\'selection_name\')!=\'\':\n if field.get_tales(\'selection_name\')!=\'\':\n
continue\n continue\n
selection_name = field.get_orig_value(\'selection_name\')\n selection_name = field.get_orig_value(\'selection_name\')\n
\n if selection_name == \'\':\n
continue\n
selection_name_dict.setdefault(\n selection_name_dict.setdefault(\n
selection_name, []).append(field_path)\n selection_name, []).append(field_path)\n
\n \n
......
2010-10-19 Kazuhiko 2010-10-19 Kazuhiko
* When checking duplicated selection names, we should use get_recursive_orig_value('selection_name') instead of getRecursiveTemplateField().get_value('selection_name'), because delegation for selection_name can end at any intermediate proxy field. * When checking duplicated selection names, we should use get_recursive_orig_value('selection_name') instead of getRecursiveTemplateField().get_value('selection_name'), because delegation for selection_name can end at any intermediate proxy field. And skip an empty selection name that is now valid for a listbox where we don't want to store a selection.
2010-10-19 yo 2010-10-19 yo
* When checking duplicated selection names, skip field libraries, as they don't matter. * When checking duplicated selection names, skip field libraries, as they don't matter.
......
624 625
\ No newline at end of file \ No newline at end of file
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment