search

DaniSchenk / moment-feiertage

MomentJS Plugin: Check if a date is a german holiday (Feiertag)
https://www.npmjs.com/package/moment-feiertage
32 stars 9 forks source link

Fix für Probleme mit Objekt-Prototypen #16

Closed ACertainCoder closed 3 years ago

ACertainCoder commented 3 years ago

resolves #15

Ich habe den Part umgeschrieben und einen weiteren Unit Test für das aktuelle Datum eingefügt. Den Fehler aus meinem eigenen Projekt konnte ich bei den Unit Tests leider nicht reproduzieren, allerdings habe ich noch keinen Test in Bezug auf das aktuelle Datum gefunden, deswegen habe ich noch einen eingefügt.

In meinem eigenen Projekt konnte ich den Fehler auf jeden Fall vorübergehend durch manuelles abändern der Datei im node_modules Ordner beheben.

(PS: ich war mir unsicher, ob ich den Version Bump direkt vornehmen soll, ggf. mache ich das nochmal neu)

DaniSchenk commented 3 years ago

hey @ACertainCoder, danke für deinen PR!

Wenn ich es richtig verstanden habe, dann war das Problem, dass die for...in Schleife auch den Prototype des Objects durchlaufen hat, oder? Alternative Lösung in Mozilla Docs

Leider konnte ich deinen Fehler auch nicht reporduzieren 🤔. Aber egal, ich push mal zu npm...

ACertainCoder commented 3 years ago

Danke dir