use applicationId instead of hardcoded app:graphPackage [152696768]

Fixed

Feature Request

Status Update

No update yet.

Description

b9...@gmail.com

created issue #1

Mar 30, 2020 05:48AM

Component used: Dynamic Navigator library

Just like the doc says:
app:graphPackage: the package name of the dynamic feature module. The package name is usually in the form of applicationId.moduleName. You can verify the package name by opening a dynamic feature APK in Android Studio's APK Analyzer and then opening the resources.arsc file, which is located in the root folder of your APK.

applicationId will changed between build variants in Gradle (dev/rc/staging), thus we can't hardcode applicationId.moduleName in xml without using placeholder, further more I think graphPackage attr could be replaced by "${applicatonId}.$moduleName", so that we don't need write this manually.

Comments

il...@google.com <il...@google.com> #2Mar 30, 2020 08:23PM

Assigned to be...@google.com.

since these are in public API (:/) we need to do this in 1.2

st...@zeppelin.co.jp <st...@zeppelin.co.jp> #3Mar 31, 2020 03:35AM

since it is already marked as deprecated, we can probably do it by now.

b9...@gmail.com <b9...@gmail.com> #4Mar 31, 2020 04:22AM

Opening diff shortly

wk...@google.com <wk...@google.com> #5Mar 31, 2020 08:55AM

Accepted by wk...@google.com.

Project: platform/frameworks/support
Branch: androidx-master-dev

commit d576cbdc911cba16638a44fd8223391a90a07ef7
Author: Mike Nakhimovich <digitalbuddha@users.noreply.github.com>
Date: Tue Aug 11 09:30:34 2020

[GH] Hide deprecated internal API.

## Proposed Changes

* `RoomDatabase.java` has protected `mCallbacks` field which is leaking in the API docs, we should @Hide it.

## Testing

Test: Ran unit tests locally

## Issues Fixed

Fixes: 76109329

This is an imported pull request from

https://github.com/androidx/androidx/pull/61.

Resolves #61
Github-Pr-Head-Sha: 6440daa3a63752c7f9d5ba2a390248cd85bc634f
GitOrigin-RevId: fe92d8466a59b44b218b6ca3cbd57dcda17992f7

Change-Id: Id599cdf5b02b32bdae0166266fb7da967598fe92

A room/runtime/api/current.ignore
M room/runtime/api/current.txt
M room/runtime/api/public_plus_experimental_current.txt
M room/runtime/api/restricted_current.txt
M room/runtime/src/main/java/androidx/room/RoomDatabase.java

https://android-review.googlesource.com/1396827

ap...@google.com <ap...@google.com> #6Apr 2, 2020 09:37AM

Marked as fixed.

Project: platform/frameworks/support
Branch: androidx-master-dev

commit 5a90b4603457fd3218e2710eb6405736b5f60a48
Author: Wojtek Kaliciński <wkal@google.com>
Date: Tue Mar 31 12:42:26 2020

Placeholder and default support in app:graphPackage

It's possible to specify ${applicationId} in the graphPackage string.

If graphPackage is omitted a default will be provided at runtime:
context.packageName + "." + moduleName

Relnote: "For dynamic graph includes you no longer need to specify
`app:graphPackage` and a default one will be used by adding `moduleName`
suffix to the `applicationId` after a dot. If you do need to customize
your `graphPackage`, an `${applicationId}` placeholder is now supported."

Test: DynamicIncludeGraphNavigatorTest
Fixes:

b/152696768

Change-Id: Ida52ce2100fbc3275bf230c5dad6362b4e36cc9f

M navigation/navigation-dynamic-features-runtime/src/main/java/androidx/navigation/dynamicfeatures/DynamicIncludeGraphNavigator.kt
M navigation/navigation-dynamic-features-runtime/src/test/java/androidx/navigation/dynamicfeatures/DynamicIncludeGraphNavigatorTest.kt
M navigation/navigation-runtime/src/main/java/androidx/navigation/NavInflater.java

https://android-review.googlesource.com/1272905

il...@google.com <il...@google.com> #7Apr 2, 2020 05:28PM

We've fixed this internally and it'll be available in Navigation 2.3.0-alpha05.

b9...@gmail.com <b9...@gmail.com> #8Apr 2, 2020 05:33PM

So quick! Thanks!👍