Promise.all () и отлов ошибок

Попробуйте вернуть отклоненное обещание.

return Promise.reject('Error when getting venue details')

При работе с обещаниями вы должны возвращать внутренние обещания вместо работы с внутренними «тезисами».

Проверь это:

fetch(`https://api.foursquare.com/v2/venues/search?${params}`)
  .then(response => response.json())
  .then(data => {
      const venueIds = data.response.venues.map(venue => venue.id);

      const venuePromises = venueIds.map(venueId => {
        fetch(`https://api.foursquare.com/v2/venues/${venueId}?${otherParams}`)
          .then(response => {
            // Must check for response.ok, because 
            // catch() does not catch 429
            if (response.ok) {
              console.info('ok')
              return response.json()
            } else {
              return Promise.reject('Error when getting venue details')
            }
          })
      });

      return Promise.all(venuePromises)
  })
  .then(venueValues => {
    const venues = venueValues.map(entry => entry.response.venue);  // Error for this line
    this.parseFsqData(venues);
  })
  .catch((e) => {console.info(e); getBackupData()})


function getBackupData() {
    console.info('backup')
}

Возвращая Promise.all в качестве значения, вы возвращаете обещание, чтобы вы могли связать дальнейшие обратные вызовы «затем». Последний улов должен захватить все отклоненные обещания.

Вам также не хватает возврата в предложении else

Надеюсь это поможет

Ваши проблемы связаны: а именно, цепочка обещаний должна быть returned. Если вы не выполняете return Promise, вы отключаете любую обработку Promise#catch вызывающего абонента, и любые ошибки в вашем коде Promise / then приведут к необработанным ошибкам отклонения обещания, например тем, что вы получили:

Uncaught (in promise) Error when getting venue details

Это неперехваченное отклонение обещания появляется в вашем коде, который обрабатывает разрешение fetch:

if (response.ok) {
  console.info('ok')
  return response.json()
} else {
  Promise.reject('Error when getting venue details')  // <----
}

Поскольку этот код используется для создания вашего массива venuePromises, его значение return будет заполнять venuePromises. Если ответ был удовлетворительным, этот элемент массива будет иметь ответ JSON от return response.json(). Если ответ не прошел, значит, оператор return не выполняется, поэтому элемент массива имеет значение undefined. Таким образом, venuePromises будет выглядеть так:

[
  { /** some object for successful response */ },
  undefined,
  { /** some other object */ },
  ...
]

Таким образом, когда к этому массиву обращается обработчик успеха вашего Promise.all, вы получаете ошибку TypeError, поскольку вы ожидали, что все элементы venuePromises будут действительными. Эта ошибка TypeError перехватывается обработчиком Promise.all.catch (поэтому она регистрируется, и вы получаете текст «резервной копии» в своем журнале).

Чтобы исправить это, вам необходимо установить return в Promise.reject, а также в Promise.all. Обратите внимание, что есть некоторые случаи неявный return, но я считаю, что лучше быть явным, особенно если оператор занимает несколько строк. Поскольку вы возвращаете оператор Promise.all, вы можете передать его .then и .catch вызывающей стороне, что приведет к уменьшению на один уровень вложенности и на один дублированный обработчик .catch.

fetch(`https://api.foursquare.com/v2/venues/search?${params}`)
    .then(response => response.json())
    .then(jsonData => {
        const venueIds = jsonData.response.venues.map(venue => venue.id);
        const venuePromises = venueIds.map(venueId => {
            let link = `https://api.foursquare.com/v2/venues/${venueId}?${otherParams}`;
            return fetch(link).then(response => {
                // Must check for response.ok, because catch() does not catch 429
                if (response.ok) {
                    console.info('ok');
                    return response.json();
                } else {
                    console.info(`FAILED: ${link}`);
                    // Return a Promise
                    return Promise.reject(`Error when getting venue details for '${venueId}'`);
                }
            });
        });

        return Promise.all(venuePromises);
    })
    .then(venueData => {
        const venues = venueData.map(entry => entry.response.venue);
        this.parseFsqData(venues);
    })
    .catch(e => {console.info(e); getBackupData()});

function getBackupData() {
    console.info('backup')
}

Я считаю, что решение довольно простое; В ответе вложенного метода выборки отсутствует оператор возврата. Вы должны избавиться от этой загадочной ошибки, как только она будет исправлена.

const venuePromises = venueIds.map(venueId => {
    <missing return statement here> fetch(`https://api.foursquare.com/v2/venues/${venueId}?${otherParams}`)
      .then(response => {