Skip to content

Commit

Permalink
Merge pull request #122 from superosku/hotfix/121-fix-grouped-query-s…
Browse files Browse the repository at this point in the history
…elect-multiple-field-validator

Fix GroupedQuerySelectMultipleField validator
  • Loading branch information
kvesteri committed Aug 29, 2017
2 parents 11e6e5c + 29418b9 commit 358333d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions tests/test_query_select_field.py
Expand Up @@ -348,6 +348,7 @@ def test_multiple_values_without_query_factory(self):

form = MyForm(DummyPostData(cities=['1', '666']))
form.cities.query = self.session.query(self.City)
assert not form.validate()
assert [x.id for x in form.cities.data] == [1]
assert not form.validate()
form.populate_obj(obj)
Expand Down
2 changes: 1 addition & 1 deletion wtforms_alchemy/fields.py
Expand Up @@ -568,7 +568,7 @@ def process_formdata(self, valuelist):
self._formdata = set(valuelist)

def pre_validate(self, form):
if self._invalid_formdata:
if self.data and self._invalid_formdata:
raise ValidationError(self.gettext('Not a valid choice'))
elif self.data:
obj_list = list(x[1] for x in self._get_object_list())
Expand Down

0 comments on commit 358333d

Please sign in to comment.