#140[30m]. Add alert on account deletion. Mark archived account grey on Accounts screen.

Author: Evgenii Kanivets

app/src/main/java/com/blogspot/e_kanivets/moneytracker/activity/account/EditAccountActivity.java

@@ -1,5 +1,7 @@
 package com.blogspot.e_kanivets.moneytracker.activity.account;
 
+import android.app.AlertDialog;
+import android.content.DialogInterface;
 import android.view.Menu;
 import android.view.MenuItem;
 import android.view.View;
@@ -89,7 +91,7 @@ void done() {
 
     private void archive() {
         if (account.equals(accountController.readDefaultAccount())) {
-            showToast("You can't archive a default account.");
+            showToast(R.string.cant_archive_default_account);
         } else {
             accountController.archive(account);
             setResult(RESULT_OK);
@@ -104,9 +106,19 @@ private void restore() {
     }
 
     private void delete() {
-        accountController.delete(account);
-        setResult(RESULT_OK);
-        finish();
+        AlertDialog.Builder builder = new AlertDialog.Builder(this);
+        builder.setTitle(R.string.delete_account_title);
+        builder.setMessage(R.string.delete_account_message);
+        builder.setPositiveButton(android.R.string.ok, new DialogInterface.OnClickListener() {
+            @Override
+            public void onClick(DialogInterface dialog, int which) {
+                accountController.delete(account);
+                setResult(RESULT_OK);
+                finish();
+            }
+        });
+        builder.setNegativeButton(android.R.string.cancel, null);
+        builder.show();
     }
 
 }

app/src/main/java/com/blogspot/e_kanivets/moneytracker/adapter/AccountAdapter.java

@@ -37,6 +37,7 @@ public class AccountAdapter extends BaseAdapter {
     private int whiteGreen;
     private int red;
     private int green;
+    private int grey;
 
     @SuppressWarnings("deprecation")
     public AccountAdapter(Context context, List<Account> accounts) {
@@ -49,6 +50,7 @@ public AccountAdapter(Context context, List<Account> accounts) {
         whiteGreen = context.getResources().getColor(R.color.white_green);
         red = context.getResources().getColor(R.color.red);
         green = context.getResources().getColor(R.color.green);
+        grey = context.getResources().getColor(R.color.grey_inactive);
     }
 
     @Override
@@ -82,7 +84,11 @@ public View getView(final int position, View convertView, ViewGroup parent) {
 
         Account account = accounts.get(position);
 
-        convertView.setBackgroundColor(account.getFullSum() >= 0.0 ? whiteGreen : whiteRed);
+        if (account.isArchived()) {
+            convertView.setBackgroundColor(grey);
+        } else {
+            convertView.setBackgroundColor(account.getFullSum() >= 0.0 ? whiteGreen : whiteRed);
+        }
 
         viewHolder.tvCurSum.setTextColor(account.getFullSum() >= 0.0 ? green : red);
         viewHolder.tvCurrency.setTextColor(account.getFullSum() >= 0.0 ? green : red);

app/src/main/res/values/colors.xml

@@ -1,6 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <resources>
     <color name="white">#ffffff</color>
+    <color name="grey_inactive">#E0E0E0</color>
 
     <color name="white_green">#f1f8e9</color>
     <color name="green_light">#8bc34a</color>

app/src/main/res/values/strings.xml

@@ -129,4 +129,7 @@
     <string name="goal">Goal</string>
     <string name="restore">Restore</string>
     <string name="archive">Archive</string>
+    <string name="cant_archive_default_account">You can\'t archive a default account.</string>
+    <string name="delete_account_title">Delete account</string>
+    <string name="delete_account_message">Are you sure about deleting this account? It can\'t be restored after.</string>
 </resources>