search

volga-volga / react-native-yamap

React Native Yandex Maps | Яндекс Карты | Yandex.MapKit implementation for react native | YandexMaps
151 stars 84 forks source link

Фикс падения андроида в removeChild #121

Closed DazzlingFame closed 2 years ago

DazzlingFame commented 2 years ago

пр фиксит это issue https://github.com/volga-volga/react-native-yamap/issues/102

Что тут:

Появилась мысль, что битые ссылки на объекты могут возникать из-за вспомогательного листа чайлдов, который тут заводится для менеджа объектов детей

Зарефакторил работу с чайлдами в вьюхе, вместо обращения к листу получаю каждый раз свежие данные через getChildAt и getChildCount

Тестировал на эмуляторе и реальных девайсах, краши воспроизвести не удалось

P.S. в пр залетело несколько линтовых изменений, если надо будет, могу убрать

DazzlingFame commented 2 years ago

У меня тут много комитов накопилось, прошу вливать сквошем

DazzlingFame commented 2 years ago

Поймал конфликты с кластерами, пошел тестировать ещё раз после мержа

DazzlingFame commented 2 years ago

Потестил, падений воспроизвести не смог, но мне нужна будет помощь с тестированием кластеров, в которые мне пришлось влезть из-за использования там childs

ownikss commented 2 years ago

Опубликовал версию 4.0.14 Кластеры погоняем, спасибо