From b69b206c6ec3082fe9fd2ac3ba019697878fe1e7 Mon Sep 17 00:00:00 2001
From: TheJoKlLa <thejoklla@gmail.com>
Date: Wed, 5 Jan 2022 00:57:59 +0100
Subject: [PATCH] Fixed MachineList Crash if not connected to server

---
 Borepin/Borepin/Base/ConnectionModelBase.cs |  4 ++
 Borepin/Borepin/Page/MachineListPage.xaml   | 42 +++++++++++----------
 2 files changed, 27 insertions(+), 19 deletions(-)

diff --git a/Borepin/Borepin/Base/ConnectionModelBase.cs b/Borepin/Borepin/Base/ConnectionModelBase.cs
index 4fe5097..e999e7d 100644
--- a/Borepin/Borepin/Base/ConnectionModelBase.cs
+++ b/Borepin/Borepin/Base/ConnectionModelBase.cs
@@ -51,6 +51,10 @@ namespace Borepin.Base
                 {
                     await _BFFHService.Reconnect().ConfigureAwait(false);
                 }
+                else if(_BFFHService.CurrentConnection == null)
+                {
+                    return false;
+                }
 
                 return true;
             }
diff --git a/Borepin/Borepin/Page/MachineListPage.xaml b/Borepin/Borepin/Page/MachineListPage.xaml
index 44bdf85..8940f48 100644
--- a/Borepin/Borepin/Page/MachineListPage.xaml
+++ b/Borepin/Borepin/Page/MachineListPage.xaml
@@ -16,26 +16,30 @@
                 <ActivityIndicator IsRunning="{Binding IsBusy}"></ActivityIndicator>
             </StackLayout>
             <StackLayout IsVisible="{Binding IsBusy, Converter={StaticResource InvertBoolConverter}}">
-                <Button Text="Scan QR-Code" Command="{Binding ScanCodeCommand}" Style="{StaticResource Style_Button_Primary}">
-                    <Button.IsVisible>
-                        <OnPlatform x:TypeArguments="x:Boolean"
-                    iOS="false"
-                    Android="true"/>
-                    </Button.IsVisible>
-                </Button>
-                <RefreshView Command="{Binding RefreshCommand}" IsRefreshing="{Binding IsRefreshing}">
-                    <ListView ItemsSource="{Binding MachineListItemViewModel_List}" IsVisible="{Binding IsConnected}" SelectionMode="None">
-                        <ListView.ItemTemplate>
-                            <DataTemplate>
-                                <ViewCell>
-                                    <views:MachineListItemView />
-                                </ViewCell>
-                            </DataTemplate>
-                        </ListView.ItemTemplate>
-                    </ListView>
-                </RefreshView>
+                <StackLayout IsVisible="{Binding IsConnected}">
+                    <Button Text="Scan QR-Code" Command="{Binding ScanCodeCommand}" Style="{StaticResource Style_Button_Primary}">
+                        <Button.IsVisible>
+                            <OnPlatform x:TypeArguments="x:Boolean"
+                            iOS="false"
+                            Android="true"/>
+                        </Button.IsVisible>
+                    </Button>
+                    <RefreshView Command="{Binding RefreshCommand}" IsRefreshing="{Binding IsRefreshing}">
+                        <ListView ItemsSource="{Binding MachineListItemViewModel_List}" IsVisible="{Binding IsConnected}" SelectionMode="None">
+                            <ListView.ItemTemplate>
+                                <DataTemplate>
+                                    <ViewCell>
+                                        <views:MachineListItemView />
+                                    </ViewCell>
+                                </DataTemplate>
+                            </ListView.ItemTemplate>
+                        </ListView>
+                    </RefreshView>
+                </StackLayout>
+                <StackLayout IsVisible="{Binding IsConnected, Converter={StaticResource InvertBoolConverter}}">
+                    <Label Text="Please connect to Server" ></Label>
+                </StackLayout>
             </StackLayout>
-            <Label Text="Please connect to Server" IsVisible="{Binding IsConnected, Converter={StaticResource InvertBoolConverter}}"></Label>
         </StackLayout>
     </ContentPage.Content>
 </ContentPage>