import android.view.LayoutInflater;
import android.view.ViewGroup;
import android.widget.LinearLayout;
-import android.widget.TableRow;
import androidx.annotation.NonNull;
import androidx.recyclerview.widget.RecyclerView;
String.format(Locale.US, "invoking notifyItemInserted(%d)", newAccountCount));
// the header is at position 0
notifyItemInserted(newAccountCount);
+ model.sendCountNotifications(); // needed after holders' positions have changed
return newAccountCount;
}
@NonNull
@Override
public void onBindViewHolder(@NonNull NewTransactionItemHolder holder, int position) {
Logger.debug("bind", String.format(Locale.US, "Binding item at position %d", position));
- holder.setData(model.getItem(position));
- Logger.debug("bind", String.format(Locale.US, "Bound item at position %d", position));
+ NewTransactionModel.Item item = model.getItem(position);
+ holder.setData(item);
+ Logger.debug("bind", String.format(Locale.US, "Bound %s item at position %d", item.getType()
+ .toString(),
+ position));
}
@Override
public int getItemCount() {
- final int itemCount = model.getAccountCount() + 2;
- Logger.debug("new-transaction",
- String.format(Locale.US, "getItemCount() returning %d", itemCount));
- return itemCount;
+ return model.getAccountCount() + 2;
}
boolean accountListIsEmpty() {
for (int i = 0; i < model.getAccountCount(); i++) {
LedgerTransactionAccount acc = model.getAccount(i);
if (!acc.getAccountName()
- .isEmpty()) return false;
- if (acc.isAmountSet()) return false;
+ .isEmpty())
+ return false;
+ if (acc.isAmountSet())
+ return false;
}
return true;
}
public void descriptionSelected(String description) {
debug("descr selected", description);
- if (!accountListIsEmpty()) return;
+ if (!accountListIsEmpty())
+ return;
String accFilter = mProfile.getPreferredAccountsFilter();
try (Cursor c = App.getDatabase()
.rawQuery(sql, params.toArray(new String[]{})))
{
- if (!c.moveToNext()) return;
+ if (!c.moveToNext())
+ return;
String profileUUID = c.getString(0);
int transactionId = c.getInt(1);
LedgerTransaction tr;
MobileLedgerProfile profile = Data.getProfile(profileUUID);
- if (profile == null) throw new RuntimeException(String.format(
- "Unable to find profile %s, which is supposed to contain " +
- "transaction %d with description %s", profileUUID, transactionId, description));
+ if (profile == null)
+ throw new RuntimeException(String.format(
+ "Unable to find profile %s, which is supposed to contain " +
+ "transaction %d with description %s", profileUUID, transactionId,
+ description));
tr = profile.loadTransaction(transactionId);
ArrayList<LedgerTransactionAccount> accounts = tr.getAccounts();
- TableRow firstNegative = null;
+ NewTransactionModel.Item firstNegative = null;
+ boolean singleNegative = false;
int negativeCount = 0;
for (int i = 0; i < accounts.size(); i++) {
LedgerTransactionAccount acc = accounts.get(i);
NewTransactionModel.Item item;
- if (model.getAccountCount() < i) {
+ if (model.getAccountCount() < i + 1) {
model.addAccount(acc);
notifyItemInserted(i + 1);
}
item.getAccount()
.setAccountName(acc.getAccountName());
- if (acc.isAmountSet()) item.getAccount()
- .setAmount(acc.getAmount());
- else item.getAccount()
- .resetAmount();
+ if (acc.isAmountSet()) {
+ item.getAccount()
+ .setAmount(acc.getAmount());
+ if (acc.getAmount() < 0) {
+ if (firstNegative == null) {
+ firstNegative = item;
+ singleNegative = true;
+ }
+ else
+ singleNegative = false;
+ }
+ }
+ else
+ item.getAccount()
+ .resetAmount();
notifyItemChanged(i + 1);
}
+
+ if (singleNegative) {
+ firstNegative.getAccount()
+ .resetAmount();
+ }
}
model.checkTransactionSubmittable(this);
model.setFocusedItem(1);
}
public void toggleAllEditing(boolean editable) {
- for (int i = 0; i < model.getAccountCount(); i++) {
- model.getItem(i + 1)
+ // item 0 is the header
+ for (int i = 0; i <= model.getAccountCount(); i++) {
+ model.getItem(i)
.setEditable(editable);
- notifyItemChanged(i + 1);
- // TODO perhaps do only one notification about the whole range [1…count]?
+ notifyItemChanged(i);
+ // TODO perhaps do only one notification about the whole range (notifyDatasetChanged)?
}
}
public void reset() {
if (presentItemCount > 2)
notifyItemRangeRemoved(3, presentItemCount - 2); // all the rest are gone
}
- public void removeItem(int pos) {
- model.removeItem(pos - 1, this);
- notifyItemRemoved(pos);
- }
}