From 9a37ab24a9c8909787e5514e06fde01f7412d8a7 Mon Sep 17 00:00:00 2001
From: ai-null <ainul.y9b@gmail.com>
Date: Thu, 11 Mar 2021 20:26:15 +0700
Subject: [PATCH] a bit of refactor

---
 .../ainul/oprek/adapter/ListItemAdapter.kt    | 14 +++++++---
 .../java/com/ainul/oprek/util/Constants.kt    |  2 ++
 .../com/ainul/oprek/util/EncryptManager.kt    |  2 +-
 .../main/res/layout/activity_detail_user.xml  | 16 -----------
 app/src/main/res/layout/activity_profile.xml  | 27 +------------------
 app/src/main/res/values/strings.xml           |  1 +
 6 files changed, 15 insertions(+), 47 deletions(-)
 delete mode 100644 app/src/main/res/layout/activity_detail_user.xml

diff --git a/app/src/main/java/com/ainul/oprek/adapter/ListItemAdapter.kt b/app/src/main/java/com/ainul/oprek/adapter/ListItemAdapter.kt
index f4dfe61..da81313 100644
--- a/app/src/main/java/com/ainul/oprek/adapter/ListItemAdapter.kt
+++ b/app/src/main/java/com/ainul/oprek/adapter/ListItemAdapter.kt
@@ -24,10 +24,7 @@ class ListItemAdapter(private val clickListener: ListItemListener) :
     ListAdapter<Project, ListItemAdapter.ListItemViewHolder>(ProjectDiffUtil()) {
 
     override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): ListItemViewHolder {
-        val layoutInflater = LayoutInflater.from(parent.context)
-        val holder = ListItemBinding.inflate(layoutInflater, parent, false)
-
-        return ListItemViewHolder(holder)
+        return ListItemViewHolder.from(parent)
     }
 
     override fun onBindViewHolder(holder: ListItemViewHolder, position: Int) {
@@ -44,5 +41,14 @@ class ListItemAdapter(private val clickListener: ListItemListener) :
             binding.project = data
             binding.executePendingBindings()
         }
+
+        companion object {
+            fun from (parent: ViewGroup): ListItemViewHolder {
+                val layoutInflater = LayoutInflater.from(parent.context)
+                val holder = ListItemBinding.inflate(layoutInflater, parent, false)
+
+                return ListItemViewHolder(holder)
+            }
+        }
     }
 }
\ No newline at end of file
diff --git a/app/src/main/java/com/ainul/oprek/util/Constants.kt b/app/src/main/java/com/ainul/oprek/util/Constants.kt
index c3c6e32..d573c84 100644
--- a/app/src/main/java/com/ainul/oprek/util/Constants.kt
+++ b/app/src/main/java/com/ainul/oprek/util/Constants.kt
@@ -7,6 +7,8 @@ object Constants {
         CANCEL(-1);
     }
 
+    const val MASTER_KEY = "oprek-master-key"
+
     // Screen navigation request code
     const val REQUEST_CODE_UPDATE_PROJECT = 1
     const val REQUEST_CODE_CHOOSE_IMAGE = 2
diff --git a/app/src/main/java/com/ainul/oprek/util/EncryptManager.kt b/app/src/main/java/com/ainul/oprek/util/EncryptManager.kt
index 097fc13..883581c 100644
--- a/app/src/main/java/com/ainul/oprek/util/EncryptManager.kt
+++ b/app/src/main/java/com/ainul/oprek/util/EncryptManager.kt
@@ -14,7 +14,7 @@ import androidx.security.crypto.MasterKey
 class EncryptManager(context: Context) {
 
     // this will be our private key for the app, also with the schema to encrypt
-    private val masterKey = MasterKey.Builder(context, "oprek-master-key")
+    private val masterKey = MasterKey.Builder(context, Constants.MASTER_KEY)
         .setKeyScheme(MasterKey.KeyScheme.AES256_GCM)
         .build()
 
diff --git a/app/src/main/res/layout/activity_detail_user.xml b/app/src/main/res/layout/activity_detail_user.xml
deleted file mode 100644
index 347b39b..0000000
--- a/app/src/main/res/layout/activity_detail_user.xml
+++ /dev/null
@@ -1,16 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<androidx.constraintlayout.widget.ConstraintLayout
-    xmlns:android="http://schemas.android.com/apk/res/android" android:layout_width="match_parent"
-    xmlns:app="http://schemas.android.com/apk/res-auto"
-    android:layout_height="match_parent">
-
-    <TextView
-        android:layout_width="match_parent"
-        app:layout_constraintTop_toTopOf="parent"
-        app:layout_constraintBottom_toBottomOf="parent"
-        android:gravity="center"
-        android:text="Detail user Activity"
-        style="@style/TextAppearance.AppCompat.Headline"
-        android:layout_height="wrap_content"/>
-
-</androidx.constraintlayout.widget.ConstraintLayout>
\ No newline at end of file
diff --git a/app/src/main/res/layout/activity_profile.xml b/app/src/main/res/layout/activity_profile.xml
index 6073f81..2af6c96 100644
--- a/app/src/main/res/layout/activity_profile.xml
+++ b/app/src/main/res/layout/activity_profile.xml
@@ -71,6 +71,7 @@
 
                 <androidx.cardview.widget.CardView
                     android:layout_width="match_parent"
+                    android:layout_marginBottom="@dimen/medium_margin"
                     android:layout_height="wrap_content">
 
                     <LinearLayout
@@ -189,32 +190,6 @@
                         </LinearLayout>
                     </LinearLayout>
                 </androidx.cardview.widget.CardView>
-
-                <androidx.cardview.widget.CardView
-                    android:layout_width="match_parent"
-                    android:layout_height="wrap_content"
-                    android:layout_marginTop="@dimen/medium_margin"
-                    android:layout_marginBottom="@dimen/medium_margin">
-
-                    <LinearLayout
-                        android:layout_width="match_parent"
-                        android:layout_height="wrap_content"
-                        android:background="@color/colorWhite"
-                        android:orientation="vertical">
-
-                        <TextView
-                            android:layout_width="wrap_content"
-                            android:layout_height="wrap_content"
-                            android:layout_marginStart="@dimen/medium_margin"
-                            android:layout_marginTop="@dimen/medium_margin"
-                            android:layout_marginBottom="@dimen/margin"
-                            android:text="Statistics"
-                            android:textAppearance="@style/TextAppearance.AppCompat.Subhead"
-                            android:textStyle="bold" />
-
-
-                    </LinearLayout>
-                </androidx.cardview.widget.CardView>
             </LinearLayout>
         </androidx.core.widget.NestedScrollView>
 
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index e15d98e..33f7c92 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -155,4 +155,5 @@
 
     <string name="dialog_alert_message">%s name can\'t be empty!</string>
     <string name="history">History</string>
+    <string name="statistics">Statistics</string>
 </resources>