feat: ability to edit badges (#429)

fossasia · Jul 10, 2019 · 80fc945 · 80fc945
1 parent a97fe71
commit 80fc945
Show file tree

Hide file tree

Showing 15 changed files with 304 additions and 45 deletions.
diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
@@ -23,8 +23,6 @@
         android:roundIcon="@mipmap/ic_launcher_round"
         android:supportsRtl="true"
         android:theme="@style/AppTheme">
-        <activity android:name=".ui.EditBadgeActivity" android:screenOrientation="landscape">
-        </activity>
         <activity
             android:name=".ui.DrawerActivity"
             android:label="@string/app_name"
@@ -48,6 +46,8 @@
                 android:name="android.app.shortcuts"
                 android:resource="@xml/shortcuts" />
         </activity>
+        <activity android:name=".ui.EditBadgeActivity" android:screenOrientation="landscape"/>
+        <activity android:name=".ui.EditClipartActivity" android:screenOrientation="landscape"/>
 
         <provider
             android:name="androidx.core.content.FileProvider"

diff --git a/app/src/main/java/org/fossasia/badgemagic/adapter/SavedClipartsAdapter.kt b/app/src/main/java/org/fossasia/badgemagic/adapter/SavedClipartsAdapter.kt
@@ -1,17 +1,19 @@
 package org.fossasia.badgemagic.adapter
 
-import android.graphics.Bitmap
+import android.content.Intent
 import android.view.LayoutInflater
 import android.view.ViewGroup
 import android.widget.Toast
 import androidx.recyclerview.widget.RecyclerView
 import androidx.databinding.DataBindingUtil
 import org.fossasia.badgemagic.R
+import org.fossasia.badgemagic.data.SavedClipart
 import org.fossasia.badgemagic.databinding.RecyclerItemSavedClipartBinding
+import org.fossasia.badgemagic.ui.EditClipartActivity
 import org.fossasia.badgemagic.viewmodels.SavedClipartViewModel
 
 class SavedClipartsAdapter(
-    private var clipartList: List<Bitmap>,
+    private var clipartList: List<SavedClipart>,
     private val viewModel: SavedClipartViewModel
 ) : RecyclerView.Adapter<SavedClipartsViewHolder>() {
     override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): SavedClipartsViewHolder {
@@ -22,14 +24,21 @@ class SavedClipartsAdapter(
     override fun getItemCount(): Int = clipartList.size
 
     override fun onBindViewHolder(holder: SavedClipartsViewHolder, position: Int) {
-        holder.bind(clipartList[position])
-        holder.getBindingView().setOnClickListener {
+        holder.bind(clipartList[position].bitmap)
+        holder.getDeleteButton().setOnClickListener {
             viewModel.deleteClipart(position)
             Toast.makeText(it.context, "Delete Clipart Successfully", Toast.LENGTH_LONG).show()
         }
+        holder.getEditButton().setOnClickListener {
+            it.context.startActivity(
+                Intent(it.context, EditClipartActivity::class.java).apply {
+                    putExtra("fileName", clipartList[position].fileName)
+                }
+            )
+        }
     }
 
-    fun setList(list: List<Bitmap>) {
+    fun setList(list: List<SavedClipart>) {
         clipartList = list
         notifyDataSetChanged()
     }

diff --git a/app/src/main/java/org/fossasia/badgemagic/adapter/SavedClipartsViewHolder.kt b/app/src/main/java/org/fossasia/badgemagic/adapter/SavedClipartsViewHolder.kt
@@ -1,7 +1,9 @@
 package org.fossasia.badgemagic.adapter
 
 import android.graphics.Bitmap
+import androidx.appcompat.widget.AppCompatImageView
 import androidx.recyclerview.widget.RecyclerView
+import org.fossasia.badgemagic.R
 import org.fossasia.badgemagic.databinding.RecyclerItemSavedClipartBinding
 
 class SavedClipartsViewHolder(private val binding: RecyclerItemSavedClipartBinding) : RecyclerView.ViewHolder(binding.root) {
@@ -10,4 +12,6 @@ class SavedClipartsViewHolder(private val binding: RecyclerItemSavedClipartBindi
     }
 
     fun getBindingView() = binding.root
+    fun getDeleteButton(): AppCompatImageView = getBindingView().findViewById(R.id.button_delete)
+    fun getEditButton(): AppCompatImageView = getBindingView().findViewById(R.id.button_edit)
 }
diff --git a/app/src/main/java/org/fossasia/badgemagic/bindings/EditBadgeViewBindings.kt b/app/src/main/java/org/fossasia/badgemagic/bindings/EditBadgeViewBindings.kt
@@ -14,12 +14,17 @@ fun setBadgeDrawState(badge: DrawBadgeLayout, drawModeState: ObservableField<Dra
     drawModeState.get()?.let { badge.changeDrawState(it) }
 }
 
-@BindingAdapter("drawingJSON")
-fun setBadgeValues(badge: DrawBadgeLayout, drawJSON: ObservableField<String>) {
+@BindingAdapter("drawingBadgeJSON")
+fun setEditBadgeValues(badge: DrawBadgeLayout, drawJSON: ObservableField<String>) {
     val badgeConfig = SendingUtils.getBadgeFromJSON(drawJSON.get() ?: "{}")
     badgeConfig?.hexStrings?.let { badge.setValue(it) }
 }
 
+@BindingAdapter("drawingClipartJSON")
+fun setEditClipartValues(badge: DrawBadgeLayout, drawJSON: ObservableField<List<String>>) {
+    drawJSON.get()?.let { badge.setValue(it) }
+}
+
 @BindingAdapter("changeColor")
 fun changeColorState(imageView: ImageView, isEnabled: ObservableBoolean) {
     imageView.setColorFilter(

diff --git a/app/src/main/java/org/fossasia/badgemagic/database/ClipArtService.kt b/app/src/main/java/org/fossasia/badgemagic/database/ClipArtService.kt
@@ -12,6 +12,7 @@ import org.koin.core.inject
 
 class ClipArtService : KoinComponent {
     private val clipArts = MutableLiveData<SparseArray<Drawable>>()
+    private val storageClipArts = MutableLiveData<HashMap<String, Drawable?>>()
     private val resourceHelper: Resource by inject()
 
     init {
@@ -46,19 +47,25 @@ class ClipArtService : KoinComponent {
                 tempSparseArray.append(index, it)
             }
         }
-        val drawablesInStorage = getClipsFromStorage()
-        drawablesInStorage.forEach {
+        val drawablesInStorage = getClipsFromStorage().value
+        drawablesInStorage?.forEach {
             tempSparseArray.append(++lastIndex, it.value)
         }
 
         return tempSparseArray
     }
 
     fun updateClipArts() {
+        storageClipArts.value = StorageUtils.getAllClips()
         clipArts.value = getAllClips()
     }
 
     fun getClipArts(): LiveData<SparseArray<Drawable>> = clipArts
 
-    fun getClipsFromStorage() = StorageUtils.getAllClips()
+    fun getClipsFromStorage(): LiveData<HashMap<String, Drawable?>> = storageClipArts
+
+    fun deleteClipart(fileName: String) {
+        StorageUtils.deleteClipart(fileName)
+        updateClipArts()
+    }
 }
diff --git a/app/src/main/java/org/fossasia/badgemagic/di/Modules.kt b/app/src/main/java/org/fossasia/badgemagic/di/Modules.kt
@@ -10,6 +10,7 @@ import org.fossasia.badgemagic.viewmodels.SettingsViewModel
 import org.fossasia.badgemagic.viewmodels.DrawViewModel
 import org.fossasia.badgemagic.viewmodels.DrawerViewModel
 import org.fossasia.badgemagic.viewmodels.EditBadgeViewModel
+import org.fossasia.badgemagic.viewmodels.EditClipartViewModel
 import org.fossasia.badgemagic.viewmodels.SavedClipartViewModel
 import org.koin.android.ext.koin.androidContext
 import org.koin.androidx.viewmodel.dsl.viewModel
@@ -20,6 +21,7 @@ val viewModelModules = module {
     viewModel { FilesViewModel(get()) }
     viewModel { SettingsViewModel(get()) }
     viewModel { EditBadgeViewModel(get()) }
+    viewModel { EditClipartViewModel(get()) }
     viewModel { DrawViewModel(get()) }
     viewModel { DrawerViewModel(get()) }
     viewModel { SavedClipartViewModel(get()) }

diff --git a/app/src/main/java/org/fossasia/badgemagic/ui/EditClipartActivity.kt b/app/src/main/java/org/fossasia/badgemagic/ui/EditClipartActivity.kt
@@ -0,0 +1,48 @@
+package org.fossasia.badgemagic.ui
+
+import android.os.Bundle
+import android.widget.Toast
+import androidx.appcompat.app.AppCompatActivity
+import androidx.databinding.DataBindingUtil
+import androidx.lifecycle.Observer
+import kotlinx.android.synthetic.main.activity_edit_clipart.*
+import org.fossasia.badgemagic.R
+import org.fossasia.badgemagic.databinding.ActivityEditClipartBinding
+import org.fossasia.badgemagic.util.Converters
+import org.fossasia.badgemagic.util.StorageUtils
+import org.fossasia.badgemagic.viewmodels.EditClipartViewModel
+import org.koin.androidx.viewmodel.ext.android.viewModel
+
+class EditClipartActivity : AppCompatActivity() {
+
+    private val editClipartViewModel by viewModel<EditClipartViewModel>()
+    private lateinit var fileName: String
+
+    override fun onCreate(savedInstanceState: Bundle?) {
+        super.onCreate(savedInstanceState)
+        val activityDrawBinding = DataBindingUtil.setContentView<ActivityEditClipartBinding>(this, R.layout.activity_edit_clipart)
+        activityDrawBinding.viewModel = editClipartViewModel
+
+        if (intent.hasExtra("fileName")) {
+            fileName = intent?.extras?.getString("fileName") ?: ""
+            editClipartViewModel.drawingJSON.set(Converters.convertDrawableToLEDHex(StorageUtils.getClipartFromPath(fileName), false))
+        }
+
+        editClipartViewModel.savedButton.observe(this, Observer {
+            if (it) {
+                if (StorageUtils.saveEditedClipart(Converters.convertStringsToLEDHex(draw_layout.getCheckedList()), fileName)) {
+                    Toast.makeText(this, R.string.clipart_saved_success, Toast.LENGTH_LONG).show()
+                    editClipartViewModel.updateClipArts()
+                } else
+                    Toast.makeText(this, R.string.clipart_saved_error, Toast.LENGTH_LONG).show()
+                finish()
+            }
+        })
+
+        editClipartViewModel.resetButton.observe(this, Observer {
+            if (it) {
+                draw_layout.resetCheckListWithDummyData()
+            }
+        })
+    }
+}
diff --git a/app/src/main/java/org/fossasia/badgemagic/ui/fragments/SavedClipartFragment.kt b/app/src/main/java/org/fossasia/badgemagic/ui/fragments/SavedClipartFragment.kt
@@ -5,9 +5,12 @@ import android.view.LayoutInflater
 import android.view.View
 import android.view.ViewGroup
 import androidx.databinding.DataBindingUtil
+import androidx.lifecycle.Observer
 import org.fossasia.badgemagic.R
+import org.fossasia.badgemagic.data.SavedClipart
 import org.fossasia.badgemagic.databinding.FragmentSavedClipartsBinding
 import org.fossasia.badgemagic.ui.base.BaseFragment
+import org.fossasia.badgemagic.util.ImageUtils
 import org.fossasia.badgemagic.viewmodels.SavedClipartViewModel
 import org.koin.androidx.viewmodel.ext.android.viewModel
 
@@ -26,4 +29,14 @@ class SavedClipartFragment : BaseFragment() {
         binding.viewModel = viewModel
         return binding.root
     }
+
+    override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
+        super.onViewCreated(view, savedInstanceState)
+
+        viewModel.getStorageClipartLiveData().observe(viewLifecycleOwner, Observer { list ->
+            viewModel.adapter.setList(
+                list.map { SavedClipart(it.key, ImageUtils.convertToBitmap(it.value)) }
+            )
+        })
+    }
 }
diff --git a/app/src/main/java/org/fossasia/badgemagic/util/Converters.kt b/app/src/main/java/org/fossasia/badgemagic/util/Converters.kt
@@ -14,7 +14,7 @@ const val DRAWABLE_START = '«'
 const val DRAWABLE_END = '»'
 
 object Converters {
-    private fun convertDrawableToLEDHex(drawableIcon: Drawable?, invertLED: Boolean): List<String> {
+    fun convertDrawableToLEDHex(drawableIcon: Drawable?, invertLED: Boolean): List<String> {
         var bm = Bitmap.createBitmap(10, 10, Bitmap.Config.ARGB_8888)
         if (drawableIcon is VectorDrawable)
             bm = ImageUtils.scaleBitmap(ImageUtils.vectorToBitmap(drawableIcon), 40)

diff --git a/app/src/main/java/org/fossasia/badgemagic/util/StorageUtils.kt b/app/src/main/java/org/fossasia/badgemagic/util/StorageUtils.kt
@@ -160,6 +160,20 @@ object StorageUtils {
         return true
     }
 
+    fun saveEditedClipart(bitmap: Bitmap, fileName: String): Boolean {
+        checkDirectory()
+        val file = File(EXTERNAL_CLIPART_DIRECTORY, fileName)
+        try {
+            val out = FileOutputStream(file)
+            bitmap.compress(Bitmap.CompressFormat.PNG, 100, out)
+            out.flush()
+            out.close()
+        } catch (e: Exception) {
+            return false
+        }
+        return true
+    }
+
     fun getAllClips(): HashMap<String, Drawable?> {
         checkDirectory()
         val list = HashMap<String, Drawable?>()
@@ -173,6 +187,10 @@ object StorageUtils {
         return list
     }
 
+    fun getClipartFromPath(filename: String): Drawable? {
+        return Drawable.createFromPath(File(EXTERNAL_CLIPART_DIRECTORY, filename).absolutePath)
+    }
+
     fun deleteClipart(fileName: String) {
         checkDirectory()
         val deleteFile = File(EXTERNAL_CLIPART_DIRECTORY, fileName)

diff --git a/app/src/main/java/org/fossasia/badgemagic/viewmodels/EditClipartViewModel.kt b/app/src/main/java/org/fossasia/badgemagic/viewmodels/EditClipartViewModel.kt
@@ -0,0 +1,48 @@
+package org.fossasia.badgemagic.viewmodels
+
+import androidx.databinding.ObservableBoolean
+import androidx.databinding.ObservableField
+import androidx.lifecycle.MutableLiveData
+import androidx.lifecycle.ViewModel
+import org.fossasia.badgemagic.data.draw_layout.DrawMode
+import org.fossasia.badgemagic.database.ClipArtService
+
+class EditClipartViewModel(
+    private val clipArtService: ClipArtService
+) : ViewModel() {
+    var drawModeState: ObservableField<DrawMode> = ObservableField(DrawMode.NOTHING)
+    var drawingJSON: ObservableField<List<String>> = ObservableField(listOf())
+
+    var drawState: ObservableBoolean = ObservableBoolean(false)
+    var eraseState: ObservableBoolean = ObservableBoolean(false)
+
+    var savedButton: MutableLiveData<Boolean> = MutableLiveData()
+    var resetButton: MutableLiveData<Boolean> = MutableLiveData()
+
+    init {
+        savedButton.value = false
+        resetButton.value = false
+    }
+
+    fun changeDrawState() {
+        drawState.set(!drawState.get())
+        eraseState.set(false)
+        drawModeState.set(if (drawState.get()) DrawMode.DRAW else DrawMode.NOTHING)
+    }
+
+    fun changeEraseState() {
+        eraseState.set(!eraseState.get())
+        drawState.set(false)
+        drawModeState.set(if (eraseState.get()) DrawMode.ERASE else DrawMode.NOTHING)
+    }
+
+    fun saveBadge() {
+        savedButton.value = true
+    }
+
+    fun changeResetState() {
+        resetButton.value = true
+    }
+
+    fun updateClipArts() = clipArtService.updateClipArts()
+}
diff --git a/app/src/main/java/org/fossasia/badgemagic/viewmodels/SavedClipartViewModel.kt b/app/src/main/java/org/fossasia/badgemagic/viewmodels/SavedClipartViewModel.kt
@@ -4,29 +4,17 @@ import androidx.lifecycle.ViewModel
 import org.fossasia.badgemagic.adapter.SavedClipartsAdapter
 import org.fossasia.badgemagic.data.SavedClipart
 import org.fossasia.badgemagic.database.ClipArtService
-import org.fossasia.badgemagic.util.ImageUtils
-import org.fossasia.badgemagic.util.StorageUtils
 
 class SavedClipartViewModel(
     private val clipArtService: ClipArtService
 ) : ViewModel() {
 
     var cliparts = listOf<SavedClipart>()
-    var adapter = SavedClipartsAdapter(cliparts.map { it.bitmap }, this)
+    var adapter = SavedClipartsAdapter(cliparts, this)
 
-    init {
-        cliparts = clipArtService.getClipsFromStorage().map { SavedClipart(it.key, ImageUtils.convertToBitmap(it.value)) }
-        adapter = SavedClipartsAdapter(cliparts.map { it.bitmap }, this)
-    }
-
-    private fun update() {
-        clipArtService.updateClipArts()
-        cliparts = clipArtService.getClipsFromStorage().map { SavedClipart(it.key, ImageUtils.convertToBitmap(it.value)) }
-        adapter.setList(cliparts.map { it.bitmap })
-    }
+    fun getStorageClipartLiveData() = clipArtService.getClipsFromStorage()
 
     fun deleteClipart(position: Int) {
-        StorageUtils.deleteClipart(cliparts[position].fileName)
-        update()
+        clipArtService.deleteClipart(cliparts[position].fileName)
     }
 }