Fixed
Status Update
Comments
il...@google.com <il...@google.com>
il...@google.com <il...@google.com> #2
Hi. Thanks for reporting this. Fixed in alpha-04
ap...@google.com <ap...@google.com> #3
Project: platform/frameworks/support
Branch: androidx-main
commit e782987543a9f8ccd485e970ddc74564b24378db
Author: Vighnesh Raut <vighnesh.raut13@gmail.com>
Date: Mon Jan 02 15:27:40 2023
fix: tab row crashes when only 1 tab is added
Bug: b/264018028
Test: Added unit test
Change-Id: I6381dbac304fc1d69d3708c6655f8b595668e93f
M tv/tv-material/src/androidTest/java/androidx/tv/material/TabRowTest.kt
M tv/tv-material/src/main/java/androidx/tv/material/TabRow.kt
https://android-review.googlesource.com/2373449
Branch: androidx-main
commit e782987543a9f8ccd485e970ddc74564b24378db
Author: Vighnesh Raut <vighnesh.raut13@gmail.com>
Date: Mon Jan 02 15:27:40 2023
fix: tab row crashes when only 1 tab is added
Bug:
Test: Added unit test
Change-Id: I6381dbac304fc1d69d3708c6655f8b595668e93f
M tv/tv-material/src/androidTest/java/androidx/tv/material/TabRowTest.kt
M tv/tv-material/src/main/java/androidx/tv/material/TabRow.kt
Description
Component used: Navigation Safe Args Kotlin Plugin
Version used: 2.4.0-alpha02, AGP 4.2.0-beta03
After upgrading to Navigation 2.4.0-alpha02 and building a minified version of my app , my app crashed with an exception that said that the
fromBundle
method couldn't be found.I use the
by navArgs()
extension, which uses reflection to access this method, so I depended on Navigation's included ProGuard keep rules to properly keep that method.I found that I could work around this issue via this commit :
Essentially just making sure that the
Companion
class is kept. After adding that ProGuard rule, thefromBundle()
class was successfully kept. I'm not sure why that helped, since I'm pretty sure that the format of thefromBundle()
didn't change since Navigation 2.3.5?Reproduction steps:
minifyEnabled true
to the./gradlew assembleDebug
and analyze the resulting APK to verify that theChooseProviderFragmentArgs.fromBundle()
method exists (it won't) 4a) Alternatively, install Muzei, Activate it as your wallpaper, then select the 'Sources' screen. Muzei should not crash.