MicroDroid / vue-materialize-datatable

A fancy Materialize CSS datatable VueJS component.
https://overcoder.dev/vue-materialize-datatable
MIT License
179 stars 67 forks source link

Fixing nested path searching issue #32

Closed SafAlhaji closed 6 years ago

SafAlhaji commented 6 years ago

Searching will not consider fields that are given a nested path (e.g. attributes.name), since getFn is overridden and looks straight into obj[path], which will give obj["attributes.name"] in the mentioned example. The dig method should be used to look into nested paths properly.

thepill commented 6 years ago

Should we make this configurable e.g. deepseach: true|false?

SafAlhaji commented 6 years ago

I think such configuration will preferably be set to true most of the times, don't think anyone would want a search that works only on first-level properties, while expecting it to work on all displayed columns.

MicroDroid commented 6 years ago

Thanks for your efforts