X-Git-Url: https://git.ktnx.net/?a=blobdiff_plain;f=app%2Fsrc%2Fmain%2Fjava%2Fnet%2Fktnx%2Fmobileledger%2Fmodel%2FMobileLedgerProfile.java;h=229e53483979a7927d5184df36d46952a8059089;hb=20c03b7a5eb152d42fbbe9ecbaae27530563b398;hp=7ae6b5a91b8d96a9c196fc52e362bee2d7e7e6bf;hpb=1aac3d8310057f17400cd194ffcd9352f3021c11;p=mobile-ledger-staging.git diff --git a/app/src/main/java/net/ktnx/mobileledger/model/MobileLedgerProfile.java b/app/src/main/java/net/ktnx/mobileledger/model/MobileLedgerProfile.java index 7ae6b5a9..229e5348 100644 --- a/app/src/main/java/net/ktnx/mobileledger/model/MobileLedgerProfile.java +++ b/app/src/main/java/net/ktnx/mobileledger/model/MobileLedgerProfile.java @@ -20,31 +20,22 @@ package net.ktnx.mobileledger.model; import android.content.res.Resources; import android.database.Cursor; import android.database.sqlite.SQLiteDatabase; -import android.os.Build; -import android.text.TextUtils; import android.util.SparseArray; import androidx.annotation.Nullable; -import androidx.lifecycle.LiveData; -import androidx.lifecycle.MutableLiveData; import net.ktnx.mobileledger.App; import net.ktnx.mobileledger.R; import net.ktnx.mobileledger.async.DbOpQueue; import net.ktnx.mobileledger.async.SendTransactionTask; -import net.ktnx.mobileledger.utils.LockHolder; -import net.ktnx.mobileledger.utils.Locker; import net.ktnx.mobileledger.utils.Logger; -import net.ktnx.mobileledger.utils.MLDB; import net.ktnx.mobileledger.utils.Misc; +import net.ktnx.mobileledger.utils.SimpleDate; import org.jetbrains.annotations.Contract; import java.util.ArrayList; -import java.util.Calendar; -import java.util.Date; import java.util.HashMap; -import java.util.Iterator; import java.util.List; import java.util.Locale; import java.util.Map; @@ -53,13 +44,8 @@ import java.util.Objects; import static net.ktnx.mobileledger.utils.Logger.debug; public final class MobileLedgerProfile { - private final MutableLiveData> displayedAccounts; - private final MutableLiveData> allTransactions; - private final MutableLiveData> displayedTransactions; // N.B. when adding new fields, update the copy-constructor below private final String uuid; - private final Locker accountsLocker = new Locker(); - private List allAccounts; private String name; private boolean permitPosting; private boolean showCommentsByDefault; @@ -73,24 +59,13 @@ public final class MobileLedgerProfile { private int themeHue; private int orderNo = -1; private SendTransactionTask.API apiVersion = SendTransactionTask.API.auto; - private Calendar firstTransactionDate; - private Calendar lastTransactionDate; private FutureDates futureDates = FutureDates.None; private boolean accountsLoaded; private boolean transactionsLoaded; // N.B. when adding new fields, update the copy-constructor below - transient private AccountListLoader loader = null; - transient private Thread displayedAccountsUpdater; - transient private AccountListSaver accountListSaver; - transient private TransactionListSaver transactionListSaver; transient private AccountAndTransactionListSaver accountAndTransactionListSaver; - private Map accountMap = new HashMap<>(); public MobileLedgerProfile(String uuid) { this.uuid = uuid; - allAccounts = new ArrayList<>(); - displayedAccounts = new MutableLiveData<>(); - allTransactions = new MutableLiveData<>(new ArrayList<>()); - displayedTransactions = new MutableLiveData<>(new ArrayList<>()); } public MobileLedgerProfile(MobileLedgerProfile origin) { uuid = origin.uuid; @@ -108,13 +83,6 @@ public final class MobileLedgerProfile { futureDates = origin.futureDates; apiVersion = origin.apiVersion; defaultCommodity = origin.defaultCommodity; - firstTransactionDate = origin.firstTransactionDate; - lastTransactionDate = origin.lastTransactionDate; - displayedAccounts = origin.displayedAccounts; - allAccounts = origin.allAccounts; - accountMap = origin.accountMap; - displayedTransactions = origin.displayedTransactions; - allTransactions = origin.allTransactions; accountsLoaded = origin.accountsLoaded; transactionsLoaded = origin.transactionsLoaded; } @@ -161,7 +129,7 @@ public final class MobileLedgerProfile { db.beginTransactionNonExclusive(); try { int orderNo = 0; - for (MobileLedgerProfile p : Data.profiles.getValue()) { + for (MobileLedgerProfile p : Objects.requireNonNull(Data.profiles.getValue())) { db.execSQL("update profiles set order_no=? where uuid=?", new Object[]{orderNo, p.getUuid()}); p.orderNo = orderNo; @@ -173,67 +141,6 @@ public final class MobileLedgerProfile { db.endTransaction(); } } - public static ArrayList mergeAccountLists(List oldList, - List newList) { - LedgerAccount oldAcc, newAcc; - ArrayList merged = new ArrayList<>(); - - Iterator oldIterator = oldList.iterator(); - Iterator newIterator = newList.iterator(); - - while (true) { - if (!oldIterator.hasNext()) { - // the rest of the incoming are new - if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.N) { - newIterator.forEachRemaining(merged::add); - } - else { - while (newIterator.hasNext()) - merged.add(newIterator.next()); - } - break; - } - oldAcc = oldIterator.next(); - - if (!newIterator.hasNext()) { - // no more incoming accounts. ignore the rest of the old - break; - } - newAcc = newIterator.next(); - - // ignore now missing old items - if (oldAcc.getName() - .compareTo(newAcc.getName()) < 0) - continue; - - // add newly found items - if (oldAcc.getName() - .compareTo(newAcc.getName()) > 0) - { - merged.add(newAcc); - continue; - } - - // two items with same account names; forward-merge UI-controlled fields - // it is important that the result list contains a new LedgerAccount instance - // so that the change is propagated to the UI - newAcc.setExpanded(oldAcc.isExpanded()); - newAcc.setAmountsExpanded(oldAcc.amountsExpanded()); - merged.add(newAcc); - } - - return merged; - } - public void mergeAccountList(List newList) { - - try (LockHolder l = accountsLocker.lockForWriting()) { - allAccounts = mergeAccountLists(allAccounts, newList); - updateAccountsMap(allAccounts); - } - } - public LiveData> getDisplayedAccounts() { - return displayedAccounts; - } @Contract(value = "null -> false", pure = true) @Override public boolean equals(@Nullable Object obj) { @@ -271,29 +178,8 @@ public final class MobileLedgerProfile { return false; if (apiVersion != p.apiVersion) return false; - if (!Objects.equals(firstTransactionDate, p.firstTransactionDate)) - return false; - if (!Objects.equals(lastTransactionDate, p.lastTransactionDate)) - return false; return futureDates == p.futureDates; } - synchronized public void scheduleAccountListReload() { - Logger.debug("async-acc", "scheduleAccountListReload() enter"); - if ((loader != null) && loader.isAlive()) { - Logger.debug("async-acc", "returning early - loader already active"); - return; - } - - Logger.debug("async-acc", "Starting AccountListLoader"); - loader = new AccountListLoader(this); - loader.start(); - } - synchronized public void abortAccountListReload() { - if (loader == null) - return; - loader.interrupt(); - loader = null; - } public boolean getShowCommentsByDefault() { return showCommentsByDefault; } @@ -421,11 +307,13 @@ public final class MobileLedgerProfile { db.endTransaction(); } } - public void storeAccount(SQLiteDatabase db, LedgerAccount acc, boolean storeUiFields) { + public void storeAccount(SQLiteDatabase db, int generation, LedgerAccount acc, + boolean storeUiFields) { // replace into is a bad idea because it would reset hidden to its default value // we like the default, but for new accounts only - String sql = "update accounts set keep = 1"; + String sql = "update accounts set generation = ?"; List params = new ArrayList<>(); + params.add(generation); if (storeUiFields) { sql += ", expanded=?"; params.add(acc.isExpanded() ? 1 : 0); @@ -436,37 +324,53 @@ public final class MobileLedgerProfile { db.execSQL(sql, params.toArray()); db.execSQL("insert into accounts(profile, name, name_upper, parent_name, level, " + - "expanded, keep) " + "select ?,?,?,?,?,0,1 where (select changes() = 0)", + "expanded, generation) select ?,?,?,?,?,0,? where (select changes() = 0)", new Object[]{uuid, acc.getName(), acc.getName().toUpperCase(), acc.getParentName(), - acc.getLevel() + acc.getLevel(), generation }); // debug("accounts", String.format("Stored account '%s' in DB [%s]", acc.getName(), uuid)); } - public void storeAccountValue(SQLiteDatabase db, String name, String currency, Float amount) { + public void storeAccountValue(SQLiteDatabase db, int generation, String name, String currency, + Float amount) { db.execSQL("replace into account_values(profile, account, " + - "currency, value, keep) values(?, ?, ?, ?, 1);", - new Object[]{uuid, name, Misc.emptyIsNull(currency), amount}); + "currency, value, generation) values(?, ?, ?, ?, ?);", + new Object[]{uuid, name, Misc.emptyIsNull(currency), amount, generation}); } - public void storeTransaction(SQLiteDatabase db, LedgerTransaction tr) { + public void storeTransaction(SQLiteDatabase db, int generation, LedgerTransaction tr) { tr.fillDataHash(); - db.execSQL("DELETE from transactions WHERE profile=? and id=?", - new Object[]{uuid, tr.getId()}); - db.execSQL("DELETE from transaction_accounts WHERE profile = ? and transaction_id=?", - new Object[]{uuid, tr.getId()}); - +// Logger.debug("storeTransaction", String.format(Locale.US, "ID %d", tr.getId())); + SimpleDate d = tr.getDate(); + db.execSQL("UPDATE transactions SET year=?, month=?, day=?, description=?, comment=?, " + + "data_hash=?, generation=? WHERE profile=? AND id=?", + new Object[]{d.year, d.month, d.day, tr.getDescription(), tr.getComment(), + tr.getDataHash(), generation, uuid, tr.getId() + }); db.execSQL("INSERT INTO transactions(profile, id, year, month, day, description, " + - "comment, data_hash, keep) values(?,?,?,?,?,?,?,?,1)", + "comment, data_hash, generation) " + + "select ?,?,?,?,?,?,?,?,? WHERE (select changes() = 0)", new Object[]{uuid, tr.getId(), tr.getDate().year, tr.getDate().month, tr.getDate().day, tr.getDescription(), tr.getComment(), - tr.getDataHash() + tr.getDataHash(), generation }); + int accountOrderNo = 1; for (LedgerTransactionAccount item : tr.getAccounts()) { + db.execSQL("UPDATE transaction_accounts SET account_name=?, amount=?, currency=?, " + + "comment=?, generation=? " + + "WHERE profile=? AND transaction_id=? AND order_no=?", + new Object[]{item.getAccountName(), item.getAmount(), + Misc.nullIsEmpty(item.getCurrency()), item.getComment(), + generation, uuid, tr.getId(), accountOrderNo + }); db.execSQL("INSERT INTO transaction_accounts(profile, transaction_id, " + - "account_name, amount, currency, comment) values(?, ?, ?, ?, ?, ?)", - new Object[]{uuid, tr.getId(), item.getAccountName(), item.getAmount(), - Misc.nullIsEmpty(item.getCurrency()), item.getComment() + "order_no, account_name, amount, currency, comment, generation) " + + "select ?, ?, ?, ?, ?, ?, ?, ? WHERE (select changes() = 0)", + new Object[]{uuid, tr.getId(), accountOrderNo, item.getAccountName(), + item.getAmount(), Misc.nullIsEmpty(item.getCurrency()), + item.getComment(), generation }); + + accountOrderNo++; } // debug("profile", String.format("Transaction %d stored", tr.getId())); } @@ -558,39 +462,43 @@ public final class MobileLedgerProfile { // debug("profile", String.format("Profile.setThemeHue(%d) called", themeHue)); this.themeHue = themeHue; } - public void markTransactionsAsNotPresent(SQLiteDatabase db) { - db.execSQL("UPDATE transactions set keep=0 where profile=?", new String[]{uuid}); - - } - private void markAccountsAsNotPresent(SQLiteDatabase db) { - db.execSQL("update account_values set keep=0 where profile=?;", new String[]{uuid}); - db.execSQL("update accounts set keep=0 where profile=?;", new String[]{uuid}); - - } - private void deleteNotPresentAccounts(SQLiteDatabase db) { - db.execSQL("delete from account_values where keep=0 and profile=?", new String[]{uuid}); - db.execSQL("delete from accounts where keep=0 and profile=?", new String[]{uuid}); - } - private void markTransactionAsPresent(SQLiteDatabase db, LedgerTransaction transaction) { - db.execSQL("UPDATE transactions SET keep = 1 WHERE profile = ? and id=?", - new Object[]{uuid, transaction.getId() - }); + public int getNextTransactionsGeneration(SQLiteDatabase db) { + int generation = 1; + try (Cursor c = db.rawQuery("SELECT generation FROM transactions WHERE profile=? LIMIT 1", + new String[]{uuid})) + { + if (c.moveToFirst()) { + generation = c.getInt(0) + 1; + } + } + return generation; } - private void markTransactionsBeforeTransactionAsPresent(SQLiteDatabase db, - LedgerTransaction transaction) { - db.execSQL("UPDATE transactions SET keep=1 WHERE profile = ? and id < ?", - new Object[]{uuid, transaction.getId() - }); - + private int getNextAccountsGeneration(SQLiteDatabase db) { + int generation = 1; + try (Cursor c = db.rawQuery("SELECT generation FROM accounts WHERE profile=? LIMIT 1", + new String[]{uuid})) + { + if (c.moveToFirst()) { + generation = c.getInt(0) + 1; + } + } + return generation; } - private void deleteNotPresentTransactions(SQLiteDatabase db) { - db.execSQL("DELETE FROM transactions WHERE profile=? AND keep = 0", new String[]{uuid}); + private void deleteNotPresentAccounts(SQLiteDatabase db, int generation) { + Logger.debug("db/benchmark", "Deleting obsolete accounts"); + db.execSQL("DELETE FROM account_values WHERE profile=? AND generation <> ?", + new Object[]{uuid, generation}); + db.execSQL("DELETE FROM accounts WHERE profile=? AND generation <> ?", + new Object[]{uuid, generation}); + Logger.debug("db/benchmark", "Done deleting obsolete accounts"); } - private void setLastUpdateStamp() { - debug("db", "Updating transaction value stamp"); - Date now = new Date(); - setLongOption(MLDB.OPT_LAST_SCRAPE, now.getTime()); - Data.lastUpdateDate.postValue(now); + private void deleteNotPresentTransactions(SQLiteDatabase db, int generation) { + Logger.debug("db/benchmark", "Deleting obsolete transactions"); + db.execSQL("DELETE FROM transaction_accounts WHERE profile=? AND generation <> ?", + new Object[]{uuid, generation}); + db.execSQL("DELETE FROM transactions WHERE profile=? AND generation <> ?", + new Object[]{uuid, generation}); + Logger.debug("db/benchmark", "Done deleting obsolete transactions"); } public void wipeAllData() { SQLiteDatabase db = App.getDatabase(); @@ -645,129 +553,20 @@ public final class MobileLedgerProfile { return null; } } - public Calendar getFirstTransactionDate() { - return firstTransactionDate; - } - public Calendar getLastTransactionDate() { - return lastTransactionDate; - } - private void applyTransactionFilter(List list) { - final String accFilter = Data.accountFilter.getValue(); - if (TextUtils.isEmpty(accFilter)) { - displayedTransactions.postValue(list); - } - else { - ArrayList newList = new ArrayList<>(); - for (LedgerTransaction tr : list) { - if (tr.hasAccountNamedLike(accFilter)) - newList.add(tr); - } - displayedTransactions.postValue(newList); - } - } - synchronized public void storeAccountListAsync(List list, - boolean storeUiFields) { - if (accountListSaver != null) - accountListSaver.interrupt(); - accountListSaver = new AccountListSaver(this, list, storeUiFields); - accountListSaver.start(); - } - public void setAndStoreAccountListFromWeb(ArrayList list) { - SQLiteDatabase db = App.getDatabase(); - db.beginTransactionNonExclusive(); - try { - markAccountsAsNotPresent(db); - for (LedgerAccount acc : list) { - storeAccount(db, acc, false); - for (LedgerAmount amt : acc.getAmounts()) { - storeAccountValue(db, acc.getName(), amt.getCurrency(), amt.getAmount()); - } - } - deleteNotPresentAccounts(db); - setLastUpdateStamp(); - db.setTransactionSuccessful(); - } - finally { - db.endTransaction(); - } - - mergeAccountList(list); - updateDisplayedAccounts(); - } - public synchronized Locker lockAccountsForWriting() { - accountsLocker.lockForWriting(); - return accountsLocker; - } - public void setAndStoreTransactionList(ArrayList list) { - storeTransactionListAsync(this, list); - SQLiteDatabase db = App.getDatabase(); - db.beginTransactionNonExclusive(); - try { - markTransactionsAsNotPresent(db); - for (LedgerTransaction tr : list) - storeTransaction(db, tr); - deleteNotPresentTransactions(db); - setLastUpdateStamp(); - db.setTransactionSuccessful(); - } - finally { - db.endTransaction(); - } - - allTransactions.postValue(list); - } - private void storeTransactionListAsync(MobileLedgerProfile mobileLedgerProfile, - List list) { - if (transactionListSaver != null) - transactionListSaver.interrupt(); - - transactionListSaver = new TransactionListSaver(this, list); - transactionListSaver.start(); - } - public void setAndStoreAccountAndTransactionListFromWeb(List accounts, - List transactions) { - storeAccountAndTransactionListAsync(accounts, transactions, false); - - mergeAccountList(accounts); - updateDisplayedAccounts(); - - allTransactions.postValue(transactions); - } - private void storeAccountAndTransactionListAsync(List accounts, - List transactions, - boolean storeAccUiFields) { + public void storeAccountAndTransactionListAsync(List accounts, + List transactions) { if (accountAndTransactionListSaver != null) accountAndTransactionListSaver.interrupt(); accountAndTransactionListSaver = - new AccountAndTransactionListSaver(this, accounts, transactions, storeAccUiFields); + new AccountAndTransactionListSaver(this, accounts, transactions); accountAndTransactionListSaver.start(); } - synchronized public void updateDisplayedAccounts() { - if (displayedAccountsUpdater != null) { - displayedAccountsUpdater.interrupt(); - } - displayedAccountsUpdater = new AccountListDisplayedFilter(this, allAccounts); - displayedAccountsUpdater.start(); - } - public List getAllAccounts() { - return allAccounts; - } - private void updateAccountsMap(List newAccounts) { - accountMap.clear(); - for (LedgerAccount acc : newAccounts) { - accountMap.put(acc.getName(), acc); - } - } - @Nullable - public LedgerAccount locateAccount(String name) { - return accountMap.get(name); - } public enum FutureDates { None(0), OneWeek(7), TwoWeeks(14), OneMonth(30), TwoMonths(60), ThreeMonths(90), SixMonths(180), OneYear(365), All(-1); - private static SparseArray map = new SparseArray<>(); + private static final SparseArray map = new SparseArray<>(); static { for (FutureDates item : FutureDates.values()) { @@ -809,222 +608,93 @@ public final class MobileLedgerProfile { } } - static class AccountListLoader extends Thread { - MobileLedgerProfile profile; - AccountListLoader(MobileLedgerProfile profile) { - this.profile = profile; - } - @Override - public void run() { - Logger.debug("async-acc", "AccountListLoader::run() entered"); - String profileUUID = profile.getUuid(); - ArrayList list = new ArrayList<>(); - HashMap map = new HashMap<>(); - - String sql = "SELECT a.name, a.expanded, a.amounts_expanded"; - sql += " from accounts a WHERE a.profile = ?"; - sql += " ORDER BY a.name"; - - SQLiteDatabase db = App.getDatabase(); - Logger.debug("async-acc", "AccountListLoader::run() connected to DB"); - try (Cursor cursor = db.rawQuery(sql, new String[]{profileUUID})) { - Logger.debug("async-acc", "AccountListLoader::run() executed query"); - while (cursor.moveToNext()) { - if (isInterrupted()) - return; - - final String accName = cursor.getString(0); -// debug("accounts", -// String.format("Read account '%s' from DB [%s]", accName, -// profileUUID)); - String parentName = LedgerAccount.extractParentName(accName); - LedgerAccount parent; - if (parentName != null) { - parent = map.get(parentName); - if (parent == null) - throw new IllegalStateException( - String.format("Can't load account '%s': parent '%s' not loaded", - accName, parentName)); - parent.setHasSubAccounts(true); - } - else - parent = null; - - LedgerAccount acc = new LedgerAccount(profile, accName, parent); - acc.setExpanded(cursor.getInt(1) == 1); - acc.setAmountsExpanded(cursor.getInt(2) == 1); - acc.setHasSubAccounts(false); - - try (Cursor c2 = db.rawQuery( - "SELECT value, currency FROM account_values WHERE profile = ?" + " " + - "AND account = ?", new String[]{profileUUID, accName})) - { - while (c2.moveToNext()) { - acc.addAmount(c2.getFloat(0), c2.getString(1)); - } - } - - list.add(acc); - map.put(accName, acc); - } - Logger.debug("async-acc", "AccountListLoader::run() query execution done"); - } - - if (isInterrupted()) - return; - - Logger.debug("async-acc", "AccountListLoader::run() posting new list"); - profile.allAccounts = list; - profile.updateAccountsMap(list); - profile.updateDisplayedAccounts(); - } - } - - static class AccountListDisplayedFilter extends Thread { - private final MobileLedgerProfile profile; - private final List list; - AccountListDisplayedFilter(MobileLedgerProfile profile, List list) { - this.profile = profile; - this.list = list; - } - @Override - public void run() { - List newDisplayed = new ArrayList<>(); - Logger.debug("dFilter", "waiting for synchronized block"); - Logger.debug("dFilter", String.format(Locale.US, - "entered synchronized block (about to examine %d accounts)", list.size())); - for (LedgerAccount a : list) { - if (isInterrupted()) { - return; - } - - if (a.isVisible()) { - newDisplayed.add(a); - } - } - if (!isInterrupted()) { - profile.displayedAccounts.postValue(newDisplayed); - } - Logger.debug("dFilter", "left synchronized block"); - } - } - - private static class AccountListSaver extends Thread { - private final MobileLedgerProfile profile; - private final List list; - private final boolean storeUiFields; - AccountListSaver(MobileLedgerProfile profile, List list, - boolean storeUiFields) { - this.list = list; - this.profile = profile; - this.storeUiFields = storeUiFields; - } - @Override - public void run() { - SQLiteDatabase db = App.getDatabase(); - db.beginTransactionNonExclusive(); - try { - profile.markAccountsAsNotPresent(db); - if (isInterrupted()) - return; - for (LedgerAccount acc : list) { - profile.storeAccount(db, acc, storeUiFields); - if (isInterrupted()) - return; - } - profile.deleteNotPresentAccounts(db); - if (isInterrupted()) - return; - profile.setLastUpdateStamp(); - db.setTransactionSuccessful(); - } - finally { - db.endTransaction(); - } - } - } - - private static class TransactionListSaver extends Thread { - private final MobileLedgerProfile profile; - private final List list; - TransactionListSaver(MobileLedgerProfile profile, List list) { - this.list = list; - this.profile = profile; - } - @Override - public void run() { - SQLiteDatabase db = App.getDatabase(); - db.beginTransactionNonExclusive(); - try { - profile.markTransactionsAsNotPresent(db); - if (isInterrupted()) - return; - for (LedgerTransaction tr : list) { - profile.storeTransaction(db, tr); - if (isInterrupted()) - return; - } - profile.deleteNotPresentTransactions(db); - if (isInterrupted()) - return; - profile.setLastUpdateStamp(); - db.setTransactionSuccessful(); - } - finally { - db.endTransaction(); - } - } - } - private static class AccountAndTransactionListSaver extends Thread { private final MobileLedgerProfile profile; private final List accounts; private final List transactions; - private final boolean storeAccUiFields; AccountAndTransactionListSaver(MobileLedgerProfile profile, List accounts, - List transactions, - boolean storeAccUiFields) { + List transactions) { this.accounts = accounts; this.transactions = transactions; this.profile = profile; - this.storeAccUiFields = storeAccUiFields; + } + public int getNextDescriptionsGeneration(SQLiteDatabase db) { + int generation = 1; + try (Cursor c = db.rawQuery("SELECT generation FROM description_history LIMIT 1", + null)) + { + if (c.moveToFirst()) { + generation = c.getInt(0) + 1; + } + } + return generation; + } + void deleteNotPresentDescriptions(SQLiteDatabase db, int generation) { + Logger.debug("db/benchmark", "Deleting obsolete descriptions"); + db.execSQL("DELETE FROM description_history WHERE generation <> ?", + new Object[]{generation}); + db.execSQL("DELETE FROM description_history WHERE generation <> ?", + new Object[]{generation}); + Logger.debug("db/benchmark", "Done deleting obsolete descriptions"); } @Override public void run() { SQLiteDatabase db = App.getDatabase(); db.beginTransactionNonExclusive(); try { - profile.markAccountsAsNotPresent(db); + int accountsGeneration = profile.getNextAccountsGeneration(db); if (isInterrupted()) return; - profile.markTransactionsAsNotPresent(db); - if (isInterrupted()) { + int transactionsGeneration = profile.getNextTransactionsGeneration(db); + if (isInterrupted()) return; - } for (LedgerAccount acc : accounts) { - profile.storeAccount(db, acc, storeAccUiFields); + profile.storeAccount(db, accountsGeneration, acc, false); if (isInterrupted()) return; + for (LedgerAmount amt : acc.getAmounts()) { + profile.storeAccountValue(db, accountsGeneration, acc.getName(), + amt.getCurrency(), amt.getAmount()); + if (isInterrupted()) + return; + } } for (LedgerTransaction tr : transactions) { - profile.storeTransaction(db, tr); - if (isInterrupted()) { + profile.storeTransaction(db, transactionsGeneration, tr); + if (isInterrupted()) return; - } } - profile.deleteNotPresentAccounts(db); + profile.deleteNotPresentTransactions(db, transactionsGeneration); if (isInterrupted()) { return; } - profile.deleteNotPresentTransactions(db); + profile.deleteNotPresentAccounts(db, accountsGeneration); if (isInterrupted()) return; - profile.setLastUpdateStamp(); + Map unique = new HashMap<>(); + + debug("descriptions", "Starting refresh"); + int descriptionsGeneration = getNextDescriptionsGeneration(db); + try (Cursor c = db.rawQuery("SELECT distinct description from transactions", + null)) + { + while (c.moveToNext()) { + String description = c.getString(0); + String descriptionUpper = description.toUpperCase(); + if (unique.containsKey(descriptionUpper)) + continue; + + storeDescription(db, descriptionsGeneration, description, descriptionUpper); + + unique.put(descriptionUpper, true); + } + } + deleteNotPresentDescriptions(db, descriptionsGeneration); db.setTransactionSuccessful(); } @@ -1032,5 +702,16 @@ public final class MobileLedgerProfile { db.endTransaction(); } } + private void storeDescription(SQLiteDatabase db, int generation, String description, + String descriptionUpper) { + db.execSQL("UPDATE description_history SET description=?, generation=? WHERE " + + "description_upper=?", new Object[]{description, generation, descriptionUpper + }); + db.execSQL( + "INSERT INTO description_history(description, description_upper, generation) " + + "select ?,?,? WHERE (select changes() = 0)", + new Object[]{description, descriptionUpper, generation + }); + } } }