NodeListTile: fix required online parameter

required named parameters cannot have a default, also the @required
has changed to required, so fix that by making 'online' non nullable
and remove the default value (since we give a value everywhere anyway)

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

diff --git a/lib/pages/main_layout_slim.dart b/lib/pages/main_layout_slim.dart
index dec5e55cda1e09c77eb060be012b8d57080383a1..31d6ef79a3afb050f9801743247b6339ddb7177d 100644 (file)
--- a/lib/pages/main_layout_slim.dart
+++ b/lib/pages/main_layout_slim.dart
@@ -377,7 +377,7 @@ class MobileDashboard extends StatelessWidget {
                     ...cState.nodes.map((node) {
                       return PveNodeListTile(
                         name: node.name,
-                        online: node.online,
+                        online: node.online ?? false,
                         type: node.type,
                         level: node.level,
                         ip: node.ip,
@@ -548,14 +548,14 @@ class MobileDashboard extends StatelessWidget {
 
 class PveNodeListTile extends StatelessWidget {
   final String name;
-  final bool? online;
+  final bool online;
   final String type;
   final String? level;
   final String? ip;
   const PveNodeListTile(
       {Key? key,
       required this.name,
-      @required this.online = false,
+      required this.online,
       required this.type,
       this.level,
       this.ip = ''})
@@ -567,8 +567,8 @@ class PveNodeListTile extends StatelessWidget {
         Renderers.getDefaultResourceIcon(type),
       ),
       title: Text(name),
-      subtitle: Text(getNodeTileSubtitle(online!, level, ip)),
-      trailing: Icon(Icons.power, color: online! ? Colors.green : Colors.grey),
+      subtitle: Text(getNodeTileSubtitle(online, level, ip)),
+      trailing: Icon(Icons.power, color: online ? Colors.green : Colors.grey),
       onTap: () => Navigator.pushNamed(context, '/nodes/$name'),
     );
   }