Функция обновления состояния в реакт асинхронная. На момент выполнения FillGoods массив AllGoodsArray будет пустой, т.к. setAllGoodsArray еще не будет выполнена и состояние еще не обновлено.
Здесь же AllGoodsArray вообще не нужен, можно в FillGoods просто передать response.data в качестве параметра.
Еще не понятно, почему некоторые переменные и функции у вас названы в стиле PascalCase, лучше придерживаться консистентности всегда
