Recurrence: RRULE not validated at add/edit time (only fails later at done) #13
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
parse_recurrence(src/cli/recurrence_parse.rs) stores the rawrrulestring without ever callingparse_rrule. A malformed or unsupported rule is accepted silently and only fails later when the user runsnext done(which then also fails the whole done operation).Fix: validate the rule when it is set (
add/edit) by parsing it throughparse_rruleand rejecting with a clear error. Add tests for malformed rules: missing FREQ,INTERVAL=0, empty string, unknown FREQ,BYMONTHDAY=0/negative, garbage.REQUIREMENTS §7 doesn't state when validation occurs — document it. Found in review.