Merge branch 'hotfix/patches'

Author: codinguser

CHANGELOG.md

@@ -1,5 +1,16 @@
 Change Log
 ===============================================================================
+Version 1.6.1 *(2015-07-08)*
+----------------------------
+* Fixed: Crash when importing some scheduled transations with custom period strings
+* Fixed: Crash when closing export progress dialog if an export error occurred
+* Fixed: Crash when creating a sub-account and changing the account type 
+* Fixed: Crash when loading backup files with no timestamp in their name
+* Fixed: Crash when app is run on devices with locale es_LG
+* Improved: Updated betterpickers library
+* Improved: New dialogs for time and date when creating transactions
+* Improved: Added translation to Ukrainian
+
 Version 1.6.0 *(2015-06-20)*
 ----------------------------
 * Feature: Scheduled backups (QIF, OFX and XML)

app/build.gradle

@@ -5,8 +5,8 @@ apply plugin: 'crashlytics'
 
 def versionMajor = 1
 def versionMinor = 6
-def versionPatch = 0
-def versionBuild = 7
+def versionPatch = 1
+def versionBuild = 0
 
 def buildTime() {
     def df = new SimpleDateFormat("yyyyMMdd")
@@ -100,13 +100,13 @@ android {
             applicationId 'org.gnucash.android.devel'
             testApplicationId 'org.gnucash.android.devel.test'
             resValue "string", "app_name", "GnuCash-devel"
-            versionName "${versionMajor}.${versionMinor}.${versionPatch}-dev${versionBuild}_${buildTime()}"
+            versionName "${versionMajor}.${versionMinor}.${versionPatch}-dev${versionBuild}_r${gitSha()}"
             resValue "string", "app_version_name", "${versionName}"
         }
 
         beta {
             resValue "string", "app_name", "GnuCash - beta"
-            versionName "${versionMajor}.${versionMinor}.${versionPatch}-beta${versionBuild}_r${gitSha()}"
+            versionName "${versionMajor}.${versionMinor}.${versionPatch}-beta${versionBuild}"
             resValue "string", "app_version_name", "${versionName}"
             buildConfigField "boolean", "USE_CRASHLYTICS", "true"
         }
@@ -154,7 +154,7 @@ dependencies {
     compile('com.android.support:support-v4:22.1.1',
             'com.actionbarsherlock:actionbarsherlock:4.4.0@aar',
             'com.viewpagerindicator:library:2.4.1@aar',
-            'com.doomonafireball.betterpickers:library:1.5.2',
+            'com.doomonafireball.betterpickers:library:1.6.0',
             'com.commonsware.cwac:merge:1.1.+',
             'com.github.PhilJay:MPAndroidChart:v2.1.0',
             'joda-time:joda-time:2.7',

app/src/androidTest/java/org/gnucash/android/test/ui/AccountsActivityTest.java

@@ -35,6 +35,7 @@
 import org.gnucash.android.db.SplitsDbAdapter;
 import org.gnucash.android.db.TransactionsDbAdapter;
 import org.gnucash.android.model.Account;
+import org.gnucash.android.model.AccountType;
 import org.gnucash.android.model.Money;
 import org.gnucash.android.receivers.AccountCreator;
 import org.gnucash.android.ui.account.AccountsActivity;
@@ -47,19 +48,24 @@
 import java.util.Currency;
 import java.util.List;
 
+import static android.support.test.espresso.Espresso.onData;
 import static android.support.test.espresso.Espresso.onView;
 import static android.support.test.espresso.action.ViewActions.clearText;
 import static android.support.test.espresso.action.ViewActions.click;
 import static android.support.test.espresso.action.ViewActions.longClick;
 import static android.support.test.espresso.action.ViewActions.scrollTo;
+import static android.support.test.espresso.action.ViewActions.swipeRight;
 import static android.support.test.espresso.action.ViewActions.typeText;
 import static android.support.test.espresso.assertion.ViewAssertions.matches;
+import static android.support.test.espresso.matcher.ViewMatchers.isChecked;
 import static android.support.test.espresso.matcher.ViewMatchers.isDisplayed;
 import static android.support.test.espresso.matcher.ViewMatchers.isNotChecked;
 import static android.support.test.espresso.matcher.ViewMatchers.withId;
 import static android.support.test.espresso.matcher.ViewMatchers.withText;
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.hamcrest.Matchers.allOf;
+import static org.hamcrest.Matchers.instanceOf;
+import static org.hamcrest.Matchers.is;
 import static org.hamcrest.Matchers.not;
 
 @RunWith(AndroidJUnit4.class)
@@ -213,6 +219,31 @@ public void testChangeParentAccount() {
         assertThat(DUMMY_ACCOUNT_UID).isEqualTo(parentUID);
     }
 
+    /**
+     * When creating a sub-account (starting from within another account), if we change the account
+     * type to another type with no accounts of that type, then the parent account list should be hidden.
+     * The account which is then created is not a sub-account, but rather a top-level account
+     */
+    @Test
+    public void shouldHideParentAccountViewWhenNoParentsExist(){
+        onView(withText(DUMMY_ACCOUNT_NAME)).perform(click());
+        onView(withId(R.id.fragment_transaction_list)).perform(swipeRight());
+        onView(withText(R.string.label_create_account)).check(matches(isDisplayed())).perform(click());
+        sleep(1000);
+        onView(withId(R.id.checkbox_parent_account)).check(matches(allOf(isChecked())));
+        onView(withId(R.id.input_account_name)).perform(typeText("Trading account"));
+        onView(withId(R.id.input_account_type_spinner)).perform(click());
+        onData(allOf(is(instanceOf(String.class)), is(AccountType.TRADING.name()))).perform(click());
+
+        onView(withId(R.id.layout_parent_account)).check(matches(not(isDisplayed())));
+        onView(withId(R.id.menu_save)).perform(click());
+
+        //no sub-accounts
+        assertThat(mAccountsDbAdapter.getSubAccountCount(DUMMY_ACCOUNT_UID)).isEqualTo(0);
+        assertThat(mAccountsDbAdapter.getSubAccountCount(mAccountsDbAdapter.getOrCreateGnuCashRootAccountUID())).isEqualTo(2);
+        assertThat(mAccountsDbAdapter.getSimpleAccountList()).extracting("mAccountType").contains(AccountType.TRADING);
+    }
+
     @Test
 	public void testEditAccount(){
 		String editedAccountName = "Edited Account";

app/src/androidTest/java/org/gnucash/android/test/ui/TransactionsActivityTest.java

@@ -16,7 +16,6 @@
 
 package org.gnucash.android.test.ui;
 
-import android.app.Fragment;
 import android.content.ContentValues;
 import android.content.Intent;
 import android.content.SharedPreferences;
@@ -28,8 +27,6 @@
 import android.support.test.runner.AndroidJUnit4;
 import android.test.ActivityInstrumentationTestCase2;
 import android.util.Log;
-import android.widget.LinearLayout;
-import android.widget.Spinner;
 
 import org.gnucash.android.R;
 import org.gnucash.android.db.AccountsDbAdapter;
@@ -58,7 +55,6 @@
 import java.util.List;
 import java.util.Locale;
 
-import static android.support.test.espresso.Espresso.onData;
 import static android.support.test.espresso.Espresso.onView;
 import static android.support.test.espresso.action.ViewActions.clearText;
 import static android.support.test.espresso.action.ViewActions.click;
@@ -69,15 +65,10 @@
 import static android.support.test.espresso.matcher.ViewMatchers.hasDescendant;
 import static android.support.test.espresso.matcher.ViewMatchers.isChecked;
 import static android.support.test.espresso.matcher.ViewMatchers.isDisplayed;
-import static android.support.test.espresso.matcher.ViewMatchers.isNotChecked;
-import static android.support.test.espresso.matcher.ViewMatchers.withChild;
 import static android.support.test.espresso.matcher.ViewMatchers.withId;
-import static android.support.test.espresso.matcher.ViewMatchers.withSpinnerText;
 import static android.support.test.espresso.matcher.ViewMatchers.withText;
-import static org.assertj.android.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.hamcrest.Matchers.allOf;
-import static org.hamcrest.Matchers.instanceOf;
 import static org.hamcrest.Matchers.is;
 import static org.hamcrest.Matchers.not;
 
@@ -191,14 +182,26 @@ public void testAddTransactionShouldRequireAmount(){
 				.perform(typeText("Lunch"));
 
 		onView(withId(R.id.menu_save)).perform(click());
-
+		sleep(500);
 		assertToastDisplayed(R.string.toast_transanction_amount_required);
 
 		int afterCount = mTransactionsDbAdapter.getTransactionsCount(DUMMY_ACCOUNT_UID);
 		assertThat(afterCount).isEqualTo(beforeCount);
 
 	}
 
+	/**
+	 * Sleep the thread for a specified period
+	 * @param millis Duration to sleep in milliseconds
+	 */
+	private void sleep(long millis) {
+		try {
+			Thread.sleep(millis);
+		} catch (InterruptedException e) {
+			e.printStackTrace();
+		}
+	}
+
 	/**
 	 * Checks that a specific toast message is displayed
 	 * @param toastString
@@ -219,7 +222,8 @@ private void validateEditTransactionFields(Transaction transaction){
 		formatter.setMinimumFractionDigits(2);
 		formatter.setMaximumFractionDigits(2);
 		onView(withId(R.id.input_transaction_amount)).check(matches(withText(formatter.format(balance.asDouble()))));
-
+		onView(withId(R.id.input_date)).check(matches(withText(TransactionFormFragment.DATE_FORMATTER.format(transaction.getTimeMillis()))));
+		onView(withId(R.id.input_time)).check(matches(withText(TransactionFormFragment.TIME_FORMATTER.format(transaction.getTimeMillis()))));
 		onView(withId(R.id.input_description)).check(matches(withText(transaction.getNote())));
 
 		validateTimeInput(transaction.getTimeMillis());

app/src/main/java/org/gnucash/android/app/GnuCashApplication.java

@@ -158,13 +158,18 @@ public static boolean shouldSaveOpeningBalances(boolean defaultValue){
      *
      * @return Default currency code string for the application
      */
-    public static String getDefaultCurrency(){
+    public static String getDefaultCurrencyCode(){
         Locale locale = Locale.getDefault();
         //sometimes the locale en_UK is returned which causes a crash with Currency
         if (locale.getCountry().equals("UK")) {
             locale = new Locale(locale.getLanguage(), "GB");
         }
 
+        //for unsupported locale es_LG
+        if (locale.getCountry().equals("LG")){
+            locale = new Locale(locale.getLanguage(), "ES");
+        }
+
         String currencyCode = "USD"; //start with USD as the default
         SharedPreferences prefs = PreferenceManager.getDefaultSharedPreferences(context);
         try { //there are some strange locales out there

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

@@ -813,7 +813,7 @@ private Money computeBalance(String accountUID, long startTimestamp, long endTim
      * @return the absolute balance of account list
      */
     public Money getAccountsBalance(List<String> accountUIDList, long startTimestamp, long endTimestamp) {
-        String currencyCode = GnuCashApplication.getDefaultCurrency();
+        String currencyCode = GnuCashApplication.getDefaultCurrencyCode();
         Money balance = Money.createZeroInstance(currencyCode);
 
         SplitsDbAdapter splitsDbAdapter = SplitsDbAdapter.getInstance();

app/src/main/java/org/gnucash/android/db/DatabaseAdapter.java

@@ -286,6 +286,7 @@ public long getID(@NonNull String uid){
      * Returns the string unique ID (GUID) of a record in the database
      * @param id long database record ID
      * @return GUID of the record
+     * @throws IllegalArgumentException if the record ID does not exist in the database
      */
     public String getUID(long id){
         Cursor cursor = mDb.query(mTableName,

app/src/main/java/org/gnucash/android/export/ExportAsyncTask.java

@@ -55,6 +55,7 @@
 import org.gnucash.android.export.xml.GncXmlExporter;
 import org.gnucash.android.model.Transaction;
 import org.gnucash.android.ui.account.AccountsActivity;
+import org.gnucash.android.ui.account.AccountsListFragment;
 import org.gnucash.android.ui.settings.SettingsActivity;
 import org.gnucash.android.ui.transaction.TransactionsActivity;
 
@@ -233,7 +234,9 @@ protected void onPostExecute(Boolean exportResult) {
 
             //now refresh the respective views
             if (mContext instanceof AccountsActivity){
-                ((AccountsActivity) mContext).getCurrentAccountListFragment().refresh();
+                AccountsListFragment fragment = ((AccountsActivity) mContext).getCurrentAccountListFragment();
+                if (fragment != null)
+                    fragment.refresh();
             }
             if (mContext instanceof TransactionsActivity){
                 ((TransactionsActivity) mContext).refresh();

app/src/main/java/org/gnucash/android/export/Exporter.java

@@ -118,6 +118,9 @@ public static String buildExportFilename(ExportFormat format) {
     public static long getExportTime(String filename){
         String[] tokens = filename.split("_");
         long timeMillis = 0;
+        if (tokens.length < 2){
+            return timeMillis;
+        }
         try {
             Date date = EXPORT_FILENAME_DATE_FORMAT.parse(tokens[0] + "_" + tokens[1]);
             timeMillis = date.getTime();

app/src/main/java/org/gnucash/android/importer/GncXmlHandler.java

@@ -191,6 +191,11 @@ public class GncXmlHandler extends DefaultHandler {
      */
     boolean mIgnoreTemplateTransaction = true;
 
+    /**
+     * Flag which notifies the handler to ignore a scheduled action because some error occurred during parsing
+     */
+    boolean mIgnoreScheduledAction = false;
+
     /**
      * Used for parsing old backup files where recurrence was saved inside the transaction.
      * Newer backup files will not require this
@@ -547,9 +552,16 @@ public void endElement(String uri, String localName, String qualifiedName) throw
                 mRecurrenceMultiplier = Integer.parseInt(characterString);
                 break;
             case GncXmlHelper.TAG_RX_PERIOD_TYPE:
-                PeriodType periodType = PeriodType.valueOf(characterString.toUpperCase());
-                periodType.setMultiplier(mRecurrenceMultiplier);
-                mScheduledAction.setPeriod(periodType);
+                try {
+                    PeriodType periodType = PeriodType.valueOf(characterString.toUpperCase());
+                    periodType.setMultiplier(mRecurrenceMultiplier);
+                    mScheduledAction.setPeriod(periodType);
+                } catch (IllegalArgumentException ex){ //the period type constant is not supported
+                    String msg = "Unsupported period constant: " + characterString;
+                    Log.e(LOG_TAG, msg);
+                    Crashlytics.logException(ex);
+                    mIgnoreScheduledAction = true;
+                }
                 break;
             case GncXmlHelper.TAG_GDATE:
                 try {
@@ -589,11 +601,13 @@ public void endElement(String uri, String localName, String qualifiedName) throw
                 }
                 break;
             case GncXmlHelper.TAG_SCHEDULED_ACTION:
-                if (mScheduledAction.getActionUID() != null)
+                if (mScheduledAction.getActionUID() != null && !mIgnoreScheduledAction) {
                     mScheduledActionsList.add(mScheduledAction);
-                int count = generateMissedScheduledTransactions(mScheduledAction);
-                Log.i(LOG_TAG, String.format("Generated %d transactions from scheduled action", count));
+                    int count = generateMissedScheduledTransactions(mScheduledAction);
+                    Log.i(LOG_TAG, String.format("Generated %d transactions from scheduled action", count));
+                }
                 mRecurrenceMultiplier = 1; //reset it, even though it will be parsed from XML each time
+                mIgnoreScheduledAction = false;
                 break;
         }