Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Factory returns not the same as the class #7846

Open
Falke-Design opened this issue Dec 8, 2021 · 1 comment · May be fixed by #8214
Open

Factory returns not the same as the class #7846

Falke-Design opened this issue Dec 8, 2021 · 1 comment · May be fixed by #8214

Comments

@Falke-Design
Copy link
Member

Falke-Design commented Dec 8, 2021

new L.Point() has a different result as L.point().

I didn't checked all factories , but same behavior in:

Mostly in the factory is the part:

if (x === undefined || x === null) {
	return x;
}

I suggest to remove this checks. What do you think?

@johnd0e
Copy link
Collaborator

johnd0e commented Dec 13, 2021

Related: #7128, #7767

@Falke-Design Falke-Design added this to To do in Validation & Checks via automation May 4, 2022
@Falke-Design Falke-Design linked a pull request May 4, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging a pull request may close this issue.

2 participants