No description provided.
Bug 1863095 Comment 0 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
After we introduced a `MailFolderId`, this patch allows using the `MailFolderId` in addition to the `MailFolder` object in all folder related functions in MV2. Following a discussion in the last add-on developer call, where the following was raised, I will change the `MailFolderId` to be an integer in one of the next patches. The reason for this is: - the current API allows using an arbitrary path, and even though the docs say to NOT create a path manually, add-on developers do that, which causes bugs and headaches. - the current `MailFolderId` has the same flaw - an integer id will make the API more robust - I also hope that I can make the integer id follow folder moves. **As a consequence, this patch enforces the usage of the `MailFolderId` in MV3, by deprecating the `MailFolder` object as input parameter.** There is a loose end which will be tackled in the next patch: Some functions (like the copy/create/move folder function) allow specifying an account to access the root folder. This will be solved by introducing a real root folder, which will have a `MailFolderId` and can be used like any other folder, and we no longer need to use accounts as "destination" for certain folder operations in MV3. I updated all except 3 tests to use `MailFolderId`s, so we can test both supported variations in MV2. In all tests which have been adjusted, I also minimized creating `MailFolder` objects or `MailFolderId`s manually