0001-fields-review-item-field-to-get-back-to-previous-beh.patch

Frédéric Péters, 27 octobre 2015 14:12

Voir les différences: en ligne côte à côte

Subject: [PATCH] fields: review item field to get back to previous behaviour
 (#8785)

In recent commits we tried to handle optional item field with an empty value
prepended to the list of options but while this allows the user to leave
optional fields uncompleted this is not the same behaviour we add.

Return to the previous behaviour.  The only way for an item field to be skipped
is now to be displayed as a <select> field with the hint put as the first
element.

 tests/test_datasource.py |  2 +-
 tests/test_fields.py     | 46 ++++++++++++++++++++++++++++++++++++++++++----
 wcs/fields.py            | 17 +++++------------
 wcs/qommon/form.py       |  2 +-
 4 files changed, 49 insertions(+), 18 deletions(-)

tests/test_datasource.py
211	211	field.add_to_form(form)
212	212	widget = form.get_widget('f1')
213	213	assert widget is not None
214		assert widget.options == [(None, '', 'None'), ('1', 'un', '1'), ('2', 'deux', '2')]
	214	assert widget.options == [('1', 'un', '1'), ('2', 'deux', '2')]

                 required=False)
         form = Form()
         field.add_to_form(form)
         assert str(form.render()).count('<option selected="selected" value="None"></option>') == 1 # None
         assert str(form.render()).count('<option') == 4 # 3 + None as first item
         assert str(form.render()).count('<option') == 3
         field = fields.ItemField(id='1', label='Foobar', items=['a', 'b', 'c'],
                 required=False, hint='Bla bla bla')
-...
         assert str(form.render()).count('<option') == 4
         # without any item
         field = fields.ItemField(id='1', label='Foobar', items=[])
         field = fields.ItemField(id='1', label='Foobar', items=None)
         form = Form()
         field.add_to_form(form)
         assert str(form.render()).count('<option') == 1
         # without any item and not being required
         field = fields.ItemField(id='1', label='Foobar', items=[], required=False)
         field = fields.ItemField(id='1', label='Foobar', items=None, required=False)
         form = Form()
         field.add_to_form(form)
         assert str(form.render()).count('<option') == 1
         # without any item but with an hint
         field = fields.ItemField(id='1', label='Foobar', items=None, hint='Bla bla bla')
         form = Form()
         field.add_to_form(form)
         assert str(form.render()).count('<option value="">Bla bla bla</option>') == 1 # ---
         assert str(form.render()).count('<option') == 1
     def test_item_render_as_radio():
         field = fields.ItemField(id='1', label='Foobar', show_as_radio=True, items=['a', 'b', 'c'])
         form = Form()
         field.add_to_form(form)
         assert str(form.render()).count('"radio"') == 3
         field = fields.ItemField(id='1', label='Foobar', items=['a', 'b', 'c'],
                 show_as_radio=True, required=False)
         form = Form()
         field.add_to_form(form)
         assert str(form.render()).count('"radio"') == 3
         field = fields.ItemField(id='1', label='Foobar', items=['a', 'b', 'c'],
                 show_as_radio=True, required=False, hint='Bla bla bla')
         form = Form()
         field.add_to_form(form)
         assert str(form.render()).count('"radio"') == 3
         # without any item
         field = fields.ItemField(id='1', label='Foobar', items=None,
                 show_as_radio=True)
         form = Form()
         field.add_to_form(form)
         assert str(form.render()).count('"radio"') == 1
         # without any item and not being required
         field = fields.ItemField(id='1', label='Foobar', items=None,
                 required=False, show_as_radio=True)
         form = Form()
         field.add_to_form(form)
         assert str(form.render()).count('"radio"') == 1

             WidgetField.__init__(self, **kwargs)
         def get_options(self):
             if not self.data_source:
                 options = self.items[:]
             else:
                 options = data_sources.get_items(self.data_source)
             if options and not self.required:
                 if type(options[0]) is str:
                     options[:0] = [None]
                 elif len(options[0]) == 2:
                     options[:0] = [(None, '')]
                 elif len(options[0]) == 3:
                     options[:0] = [(None, '', None)]
             return options
             if self.data_source:
                 return data_sources.get_items(self.data_source)
             if self.items:
                 return self.items[:]
             return []
         def perform_more_widget_changes(self, form, kwargs, edit = True):
             real_data_source = data_sources.get_real(self.data_source)

             if not self.separate_hint() and self.hint:
                 r = htmltag('option', value='', selected=None)
                 tags.append(r + htmlescape(self.hint) + htmltext('</option>'))
                 if not self.required:
                 if self.options[0][0] is None:
                     # hint has been put as first element, skip the default empty
                     # value.
                     options = self.options[1:]
+    -

Projet

Général

Profil

Produits Entr'ouvert » w.c.s.

0001-fields-review-item-field-to-get-back-to-previous-beh.patch