fix(community_management): use ValueWrapper to set nullable values

- Replace manual Engagement creation with originalEngagement.copyWith()
- Use ValueWrapper to explicitly set comment to null or resolved status
- Remove unnecessary TODO comment
- Update core dependency to latest version

Author: fulleni

lib/community_management/bloc/community_management_bloc.dart

@@ -298,8 +298,10 @@ class CommunityManagementBloc
       if (originalEngagement.comment == null) return;
 
       final updatedEngagement = originalEngagement.copyWith(
-        comment: originalEngagement.comment!.copyWith(
-          status: ModerationStatus.resolved,
+        comment: ValueWrapper(
+          originalEngagement.comment!.copyWith(
+            status: ModerationStatus.resolved,
+          ),
         ),
       );
 
@@ -334,21 +336,9 @@ class CommunityManagementBloc
         (e) => e.id == event.engagementId,
       );
 
-      // TODO(fulleni): Fix the following BUG
-      // The original Engagement.copyWith method has a bug where
-      // `comment: comment ?? this.comment` prevents setting the comment to null.
-      // To work around this without modifying the core model, we must create a
-      // new instance of Engagement manually, copying all properties and
-      // explicitly setting the comment to null.
-      final updatedEngagement = Engagement(
-        id: originalEngagement.id,
-        userId: originalEngagement.userId,
-        entityId: originalEngagement.entityId,
-        entityType: originalEngagement.entityType,
-        reaction: originalEngagement.reaction,
-        createdAt: originalEngagement.createdAt,
-        updatedAt: DateTime.now(),
-        comment: null, // Explicitly set to null to reject/delete the comment.
+      final updatedEngagement = originalEngagement.copyWith(
+        // Use ValueWrapper to explicitly set the comment to null.
+        comment: const ValueWrapper(null),
       );
 
       final updatedEngagements = List<Engagement>.from(state.engagements)

pubspec.lock

@@ -89,8 +89,8 @@ packages:
     dependency: "direct main"
     description:
       path: "."
-      ref: "8bae6eb17369b76f72961870a22ee13d3073fa61"
-      resolved-ref: "8bae6eb17369b76f72961870a22ee13d3073fa61"
+      ref: b3bf1fbe84c51006c55c0eed7fb5de2f11a95208
+      resolved-ref: b3bf1fbe84c51006c55c0eed7fb5de2f11a95208
       url: "https://github.com/flutter-news-app-full-source-code/core.git"
     source: git
     version: "1.3.1"

pubspec.yaml

@@ -94,7 +94,7 @@ dependency_overrides:
   core:
     git:
       url: https://github.com/flutter-news-app-full-source-code/core.git
-      ref: 8bae6eb17369b76f72961870a22ee13d3073fa61
+      ref: b3bf1fbe84c51006c55c0eed7fb5de2f11a95208
   http_client:
     git:
       url: https://github.com/flutter-news-app-full-source-code/http-client.git