diff --git a/packages/flutter_tools/gradle/src/main/groovy/flutter.groovy b/packages/flutter_tools/gradle/src/main/groovy/flutter.groovy index b75a4601a6c..4d0168ad315 100644 --- a/packages/flutter_tools/gradle/src/main/groovy/flutter.groovy +++ b/packages/flutter_tools/gradle/src/main/groovy/flutter.groovy @@ -48,7 +48,7 @@ class FlutterExtension { public final int compileSdkVersion = 35 /** Sets the minSdkVersion used by default in Flutter app projects. */ - public final int minSdkVersion = 21 + public final int minSdkVersion = 21 /** * Sets the targetSdkVersion used by default in Flutter app projects. @@ -196,7 +196,7 @@ class FlutterPlugin implements Plugin { } } - String flutterRootPath = resolveProperty("flutter.sdk", System.env.FLUTTER_ROOT) + String flutterRootPath = resolveProperty("flutter.sdk", System.getenv("FLUTTER_ROOT")) if (flutterRootPath == null) { throw new GradleException("Flutter SDK not found. Define location with flutter.sdk in the local.properties file or with a FLUTTER_ROOT environment variable.") } @@ -211,11 +211,11 @@ class FlutterPlugin implements Plugin { engineRealm = Paths.get(flutterRoot.absolutePath, "bin", "internal", "engine.realm").toFile().text.trim() if (engineRealm) { - engineRealm = engineRealm + "/" + engineRealm += "/" } // Configure the Maven repository. - String hostedRepository = System.env.FLUTTER_STORAGE_BASE_URL ?: DEFAULT_MAVEN_HOST + String hostedRepository = System.getenv("FLUTTER_STORAGE_BASE_URL") ?: DEFAULT_MAVEN_HOST String repository = useLocalEngine() ? project.property(propLocalEngineRepo) : "$hostedRepository/${engineRealm}download.flutter.io" @@ -239,11 +239,8 @@ class FlutterPlugin implements Plugin { } } - String flutterVersionCode = localProperties.getProperty("flutter.versionCode") - extension.flutterVersionCode = flutterVersionCode ?: "1" - - String flutterVersionName = localProperties.getProperty("flutter.versionName") - extension.flutterVersionName = flutterVersionName ?: "1.0" + extension.flutterVersionCode = localProperties.getProperty("flutter.versionCode", "1") + extension.flutterVersionName = localProperties.getProperty("flutter.versionName", "1.0") this.addFlutterTasks(project) forceNdkDownload(project, flutterRootPath) @@ -471,7 +468,7 @@ class FlutterPlugin implements Plugin { // Warning: The name of this task is used by AndroidBuilder.outputsAppLinkSettings project.tasks.register("output${variant.name.capitalize()}AppLinkSettings") { description "stores app links settings for the given build variant of this Android project into a json file." - variant.outputs.all { output -> + variant.outputs.configureEach { output -> // Deeplinks are defined in AndroidManifest.xml and is only available after // `processResourcesProvider`. Object processResources = output.hasProperty(propProcessResourcesProvider) ? @@ -482,7 +479,7 @@ class FlutterPlugin implements Plugin { AppLinkSettings appLinkSettings = new AppLinkSettings() appLinkSettings.applicationId = variant.applicationId appLinkSettings.deeplinks = [] as Set - variant.outputs.all { output -> + variant.outputs.configureEach { output -> Object processResources = output.hasProperty(propProcessResourcesProvider) ? output.processResourcesProvider.get() : output.processResources def manifest = new XmlParser().parse(processResources.manifestFile) @@ -1011,9 +1008,7 @@ class FlutterPlugin implements Plugin { if (localProperties == null) { localProperties = readPropertiesIfExist(new File(project.projectDir.parentFile, "local.properties")) } - String result = project.hasProperty(name) ? project.property(name) : null - result = result ?: localProperties?.getProperty(name) - return result ?: defaultValue + return project.findProperty(name) ?: localProperties?.getProperty(name, defaultValue) } private List getTargetPlatforms() { @@ -1239,7 +1234,7 @@ class FlutterPlugin implements Plugin { // for only the output APK, not for the variant itself. Skipping this step simply // causes Gradle to use the value of variant.versionCode for the APK. // For more, see https://developer.android.com/studio/build/configure-apk-splits - Integer abiVersionCode = ABI_VERSION.get(output.getFilter(OutputFile.ABI)) + Integer abiVersionCode = ABI_VERSION[output.getFilter(OutputFile.ABI)] if (abiVersionCode != null) { output.versionCodeOverride = abiVersionCode * 1000 + variant.versionCode @@ -1304,7 +1299,7 @@ class FlutterPlugin implements Plugin { validateDeferredComponents(validateDeferredComponentsValue) flavor(flavorValue) } - Task compileTask = compileTaskProvider.get(); + Task compileTask = compileTaskProvider.get() File libJar = project.file(project.layout.buildDirectory.dir("$INTERMEDIATES_DIR/flutter/${variant.name}/libs.jar")) TaskProvider packJniLibsTaskProvider = project.tasks.register("packJniLibs${FLUTTER_BUILD_PREFIX}${variant.name.capitalize()}", Jar) { destinationDirectory = libJar.parentFile @@ -1331,7 +1326,7 @@ class FlutterPlugin implements Plugin { } } } - Task packJniLibsTask = packJniLibsTaskProvider.get(); + Task packJniLibsTask = packJniLibsTaskProvider.get() addApiDependencies(project, variant.name, project.files { packJniLibsTask }) @@ -1370,7 +1365,7 @@ class FlutterPlugin implements Plugin { mergeAssets.mustRunAfter("clean${mergeAssets.name.capitalize()}") into(mergeAssets.outputDir) } - Task copyFlutterAssetsTask = copyFlutterAssetsTaskProvider.get(); + Task copyFlutterAssetsTask = copyFlutterAssetsTaskProvider.get() if (!isUsedAsSubproject) { def variantOutput = variant.outputs.first() def processResources = variantOutput.hasProperty(propProcessResourcesProvider) ? @@ -1569,7 +1564,7 @@ class IntentFilterCheck { class Deeplink { String scheme, host, path IntentFilterCheck intentFilterCheck - boolean equals(o) { + boolean equals(Object o) { if (o == null) { throw new NullPointerException() }