fix createAsync wrapper, strict ordering. Settings handler remove unn…

…ecessary second param
nodemailer · Oct 23, 2023 · c389ea6 · c389ea6
1 parent 61645e4
commit c389ea6
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/mailbox-handler.js b/lib/mailbox-handler.js
@@ -26,7 +26,7 @@ class MailboxHandler {
 
     create(user, path, opts, callback) {
         this.createAsync(user, path, opts)
-            .then(mailboxData => callback(null, ...Object.values(mailboxData)))
+            .then(mailboxData => callback(null, ...[mailboxData.status, mailboxData.id]))
             .catch(err => callback(err));
     }
 
@@ -51,7 +51,7 @@ class MailboxHandler {
 
         const mailboxCountForUser = await this.database.collection('mailboxes').countDocuments({ user });
 
-        if (mailboxCountForUser > (await this.settingsHandler.get('const:max:mailboxes', {}))) {
+        if (mailboxCountForUser > (await this.settingsHandler.get('const:max:mailboxes'))) {
             const err = new Error('Mailbox creation failed with code ReachedMailboxCountLimit. Max mailboxes count reached.');
             err.code = 'CANNOT';
             err.responseCode = 400;