Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

crash on disabledSeconds is not a function

Offen
  • Stimmt etwas nicht?
#694 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
1/5
Geschätzter Aufwand
Unter einer Stunde
Anfängerfreundlichkeit
45/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Veraltet
Tech-Stack
javascript, react
Bereich
frontend

Rechercherichtung

Beginne in node_modules/rc-picker/es/hooks/useTimeSelection.js und untersuche, wie disabledMinutes und disabledSeconds während der Zeitauswahl aufgerufen werden. Reproduziere den gemeldeten Absturz mit einem disabledSeconds-Wert, der keine Funktion ist, und überprüfe anschließend, dass gültige disabled-Callbacks weiterhin funktionieren und der Fall mit einem Nicht-Funktionswert nicht mehr abstürzt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Hi! 👋

Firstly, thanks for your work on this project! 🙂

Today I used patch-package to patch [email protected] for the project I'm working on.

Here is the diff that solved my problem:

diff --git a/node_modules/rc-picker/es/hooks/useTimeSelection.js b/node_modules/rc-picker/es/hooks/useTimeSelection.js
index 9c4d47c..8f272be 100644

@@ -14,7 +14,7 @@ export default function useTimeSelection(_ref) {
     var mergedHour = newHour < 0 ? generateConfig.getHour(now) : newFormattedHour;
     var mergedMinute = newMinute < 0 ? generateConfig.getMinute(now) : newMinute;
     var mergedSecond = newSecond < 0 ? generateConfig.getSecond(now) : newSecond;
-    var newDisabledMinutes = disabledMinutes && disabledMinutes(mergedHour);
+    var newDisabledMinutes = disabledMinutes && typeof disabledMinutes=== "function" && disabledMinutes(mergedHour);
     if (newDisabledMinutes !== null && newDisabledMinutes !== void 0 && newDisabledMinutes.includes(mergedMinute)) {
       // find the first available minute in minutes
       var availableMinute = minutes.find(function (i) {
@@ -26,7 +26,7 @@ export default function useTimeSelection(_ref) {
         return null;
       }
     }
-    var newDisabledSeconds = disabledSeconds && disabledSeconds(mergedHour, mergedMinute);
+    var newDisabledSeconds = disabledSeconds && typeof disabledSeconds === "function" && disabledSeconds(mergedHour, mergedMinute);
     if (newDisabledSeconds !== null && newDisabledSeconds !== void 0 && newDisabledSeconds.includes(mergedSecond)) {
       // find the first available second in seconds
       var availableSecond = seconds.find(function (i) {

This issue body was partially generated by patch-package.

Vorherrschende Sprache
TypeScript
Sterne
304
Forks
347
Ø Merge
1 T. 2 Std.
Gemergte PRs (30 T.)
2

Entwicklungsumgebung

Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus react-component/picker

Alle Issues in react-component/picker

Ähnliche Issues

Weitere Issues zu TypeScript

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.