CoarseFineLocation lint rule doesn't account for a maxSdkVersion attribute [383595395]

Fixed

Bug

Status Update

No update yet.

Description

ey...@gmail.com

created issue #1

Dec 12, 2024 05:10AM

DESCRIBE THE ISSUE IN DETAIL:

I need to request ACCESS_FINE_LOCATION permission in my app on Android versions 30 and below in order to use BLE scanning.

Since I don't need that for versions 31 and above, I include the permission in my manifest with a maxSdkVersion:

  <uses-permission
    android:name="android.permission.ACCESS_FINE_LOCATION"
    android:maxSdkVersion="30" />

This triggers a violation of lint's CoarseFineLocation.

Comments

xi...@google.com <xi...@google.com> Dec 12, 2024 04:48PM

Assigned to an...@google.com.

tn...@google.com <tn...@google.com> Dec 12, 2024 10:07PM

Reassigned to tn...@google.com.

me...@google.com <me...@google.com> Dec 13, 2024 04:07PM

Reassigned to tn...@google.com.

tn...@google.com <tn...@google.com> #2Dec 13, 2024 06:23PM

Marked as fixed.

George, you fixed this issue now right?

an...@google.com <an...@google.com> #3Dec 23, 2024 09:19PM

I see the release note in compose-ui 1.7.3:

Fixed exception thrown when the IME is active on an external View and the Next action is used to enter focus on the ComposeView. The clearFocus() behavior aligns with View behavior on API < 28, where clearFocus() can cause the default View to be focused.

However, we can still reproduce this crash with BOM 2024.09.03 and compose-ui 1.7.3.

aa...@gmail.com <aa...@gmail.com> #4Jan 5, 2025 02:19AM

We are also seeing this crash in prod after bumping to 1.7+

Issue 383595395