From 0bfa839b88d1ae83504c42200bec41cead9f61db Mon Sep 17 00:00:00 2001
From: Vishesh Handa <me@vhanda.in>
Date: Sun, 28 Jul 2019 03:01:16 +0200
Subject: [PATCH] Fix GitRepo.clone parameters

This is why it sucks to have 2 fields both of the same type and not have
some kind of automated testing for this part of the code.
---
 lib/gitapp.dart                       | 2 +-
 lib/screens/githostsetup_screens.dart | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/gitapp.dart b/lib/gitapp.dart
index 051bb386..2e7cc760 100644
--- a/lib/gitapp.dart
+++ b/lib/gitapp.dart
@@ -56,7 +56,7 @@ class GitApp extends StatelessWidget {
         child: Text("Git Clone"),
         onPressed: () async {
           try {
-            await GitRepo.clone(cloneUrl, basePath);
+            await GitRepo.clone(basePath, cloneUrl);
             _sendSuccess();
           } on GitException catch (ex) {
             print(ex);
diff --git a/lib/screens/githostsetup_screens.dart b/lib/screens/githostsetup_screens.dart
index 8a1d5ec1..8782915f 100644
--- a/lib/screens/githostsetup_screens.dart
+++ b/lib/screens/githostsetup_screens.dart
@@ -382,7 +382,7 @@ class GitHostSetupScreenState extends State<GitHostSetupScreen> {
 
     String error;
     try {
-      await GitRepo.clone(_gitCloneUrl, "journal");
+      await GitRepo.clone("journal", _gitCloneUrl);
     } on GitException catch (e) {
       error = e.cause;
     }