search

dncrews / angular-elastic-builder

This is an Angular.js directive for building an Elasticsearch query. You just give it the fields and can generate a query for it.
Other
83 stars 38 forks source link

checkboxes (for type multi) are deselected when query is parsed #5

Closed oguennec closed 8 years ago

oguennec commented 8 years ago

@dncrews : When query is parsed multi values in query group are now converted to false whereas they should be converted to true so the checkboxes show as checked. This regression has been introduced with commit e79a94b.

tienvx commented 8 years ago

I can confirm that this will be fixed by the commit of @oguennec

ohneworte commented 8 years ago

Is #5 truly fixed? It's still not working for me and also not in the demo (http://dncrews.com/angular-elastic-builder/examples) where test.state.multi is filled with:

{
  'terms': {
    'test.state.multi': [ 'AZ', 'CT' ]
  }
}
dncrews commented 7 years ago

@ohneworte I forgot to update the example at that time. Can you reverify?

ohneworte commented 7 years ago

@dncrews, sorry, but I don't see any changes - should I? In the demo of your website (http://dncrews.com/angular-elastic-builder/examples) the checkboxes in "test.state.multi" are still empty, although initialized differently (see http://dncrews.com/angular-elastic-builder/examples/exampleApp.js).