Skip to content

Commit e2eb39b

Browse files
authored
Merge pull request #17201 from wordpress-mobile/analysis/image-editor-all-warnings-as-errors
[Compile Warnings As Errors] Image Editor Module - Resolve Warnings & Enable All Warnings as Errors
2 parents 2d659ab + 30a06c2 commit e2eb39b

3 files changed

Lines changed: 6 additions & 1 deletion

File tree

libs/image-editor/build.gradle

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,10 @@ android {
1818
vectorDrawables.useSupportLibrary = true
1919
}
2020

21+
kotlinOptions {
22+
allWarningsAsErrors = true
23+
}
24+
2125
sourceSets {
2226
main.java.srcDirs += 'src/main/kotlin'
2327
test.java.srcDirs += 'src/test/kotlin'

libs/image-editor/src/main/kotlin/org/wordpress/android/imageeditor/crop/CropViewModel.kt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@ class CropViewModel : ViewModel() {
4444
private var shouldReturnToPreviewScreen: Boolean = false
4545
private var isStarted = false
4646

47+
@Suppress("DEPRECATION")
4748
private val cropOptions by lazy {
4849
Options().apply {
4950
setShowCropGrid(true)

libs/image-editor/src/main/kotlin/org/wordpress/android/imageeditor/preview/PreviewImageViewModel.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -316,7 +316,7 @@ class PreviewImageViewModel : ViewModel() {
316316
private fun getOutputData() = (uiState.value?.viewPagerItemsStates?.map { OutputData(it.data.highResImageUrl) }
317317
?: emptyList())
318318

319-
private fun isFileUrl(url: String): Boolean = url.toLowerCase(Locale.ROOT).startsWith(FILE_BASE)
319+
private fun isFileUrl(url: String): Boolean = url.lowercase(Locale.ROOT).startsWith(FILE_BASE)
320320

321321
data class ImageData(
322322
val id: Long = UUID.randomUUID().hashCode().toLong(),

0 commit comments

Comments
 (0)