Apply official Kotlin code style.

Author: android10

README.md

@@ -60,13 +60,9 @@ Here are some useful Gradle/adb commands for executing this example:
 ## Discussions
 Refer to the issues section: https://github.com/android10/Android-CleanArchitecture-Kotlin/issues
 
-## Code style
-Here you can download and install the java codestyle.
-https://github.com/android10/java-code-styles
-
 ## License
 
- Copyright 2018 Fernando Cejas
+ Copyright 2021 Fernando Cejas
 
  Licensed under the Apache License, Version 2.0 (the "License");
  you may not use this file except in compliance with the License.

app/src/main/kotlin/com/fernandocejas/sample/core/di/ApplicationModule.kt

@@ -31,22 +31,27 @@ import javax.inject.Singleton
 @InstallIn(SingletonComponent::class)
 class ApplicationModule {
 
- @Provides @Singleton fun provideRetrofit(): Retrofit {
+ @Provides
+ @Singleton
+ fun provideRetrofit(): Retrofit {
  return Retrofit.Builder()
-  .baseUrl("https://raw.githubusercontent.com/android10/Sample-Data/master/Android-CleanArchitecture-Kotlin/")
-  .client(createClient())
-  .addConverterFactory(GsonConverterFactory.create())
-  .build()
+ .baseUrl("https://raw.githubusercontent.com/android10/Sample-Data/master/Android-CleanArchitecture-Kotlin/")
+ .client(createClient())
+ .addConverterFactory(GsonConverterFactory.create())
+ .build()
  }
 
  private fun createClient(): OkHttpClient {
  val okHttpClientBuilder: OkHttpClient.Builder = OkHttpClient.Builder()
  if (BuildConfig.DEBUG) {
- val loggingInterceptor = HttpLoggingInterceptor().setLevel(HttpLoggingInterceptor.Level.BASIC)
+ val loggingInterceptor =
+ HttpLoggingInterceptor().setLevel(HttpLoggingInterceptor.Level.BASIC)
  okHttpClientBuilder.addInterceptor(loggingInterceptor)
  }
  return okHttpClientBuilder.build()
  }
 
- @Provides @Singleton fun provideMoviesRepository(dataSource: MoviesRepository.Network): MoviesRepository = dataSource
+ @Provides
+ @Singleton
+ fun provideMoviesRepository(dataSource: MoviesRepository.Network): MoviesRepository = dataSource
 }

app/src/main/kotlin/com/fernandocejas/sample/core/exception/Failure.kt

@@ -24,5 +24,5 @@ sealed class Failure {
  object ServerError : Failure()
 
  /** * Extend this class for feature specific failures.*/
- abstract class FeatureFailure: Failure()
+ abstract class FeatureFailure : Failure()
 }

app/src/main/kotlin/com/fernandocejas/sample/core/extension/Context.kt

@@ -18,5 +18,6 @@ package com.fernandocejas.sample.core.extension
 import android.content.Context
 import android.net.ConnectivityManager
 
-val Context.connectivityManager: ConnectivityManager get() =
- this.getSystemService(Context.CONNECTIVITY_SERVICE) as ConnectivityManager
+val Context.connectivityManager: ConnectivityManager
+ get() =
+ this.getSystemService(Context.CONNECTIVITY_SERVICE) as ConnectivityManager

app/src/main/kotlin/com/fernandocejas/sample/core/extension/Fragment.kt

@@ -24,7 +24,7 @@ import com.fernandocejas.sample.core.platform.BaseFragment
 import kotlinx.android.synthetic.main.activity_layout.*
 
 inline fun FragmentManager.inTransaction(func: FragmentTransaction.() -> FragmentTransaction) =
-  beginTransaction().func().commit()
+ beginTransaction().func().commit()
 
 fun BaseFragment.close() = fragmentManager?.popBackStack()
 

app/src/main/kotlin/com/fernandocejas/sample/core/extension/Lifecycle.kt

@@ -21,7 +21,7 @@ import androidx.lifecycle.Observer
 import com.fernandocejas.sample.core.exception.Failure
 
 fun <T : Any, L : LiveData<T>> LifecycleOwner.observe(liveData: L, body: (T?) -> Unit) =
-  liveData.observe(this, Observer(body))
+ liveData.observe(this, Observer(body))
 
 fun <L : LiveData<Failure>> LifecycleOwner.failure(liveData: L, body: (Failure?) -> Unit) =
-  liveData.observe(this, Observer(body))
+ liveData.observe(this, Observer(body))

app/src/main/kotlin/com/fernandocejas/sample/core/extension/View.kt

@@ -35,12 +35,16 @@ fun View.cancelTransition() {
 
 fun View.isVisible() = this.visibility == View.VISIBLE
 
-fun View.visible() { this.visibility = View.VISIBLE }
+fun View.visible() {
+ this.visibility = View.VISIBLE
+}
 
-fun View.invisible() { this.visibility = View.GONE }
+fun View.invisible() {
+ this.visibility = View.GONE
+}
 
 fun ViewGroup.inflate(@LayoutRes layoutRes: Int): View =
-  LayoutInflater.from(context).inflate(layoutRes, this, false)
+ LayoutInflater.from(context).inflate(layoutRes, this, false)
 
 fun ImageView.loadFromUrl(url: String) =
  Glide.with(this.context.applicationContext)
@@ -53,7 +57,8 @@ fun ImageView.loadUrlAndPostponeEnterTransition(url: String, activity: FragmentA
  Glide.with(context.applicationContext).load(url).into(target)
 }
 
-private class ImageViewBaseTarget (var imageView: ImageView?, var activity: FragmentActivity?) : BaseTarget<Drawable>() {
+private class ImageViewBaseTarget(var imageView: ImageView?, var activity: FragmentActivity?) :
+ BaseTarget<Drawable>() {
  override fun onLoadFailed(errorDrawable: Drawable?) {
  super.onLoadFailed(errorDrawable)
  activity?.supportStartPostponedEnterTransition()

app/src/main/kotlin/com/fernandocejas/sample/core/functional/Either.kt

@@ -49,7 +49,7 @@ sealed class Either<out L, out R> {
  */
  fun <L> left(a: L) = Either.Left(a)
 
- 
+
  /**
  * Creates a Left type.
  * @see Right

app/src/main/kotlin/com/fernandocejas/sample/core/interactor/UseCase.kt

@@ -31,7 +31,11 @@ abstract class UseCase<out Type, in Params> where Type : Any {
 
  abstract suspend fun run(params: Params): Either<Failure, Type>
 
- operator fun invoke(params: Params, scope: CoroutineScope = GlobalScope, onResult: (Either<Failure, Type>) -> Unit = {}) {
+ operator fun invoke(
+ params: Params,
+ scope: CoroutineScope = GlobalScope,
+ onResult: (Either<Failure, Type>) -> Unit = {}
+ ) {
  scope.launch(Dispatchers.Main) {
  val deferred = async(Dispatchers.IO) {
  run(params)

app/src/main/kotlin/com/fernandocejas/sample/core/navigation/Navigator.kt

@@ -23,12 +23,12 @@ import android.view.View
 import android.widget.ImageView
 import androidx.core.app.ActivityOptionsCompat
 import androidx.fragment.app.FragmentActivity
+import com.fernandocejas.sample.core.extension.empty
 import com.fernandocejas.sample.features.login.Authenticator
 import com.fernandocejas.sample.features.login.LoginActivity
 import com.fernandocejas.sample.features.movies.MovieDetailsActivity
 import com.fernandocejas.sample.features.movies.MovieView
 import com.fernandocejas.sample.features.movies.MoviesActivity
-import com.fernandocejas.sample.core.extension.empty
 import javax.inject.Inject
 import javax.inject.Singleton
 
@@ -37,7 +37,8 @@ import javax.inject.Singleton
 class Navigator
 @Inject constructor(private val authenticator: Authenticator) {
 
- private fun showLogin(context: Context) = context.startActivity(LoginActivity.callingIntent(context))
+ private fun showLogin(context: Context) =
+ context.startActivity(LoginActivity.callingIntent(context))
 
  fun showMain(context: Context) {
  when (authenticator.userLoggedIn()) {
@@ -46,13 +47,14 @@ class Navigator
  }
  }
 
- private fun showMovies(context: Context) = context.startActivity(MoviesActivity.callingIntent(context))
+ private fun showMovies(context: Context) =
+ context.startActivity(MoviesActivity.callingIntent(context))
 
  fun showMovieDetails(activity: FragmentActivity, movie: MovieView, navigationExtras: Extras) {
  val intent = MovieDetailsActivity.callingIntent(activity, movie)
  val sharedView = navigationExtras.transitionSharedElement as ImageView
  val activityOptions = ActivityOptionsCompat
-  .makeSceneTransitionAnimation(activity, sharedView, sharedView.transitionName)
+ .makeSceneTransitionAnimation(activity, sharedView, sharedView.transitionName)
  activity.startActivity(intent, activityOptions.toBundle())
  }
 
@@ -70,7 +72,10 @@ class Navigator
  private fun createYoutubeIntent(videoUrl: String): Intent {
  val videoId = when {
  videoUrl.startsWith(VIDEO_URL_HTTP) -> videoUrl.replace(VIDEO_URL_HTTP, String.empty())
- videoUrl.startsWith(VIDEO_URL_HTTPS) -> videoUrl.replace(VIDEO_URL_HTTPS, String.empty())
+ videoUrl.startsWith(VIDEO_URL_HTTPS) -> videoUrl.replace(
+ VIDEO_URL_HTTPS,
+ String.empty()
+ )
  else -> videoUrl
  }