j-hellenberg commented on PR #2588:
URL: https://github.com/apache/james-project/pull/2588#issuecomment-2577340477
> `FileIntoAction` follows sieve convention to use `/` as a folder delimiter
and needs a translation to the actual deleimiter. Email, i know...
I saw that the `execute` method does
`destinationMailbox.replace(FileIntoAction.HIERARCHY_DELIMITER, '/')`
(`destinationMailbox.replace('.', '/')`, in other words).
So, to be clear: If the `MailboxConstants.FOLDER_DELIMITER` is anything
except '/', this line needs to be
`destinationMailbox.replace(MailboxConstants.FOLDER_DELIMITER, '/')` (and it
works fine if delimiter is already `/`, in which case this is a noop)?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org
For additional commands, e-mail: notifications-h...@james.apache.org
