-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
NullPointerException при открытии видео из списка "Новое" или из недобавленного в базу плейлиста #32
Comments
Пусть будет здесь же: Если открывать так Питрубовский ролик, при при загрузке он не вылетает и даже играет, но остаётся кнопка "вправо" при нажатие на которую вылетает эксепшен (во-первых, экспепшен в любом случае не должен вылетать, во-вторых, в этом режиме кнопки вправо не должно быть):
|
Здесь же: в режиме проигрывания недобавленного ролика в меню остаётся звездочка и элементы "добавить в блеклист", "скопировать адрес плейлиста" и т.п. (их лучше убрать) |
Первый эксепшен исчез на одном из коммитов после 0.1.0. Специально не исправлял, но в этой области были правки, ну и ладно. Проверил: в текущей версии открываю видео из недобавленного плейлиста - не вылетает, откатился на версию 0.1.0, открываю это же видео - вылетает, возвращаюсь к текущей версии - опять не вылетает. С первым откатом до 0.1.0 проебал базу с плейлистами и историей просмотров на телефоне, которая копилась больше года начиная с первых версий до релиза, как жаль. Во-первых, в VideoDatabase.java нужно было выключить fallbackToDestructiveMigration() - он грохает базу при несовместимости версий базы, в 0.2.0 версия базы как раз поменялась (сейчас выключил, так больше происходить не будет, но только при откате на версии старше 0.1.0, откат на 0.1.0 теперь всегда будет обнулять базу) Во-вторых, нужно было скачать файл с базой данных из /data/data/su.sadrobot.yashlang/databases/video-db в телефоне в FileExplorer в Android Studio, ведь только вчера его выкачивал так, но, к сожалению, только с эмулятора, а не с телефона, ай, ай. |
исходный баг закрываю, остальное перенес сюда: #37 |
The text was updated successfully, but these errors were encountered: