Merge pull request #32 from millotp/main

fix: 0 is forbidden for day
adhocore · Sep 20, 2023 · 06e7337 · 06e7337
2 parents f878463 + 8aee396
commit 06e7337
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/checker.go b/checker.go
@@ -81,14 +81,14 @@ func (c *SegmentChecker) isOffsetDue(offset string, val, pos int) (bool, error)
 		return false, err
 	}
 
-	if !isWeekDay && (val == 0 || nval == 0) {
-		return nval == 0 && val == 0, nil
-	}
-
 	if nval < bounds[0] || nval > bounds[1] {
 		return false, fmt.Errorf("segment#%d: '%s' out of bounds(%d, %d)", pos, offset, bounds[0], bounds[1])
 	}
 
+	if !isWeekDay && (val == 0 || nval == 0) {
+		return nval == 0 && val == 0, nil
+	}
+
 	return nval == val || (isWeekDay && nval == 7 && val == 0), nil
 }
 

diff --git a/gronx_test.go b/gronx_test.go
@@ -243,6 +243,7 @@ func errcases() []Case {
 		{"1-Z * * * *", "2011-07-01 00:01:00", false, ""},
 		{"1-Z/2 * * * *", "2011-07-01 00:01:00", false, ""},
 		{"Z-Z/2 * * * *", "2011-07-01 00:01:00", false, ""},
+		{"* * 0 * *", "2011-07-01 00:01:00", false, ""},
 		{"* * * W * *", "", false, ""},
 		{"* * * ZW * *", "", false, ""},
 		{"* * * * 4W", "2011-07-01 00:00:00", false, ""},