fix: handle postgrest special characters in filters

[inian]

fixes #164

This only quotes values and not column names. Previous bugs were due to quoting column names.

I also add in a check to only quote the value if the filter value contains a postgrest special character.

[soedirgo]

LGTM 👍 though I'd wait for steve's response on properly quoting " itself

[steve-chavez]

Something I've noticed is that the cleanFilterArray function is also applied to the cs, cd and ov operators and those don't need special handling. Their filter value is just a string, cleanFilterArray could be removed from them.

postgrest-js/test/filters.ts

Lines 65 to 68 in e5f5dfa

	test('cs', async () => {
	const res = await postgrest.from('users').select().cs('age_range', '[1,2)')
	expect(res).toMatchSnapshot()
	})

postgrest-js/test/filters.ts

Lines 70 to 73 in e5f5dfa

	test('cd', async () => {
	const res = await postgrest.from('users').select().cd('age_range', '[1,2)')
	expect(res).toMatchSnapshot()
	})

postgrest-js/test/filters.ts

Lines 100 to 103 in e5f5dfa

	test('ov', async () => {
	const res = await postgrest.from('users').select().ov('age_range', '[2,25)')
	expect(res).toMatchSnapshot()
	})

[soedirgo]

Gotcha, thanks Steve!

[soedirgo]

Should be good now @inian, feel free to merge 👍

[inian]

Thanks Bobbie and Steve!

[github-actions]

🎉 This PR is included in version 0.26.1 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀