Revert "refactor: Avoid unnecessary bool comparison"

This reverts commit 1cd3a91037.

analysis_options.yaml

@@ -37,7 +37,6 @@ dart_code_linter:
     - avoid-unnecessary-type-assertions
     - avoid-unnecessary-type-casts
     - avoid-unrelated-type-assertions
-    - no-boolean-literal-compare
     - no-equal-then-else
     - prefer-first
     - prefer-last

lib/pages/bootstrap/bootstrap_dialog.dart

@@ -161,7 +161,8 @@ class BootstrapDialogState extends State<BootstrapDialog> {
     Widget body = const Center(child: CircularProgressIndicator.adaptive());
     titleText = L10n.of(context).loadingPleaseWait;
 
-    if (bootstrap.newSsssKey?.recoveryKey != null && _recoveryKeyStored) {
+    if (bootstrap.newSsssKey?.recoveryKey != null &&
+        _recoveryKeyStored == false) {
       final key = bootstrap.newSsssKey!.recoveryKey;
       titleText = L10n.of(context).recoveryKey;
       return LoginScaffold(

lib/pages/chat/chat.dart

@@ -249,9 +249,9 @@ class ChatController extends State<ChatPageWithRoom>
     }
     if (!scrollController.hasClients) return;
     if (timeline?.allowNewEvent == false ||
-        scrollController.position.pixels > 0 && _scrolledUp) {
+        scrollController.position.pixels > 0 && _scrolledUp == false) {
       setState(() => _scrolledUp = true);
-    } else if (scrollController.position.pixels <= 0 && _scrolledUp) {
+    } else if (scrollController.position.pixels <= 0 && _scrolledUp == true) {
       setState(() => _scrolledUp = false);
       setReadMarker();
     }
@@ -919,7 +919,7 @@ class ChatController extends State<ChatPageWithRoom>
     final clients = Matrix.of(context).currentBundle;
     for (final event in selectedEvents) {
       if (!event.status.isSent) return false;
-      if (event.canRedact &&
+      if (event.canRedact == false &&
           !(clients!.any((cl) => event.senderId == cl!.userID))) {
         return false;
       }

lib/pages/chat/chat_input_row.dart

@@ -317,7 +317,7 @@ class ChatInputRow extends StatelessWidget {
                         autofocus: !PlatformInfos.isMobile,
                         keyboardType: TextInputType.multiline,
                         textInputAction:
-                            AppSettings.sendOnEnter.value &&
+                            AppSettings.sendOnEnter.value == true &&
                                 PlatformInfos.isMobile
                             ? TextInputAction.send
                             : null,

lib/pages/chat/chat_view.dart

@@ -364,7 +364,7 @@ class ChatView extends StatelessWidget {
                                     ? theme.colorScheme.tertiaryContainer
                                     : theme.colorScheme.surfaceContainerHigh,
                                 borderRadius: BorderRadius.circular(32),
-                                child: controller.room.isAbandonedDMRoom
+                                child: controller.room.isAbandonedDMRoom == true
                                     ? Row(
                                         mainAxisAlignment: .spaceEvenly,
                                         children: [

lib/pages/chat/events/audio_player.dart

@@ -179,7 +179,7 @@ class AudioPlayerState extends State<AudioPlayerWidget> {
             matrixFile.mimeType.toLowerCase() == 'audio/ogg') {
           Logs().v('Convert ogg audio file for iOS...');
           final convertedFile = File('${file.path}.caf');
-          if (await convertedFile.exists()) {
+          if (await convertedFile.exists() == false) {
             OpusCaf().convertOpusToCaf(file.path, convertedFile.path);
           }
           file = convertedFile;

lib/pages/chat/recording_view_model.dart

@@ -55,7 +55,7 @@ class RecordingViewModelState extends State<RecordingViewModel> {
         return;
       }
     }
-    if (await AudioRecorder().hasPermission()) return;
+    if (await AudioRecorder().hasPermission() == false) return;
 
     final audioRecorder = _audioRecorder ??= AudioRecorder();
     setState(() {});
@@ -76,7 +76,7 @@ class RecordingViewModelState extends State<RecordingViewModel> {
       }
 
       final result = await audioRecorder.hasPermission();
-      if (!result) {
+      if (result != true) {
         showOkAlertDialog(
           context: context,
           title: L10n.of(context).oopsSomethingWentWrong,

lib/pages/chat_access_settings/chat_access_settings_controller.dart

@@ -320,7 +320,7 @@ class ChatAccessSettingsController extends State<ChatAccessSettings> {
     try {
       await room.client.setRoomVisibilityOnDirectory(
         room.id,
-        visibility: visibility ? Visibility.public : Visibility.private,
+        visibility: visibility == true ? Visibility.public : Visibility.private,
       );
       setState(() {});
     } catch (e, s) {

lib/pages/chat_list/chat_list.dart

@@ -168,8 +168,9 @@ class ChatListController extends State<ChatList>
       initialText: searchServer,
       keyboardType: TextInputType.url,
       autocorrect: false,
-      validator: (server) =>
+      validator: (server) => server.contains('.') == true
-          server.contains('.') ? null : L10n.of(context).invalidServerName,
+          ? null
+          : L10n.of(context).invalidServerName,
     );
     if (newServer == null) return;
     Matrix.of(context).store.setString(_serverStoreNamespace, newServer);
@@ -202,9 +203,10 @@ class ChatListController extends State<ChatList>
 
       if (searchQuery.isValidMatrixId &&
           searchQuery.sigil == '#' &&
-          !roomSearchResult.chunk.any(
+          roomSearchResult.chunk.any(
                 (room) => room.canonicalAlias == searchQuery,
-          )) {
+              ) ==
+              false) {
         final response = await client.getRoomIdByAlias(searchQuery);
         final roomId = response.roomId;
         if (roomId != null) {

lib/pages/chat_list/status_msg_list.dart

@@ -314,7 +314,7 @@ extension on CachedPresence {
           ? DateTime.now()
           : DateTime.fromMillisecondsSinceEpoch(0));
 
-  LinearGradient get gradient => presence.isOnline
+  LinearGradient get gradient => presence.isOnline == true
       ? LinearGradient(
           colors: [Colors.green, Colors.green.shade200, Colors.green.shade900],
           begin: Alignment.topLeft,

lib/pages/image_viewer/image_viewer.dart

@@ -110,7 +110,7 @@ class ImageViewerController extends State<ImageViewer> {
 
   /// Go back if user swiped it away
   void onInteractionEnds(ScaleEndDetails endDetails) {
-    if (!PlatformInfos.usesTouchscreen) {
+    if (PlatformInfos.usesTouchscreen == false) {
       if (endDetails.velocity.pixelsPerSecond.dy >
           MediaQuery.sizeOf(context).height * maxScaleFactor) {
         Navigator.of(context, rootNavigator: false).pop();

lib/pages/image_viewer/video_player.dart

@@ -73,7 +73,7 @@ class EventVideoPlayerState extends State<EventVideoPlayer> {
           widget.event.attachmentOrThumbnailMxcUrl()!.pathSegments.last,
         );
         final file = File('${tempDir.path}/${fileName}_${videoFile.name}');
-        if (!await file.exists()) {
+        if (await file.exists() == false) {
           await file.writeAsBytes(videoFile.bytes);
         }
         videoPlayerController = VideoPlayerController.file(file);

lib/utils/matrix_sdk_extensions/flutter_matrix_dart_sdk_database/cipher.dart

@@ -50,7 +50,7 @@ Future<String?> getDatabaseCipher() async {
 void _sendNoEncryptionWarning(Object exception) async {
   final isStored = AppSettings.noEncryptionWarningShown.value;
 
-  if (isStored) return;
+  if (isStored == true) return;
 
   final l10n = await lookupL10n(PlatformDispatcher.instance.locale);
   ClientManager.sendInitNotification(

lib/utils/voip_plugin.dart

@@ -117,7 +117,7 @@ class VoipPlugin with WidgetsBindingObserver implements WebRTCDelegate {
 
         await matrix.store.setString(
           'wasForeground',
-          wasForeground ? 'true' : 'false',
+          wasForeground == true ? 'true' : 'false',
         );
         FlutterForegroundTask.setOnLockScreenVisibility(true);
         FlutterForegroundTask.wakeUpScreen();