Skip to content

Commit 8935d90

Browse files
refactor: update getSendMessageDraftRequestConfig() to assemble correct url
explicitly states that method is POST now, too refs conjoon/lumen-app-email#58
1 parent 7faa170 commit 8935d90

File tree

2 files changed

+14
-4
lines changed

2 files changed

+14
-4
lines changed

src/view/mail/message/editor/MessageEditorViewController.js

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -601,9 +601,19 @@ Ext.define("conjoon.cn_mail.view.mail.message.editor.MessageEditorViewController
601601
throw("\"baseAddress\" must be a string");
602602
}
603603

604+
let url = [
605+
baseAddress,
606+
"MailAccounts",
607+
encodeURIComponent(messageDraft.getCompoundKey().getMailAccountId()),
608+
"MailFolders",
609+
encodeURIComponent(messageDraft.getCompoundKey().getMailFolderId()),
610+
"MessageItems",
611+
encodeURIComponent(messageDraft.getCompoundKey().getId())
612+
].join("/");
613+
604614
return {
605-
url: l8.unify(baseAddress + "/SendMessage", "/", "://"),
606-
params: messageDraft.getCompoundKey().toObject()
615+
url: l8.unify(url, "/", "://"),
616+
method: "POST"
607617
};
608618

609619
},

tests/src/view/mail/message/editor/MessageEditorViewControllerTest_03.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,8 +80,8 @@ StartTest(async t => {
8080
t.expect(
8181
controller.getSendMessageDraftRequestConfig(messageDraft, ".//cn_mail///")
8282
).toEqual({
83-
url: "./cn_mail/SendMessage",
84-
params: messageDraft.getCompoundKey().toObject()
83+
url: "./cn_mail/1/1/1",
84+
method: "POST"
8585
});
8686

8787
let tmp = Ext.Ajax.request;

0 commit comments

Comments
 (0)