fix : 🐛 Fixed onEventTap in MonthView to ignore when onEventTap is null.(#278)

Author: Sahil-Simform

CHANGELOG.md

@@ -2,6 +2,8 @@
 
 - Adds clear method to `EventController`.
 - Adds `onEventTapDetails`, `onEventDoubleTapDetails` & `onEventLongTapDetails` gesture recognizers to get tap details. [#390](https://github.com/SimformSolutionsPvtLtd/flutter_calendar_view/issues/390)
+- Fixed `onEventTap` in `MonthView` to ignore when it is
+  null. [#278](https://github.com/SimformSolutionsPvtLtd/flutter_calendar_view/issues/278)
 
 # [1.4.0 - 7 Jan 2025](https://github.com/SimformSolutionsPvtLtd/flutter_calendar_view/tree/1.4.0)
 

lib/src/components/month_view_components.dart

@@ -173,28 +173,32 @@ class FilledCell<T extends Object?> extends StatelessWidget {
                     children: List.generate(
                       events.length,
                       (index) => GestureDetector(
-                        onTap: () => onTileTap?.call(events[index], date),
-                        onLongPress: () =>
-                            onTileLongTap?.call(events[index], date),
-                        onDoubleTap: () =>
-                            onTileDoubleTap?.call(events[index], date),
-                        onTapUp: (details) => onTileTapDetails?.call(
-                          events[index],
-                          date,
-                          details,
-                        ),
-                        onLongPressStart: (details) =>
-                            onTileLongTapDetails?.call(
-                          events[index],
-                          date,
-                          details,
-                        ),
-                        onDoubleTapDown: (details) =>
-                            onTileDoubleTapDetails?.call(
-                          events[index],
-                          date,
-                          details,
-                        ),
+                        onTap: onTileTap.safeVoidCall(events[index], date),
+                        onLongPress:
+                            onTileLongTap.safeVoidCall(events[index], date),
+                        onDoubleTap:
+                            onTileDoubleTap.safeVoidCall(events[index], date),
+                        onTapUp: onTileTapDetails == null
+                            ? null
+                            : (details) => onTileTapDetails?.call(
+                                  events[index],
+                                  date,
+                                  details,
+                                ),
+                        onLongPressStart: onTileLongTapDetails == null
+                            ? null
+                            : (details) => onTileLongTapDetails?.call(
+                                  events[index],
+                                  date,
+                                  details,
+                                ),
+                        onDoubleTapDown: onTileDoubleTapDetails == null
+                            ? null
+                            : (details) => onTileDoubleTapDetails?.call(
+                                  events[index],
+                                  date,
+                                  details,
+                                ),
                         child: Container(
                           decoration: BoxDecoration(
                             color: events[index].color,

lib/src/day_view/day_view.dart

@@ -656,8 +656,10 @@ class DayViewState<T extends Object?> extends State<DayView<T>> {
   /// Default timeline builder this builder will be used if
   /// [widget.eventTileBuilder] is null
   ///
-  Widget _defaultTimeLineBuilder(date) => DefaultTimeLineMark(
-      date: date, timeStringBuilder: widget.timeStringBuilder);
+  Widget _defaultTimeLineBuilder(DateTime date) => DefaultTimeLineMark(
+        date: date,
+        timeStringBuilder: widget.timeStringBuilder,
+      );
 
   /// Default timeline builder. This builder will be used if
   /// [widget.eventTileBuilder] is null

lib/src/extensions.dart

@@ -252,3 +252,20 @@ void debugLog(String message) {
     return false;
   }(), '');
 }
+
+/// For callbacks with one argument
+extension NullableCallback1<A> on void Function(A)? {
+  VoidCallback? safeVoidCall(A a) => this == null ? null : () => this!(a);
+}
+
+/// For callbacks with two arguments
+extension NullableCallback2<A, B> on void Function(A, B)? {
+  VoidCallback? safeVoidCall(A a, B b) =>
+      this == null ? null : () => this!(a, b);
+}
+
+/// For callbacks with three arguments
+extension NullableCallback3<A, B, C> on void Function(A, B, C)? {
+  VoidCallback? safeVoidCall(A a, B b, C c) =>
+      this == null ? null : () => this!(a, b, c);
+}

lib/src/month_view/month_view.dart

@@ -582,11 +582,11 @@ class MonthViewState<T extends Object?> extends State<MonthView<T>> {
 
   /// Default cell builder. Used when [widget.cellBuilder] is null
   Widget _defaultCellBuilder(
-    date,
+    DateTime date,
     List<CalendarEventData<T>> events,
-    isToday,
-    isInMonth,
-    hideDaysNotInMonth,
+    bool isToday,
+    bool isInMonth,
+    bool hideDaysNotInMonth,
   ) {
     if (hideDaysNotInMonth) {
       return FilledCell<T>(