Expose defaultCleanOptions about meteor-collection2

[Prinzhorn]

Found it https://github.com/longshotlabs/simpl-schema#set-default-options-for-one-schema

[Prinzhorn]

Actually, let's keep it open for the global version (exposing

meteor-collection2/package/collection2/collection2.js

Lines 18 to 24 in 84a97a6

	const defaultCleanOptions = {
	filter: true,
	autoConvert: true,
	removeEmptyStrings: true,
	trimStrings: true,
	removeNullsFromArrays: false,
	};

somehow). Because I still need to put this in every single constructor.

[harryadel]

Actually, let's keep it open for the global version (exposing somehow). Because I still need to put this in every single constructor.

If the sole purpose of keeping this issue open is for future reference then it's better to close it.

[Prinzhorn]

If the sole purpose of keeping this issue open is for future reference then it's better to close it.

No, as a feature request to expose the defaultCleanOptions options. The original issue where basically two feature requests of which one I just didn't find originally and it already exists.

[harryadel]

Please bear with me as I try to understand this.

No, as a feature request to expose the defaultCleanOptions options

Isn't already exposed as you can assign it along with your schema? Can you please give me a code example of what you want to achieve? Thanks.

[Prinzhorn]

Can you please give me a code example of what you want to achieve?

I want the same sane options for every single schema without having to put it into every single SimpleSchema constructor (can be forgotten or inconsistent -> source of bugs). If I could update defaultCleanOptions then this would work for all schemas automatically.

Replacing const defaultCleanOptions with Collection2.defaultCleanOptions for example would allow exactly that.

master...Prinzhorn:patch-1

[harryadel]

Isn't this already achievable via https://github.com/longshotlabs/simpl-schema#set-default-options-for-all-schemas ?

[harryadel]

I've a PR in the works but while I was about to add notes to the README I found about the ability to set this in simpl-schema as referenced in my previous comment. @Prinzhorn

[Prinzhorn]

@harryadel thanks for looking into this. It does not quite solve the same problem. Because what if I'm using the simpl-schema package in the same project for other unrelated purposes. Your draft PR would allow the same options for all the attachSchema use-cases while not messing with the rest.

[harryadel]

I see. If that's the case then I'll take your word for it. Please review #434 There's something off with the tests that I'm fixing right now. Once it's set and you reviewed it we can merge and release a new version.