v0.6.21 TypeError: Cannot convert object to primitive value about flexsearch HOT 11 CLOSED

Supporting both formats is possible. The doc configuration needs a way to keep JSON standard compatibility.

from flexsearch.

@mahnunchik Thanks for the report. Could you provide me the used configuration or a short example please?

from flexsearch.

Hi @ts-thomas

import FlexSearch from 'flexsearch';
import waypoints from './data/waypoints';

const index = new FlexSearch({
  doc: {
    id: "code",
    field: {
      code: {
        split: /\s+/,
        tokenize: "strict",
      },
      title: {
        split: /[\s()-]+/,
        tokenize: "forward",
      },
    },
  },
});

//console.time('index');
index.add(waypoints);
//console.timeEnd('index');

export default index;

Data: https://gist.github.com/mahnunchik/6920a1039a3da4ef22177521a800642d

from flexsearch.

@mahnunchik Sorry for the late answer. The issue here was the split-field needs a string value (to keep json compatibility):

const index = new FlexSearch({
  doc: {
    id: "code",
    field: {
      code: {
        split: "/\s+/",
        tokenize: "strict",
      },
      title: {
        split: "/[\s()-]+/",
        tokenize: "forward",
      },
    },
  },
});

from flexsearch.

@ts-thomas it would be more predictable to have usual regexp in normal js configuration, not a string of js regexp. Or support both formats.

from flexsearch.

Hi @ts-thomas

It seems "/\s+/" from your comment not works as expected in 0.6.22.

from flexsearch.

Hi @ts-thomas
it doesn't work for me as well. the content.split(this.split) not works as this.split is just a string and not regex.
if i use regex in the field definition the clone_object is not working correctly:
clone_object({a:/a/}) = {a:{}}

from flexsearch.

patch to clone_object:

		function clone_object(obj) {
			const clone = create_object();

			for (const key in obj) {
				if (obj.hasOwnProperty(key)) {
					const value = obj[key];

					if (is_array(value)) {
						clone[key] = value.slice(0);
					} else if (value instanceof RegExp) {
						clone[key] = value;
					} else if (is_object(value)) {
						clone[key] = clone_object(value);
					} else {
						clone[key] = value;
					}
				}
			}

			return clone;
		}

from flexsearch.

@wamir79 thanks for providing a fix.

from flexsearch.

Should this issue be closed?

from flexsearch.

Please change over to the version >= 0.7.x which solves that issue, thanks a lot.

from flexsearch.