Miscellaneous Cleanup 20241110 #2

Author: rayliverified

lib/src/transformers/node_transformers/passive_text_field_transformer.dart

@@ -504,8 +504,6 @@ class _PassiveTextFieldWidgetState extends State<PassiveTextFieldWidget> {
           borderRadius: inputBorder.cornerRadius.borderRadius,
           borderSide: inputBorder.borderSide.toFlutter(),
         );
-      default:
-        return null;
     }
   }
 

lib/src/transformers/utils/condition_visitors.dart

@@ -1,5 +1,4 @@
 import 'package:codelessly_api/codelessly_api.dart';
-import 'package:collection/collection.dart';
 
 /// A visitor that returns the list of variable names used in a condition.
 class ConditionVariablesVisitor
@@ -67,7 +66,7 @@ class ConditionVariablesVisitor
     return variablePathRegex
         .allMatches(part.valueString)
         .map((match) => match.namedGroup('name'))
-        .whereNotNull()
+        .nonNulls
         .where((element) => !excludedVariableNames.contains(element))
         .toSet();
   }
@@ -101,7 +100,7 @@ class ConditionVariablesVisitor
           (value) => variablePathRegex
               .allMatches(value.value.toString())
               .map((match) => match.namedGroup('name'))
-              .whereNotNull()
+              .nonNulls
               .where((element) => !excludedVariableNames.contains(element)),
         )
         .toSet();
@@ -137,7 +136,7 @@ class ConditionNodesVisitor
   Set<String>? visitCondition(Condition condition) => {
         ...condition.actions
             .map((action) => action.accept<String>(this))
-            .whereNotNull()
+            .nonNulls
             .toSet(),
       };
 
@@ -146,13 +145,13 @@ class ConditionNodesVisitor
         ...condition.ifCondition.actions,
         ...condition.elseIfConditions.expand((condition) => condition.actions),
         ...condition.elseCondition?.actions ?? [],
-      }.map((action) => action.accept<String>(this)).whereNotNull().toSet();
+      }.map((action) => action.accept<String>(this)).nonNulls.toSet();
 
   @override
   Set<String>? visitElseCondition(ElseCondition condition) {
     return condition.actions
         .map((action) => action.accept<String>(this))
-        .whereNotNull()
+        .nonNulls
         .toSet();
   }
 

lib/src/utils/extensions.dart

@@ -269,8 +269,6 @@ extension TextOverflowHelper on TextOverflowC {
         return TextOverflow.ellipsis;
       case TextOverflowC.visible:
         return TextOverflow.visible;
-      default:
-        throw Exception('Unknown TextOverflowc value: $this');
     }
   }
 
@@ -462,8 +460,6 @@ extension CMainAxisAlignmentHelper on MainAxisAlignmentC {
         return flutter.MainAxisAlignment.spaceAround;
       case MainAxisAlignmentC.spaceEvenly:
         return flutter.MainAxisAlignment.spaceEvenly;
-      default:
-        return flutter.MainAxisAlignment.start;
     }
   }
 }
@@ -481,8 +477,6 @@ extension CCrossAxisAlignmentHelper on CrossAxisAlignmentC {
         return flutter.CrossAxisAlignment.stretch;
       case CrossAxisAlignmentC.baseline:
         return flutter.CrossAxisAlignment.baseline;
-      default:
-        return flutter.CrossAxisAlignment.start;
     }
   }
 }
@@ -661,15 +655,12 @@ extension VariableDataListExtensions<T extends VariableData> on Iterable<T> {
 
   List<String> nameToId(List<String> names) => names
       .map((name) => findByNameOrNull(name))
-      .whereNotNull()
+      .nonNulls
       .map((e) => e.id)
       .toList();
 
-  List<String> idToName(List<String> ids) => ids
-      .map((id) => findByIdOrNull(id))
-      .whereNotNull()
-      .map((e) => e.name)
-      .toList();
+  List<String> idToName(List<String> ids) =>
+      ids.map((id) => findByIdOrNull(id)).nonNulls.map((e) => e.name).toList();
 
   List<String> toIds() => map((e) => e.id).toList();
 
@@ -827,8 +818,6 @@ extension FitExtension on Fit {
         return BoxFit.none;
       case Fit.scaleDown:
         return BoxFit.scaleDown;
-      default:
-        return BoxFit.contain;
     }
   }
 }
@@ -1074,8 +1063,6 @@ extension BoxHeightStyleEnumExtensions on BoxHeightStyleEnum {
         return BoxHeightStyle.strut;
       case BoxHeightStyleEnum.tight:
         return BoxHeightStyle.tight;
-      default:
-        return BoxHeightStyle.tight;
     }
   }
 }
@@ -1087,8 +1074,6 @@ extension BoxWidthStyleEnumExtensions on BoxWidthStyleEnum {
         return BoxWidthStyle.max;
       case BoxWidthStyleEnum.tight:
         return BoxWidthStyle.tight;
-      default:
-        return BoxWidthStyle.tight;
     }
   }
 }
@@ -1592,7 +1577,7 @@ extension ActionModelExt on ActionModel {
         strings.addAll(action.values
             .map(
                 (value) => value.value is String ? value.value as String : null)
-            .whereNotNull());
+            .nonNulls);
       case ActionType.setVariable:
         final SetVariableAction action = this as SetVariableAction;
         strings.addAll([action.index, action.mapKey, action.newValue]);

lib/src/utils/utils.dart

@@ -1,6 +1,5 @@
 import 'package:cloud_firestore/cloud_firestore.dart';
 import 'package:codelessly_api/codelessly_api.dart';
-import 'package:collection/collection.dart';
 import 'package:equatable/equatable.dart';
 import 'package:flutter/widgets.dart';
 import 'package:http/http.dart' as http;
@@ -331,7 +330,7 @@ Set<String> getApisToLoadForCanvas(CanvasNode canvasNode) {
       .map((e) => e.action)
       .whereType<ApiCallAction>()
       .map((e) => e.apiId)
-      .whereNotNull()
+      .nonNulls
       .toSet();
 }