Commit b43750df authored by Tomáš Peterka's avatar Tomáš Peterka

[base_edit] if back to elif, more pep8 compliant, spelling

parent e308b004
...@@ -149,7 +149,7 @@ def editMatrixBox(matrixbox_field, matrixbox): ...@@ -149,7 +149,7 @@ def editMatrixBox(matrixbox_field, matrixbox):
tab_ids = map(lambda x: x[0], tabs) tab_ids = map(lambda x: x[0], tabs)
extra_dimension_category_list_list = [[category for category, label in dimension_list] for dimension_list in extra_dimension_list_list] extra_dimension_category_list_list = [[category for category, label in dimension_list] for dimension_list in extra_dimension_list_list]
# There are 3 cases # There are 4 cases
# Case 1: we do 1 dimensional matrix # Case 1: we do 1 dimensional matrix
# Case 2: we do 2 dimensional matrix # Case 2: we do 2 dimensional matrix
# Case 3: we do 2 dimensional matrix + tabs # Case 3: we do 2 dimensional matrix + tabs
...@@ -227,7 +227,7 @@ try: ...@@ -227,7 +227,7 @@ try:
if(field_meta_type == 'ListBox'): if(field_meta_type == 'ListBox'):
editListBox(field, request.get(field.id)) editListBox(field, request.get(field.id))
if(field_meta_type == 'MatrixBox'): elif(field_meta_type == 'MatrixBox'):
editMatrixBox(field, request.get(field.id)) editMatrixBox(field, request.get(field.id))
# Return parsed values # Return parsed values
......
...@@ -6,7 +6,7 @@ ...@@ -6,7 +6,7 @@
:param silent: int (0|1) means that the edit action is not invoked by a form :param silent: int (0|1) means that the edit action is not invoked by a form
submit but rather by an internal code thus the return value submit but rather by an internal code thus the return value
contains as much usefull info as possible contains as much useful info as possible
TODO: split the generic form validation logic TODO: split the generic form validation logic
from the context update logic from the context update logic
...@@ -101,7 +101,7 @@ def editMatrixBox(matrixbox_field, matrixbox): ...@@ -101,7 +101,7 @@ def editMatrixBox(matrixbox_field, matrixbox):
portal_type = matrixbox_field.get_value('cell_portal_type') portal_type = matrixbox_field.get_value('cell_portal_type')
getter_method = matrixbox_field.get_value('getter_method') getter_method = matrixbox_field.get_value('getter_method')
if getter_method not in (None, ''): if getter_method not in (None, ''):
matrix_context = getattr(context,getter_method)() matrix_context = getattr(context, getter_method)()
else: else:
matrix_context = context matrix_context = context
...@@ -147,7 +147,7 @@ def editMatrixBox(matrixbox_field, matrixbox): ...@@ -147,7 +147,7 @@ def editMatrixBox(matrixbox_field, matrixbox):
tab_ids = map(lambda x: x[0], tabs) tab_ids = map(lambda x: x[0], tabs)
extra_dimension_category_list_list = [[category for category, label in dimension_list] for dimension_list in extra_dimension_list_list] extra_dimension_category_list_list = [[category for category, label in dimension_list] for dimension_list in extra_dimension_list_list]
# There are 3 cases # There are 4 cases
# Case 1: we do 1 dimensional matrix # Case 1: we do 1 dimensional matrix
# Case 2: we do 2 dimensional matrix # Case 2: we do 2 dimensional matrix
# Case 3: we do 2 dimensional matrix + tabs # Case 3: we do 2 dimensional matrix + tabs
...@@ -225,7 +225,7 @@ try: ...@@ -225,7 +225,7 @@ try:
# Kato: following fields should implement 'Editors' so we can delete those lines # Kato: following fields should implement 'Editors' so we can delete those lines
if(field_meta_type == 'ListBox'): if(field_meta_type == 'ListBox'):
editListBox(field, request.get(field.id)) editListBox(field, request.get(field.id))
if(field_meta_type == 'MatrixBox'): elif(field_meta_type == 'MatrixBox'):
editMatrixBox(field, request.get(field.id)) editMatrixBox(field, request.get(field.id))
# Return parsed values # Return parsed values
......
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