#168. Migrate ListView to RecyclerView in MainActivity.

app/src/main/java/com/blogspot/e_kanivets/moneytracker/MtApp.java

@@ -6,7 +6,6 @@
 import com.blogspot.e_kanivets.moneytracker.di.DaggerAppComponent;
 import com.blogspot.e_kanivets.moneytracker.di.module.ControllerModule;
 import com.blogspot.e_kanivets.moneytracker.di.module.repo.CachedRepoModule;
-
 import com.blogspot.e_kanivets.moneytracker.util.AnswersProxy;
 import com.crashlytics.android.Crashlytics;
 

app/src/main/java/com/blogspot/e_kanivets/moneytracker/activity/ReportActivity.java

@@ -74,7 +74,7 @@ protected void initViews() {
         initSpinnerCurrency();
 
         shortSummaryPresenter = new ShortSummaryPresenter(ReportActivity.this);
-        expandableListView.addHeaderView(shortSummaryPresenter.create(false));
+        expandableListView.addHeaderView(shortSummaryPresenter.create(false, null));
     }
 
     private void update(String currency) {

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

@@ -22,30 +22,36 @@
 import butterknife.OnItemClick;
 
 public class AccountsActivity extends BaseBackActivity {
-    @SuppressWarnings("unused") private static final String TAG = "AccountsActivity";
+    @SuppressWarnings("unused")
+    private static final String TAG = "AccountsActivity";
 
     private static final int REQUEST_ADD_ACCOUNT = 1;
     private static final int REQUEST_TRANSFER = 2;
     private static final int REQUEST_EDIT_ACCOUNT = 3;
 
-    @Inject AccountController accountController;
+    @Inject
+    AccountController accountController;
 
     private AccountsSummaryPresenter summaryPresenter;
 
-    @BindView(R.id.listView) ListView listView;
+    @BindView(R.id.listView)
+    ListView listView;
 
-    @Override protected int getContentViewId() {
+    @Override
+    protected int getContentViewId() {
         return R.layout.activity_accounts;
     }
 
-    @Override protected boolean initData() {
+    @Override
+    protected boolean initData() {
         boolean result = super.initData();
         getAppComponent().inject(AccountsActivity.this);
         summaryPresenter = new AccountsSummaryPresenter(AccountsActivity.this);
         return result;
     }
 
-    @Override protected void initViews() {
+    @Override
+    protected void initViews() {
         super.initViews();
 
         listView.addHeaderView(summaryPresenter.create());
@@ -54,23 +60,23 @@ public class AccountsActivity extends BaseBackActivity {
         update();
     }
 
-    @Override public boolean onCreateOptionsMenu(Menu menu) {
+    @Override
+    public boolean onCreateOptionsMenu(Menu menu) {
         getMenuInflater().inflate(R.menu.menu_accounts, menu);
         return true;
     }
 
-    @Override public boolean onOptionsItemSelected(MenuItem item) {
-        switch (item.getItemId()) {
-            case R.id.action_transfer:
-                makeTransfer();
-                return true;
-
-            default:
-                return super.onOptionsItemSelected(item);
+    @Override
+    public boolean onOptionsItemSelected(MenuItem item) {
+        if (item.getItemId() == R.id.action_transfer) {
+            makeTransfer();
+            return true;
         }
+        return super.onOptionsItemSelected(item);
     }
 
-    @OnItemClick(R.id.listView) public void onAccountClick(int position) {
+    @OnItemClick(R.id.listView)
+    public void onAccountClick(int position) {
         Account account = accountController.readAll().get(position - 1);
         startActivityForResult(EditAccountActivity.Companion.newIntent(this, account), REQUEST_EDIT_ACCOUNT);
     }
@@ -80,13 +86,15 @@ public void makeTransfer() {
         startActivityForResult(new Intent(AccountsActivity.this, TransferActivity.class), REQUEST_TRANSFER);
     }
 
-    @OnClick(R.id.btn_add_account) public void addAccount() {
+    @OnClick(R.id.btn_add_account)
+    public void addAccount() {
         AnswersProxy.get().logButton("Add Account");
         Intent intent = new Intent(AccountsActivity.this, AddAccountActivity.class);
         startActivityForResult(intent, REQUEST_ADD_ACCOUNT);
     }
 
-    @Override public void onActivityResult(int requestCode, int resultCode, Intent data) {
+    @Override
+    public void onActivityResult(int requestCode, int resultCode, Intent data) {
         super.onActivityResult(requestCode, resultCode, data);
 
         if (resultCode == AppCompatActivity.RESULT_OK) {

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

@@ -7,6 +7,7 @@ import android.content.Intent
 import android.support.v4.view.ViewPager.OnPageChangeListener
 import android.view.Menu
 import android.view.MenuItem
+import android.view.View
 import com.blogspot.e_kanivets.moneytracker.R
 import com.blogspot.e_kanivets.moneytracker.activity.account.edit.fragment.AccountOperationsFragment
 import com.blogspot.e_kanivets.moneytracker.activity.account.edit.fragment.EditAccountFragment
@@ -18,6 +19,8 @@ import kotlinx.android.synthetic.main.activity_edit_account.fabDone
 import kotlinx.android.synthetic.main.activity_edit_account.tabLayout
 import kotlinx.android.synthetic.main.activity_edit_account.viewPager
 import javax.inject.Inject
+import android.view.inputmethod.InputMethodManager
+import kotlinx.android.synthetic.main.fragment_edit_account.*
 
 class EditAccountActivity : BaseBackActivity() {
 
@@ -55,12 +58,30 @@ class EditAccountActivity : BaseBackActivity() {
             override fun onPageScrolled(position: Int, positionOffset: Float, positionOffsetPixels: Int) {}
 
             override fun onPageSelected(position: Int) {
-                if (position == 0) fabDone.show() else fabDone.hide()
+                if (position == 0) {
+                    fabDone.show()
+                    showKeyboard()
+                } else {
+                    fabDone.hide()
+                    hideKeyboard()
+                }
             }
-
         })
     }
 
+    private fun hideKeyboard() {
+        val view: View? = currentFocus
+        if (view != null) {
+            val imm = getSystemService(Activity.INPUT_METHOD_SERVICE) as InputMethodManager
+            imm.hideSoftInputFromWindow(view.windowToken, 0)
+        }
+    }
+
+    private fun showKeyboard() {
+        val imm = getSystemService(Activity.INPUT_METHOD_SERVICE) as InputMethodManager
+        imm.showSoftInput(etTitle, 0)
+    }
+
     override fun onCreateOptionsMenu(menu: Menu): Boolean {
         menuInflater.inflate(if (account.isArchived) R.menu.menu_archived_account else R.menu.menu_account, menu)
         return true

app/src/main/java/com/blogspot/e_kanivets/moneytracker/activity/account/edit/fragment/AccountOperationsFragment.kt

@@ -30,11 +30,11 @@ class AccountOperationsFragment : BaseFragment() {
 
     override fun initData() {
         appComponent.inject(this@AccountOperationsFragment)
-        arguments?.let { arguments -> account = arguments.getParcelable(AccountOperationsFragment.KEY_ACCOUNT) }
+        arguments?.let { arguments -> account = arguments.getParcelable(KEY_ACCOUNT) }
     }
 
     override fun initViews(view: View) {
-        listView.adapter = RecordAdapter(activity, getRecords())
+        recyclerView.adapter = RecordAdapter(requireContext(), getRecords(), false, null)
     }
 
     private fun getRecords(): List<Record> {
@@ -83,4 +83,5 @@ class AccountOperationsFragment : BaseFragment() {
         }
 
     }
+
 }

app/src/main/java/com/blogspot/e_kanivets/moneytracker/activity/charts/fragment/GraphFragment.java

@@ -107,5 +107,4 @@ private void initViews(@Nullable View rootView) {
             barChart.setHighlightPerTapEnabled(false);
         }
     }
-
 }

app/src/main/java/com/blogspot/e_kanivets/moneytracker/activity/charts/fragment/SummaryFragment.java

@@ -72,5 +72,4 @@ private void initViews(@Nullable View rootView) {
             listView.setAdapter(new MonthSummaryAdapter(getActivity(), monthReport));
         }
     }
-
 }

app/src/main/java/com/blogspot/e_kanivets/moneytracker/activity/record/MainActivity.kt

@@ -18,7 +18,6 @@ import com.blogspot.e_kanivets.moneytracker.entity.Period
 import com.blogspot.e_kanivets.moneytracker.entity.data.Record
 import com.blogspot.e_kanivets.moneytracker.report.ReportMaker
 import com.blogspot.e_kanivets.moneytracker.ui.AppRateDialog
-import com.blogspot.e_kanivets.moneytracker.ui.presenter.ShortSummaryPresenter
 import com.blogspot.e_kanivets.moneytracker.util.AnswersProxy
 import kotlinx.android.synthetic.main.activity_main.*
 import kotlinx.android.synthetic.main.content_main.*
@@ -28,6 +27,7 @@ class MainActivity : BaseDrawerActivity() {
 
     private lateinit var recordList: List<Record>
     private lateinit var period: Period
+    private lateinit var recordAdapter: RecordAdapter
 
     @Inject
     lateinit var recordController: RecordController
@@ -44,8 +44,6 @@ class MainActivity : BaseDrawerActivity() {
     @Inject
     lateinit var formatController: FormatController
 
-    private lateinit var summaryPresenter: ShortSummaryPresenter
-
     private lateinit var tvDefaultAccountTitle: TextView
     private lateinit var tvDefaultAccountSum: TextView
     private lateinit var tvCurrency: TextView
@@ -57,7 +55,6 @@ class MainActivity : BaseDrawerActivity() {
         appComponent.inject(this)
 
         preferenceController.addLaunchCount()
-        summaryPresenter = ShortSummaryPresenter(this)
 
         return super.initData()
     }
@@ -73,9 +70,11 @@ class MainActivity : BaseDrawerActivity() {
         tvDefaultAccountSum = navigationView.getHeaderView(0).findViewById(R.id.tvDefaultAccountSum)
         tvCurrency = navigationView.getHeaderView(0).findViewById(R.id.tvCurrency)
 
-        val summaryView = summaryPresenter.create(true)
-        listView.addHeaderView(summaryView)
-        summaryView.setOnClickListener { showReport() }
+        recordAdapter = RecordAdapter(this, listOf(), true) { position ->
+            if (position == 0) showReport()
+            else editRecord(position)
+        }
+        recyclerView.adapter = recordAdapter
 
         spinner.setPeriodSelectedListener { period ->
             this.period = period
@@ -85,16 +84,13 @@ class MainActivity : BaseDrawerActivity() {
 
         spinner.setPeriod(periodController.readLastUsedPeriod())
 
-        listView.setOnItemClickListener { _, _, position, _ ->
-            editRecord(position)
-        }
-
         btnAddExpense.setOnClickListener { addExpense() }
         btnAddIncome.setOnClickListener { addIncome() }
     }
 
     private fun editRecord(position: Int) {
         AnswersProxy.get().logButton("Edit Record")
+
         // Minus one because of list view's header view
         val record = recordList[position - 1]
         startAddRecordActivity(record, AddRecordActivity.Mode.MODE_EDIT, record.type)
@@ -125,7 +121,7 @@ class MainActivity : BaseDrawerActivity() {
                 REQUEST_ACTION_RECORD -> update()
 
                 REQUEST_BACKUP -> {
-                    appComponent.inject(this@MainActivity)
+                    appComponent.inject(this)
                     update()
                 }
 
@@ -139,13 +135,12 @@ class MainActivity : BaseDrawerActivity() {
         recordList = recordController.getRecordsForPeriod(period)
         recordList = recordList.reversed()
 
-        listView.adapter = RecordAdapter(this, recordList)
-
         val currency = currencyController.readDefaultCurrency()
 
         val reportMaker = ReportMaker(rateController)
         val report = reportMaker.getRecordReport(currency, period, recordList)
-        summaryPresenter.update(report, currency, reportMaker.currencyNeeded(currency, recordList))
+
+        recordAdapter.setRecords(recordList, report, currency, reportMaker.currencyNeeded(currency, recordList))
 
         fillDefaultAccount()
     }
@@ -174,7 +169,6 @@ class MainActivity : BaseDrawerActivity() {
     }
 
     companion object {
-        private const val TAG = "MainActivity"
         private const val REQUEST_ACTION_RECORD = 6
     }
 

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

@@ -101,7 +101,7 @@ public View getView(final int position, View convertView, ViewGroup parent) {
     }
 
     public static class ViewHolder {
-        @BindView(R.id.tv_title)
+        @BindView(R.id.tvTitle)
         TextView tvTitle;
         @BindView(R.id.tv_cur_sum)
         TextView tvCurSum;

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

@@ -98,7 +98,7 @@ protected void publishResults(CharSequence constraint, FilterResults results) {
     }
 
     public static class ViewHolder {
-        @BindView(R.id.tv_category)
+        @BindView(R.id.tvCategory)
         TextView tvCategory;
         @BindView(R.id.iv_cancel)
         View ivCancel;

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

@@ -95,9 +95,9 @@ private void customizeView(View view, Map<String, String> values, boolean groupV
     }
 
     public static class ViewHolder {
-        @BindView(R.id.tv_category)
+        @BindView(R.id.tvCategory)
         TextView tvCategory;
-        @BindView(R.id.tv_total)
+        @BindView(R.id.tvTotal)
         TextView tvTotal;
 
         public ViewHolder(View view) {

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

@@ -99,11 +99,11 @@ public View getView(int position, View convertView, ViewGroup parent) {
     }
 
     public static class ViewHolder {
-        @BindView(R.id.tv_month)
+        @BindView(R.id.tvMonth)
         TextView tvMonth;
-        @BindView(R.id.tv_total_income)
+        @BindView(R.id.tvTotalIncome)
         TextView tvTotalIncome;
-        @BindView(R.id.tv_total_expense)
+        @BindView(R.id.tvTotalExpense)
         TextView tvTotalExpense;
 
         public ViewHolder(View view) {