[asterisk-bugs] [JIRA] (ASTERISK-24927) app_voicemail (IMAP support) function save_to_folder: creates wrong folder

[Rusty Newton (JIRA)]

    [ https://issues.asterisk.org/jira/browse/ASTERISK-24927?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=225736#comment-225736 ] 

Rusty Newton commented on ASTERISK-24927:
-----------------------------------------

I posted our full canned response just for reference and if you want to do things yourself (which helps the Asterisk community out). If you want to simply attach your patch then we can walk it through.

It is always faster if you can push it through where you are able.

> app_voicemail (IMAP support) function save_to_folder: creates wrong folder
> --------------------------------------------------------------------------
>
>                 Key: ASTERISK-24927
>                 URL: https://issues.asterisk.org/jira/browse/ASTERISK-24927
>             Project: Asterisk
>          Issue Type: Bug
>      Security Level: None
>          Components: Applications/app_voicemail/IMAP
>    Affects Versions: 13.2.0
>            Reporter: Alexei Gradinari
>            Severity: Minor
>
> If try to move message to Cust1 (number 5) the function save_to_folder tries to create Greeting folder instead of Cust1.
> The function save_to_folder calls imap_mailbox_name to get mailbox name by box id:
> {code}
>         imap_mailbox_name(mailbox, sizeof(mailbox), vms, box, 1);
> {code}
> then tries to create IMAP folder:
> {code}
>         if (mail_create(vms->mailstream, mailbox) == NIL)
> {code}
> The function imap_mailbox_name set mailbox name based on enum vm_box:
> {code}
>         if (box == NEW_FOLDER || box == OLD_FOLDER)
>                 snprintf(spec, len, "%s%s", tmp, use_folder? vms->imapfolder: "INBOX");
>         else if (box == GREETINGS_FOLDER)
>                 snprintf(spec, len, "%s%s", tmp, greetingfolder);
> {code}
> But GREETINGS_FOLDER value is 5 and imap_mailbox_name set mailbox as Greeting instead of Cust1
> To fix it we have to set GREETINGS_FOLDER value other then folders numbers
> {code}
> enum vm_box {
>         NEW_FOLDER = 0,
>         OLD_FOLDER = 1,
>         WORK_FOLDER = 2,
>         FAMILY_FOLDER = 3,
>         FRIENDS_FOLDER = 4,
>         GREETINGS_FOLDER = -1
> };
> {code}



--
This message was sent by Atlassian JIRA
(v6.2#6252)