Adding required fields to all widgets, which are missing

deepikahr · deepikahr · commit 1d3b5d14986f · 2021-03-04T13:55:00.000+05:30
diff --git a/lib/components/accordian/gf_accordian.dart b/lib/components/accordian/gf_accordian.dart
@@ -5,18 +5,19 @@ class GFAccordion extends StatefulWidget {
   /// An accordion is used to show (and hide) content. Use [showAccordion] to hide & show the accordion content.
   const GFAccordion(
       {Key key,
-      this.titleChild,
-      this.content,
+        this.title,
+        this.content,
+        this.titleChild,
+        this.contentChild,
       this.collapsedTitleBackgroundColor = GFColors.WHITE,
       this.expandedTitleBackgroundColor = const Color(0xFFE0E0E0),
       this.collapsedIcon = const Icon(Icons.keyboard_arrow_down),
       this.expandedIcon = const Icon(Icons.keyboard_arrow_up),
-      this.title,
+
       this.textStyle = const TextStyle(color: Colors.black, fontSize: 16),
       this.titlePadding = const EdgeInsets.all(10),
       this.contentBackgroundColor,
       this.contentPadding = const EdgeInsets.all(10),
-      this.contentChild,
       this.titleBorder = const Border(),
       this.contentBorder = const Border(),
       this.margin,
diff --git a/lib/components/alert/gf_alert.dart b/lib/components/alert/gf_alert.dart
@@ -6,14 +6,14 @@ class GFAlert extends StatefulWidget {
   /// Alert has to be wrap inside the body like [GFFloatingWidget]. See [GFFloatingWidget]
   const GFAlert(
       {Key key,
+        this.content,
+        this.title,
       this.child,
       this.backgroundColor,
-      this.content,
       this.width,
       this.type = GFAlertType.basic,
       this.alignment,
       this.contentChild,
-      this.title,
       this.bottombar,
       this.contentTextStyle = const TextStyle(color: Colors.black87),
       this.titleTextStyle = const TextStyle(
diff --git a/lib/components/animation/gf_animation.dart b/lib/components/animation/gf_animation.dart
@@ -5,9 +5,9 @@ class GFAnimation extends StatefulWidget {
   /// Wrap any widget with [GFAnimation] to animate your widget. see [GFAnimationType] for types of animations.
   const GFAnimation({
     Key key,
+    @required this.child,
     this.duration,
     this.alignment,
-    this.child,
     this.curve,
     this.type,
     this.width,
@@ -33,7 +33,8 @@ class GFAnimation extends StatefulWidget {
     this.changedWidth,
     this.changedHeight,
     this.reverseDuration,
-  }) : super(key: key);
+  }) : assert(child != null),
+        super(key: key);
 
   /// The duration for animation to perform
   final Duration duration;
diff --git a/lib/components/badge/gf_icon_badge.dart b/lib/components/badge/gf_icon_badge.dart
@@ -8,7 +8,9 @@ class GFIconBadge extends StatefulWidget {
     this.padding = const EdgeInsets.symmetric(horizontal: 8),
     @required this.child,
     @required this.counterChild,
-  })  : assert(padding != null),
+  })  : assert(child != null),
+        assert(counterChild != null),
+        assert(padding != null),
         super(key: key);
 
   /// child of type [Widget] is used to show icon.
diff --git a/lib/components/button/gf_button_bar.dart b/lib/components/button/gf_button_bar.dart
@@ -15,9 +15,10 @@ class GFButtonBar extends StatelessWidget {
     this.crossAxisAlignment = WrapCrossAlignment.start,
     this.textDirection,
     this.verticalDirection = VerticalDirection.down,
-    this.children = const <Widget>[],
+    @required this.children,
     this.padding = const EdgeInsets.all(0),
-  }) : super(key: key);
+  }) : assert(children != null),
+        super(key: key);
 
   /// The empty space that surrounds the buttonBar. Default's to padding.zero
   final EdgeInsetsGeometry padding;
diff --git a/lib/components/checkbox/gf_checkbox.dart b/lib/components/checkbox/gf_checkbox.dart
@@ -11,8 +11,8 @@ class GFCheckbox extends StatefulWidget {
       this.inactiveBgColor = GFColors.WHITE,
       this.activeBorderColor = GFColors.WHITE,
       this.inactiveBorderColor = GFColors.DARK,
-      this.onChanged,
-      this.value,
+      @required this.onChanged,
+      @required this.value,
       this.activeIcon = const Icon(
         Icons.check,
         size: 20,
@@ -22,7 +22,9 @@ class GFCheckbox extends StatefulWidget {
       this.customBgColor = GFColors.SUCCESS,
       this.autofocus = false,
       this.focusNode})
-      : assert(autofocus != null),
+      : assert(value != null),
+        assert(onChanged != null),
+        assert(autofocus != null),
         super(key: key);
 
   /// type of [GFCheckboxType] which is of four type is basic, square, circular and custom
diff --git a/lib/components/checkbox_list_tile/gf_checkbox_list_tile.dart b/lib/components/checkbox_list_tile/gf_checkbox_list_tile.dart
@@ -35,6 +35,7 @@ class GFCheckboxListTile extends StatelessWidget {
     this.selected = false,
     this.autofocus = false,
   })  : assert(value != null),
+        assert(onChanged != null),
         assert(selected != null),
         assert(autofocus != null),
         super(key: key);
diff --git a/lib/components/dropdown/gf_dropdown.dart b/lib/components/dropdown/gf_dropdown.dart
@@ -32,7 +32,8 @@ class GFDropdown<T> extends StatefulWidget {
       this.border = const BorderSide(
           color: Colors.transparent, width: 1, style: BorderStyle.solid),
       this.dropdownButtonColor = GFColors.WHITE})
-      : super(key: key);
+      : assert(onChanged != null),
+        super(key: key);
 
   final DropdownButtonBuilder selectedItemBuilder;
   final List<DropdownMenuItem<T>> items;
diff --git a/lib/components/image/gf_image_overlay.dart b/lib/components/image/gf_image_overlay.dart
@@ -12,7 +12,7 @@ class GFImageOverlay extends StatelessWidget {
     this.color,
     this.padding,
     this.margin,
-    this.image,
+    @required this.image,
     this.child = const Text(''),
     this.alignment,
     this.borderRadius,
@@ -22,6 +22,7 @@ class GFImageOverlay extends StatelessWidget {
     this.border,
     this.shape = BoxShape.rectangle,
   })  : assert(shape != null),
+        assert(image != null),
         super(key: key);
 
   /// define image's [double] height
diff --git a/lib/components/loader/gf_loader.dart b/lib/components/loader/gf_loader.dart
@@ -64,22 +64,22 @@ class GFLoader extends StatefulWidget {
 
 class _GFLoaderState extends State<GFLoader>
     with SingleTickerProviderStateMixin {
-  Animation<double> loaderanimation1;
-  Animation<double> loaderanimation2;
-  Animation<double> loaderanimation3;
+  Animation<double> loaderAnimation1;
+  Animation<double> loaderAnimation2;
+  Animation<double> loaderAnimation3;
   AnimationController controller;
   Interval interval = const Interval(0.1, 0.81, curve: Curves.linear);
 
   @override
   void initState() {
     controller = AnimationController(duration: widget.duration, vsync: this);
-    loaderanimation1 = Tween<double>(begin: 0, end: 1).animate(
+    loaderAnimation1 = Tween<double>(begin: 0, end: 1).animate(
       CurvedAnimation(parent: controller, curve: interval),
     );
-    loaderanimation2 = Tween<double>(begin: 0, end: 1).animate(
+    loaderAnimation2 = Tween<double>(begin: 0, end: 1).animate(
       CurvedAnimation(parent: controller, curve: interval),
     );
-    loaderanimation3 = Tween<double>(begin: 0, end: 1).animate(
+    loaderAnimation3 = Tween<double>(begin: 0, end: 1).animate(
       CurvedAnimation(parent: controller, curve: interval),
     );
     controller.addListener(() {
@@ -117,12 +117,12 @@ class _GFLoaderState extends State<GFLoader>
                         mainAxisAlignment: MainAxisAlignment.center,
                         children: <Widget>[
                           Opacity(
-                            opacity: loaderanimation1.value <= 0.3
-                                ? 2.5 * loaderanimation1.value
-                                : (loaderanimation1.value > 0.30 &&
-                                        loaderanimation1.value <= 0.70)
+                            opacity: loaderAnimation1.value <= 0.3
+                                ? 2.5 * loaderAnimation1.value
+                                : (loaderAnimation1.value > 0.30 &&
+                                        loaderAnimation1.value <= 0.70)
                                     ? 1.0
-                                    : 2.5 - (2.5 * loaderanimation1.value),
+                                    : 2.5 - (2.5 * loaderAnimation1.value),
                             child: Padding(
                               padding: const EdgeInsets.only(right: 8),
                               child: Loader(
@@ -135,12 +135,12 @@ class _GFLoaderState extends State<GFLoader>
                             ),
                           ),
                           Opacity(
-                            opacity: loaderanimation2.value <= 0.3
-                                ? 2.5 * loaderanimation2.value
-                                : (loaderanimation2.value > 0.30 &&
-                                        loaderanimation2.value <= 0.70)
+                            opacity: loaderAnimation2.value <= 0.3
+                                ? 2.5 * loaderAnimation2.value
+                                : (loaderAnimation2.value > 0.30 &&
+                                        loaderAnimation2.value <= 0.70)
                                     ? 1.0
-                                    : 2.5 - (2.5 * loaderanimation2.value),
+                                    : 2.5 - (2.5 * loaderAnimation2.value),
                             child: Padding(
                               padding: const EdgeInsets.only(right: 8),
                               child: Loader(
@@ -152,12 +152,12 @@ class _GFLoaderState extends State<GFLoader>
                             ),
                           ),
                           Opacity(
-                            opacity: loaderanimation3.value <= 0.3
-                                ? 2.5 * loaderanimation3.value
-                                : (loaderanimation3.value > 0.30 &&
-                                        loaderanimation3.value <= 0.70)
+                            opacity: loaderAnimation3.value <= 0.3
+                                ? 2.5 * loaderAnimation3.value
+                                : (loaderAnimation3.value > 0.30 &&
+                                        loaderAnimation3.value <= 0.70)
                                     ? 1.0
-                                    : 2.5 - (2.5 * loaderanimation3.value),
+                                    : 2.5 - (2.5 * loaderAnimation3.value),
                             child: Padding(
                               padding: const EdgeInsets.only(right: 8),
                               child: Loader(
diff --git a/lib/components/radio/gf_radio.dart b/lib/components/radio/gf_radio.dart
@@ -28,7 +28,10 @@ class GFRadio<T> extends StatefulWidget {
       this.autofocus = false,
       this.focusNode,
       this.toggleable = false})
-      : assert(autofocus != null),
+      : assert(value != null),
+        assert(groupValue != null),
+        assert(onChanged != null),
+        assert(autofocus != null),
         assert(toggleable != null),
         super(key: key);
 
diff --git a/lib/components/radio_list_tile/gf_radio_list_tile.dart b/lib/components/radio_list_tile/gf_radio_list_tile.dart
@@ -42,7 +42,10 @@ class GFRadioListTile<T> extends StatelessWidget {
     this.selected = false,
     this.focusColor,
     this.hoverColor,
-  })  : assert(autofocus != null),
+  })  : assert(value != null),
+        assert(groupValue != null),
+        assert(onChanged != null),
+        assert(autofocus != null),
         assert(toggleable != null),
         assert(enabled != null),
         assert(selected != null),
diff --git a/lib/components/rating/gf_rating.dart b/lib/components/rating/gf_rating.dart
@@ -7,10 +7,10 @@ class GFRating extends StatefulWidget {
   /// [GFRating] to show ratings with many custimazation options.
   const GFRating({
     Key key,
-    this.onChanged,
+    @required this.onChanged,
+    @required this.value,
     this.itemCount = 5,
     this.spacing = 0.0,
-    this.value = 0.0,
     this.defaultIcon,
     this.color,
     this.borderColor,
@@ -25,6 +25,7 @@ class GFRating extends StatefulWidget {
     this.margin = const EdgeInsets.symmetric(vertical: 16),
     this.padding = const EdgeInsets.symmetric(horizontal: 16),
   })  : assert(value != null),
+        assert(onChanged != null),
         super(key: key);
 
   /// defines total number of rating items
diff --git a/lib/components/slidable/gf_slidable.dart b/lib/components/slidable/gf_slidable.dart
diff --git a/lib/components/tabs/gf_segment_tabs.dart b/lib/components/tabs/gf_segment_tabs.dart
@@ -12,7 +12,8 @@ class GFSegmentTabs extends StatefulWidget {
   const GFSegmentTabs(
       {Key key,
       @required this.length,
-      // this.initialIndex = 0,
+        @required this.tabs,
+        @required this.tabController,
       this.height,
       this.width,
       this.border,
@@ -28,17 +29,11 @@ class GFSegmentTabs extends StatefulWidget {
       this.labelPadding,
       this.unselectedLabelColor,
       this.unselectedLabelStyle,
-      this.tabs,
-      this.tabController})
+      })
       : assert(length != null && length >= 0),
-        // assert(initialIndex != null &&
-        //     initialIndex >= 0 &&
-        //     (length == 0 || initialIndex < length)),
+        assert(tabs != null),
         super(key: key);
 
-  // /// The initial index of the selected tab. Defaults to zero.
-  // final int initialIndex;
-
   /// The total number of tabs. Typically greater than one. Must match [TabBar.tabs]'s and
   /// [TabBarView.children]'s length.
   final int length;
diff --git a/lib/components/tabs/gf_tabbar.dart b/lib/components/tabs/gf_tabbar.dart
@@ -15,8 +15,9 @@ class GFTabBar extends StatefulWidget {
   /// [TabController.length].
   const GFTabBar({
     Key key,
-    // this.initialIndex = 0,
     @required this.length,
+    @required this.tabs,
+    @required this.controller,
     this.isScrollable = false,
     this.tabBarHeight,
     this.tabBarColor,
@@ -30,19 +31,12 @@ class GFTabBar extends StatefulWidget {
     this.labelPadding,
     this.unselectedLabelColor,
     this.unselectedLabelStyle,
-    this.tabs,
-    this.controller,
     this.shape,
   })  : assert(length != null && length >= 0),
         assert(isScrollable != null),
-        // assert(initialIndex != null &&
-        //     initialIndex >= 0 &&
-        //     (length == 0 || initialIndex < length)),
+        assert(tabs != null),
         super(key: key);
 
-  // /// The initial index of the selected tab. Defaults to zero.
-  // final int initialIndex;
-
   /// The total number of tabs. Typically greater than one. Must match [TabBar.tabs]'s and
   /// [TabBarView.children]'s length.
   final int length;
diff --git a/lib/components/tabs/gf_tabbar_view.dart b/lib/components/tabs/gf_tabbar_view.dart
@@ -13,7 +13,7 @@ class GFTabBarView extends StatefulWidget {
   const GFTabBarView({
     Key key,
     @required this.children,
-    this.controller,
+    @required this.controller,
     this.physics,
     this.height,
     this.dragStartBehavior = DragStartBehavior.start,
diff --git a/lib/components/tabs/gf_tabs.dart b/lib/components/tabs/gf_tabs.dart
@@ -15,6 +15,9 @@ class GFTabs extends StatefulWidget {
     Key key,
     this.initialIndex = 0,
     @required this.length,
+    @required this.tabs,
+    @required this.tabBarView,
+    @required this.controller,
     this.isScrollable = false,
     this.height,
     this.tabBarColor,
@@ -28,16 +31,15 @@ class GFTabs extends StatefulWidget {
     this.labelPadding,
     this.unselectedLabelColor,
     this.unselectedLabelStyle,
-    this.tabBarView,
-    this.tabs,
-    this.controller,
     this.tabBarHeight,
     this.shape,
   })  : assert(length != null && length >= 0),
         assert(isScrollable != null),
         assert(initialIndex != null &&
             initialIndex >= 0 &&
             (length == 0 || initialIndex < length)),
+        assert(tabs != null),
+        assert(tabBarView != null),
         super(key: key);
 
   /// The initial index of the selected tab. Defaults to zero.
diff --git a/lib/components/toggle/gf_toggle.dart b/lib/components/toggle/gf_toggle.dart
@@ -30,6 +30,7 @@ class GFToggle extends StatefulWidget {
       this.borderRadius,
       this.duration = const Duration(milliseconds: 400)})
       : assert(value != null),
+        assert(onChanged != null),
         super(key: key);
 
   ///type [String] used to add custom text i.e, ON,ENABLE
diff --git a/lib/getwidget.dart b/lib/getwidget.dart
@@ -36,7 +36,6 @@ export 'package:getwidget/components/radio_list_tile/gf_radio_list_tile.dart';
 export 'package:getwidget/components/rating/gf_rating.dart';
 export 'package:getwidget/components/search_bar/gf_search_bar.dart';
 export 'package:getwidget/components/shimmer/gf_shimmer.dart';
-export 'package:getwidget/components/slidable/gf_slidable.dart';
 export 'package:getwidget/components/sticky_header/gf_sticky_header.dart';
 export 'package:getwidget/components/sticky_header/gf_sticky_header_builder.dart';
 export 'package:getwidget/components/sticky_header/render_gf_sticky_header.dart';
diff --git a/test/alert_test.dart b/test/alert_test.dart
@@ -16,7 +16,8 @@ void main() {
     final GFAlert alert = GFAlert(
         backgroundColor: bgColor,
         contentChild: contentChild,
-        child: childWidget);
+        child: childWidget
+    );
 
     final TestApp app = TestApp(alert);
 
diff --git a/test/animation_test.dart b/test/animation_test.dart
diff --git a/test/rating_test.dart b/test/rating_test.dart
diff --git a/test/segment_tabs_test.dart b/test/segment_tabs_test.dart
diff --git a/test/tabbar_test.dart b/test/tabbar_test.dart
diff --git a/test/tabbarview_test.dart b/test/tabbarview_test.dart
diff --git a/test/tabs_test.dart b/test/tabs_test.dart
