Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
29 commits
Select commit Hold shift + click to select a range
44afe7a
Update gradle and run agp updater on all camera* examples
reidbaker Apr 25, 2023
85bd44a
More updates of agp and gradle
reidbaker Apr 25, 2023
56b45cd
agp/gradle update for quick actions
reidbaker Apr 25, 2023
36df790
More updates of agp and gradle
reidbaker Apr 25, 2023
0e83599
More updates of agp and gradle for image_picker, also add exported an…
reidbaker Apr 25, 2023
974a4b3
More updates of agp and gradle for image_picker in app purchase
reidbaker Apr 25, 2023
7ec81cf
More updates of agp and gradle for url_launcher
reidbaker Apr 25, 2023
6ec0960
More updates of agp and gradle for video_player
reidbaker Apr 25, 2023
88bdb81
More updates of agp and gradle for video_player local auth
reidbaker Apr 25, 2023
0d47349
More updates of agp and gradle for local_auth
reidbaker Apr 25, 2023
2954607
More updates of agp and gradle for google_sign_in
reidbaker Apr 25, 2023
087ad0d
More updates of agp and gradle for webview flutter
reidbaker Apr 25, 2023
38dc140
More updates of agp and gradle for webview flutter, path_provider_and…
reidbaker Apr 25, 2023
5bd7755
update to task registration
reidbaker Apr 25, 2023
3b07ba2
Merge branch 'main' into i122213-run-agp-update-for-many
reidbaker Apr 25, 2023
2f12f9f
Fix lints that were added by updated toolchain, mostly camera
reidbaker Apr 26, 2023
ed33296
Fix lints that were added by updated toolchain, update baselines
reidbaker Apr 26, 2023
96439a8
Ran formatter for modified log lines
reidbaker Apr 26, 2023
fbf49e3
expanded lint baseline for in_app_purchase
reidbaker Apr 26, 2023
8db9d8b
Merge branch 'main' into i122213-run-agp-update-for-many
reidbaker Apr 26, 2023
e7da527
Add required changelogs
reidbaker Apr 27, 2023
bbfe2ea
Merge branch 'main' into i122213-run-agp-update-for-many
reidbaker Apr 27, 2023
41c1757
Add required changelogs in app purchase
reidbaker Apr 27, 2023
51d3e35
Add required changelogs in app purchase
reidbaker Apr 27, 2023
4a2299a
Merge remote-tracking branch 'upstream/main' into i122213-run-agp-upd…
reidbaker Apr 28, 2023
843302b
Update pubspecs
reidbaker Apr 28, 2023
ed7174f
Merge remote-tracking branch 'origin/main' into i122213-run-agp-updat…
reidbaker Apr 28, 2023
73e0d7d
Update changelog
reidbaker Apr 28, 2023
d850d2c
Merge branch 'main' into i122213-run-agp-update-for-many
reidbaker Apr 28, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix lints that were added by updated toolchain, mostly camera
  • Loading branch information
reidbaker committed Apr 26, 2023
commit 2f12f9f1fc748b30209fbfaaef198272042b4a5b
3 changes: 3 additions & 0 deletions packages/camera/camera_android/android/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,9 @@ android {
targetCompatibility JavaVersion.VERSION_1_8
}

lint {
baseline = file("lint-baseline.xml")
}

testOptions {
unitTests.includeAndroidResources = true
Expand Down
26 changes: 26 additions & 0 deletions packages/camera/camera_android/android/lint-baseline.xml
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
<?xml version="1.0" encoding="UTF-8"?>
<issues format="6" by="lint 7.4.2" type="baseline" client="gradle" dependencies="false" name="AGP (7.4.2)" variant="all" version="7.4.2">

<issue
id="OldTargetApi"
message="Not targeting the latest versions of Android; compatibility modes apply. Consider testing and updating this version. Consult the android.os.Build.VERSION_CODES javadoc for details."
errorLine1=" targetSdkVersion 31"
errorLine2=" ~~~~~~~~~~~~~~~~~~~">
<location
file="build.gradle"
line="34"
column="9"/>
</issue>

<issue
id="ObsoleteSdkInt"
message="Unnecessary; SDK_INT is always >= 21"
errorLine1=" @TargetApi(Build.VERSION_CODES.LOLLIPOP)"
errorLine2=" ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
<location
file="src/main/java/io/flutter/plugins/camera/CameraDeviceWrapper.java"
line="27"
column="3"/>
</issue>

</issues>
Original file line number Diff line number Diff line change
Expand Up @@ -322,15 +322,15 @@ public void onOpened(@NonNull CameraDevice device) {
cameraDevice = new DefaultCameraDeviceWrapper(device);
try {
startPreview();
if (!recordingVideo) // only send initialization if we werent already recording and switching cameras
dartMessenger.sendCameraInitializedEvent(
if (!recordingVideo) { // only send initialization if we werent already recording and switching cameras
dartMessenger.sendCameraInitializedEvent(
resolutionFeature.getPreviewSize().getWidth(),
resolutionFeature.getPreviewSize().getHeight(),
cameraFeatures.getExposureLock().getValue(),
cameraFeatures.getAutoFocus().getValue(),
cameraFeatures.getExposurePoint().checkIsSupported(),
cameraFeatures.getFocusPoint().checkIsSupported());

}
} catch (Exception e) {
if (BuildConfig.DEBUG) {
Log.i(TAG, "open | onOpened error: " + e.getMessage());
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,8 @@ public long addHostCreatedInstance(Object instance) {

if (containsInstance(instance)) {
throw new IllegalArgumentException(
String.format("Instance of `%s` has already been added.", instance.getClass()));
"Instance of " + instance.getClass() + " has already been added."
);
}
final long identifier = nextIdentifier++;
addInstance(instance, identifier);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ private void onLaunch(@NonNull MethodCall call, @NonNull Result result, @NonNull
} else if (launchStatus == LaunchStatus.ACTIVITY_NOT_FOUND) {
result.error(
"ACTIVITY_NOT_FOUND",
String.format("No Activity found to handle intent { %s }", url),
"No Activity found to handle intent { " + url + " }",
null);
} else {
result.success(true);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,8 @@ public long addHostCreatedInstance(@NonNull Object instance) {

if (containsInstance(instance)) {
throw new IllegalArgumentException(
String.format("Instance of `%s` has already been added.", instance.getClass()));
"Instance of " + instance.getClass() + " has already been added."
);
}
final long identifier = nextIdentifier++;
addInstance(instance, identifier);
Expand Down