Почему следующее обещание JavaScript возвращает все вложения электронной почты?

Question

Здесь я извлекаю следующий фрагмент кода:

https://github.com/chadxz/imap-simple

imaps.connect(config).then(function (connection) {

connection.openBox('INBOX').then(function () {

    // Fetch emails from the last 24h
    var delay = 24 * 3600 * 1000;
    var yesterday = new Date();
    yesterday.setTime(Date.now() - delay);
    yesterday = yesterday.toISOString();
    var searchCriteria = ['UNSEEN', ['SINCE', yesterday]];
    var fetchOptions = { bodies: ['HEADER.FIELDS (FROM TO SUBJECT DATE)'], struct: true };

    // retrieve only the headers of the messages
    return connection.search(searchCriteria, fetchOptions);
}).then(function (messages) {

    var attachments = [];

    messages.forEach(function (message) {
        var parts = imaps.getParts(message.attributes.struct);
        attachments = attachments.concat(parts.filter(function (part) {
            return part.disposition && part.disposition.type.toUpperCase() === 'ATTACHMENT';
        }).map(function (part) {
            // retrieve the attachments only of the messages with attachments
            return connection.getPartData(message, part)
                .then(function (partData) {
                    return {
                        filename: part.disposition.params.filename,
                        data: partData
                    };
                });
        }));
    });

    return Promise.all(attachments);
}).then(function (attachments) {
    console.log(attachments);
    // =>
    //    [ { filename: 'cats.jpg', data: Buffer() },
    //      { filename: 'pay-stub.pdf', data: Buffer() } ]
});

При тестировании этого фрагмента кода с двумя электронными письмами и двумя вложениями конечный результат печатает один массив с четырьмя вложениями.

Как изменить код, чтобы получить отдельный массив для каждого письма, содержащий только вложения для этого письма?

Answer 1

Вы можете попробовать следующее:

messages.forEach(function (message) {
      var parts = imaps.getParts(message.attributes.struct);
                                      //concat array to attachments
      attachments = attachments.concat([parts.filter(function (part) {
          return part.disposition && part.disposition.type.toUpperCase() === 'ATTACHMENT';
      }).map(function (part) {
          // retrieve the attachments only of the messages with attachments
          return connection.getPartData(message, part)
              .then(function (partData) {
                  return {
                      filename: part.disposition.params.filename,
                      data: partData
                  };
              });
      })]/**close bracket for concat array */);
});

return Promise.all(//attachments is array of arrays
  attachments.map(
    attachments=>Promise.all(attachments)
  )
)

Вот как вы можете сделать это, используя карту:

Promise.all(
  messages.map(
    message=>
      Promise.all(
        imaps.getParts(message.attributes.struct).filter(
          part=>
            part.disposition && part.disposition.type.toUpperCase() === 'ATTACHMENT'
        ).map(
          part=>
            connection.getPartData(message, part)
            .then(
              partData=>({
                filename: part.disposition.params.filename,
                data: partData
              })
            )

        )
      )
  )
)