Merge branch 'hotfix/patches'

Author: codinguser

CHANGELOG.md

@@ -1,5 +1,10 @@
 Change Log
 ===============================================================================
+Version 2.1.7 *(2017-04-18)*
+----------------------------
+* Properly handle crashes during migration of backup/export files to new location
+* Update README and CONTRIBUTORS
+
 Version 2.1.6 *(2017-04-15)*
 ----------------------------
 * Fixed #664: Rotating device in transaction view causes crash

CONTRIBUTORS.md

@@ -7,35 +7,59 @@ project through the early phases (as Google Summer of Code project 2012) and pro
 * Ngewi Fet <[email protected]> - Project maintainer
 * Yongxin Wang <[email protected]>
 * Oleksandr Tyshkovets <[email protected]>
-* Àlex Magaz Graça <rivaldi8@gmail.com>
+* Àlex Magaz Graça <alexandre.magaz@gmail.com>
 
 ### Other Contributors
-The following (incomplete list of) people (in no particular order) contributed (patches and translations) to GnuCash Android:
-* Christian Stimming <[email protected]>
-* Cristian Marchi <[email protected]>
-* Menelaos Maglis <[email protected]>
-* Kjell Thomas Pedersen <[email protected]>
-* Alexander Galanin <[email protected]>
-* Jorge Martínez López <[email protected]>
-* Israel Buitron <[email protected]>
-* Geert Janssens <[email protected]>
-* Nicolas Barranger <[email protected]>
-* Sigurd Gartmann <[email protected]>
-* Pedro Abel <[email protected]>
-* windwarrior <[email protected]>
-* Alex Lei <[email protected]>
-* Matthew Hague <[email protected]>
-* Spanti Nicola <[email protected]>
-* Jesse Shieh <[email protected]>
-* Terry Chung <[email protected]>
-* Caesar Wirth <[email protected]>
-* Alceu Rodrigues Neto <[email protected]>
-* Carlo Zancanaro <[email protected]>
-* Eric Daly <[email protected]>
-* Weslly Oliveira <[email protected]>
-* Felipe Morato <[email protected]>
-* Alceu Rodrigues Neto <[email protected]>
-* Salama AB <[email protected]>
-* Juan Villa <[email protected]>
+The following people (in alphabetical order) contributed (commits on GitHub) to GnuCash Android:
+* aerkefiende
+* Alceu Rodrigues Neto
+* Aleksey Ivanovski
+* Àlex Magaz Graça
+* Alexander Galanin
+* Alexandr Makaric
+* Caesar Wirth
+* Carlo Zancanaro
+* choni [email protected]
+* Christian Stimming
+* Cristian Marchi
+* David Landry
+* Eric Daly
+* Falk Brockmann
+* Felipe Morato
+* Geert Janssens
+* Jörg Möller
+* Israel Buitron
+* Jesse Shieh
+* Jorge Martínez López
+* Juan Villa
+* Kjell Thomas Pedersen
+* Lian Kai
+* lxb [email protected]
+* Alex Lei
+* Mark Haanen
+* Matthew Hague
+* Menelaos Maglis
+* [email protected]
+* Ngewi Fet
+* Nicolas Barranger
+* Oleg Kosmakov
+* Oleksandr Tyshkovets
+* PBNeves [email protected]
+* Pedro Abel
+* Salama AB
+* Sandro Santilli
+* Sigurd Gartmann
+* smisger [email protected]
+* Spanti Nicola
+* Stephan Windmüller
+* Terry Chung
+* thesebas [email protected]
+* Vladimir Rutsky
+* Weslly Oliveira
+* windwarrior [email protected]
+* yapiti [email protected]
+* Yongxin Wang
 
-Please visit https://crowdin.com/project/gnucash-android for a more complete list of translation contributions
+
+### Translators
+Please visit https://crowdin.com/project/gnucash-android for a more complete list of translation contributions

README.md

@@ -7,6 +7,10 @@
 GnuCash Android is a companion expense-tracker application for GnuCash (desktop) designed for Android.
 It allows you to record transactions on-the-go and later import the data into GnuCash for the desktop.
 
+Accounts            |  Transactions          |  Reports
+:-------------------------:|:-------------------------:|:-------------------------:
+![Accounts List](docs/images/v2.0.0_home.png)  |  ![Transactions List](docs/images/v2.0.0_transactions_list.png) |  ![Reports](docs/images/v2.0.0_reports.png)
+
 The application supports Android 2.3.3 Gingerbread (API level 10) and above.
 
 Features include:
@@ -66,7 +70,7 @@ The app is configured to allow you to install a development and production versi
 
 ### With Android Studio
 The easiest way to build is to install [Android Studio](https://developer.android.com/sdk/index.html) v2.+
-with [Gradle](https://www.gradle.org/) v2.10
+with [Gradle](https://www.gradle.org/) v3.4.1
 Once installed, then you can import the project into Android Studio:
 
 1. Open `File`
@@ -89,7 +93,7 @@ Please visit [CrowdIn](https://crowdin.com/project/gnucash-android) in order to
 
 * You could as well contribute code, fixing bugs, new features or automated tests. Pull requests are always welcome.
 Take a look at the [bug tracker](https://github.com/codinguser/gnucash-android/issues?state=open)
-for ideas where to start.
+for ideas where to start. Also take a look at the [contribution guidlines](https://github.com/codinguser/gnucash-android/blob/master/.github/CONTRIBUTING.md)
 
 For development, it is recommended to use the Android Studio for development which is available for free.
 Import the project into the IDE using the build.gradle file. The IDE will resolve dependencies automatically.

app/build.gradle

@@ -6,7 +6,7 @@ apply plugin: 'android-apt'
 
 def versionMajor = 2
 def versionMinor = 1
-def versionPatch = 6
+def versionPatch = 7
 def versionBuild = 0
 
 def buildTime() {

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

@@ -1488,10 +1488,25 @@ static int upgradeDbToVersion13(SQLiteDatabase db){
      * @param srcDir Source directory which should already exist
      * @param dstDir Destination directory which should already exist
      * @see #moveFile(File, File)
+     * @throws IOException if the {@code srcDir} does not exist or {@code dstDir} could not be created
+     * @throws IllegalArgumentException if {@code srcDir} is not a directory
      */
-    private static void moveDirectory(File srcDir, File dstDir){
-        if (!srcDir.exists() || !srcDir.isDirectory() || !dstDir.isDirectory() || !dstDir.exists()){
-            throw new IllegalArgumentException("Source is not a directory, use MigrationHelper.moveFile(...)");
+    private static void moveDirectory(File srcDir, File dstDir) throws IOException {
+        if (!srcDir.isDirectory()){
+            throw new IllegalArgumentException("Source is not a directory: " + srcDir.getPath());
+        }
+
+        if (!srcDir.exists()){
+            String msg = String.format(Locale.US, "Source directory %s does not exist", srcDir.getPath());
+            Log.e(LOG_TAG, msg);
+            throw new IOException(msg);
+        }
+
+        if (!dstDir.exists() || !dstDir.isDirectory()){
+            Log.w(LOG_TAG, "Target directory does not exist. Attempting to create..." + dstDir.getPath());
+            if (!dstDir.mkdirs()){
+                throw new IOException(String.format("Target directory %s does not exist and could not be created", dstDir.getPath()));
+            }
         }
 
         for (File src : srcDir.listFiles()){
@@ -1521,7 +1536,7 @@ private static void moveDirectory(File srcDir, File dstDir){
      *     It moves the backup files to a new backup location which does not require SD CARD write permission
      * </p>
      * @param db SQLite database to be upgraded
-     * @return
+     * @return New database version
      */
     public static int upgradeDbToVersion14(SQLiteDatabase db){
         Log.i(DatabaseHelper.LOG_TAG, "Upgrading database to version 14");
@@ -1534,17 +1549,20 @@ public static int upgradeDbToVersion14(SQLiteDatabase db){
             public void run() {
                 File srcDir = new File(Exporter.LEGACY_BASE_FOLDER_PATH);
                 File dstDir = new File(Exporter.BASE_FOLDER_PATH);
-                moveDirectory(srcDir, dstDir);
-                File readmeFile = new File(Exporter.LEGACY_BASE_FOLDER_PATH, "README.txt");
-                FileWriter writer = null;
                 try {
+                    moveDirectory(srcDir, dstDir);
+                    File readmeFile = new File(Exporter.LEGACY_BASE_FOLDER_PATH, "README.txt");
+                    FileWriter writer = null;
                     writer = new FileWriter(readmeFile);
                     writer.write("Backup files have been moved to " + dstDir.getPath() +
                             "\nYou can now delete this folder");
                     writer.flush();
-                } catch (IOException e) {
-                    e.printStackTrace();
-                    Log.e(LOG_TAG, "Error creating README file");
+                } catch (IOException | IllegalArgumentException ex) {
+                    ex.printStackTrace();
+                    String msg = String.format("Error moving files from %s to %s", srcDir.getPath(), dstDir.getPath());
+                    Log.e(LOG_TAG, msg);
+                    Crashlytics.log(msg);
+                    Crashlytics.logException(ex);
                 }
 
             }