Skip to content

Commit f606a51

Browse files
codingusercodinguser
committed
Merge branch 'release/v1.2.2'
2 parents 53937c8 + c1ff793 commit f606a51

12 files changed

Lines changed: 170 additions & 132 deletions

File tree

‎CHANGELOG.md‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,12 @@
11
Change Log
22
===============================================================================
3+
Version 1.2.2 *(2013-06-23)*
4+
----------------------------
5+
* Fixed: bug with importing accounts
6+
* Fixed: deleting an account renders sub-accounts inaccessible.
7+
* Fixed: impossible to scroll new accounts screen
8+
* Updated Brazilian Portuguese translations
9+
310
Version 1.2.1 *(2013-06-22)*
411
----------------------------
512
* Fixed: crash when opening Settings on devices with Gingerbread or earlier

‎app/AndroidManifest.xml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,8 @@
1717

1818
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
1919
package="org.gnucash.android"
20-
android:versionCode="9"
21-
android:versionName="1.2.1" >
20+
android:versionCode="10"
21+
android:versionName="1.2.2" >
2222

2323
<uses-sdk android:minSdkVersion="8" android:targetSdkVersion="15"/>
2424

‎app/pom.xml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
<description>Gnucash Android companion application</description>
2323

2424
<parent>
25-
<version>1.2.1</version>
25+
<version>1.2.2</version>
2626
<groupId>org.gnucash.android</groupId>
2727
<artifactId>gnucash-android-parent</artifactId>
2828
</parent>

‎app/res/layout/fragment_new_account.xml‎

Lines changed: 84 additions & 80 deletions
Original file line numberDiff line numberDiff line change
@@ -15,85 +15,89 @@
1515
limitations under the License.
1616
-->
1717

18-
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
19-
android:layout_width="match_parent"
20-
android:layout_height="match_parent"
21-
android:orientation="vertical" >
22-
23-
<EditText
24-
android:id="@+id/edit_text_account_name"
25-
android:layout_width="match_parent"
26-
android:layout_height="wrap_content"
27-
android:layout_marginTop="10dp"
28-
android:layout_marginLeft="10dp"
29-
android:layout_marginRight="10dp"
30-
android:layout_marginBottom="20dp"
31-
android:ems="15"
32-
android:inputType="textCapWords"
33-
android:minHeight="?android:attr/listPreferredItemHeight"
34-
android:singleLine="true"
35-
android:ellipsize="start"
36-
android:hint="@string/label_account_name"
37-
style="@style/ListItemText">
38-
<requestFocus />
39-
</EditText>
40-
41-
<TextView
42-
android:text="@string/label_account_currency"
43-
android:layout_height="wrap_content"
44-
android:layout_width="wrap_content"
45-
android:layout_marginBottom="-8dip"
46-
android:layout_marginLeft="10dp"
47-
style="@style/TextAppearance.EditTransaction_LabelSmall" />
48-
49-
<Spinner android:id="@+id/input_currency_spinner"
50-
android:layout_width="match_parent"
51-
android:layout_height="wrap_content"
52-
android:layout_marginLeft="10dp"
53-
android:layout_marginRight="10dp"
54-
android:layout_marginBottom="20dp"
55-
style="@style/ListItem" />
18+
<ScrollView xmlns:android="http://schemas.android.com/apk/res/android"
19+
android:layout_width="match_parent"
20+
android:layout_height="match_parent">
21+
<LinearLayout
22+
android:layout_width="match_parent"
23+
android:layout_height="match_parent"
24+
android:orientation="vertical" >
5625

57-
<TextView
58-
android:text="@string/label_account_type"
59-
android:layout_height="wrap_content"
60-
android:layout_width="wrap_content"
61-
android:layout_marginBottom="-8dip"
62-
android:layout_marginLeft="10dp"
63-
style="@style/TextAppearance.EditTransaction_LabelSmall" />
26+
<EditText
27+
android:id="@+id/edit_text_account_name"
28+
android:layout_width="match_parent"
29+
android:layout_height="wrap_content"
30+
android:layout_marginTop="10dp"
31+
android:layout_marginLeft="10dp"
32+
android:layout_marginRight="10dp"
33+
android:layout_marginBottom="20dp"
34+
android:ems="15"
35+
android:inputType="textCapWords"
36+
android:minHeight="?android:attr/listPreferredItemHeight"
37+
android:singleLine="true"
38+
android:ellipsize="start"
39+
android:hint="@string/label_account_name"
40+
style="@style/ListItemText">
41+
<requestFocus />
42+
</EditText>
6443

65-
<Spinner android:id="@+id/input_account_type_spinner"
66-
android:layout_width="match_parent"
67-
android:layout_height="wrap_content"
68-
android:layout_marginLeft="10dp"
69-
android:layout_marginRight="10dp"
70-
android:layout_marginBottom="20dp"
71-
style="@style/ListItem" />
72-
73-
<TextView android:id="@+id/label_parent_account"
74-
android:text="@string/label_parent_account"
75-
android:layout_height="wrap_content"
76-
android:layout_width="wrap_content"
77-
android:layout_marginLeft="10dp"
78-
android:layout_marginBottom="-8dip"
79-
style="@style/TextAppearance.EditTransaction_LabelSmall" />
80-
81-
<LinearLayout android:id="@+id/layout_parent_account"
82-
android:layout_width="match_parent"
83-
android:layout_height="wrap_content"
84-
style="@style/ListItem"
85-
android:orientation="horizontal" >
86-
87-
<CheckBox android:id="@+id/checkbox"
88-
android:layout_width="wrap_content"
89-
android:layout_height="wrap_content"
90-
android:focusable="false" />
91-
<Spinner
92-
android:id="@+id/input_parent_account"
93-
android:layout_width="match_parent"
94-
android:layout_height="wrap_content"
95-
android:layout_marginLeft="10dp"
96-
android:layout_marginRight="10dp"
97-
android:minHeight="?android:attr/listPreferredItemHeight" />
98-
</LinearLayout>
99-
</LinearLayout>
44+
<TextView
45+
android:text="@string/label_account_currency"
46+
android:layout_height="wrap_content"
47+
android:layout_width="wrap_content"
48+
android:layout_marginBottom="-8dip"
49+
android:layout_marginLeft="10dp"
50+
style="@style/TextAppearance.EditTransaction_LabelSmall" />
51+
52+
<Spinner android:id="@+id/input_currency_spinner"
53+
android:layout_width="match_parent"
54+
android:layout_height="wrap_content"
55+
android:layout_marginLeft="10dp"
56+
android:layout_marginRight="10dp"
57+
android:layout_marginBottom="20dp"
58+
style="@style/ListItem" />
59+
60+
<TextView
61+
android:text="@string/label_account_type"
62+
android:layout_height="wrap_content"
63+
android:layout_width="wrap_content"
64+
android:layout_marginBottom="-8dip"
65+
android:layout_marginLeft="10dp"
66+
style="@style/TextAppearance.EditTransaction_LabelSmall" />
67+
68+
<Spinner android:id="@+id/input_account_type_spinner"
69+
android:layout_width="match_parent"
70+
android:layout_height="wrap_content"
71+
android:layout_marginLeft="10dp"
72+
android:layout_marginRight="10dp"
73+
android:layout_marginBottom="20dp"
74+
style="@style/ListItem" />
75+
76+
<TextView android:id="@+id/label_parent_account"
77+
android:text="@string/label_parent_account"
78+
android:layout_height="wrap_content"
79+
android:layout_width="wrap_content"
80+
android:layout_marginLeft="10dp"
81+
android:layout_marginBottom="-8dip"
82+
style="@style/TextAppearance.EditTransaction_LabelSmall" />
83+
84+
<LinearLayout android:id="@+id/layout_parent_account"
85+
android:layout_width="match_parent"
86+
android:layout_height="wrap_content"
87+
style="@style/ListItem"
88+
android:orientation="horizontal" >
89+
90+
<CheckBox android:id="@+id/checkbox"
91+
android:layout_width="wrap_content"
92+
android:layout_height="wrap_content"
93+
android:focusable="false" />
94+
<Spinner
95+
android:id="@+id/input_parent_account"
96+
android:layout_width="match_parent"
97+
android:layout_height="wrap_content"
98+
android:layout_marginLeft="10dp"
99+
android:layout_marginRight="10dp"
100+
android:minHeight="?android:attr/listPreferredItemHeight" />
101+
</LinearLayout>
102+
</LinearLayout>
103+
</ScrollView>

‎app/res/values-pt-rBR/strings.xml‎

Lines changed: 39 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -308,53 +308,52 @@
308308
<string name="title_whats_new">O que há de novo</string>
309309
<string name="whats_new">
310310
<b>Version 1.2.0 - 20.06.2013</b>\n\n
311-
- Import GnuCash desktop account structure\n
312-
- Nested display of account hierarchy\n
313-
- Options for deleting all accounts/transactions\n
314-
- Preliminary support for account types\n
315-
- Account balance now takes sub-accounts into consideration\n
316-
- Numerous bug fixes\n
311+
- Importação da estrutura de contas do GnuCash para desktop\n
312+
- Apresentação de contas alinhadas hierarquicamente\n
313+
- Opções para deletar todas as contas/transações\n
314+
- Suporte preliminar para tipos de contas\n
315+
- Saldos das sub-contas agora são considerados ao apresentar o saldo da conta principal\n
316+
- Diversas correções de bugs\n
317317
</string>
318318
<string name="label_dismiss">Descartar</string>
319-
<string name="toast_transanction_amount_required">Enter an amount to save the transaction</string>
320-
<string name="menu_import_accounts">Import GnuCash Accounts</string>
321-
<string name="btn_import_accounts">Import Accounts</string>
322-
<string name="toast_error_importing_accounts">An error occurred while importing the GnuCash accounts</string>
323-
<string name="toast_success_importing_accounts">GnuCash accounts successfully imported</string>
324-
<string name="summary_import_accounts">Import account structure exported from GnuCash desktop</string>
325-
<string name="title_import_accounts">Import GnuCash accounts</string>
326-
<string name="summary_delete_all_accounts">Delete all accounts in the database. All transactions will be deleted as
327-
well.
319+
<string name="toast_transanction_amount_required">Adicione um valor para salvar a transação</string>
320+
<string name="menu_import_accounts">Importar Contas do GnuCash</string>
321+
<string name="btn_import_accounts">Importar Contas</string>
322+
<string name="toast_error_importing_accounts">Um erro ocorreu ao importar as contas do GnuCash</string>
323+
<string name="toast_success_importing_accounts">Contas GnuCash importadas com sucesso</string>
324+
<string name="summary_import_accounts">Importar estrutura de contas exportadas do GnuCash para desktop</string>
325+
<string name="title_import_accounts">Importar contas do GnuCash</string>
326+
<string name="summary_delete_all_accounts">Deletar todas as contas. Todas as transações também serão deletadas.
328327
</string>
329-
<string name="title_delete_all_accounts">Delete all accounts</string>
330-
<string name="header_account_settings">Accounts</string>
331-
<string name="toast_all_accounts_deleted">All accounts have been successfully deleted</string>
332-
<string name="confirm_delete_all_accounts">Are you sure you want to delete all accounts and transactions? \nThis
333-
operation cannot be undone!
328+
<string name="title_delete_all_accounts">Deletar todas as contas</string>
329+
<string name="header_account_settings">Contas</string>
330+
<string name="toast_all_accounts_deleted">Todas as contas foram deletadas com sucesso</string>
331+
<string name="confirm_delete_all_accounts">Tem certeza que deseja deletar todas as contas e transações? \nEsta
332+
operação não pode ser desfeita!
334333
</string>
335-
<string name="label_account_type">Account Type</string>
336-
<string name="summary_delete_all_transactions">All transactions in all accounts will be deleted!</string>
337-
<string name="title_delete_all_transactions">Delete all transactions</string>
338-
<string name="toast_all_transactions_deleted">All transactions successfully deleted!</string>
339-
<string name="title_progress_importing_accounts">Importing accounts</string>
340-
<string name="toast_tap_again_to_confirm_delete">Tap again to confirm. ALL entries will be deleted!!</string>
334+
<string name="label_account_type">Tipo da conta</string>
335+
<string name="summary_delete_all_transactions">Todas as transações em todas as contas serão deletadas!</string>
336+
<string name="title_delete_all_transactions">Deletar todas as transações</string>
337+
<string name="toast_all_transactions_deleted">Todas as transações foram deletadas com sucesso!</string>
338+
<string name="title_progress_importing_accounts">Importando contas</string>
339+
<string name="toast_tap_again_to_confirm_delete">Toque novamente para confirmar. TODAS as entradas serão deletadas!!!</string>
341340
<plurals name="label_sub_accounts">
342-
<item quantity="one">%d sub-account</item>
343-
<item quantity="other">%d sub-accounts</item>
341+
<item quantity="one">%d sub-contas</item>
342+
<item quantity="other">%d sub-contas</item>
344343
</plurals>
345344
<string-array name="account_type_entry_values">
346-
<item>CASH</item>
347-
<item>BANK</item>
348-
<item>CREDIT CARD</item>
349-
<item>ASSET</item>
350-
<item>LIABILITY</item>
351-
<item>INCOME</item>
352-
<item>EXPENSE</item>
353-
<item>PAYABLE</item>
354-
<item>RECEIVABLE</item>
355-
<item>EQUITY</item>
345+
<item>DINHEIRO</item>
346+
<item>BANCO</item>
347+
<item>CARTÃO DE CRÉDITO</item>
348+
<item>ATIVO</item>
349+
<item>PASSIVO</item>
350+
<item>RECEITA</item>
351+
<item>DESPESA</item>
352+
<item>CONTAS A PAGAR</item>
353+
<item>CONTAS A RECEBER</item>
354+
<item>PATRIMÔNIO LÍQUIDO</item>
356355
<item>CURRENCY</item>
357-
<item>STOCK</item>
358-
<item>MUTUAL FUND</item>
356+
<item>AÇÕES</item>
357+
<item>FUNDO MÚTUO</item>
359358
</string-array>
360359
</resources>

‎app/res/values/strings.xml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -567,6 +567,6 @@
567567
<item>EQUITY</item>
568568
<item>CURRENCY</item>
569569
<item>STOCK</item>
570-
<item>MUTUAL_FUND</item>
570+
<item>MUTUAL</item>
571571
</string-array>
572572
</resources>

‎app/src/org/gnucash/android/data/Account.java‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ public class Account {
5252
* they are currently not used except for exporting
5353
*/
5454
public enum AccountType {CASH, BANK, CREDIT, ASSET, LIABILITY, INCOME, EXPENSE,
55-
PAYABLE, RECEIVABLE, EQUITY, CURRENCY, STOCK, MUTUAL_FUND, ROOT};
55+
PAYABLE, RECEIVABLE, EQUITY, CURRENCY, STOCK, MUTUAL, ROOT};
5656

5757
/**
5858
* Accounts types which are used by the OFX standard
@@ -340,7 +340,7 @@ public static OfxAccountType convertToOfxAccountType(AccountType accountType){
340340
case ASSET:
341341
return OfxAccountType.SAVINGS;
342342

343-
case MUTUAL_FUND:
343+
case MUTUAL:
344344
case STOCK:
345345
case EQUITY:
346346
case CURRENCY:

‎app/src/org/gnucash/android/db/AccountsDbAdapter.java‎

Lines changed: 29 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ public long addAccount(Account account){
9090
}
9191
return rowId;
9292
}
93-
93+
9494
/**
9595
* Deletes an account with database id <code>rowId</code>
9696
* All the transactions in the account will also be deleted
@@ -112,7 +112,26 @@ public boolean destructiveDeleteAccount(long rowId){
112112
result &= deleteRecord(DatabaseHelper.ACCOUNTS_TABLE_NAME, rowId);
113113
return result;
114114
}
115-
115+
116+
/**
117+
* Reassigns all accounts with parent UID <code>oldParentUID</code> to <code>newParentUID</code>
118+
* @param oldParentUID Old parent account Unique ID
119+
* @param newParentUID Unique ID of new parent account
120+
* @return Number of records which are modified
121+
*/
122+
public int reassignParent(String oldParentUID, String newParentUID){
123+
ContentValues contentValues = new ContentValues();
124+
if (newParentUID == null)
125+
contentValues.putNull(DatabaseHelper.KEY_PARENT_ACCOUNT_UID);
126+
else
127+
contentValues.put(DatabaseHelper.KEY_PARENT_ACCOUNT_UID, newParentUID);
128+
129+
return mDb.update(DatabaseHelper.ACCOUNTS_TABLE_NAME,
130+
contentValues,
131+
DatabaseHelper.KEY_PARENT_ACCOUNT_UID + "= '" + oldParentUID + "' ",
132+
null);
133+
}
134+
116135
/**
117136
* Deletes an account while preserving the linked transactions
118137
* Reassigns all transactions belonging to the account with id <code>rowId</code> to
@@ -329,9 +348,16 @@ public Cursor fetchRecord(long rowId) {
329348
return fetchRecord(DatabaseHelper.ACCOUNTS_TABLE_NAME, rowId);
330349
}
331350

351+
/**
352+
* Deletes an account and its transactions from the database.
353+
* This is equivalent to calling {@link #destructiveDeleteAccount(long)}
354+
* @param rowId ID of record to be deleted
355+
* @return <code>true</code> if successful, <code>false</code> otherwise
356+
*/
332357
@Override
333358
public boolean deleteRecord(long rowId) {
334-
return deleteRecord(DatabaseHelper.ACCOUNTS_TABLE_NAME, rowId);
359+
return destructiveDeleteAccount(rowId);
360+
//return deleteRecord(DatabaseHelper.ACCOUNTS_TABLE_NAME, rowId);
335361
}
336362

337363
/**

‎app/src/org/gnucash/android/ui/accounts/AccountsListFragment.java‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -276,8 +276,10 @@ public void tryDeleteAccount(long rowId) {
276276
* @param rowId Record ID of the account to be deleted
277277
*/
278278
protected void deleteAccount(long rowId) {
279+
String accountUID = mAccountsDbAdapter.getAccountUID(rowId);
279280
boolean deleted = mAccountsDbAdapter.destructiveDeleteAccount(rowId);
280281
if (deleted) {
282+
mAccountsDbAdapter.reassignParent(accountUID, null);
281283
Toast.makeText(getActivity(), R.string.toast_account_deleted, Toast.LENGTH_SHORT).show();
282284
WidgetConfigurationActivity.updateAllWidgets(getActivity().getApplicationContext());
283285
}

‎app/src/org/gnucash/android/util/GnucashAccountXmlHandler.java‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -136,8 +136,8 @@ public static void parse(Context context, InputStream accountsInputStream){
136136
xr.parse(new InputSource(bos));
137137

138138
} catch (Exception e) {
139-
Toast.makeText(context, R.string.toast_error_importing_accounts, Toast.LENGTH_LONG).show();
140139
e.printStackTrace();
140+
Toast.makeText(context, R.string.toast_error_importing_accounts, Toast.LENGTH_LONG).show();
141141
}
142142
}
143143
}

0 commit comments

Comments
 (0)