Skip to content

managesieve: handle string-list format values for date test#10075

Closed
johndoh wants to merge 1 commit intoroundcube:masterfrom
johndoh:vacationui
Closed

managesieve: handle string-list format values for date test#10075
johndoh wants to merge 1 commit intoroundcube:masterfrom
johndoh:vacationui

Conversation

@johndoh
Copy link
Copy Markdown
Contributor

@johndoh johndoh commented Jan 24, 2026

possible fix for #10074 by forcing the date argument to always be a string.

@johndoh
Copy link
Copy Markdown
Contributor Author

johndoh commented Jan 24, 2026

failed test refers to program/actions/mail/folder_purge.php and does not appear to be related to this PR

@alecpl
Copy link
Copy Markdown
Member

alecpl commented Jan 25, 2026

This is not a proper fix because we have support for string-list in the general filter UI. We only needed a fix for the rcube_sieve_vacation class. Fixed in 719b8be.

There's another issue you could take a look at - #10076

@alecpl alecpl closed this Jan 25, 2026
@johndoh johndoh deleted the vacationui branch January 25, 2026 09:32
c4539 pushed a commit to c4539/roundcubemail that referenced this pull request Feb 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants