TF-3243 Fix email without content

core/lib/presentation/utils/html_transformer/dom/add_lazy_loading_for_background_image_transformers.dart

@@ -1,5 +1,6 @@
 import 'package:core/data/network/dio_client.dart';
 import 'package:core/presentation/utils/html_transformer/base/dom_transformer.dart';
+import 'package:core/utils/app_logger.dart';
 import 'package:html/dom.dart';
 
 class AddLazyLoadingForBackgroundImageTransformer extends DomTransformer {
@@ -11,7 +12,11 @@ class AddLazyLoadingForBackgroundImageTransformer extends DomTransformer {
     required DioClient dioClient,
     Map<String, String>? mapUrlDownloadCID,
   }) async {
+    try {
       final elements = document.querySelectorAll('[style*="background-image"]');
+
+      if (elements.isEmpty) return;
+
       await Future.wait(elements.map((element) async {
         var exStyle = element.attributes['style'];
         final imageUrls = findImageUrlFromStyleTag(exStyle!);
@@ -22,5 +27,8 @@ class AddLazyLoadingForBackgroundImageTransformer extends DomTransformer {
           element.attributes.addAll({'lazy': ''});
         }
       }));
+    } catch (e) {
+      logError('$runtimeType::process:Exception = $e');
+    }
   }
 }

core/lib/presentation/utils/html_transformer/dom/block_code_transformers.dart

@@ -1,6 +1,7 @@
 
 import 'package:core/data/network/dio_client.dart';
 import 'package:core/presentation/utils/html_transformer/base/dom_transformer.dart';
+import 'package:core/utils/app_logger.dart';
 import 'package:html/dom.dart';
 
 class BlockCodeTransformer extends DomTransformer {
@@ -13,7 +14,11 @@ class BlockCodeTransformer extends DomTransformer {
     required DioClient dioClient,
     Map<String, String>? mapUrlDownloadCID,
   }) async {
+    try {
       final codeElements = document.getElementsByTagName('pre');
+
+      if (codeElements.isEmpty) return;
+
       await Future.wait(codeElements.map((element) async {
         element.attributes['style'] = '''
           display: block;
@@ -30,5 +35,8 @@ class BlockCodeTransformer extends DomTransformer {
           overflow: auto;
         ''';
       }));
+    } catch (e) {
+      logError('$runtimeType::process:Exception = $e');
+    }
   }
 }

core/lib/presentation/utils/html_transformer/dom/block_quoted_transformers.dart

@@ -1,6 +1,7 @@
 
 import 'package:core/data/network/dio_client.dart';
 import 'package:core/presentation/utils/html_transformer/base/dom_transformer.dart';
+import 'package:core/utils/app_logger.dart';
 import 'package:html/dom.dart';
 
 class BlockQuotedTransformer extends DomTransformer {
@@ -13,7 +14,11 @@ class BlockQuotedTransformer extends DomTransformer {
     required DioClient dioClient,
     Map<String, String>? mapUrlDownloadCID,
   }) async {
+    try {
       final quotedElements = document.getElementsByTagName('blockquote');
+
+      if (quotedElements.isEmpty) return;
+
       await Future.wait(quotedElements.map((quotedElement) async {
         quotedElement.attributes['style'] = '''
           margin-left: 4px;
@@ -23,5 +28,8 @@ class BlockQuotedTransformer extends DomTransformer {
           border-left: 2px solid #eee;
         ''';
       }));
+    } catch (e) {
+      logError('$runtimeType::process:Exception = $e');
+    }
   }
 }

core/lib/presentation/utils/html_transformer/dom/hide_draft_signature_transformer.dart

@@ -1,5 +1,6 @@
 import 'package:core/data/network/dio_client.dart';
 import 'package:core/presentation/utils/html_transformer/base/dom_transformer.dart';
+import 'package:core/utils/app_logger.dart';
 import 'package:html/dom.dart';
 
 class HideDraftSignatureTransformer extends DomTransformer {
@@ -12,6 +13,7 @@ class HideDraftSignatureTransformer extends DomTransformer {
     required DioClient dioClient,
     Map<String, String>? mapUrlDownloadCID
   }) async {
+    try {
       final signature = document.querySelector('div.tmail-signature');
       if (signature == null) return;
       final currentStyle = signature.attributes['style']?.trim();
@@ -22,5 +24,8 @@ class HideDraftSignatureTransformer extends DomTransformer {
       } else {
         signature.attributes['style'] = '$currentStyle; display: none;';
       }
+    } catch (e) {
+      logError('$runtimeType::process:Exception = $e');
+    }
   }
 }

core/lib/presentation/utils/html_transformer/dom/image_transformers.dart

@@ -5,6 +5,7 @@ import 'package:core/data/network/dio_client.dart';
 import 'package:core/data/utils/compress_file_utils.dart';
 import 'package:core/presentation/extensions/html_extension.dart';
 import 'package:core/presentation/utils/html_transformer/base/dom_transformer.dart';
+import 'package:core/utils/app_logger.dart';
 import 'package:core/utils/platform_info.dart';
 import 'package:dio/dio.dart';
 import 'package:flutter/foundation.dart';
@@ -20,7 +21,11 @@ class ImageTransformer extends DomTransformer {
     required DioClient dioClient,
     Map<String, String>? mapUrlDownloadCID,
   }) async {
+    try {
       final imageElements = document.querySelectorAll('img');
+
+      if (imageElements.isEmpty) return;
+
       await Future.wait(imageElements.map((imageElement) async {
         var exStyle = imageElement.attributes['style'];
         if (exStyle != null) {
@@ -55,6 +60,9 @@ class ImageTransformer extends DomTransformer {
           }
         }
       }));
+    } catch (e) {
+      logError('$runtimeType::process:Exception = $e');
+    }
   }
 
   Future<String?> _convertCidToBase64Image({

core/lib/presentation/utils/html_transformer/dom/remove_collapsed_signature_button_transformers.dart

@@ -1,6 +1,7 @@
 
 import 'package:core/data/network/dio_client.dart';
 import 'package:core/presentation/utils/html_transformer/base/dom_transformer.dart';
+import 'package:core/utils/app_logger.dart';
 import 'package:html/dom.dart';
 
 class RemoveCollapsedSignatureButtonTransformer extends DomTransformer {
@@ -13,9 +14,16 @@ class RemoveCollapsedSignatureButtonTransformer extends DomTransformer {
     required DioClient dioClient,
     Map<String, String>? mapUrlDownloadCID,
   }) async {
+    try {
       final elements = document.querySelectorAll('.tmail-signature-button');
+
+      if (elements.isEmpty) return;
+
       await Future.wait(elements.map((element) async {
         element.remove();
       }));
+    } catch (e) {
+      logError('$runtimeType::process:Exception = $e');
+    }
   }
 }

core/lib/presentation/utils/html_transformer/dom/remove_lazy_loading_for_background_image_transformers.dart

@@ -1,5 +1,6 @@
 import 'package:core/data/network/dio_client.dart';
 import 'package:core/presentation/utils/html_transformer/base/dom_transformer.dart';
+import 'package:core/utils/app_logger.dart';
 import 'package:html/dom.dart';
 
 class RemoveLazyLoadingForBackgroundImageTransformer extends DomTransformer {
@@ -11,7 +12,11 @@ class RemoveLazyLoadingForBackgroundImageTransformer extends DomTransformer {
     required DioClient dioClient,
     Map<String, String>? mapUrlDownloadCID,
   }) async {
+    try {
       final elements = document.querySelectorAll('[lazy]');
+
+      if (elements.isEmpty) return;
+
       await Future.wait(elements.map((element) async {
         var exStyle = element.attributes['style'];
         final dataSrc = element.attributes['data-src'];
@@ -25,5 +30,8 @@ class RemoveLazyLoadingForBackgroundImageTransformer extends DomTransformer {
         element.attributes.remove('data-src');
         element.attributes.remove('lazy');
       }));
+    } catch (e) {
+      logError('$runtimeType::process:Exception = $e');
+    }
   }
 }

core/lib/presentation/utils/html_transformer/dom/remove_lazy_loading_image_transformers.dart

@@ -1,5 +1,6 @@
 import 'package:core/data/network/dio_client.dart';
 import 'package:core/presentation/utils/html_transformer/base/dom_transformer.dart';
+import 'package:core/utils/app_logger.dart';
 import 'package:html/dom.dart';
 
 class RemoveLazyLoadingImageTransformer extends DomTransformer {
@@ -11,9 +12,16 @@ class RemoveLazyLoadingImageTransformer extends DomTransformer {
     required DioClient dioClient,
     Map<String, String>? mapUrlDownloadCID,
   }) async {
+    try {
       final elements = document.querySelectorAll('img[loading]');
+
+      if (elements.isEmpty) return;
+
       await Future.wait(elements.map((element) async {
         element.attributes.remove('loading');
       }));
+    } catch (e) {
+      logError('$runtimeType::process:Exception = $e');
+    }
   }
 }

core/lib/presentation/utils/html_transformer/dom/remove_max_width_in_image_style_transformers.dart

@@ -1,5 +1,6 @@
 import 'package:core/data/network/dio_client.dart';
 import 'package:core/presentation/utils/html_transformer/base/dom_transformer.dart';
+import 'package:core/utils/app_logger.dart';
 import 'package:html/dom.dart';
 
 class RemoveMaxWidthInImageStyleTransformer extends DomTransformer {
@@ -12,7 +13,11 @@ class RemoveMaxWidthInImageStyleTransformer extends DomTransformer {
     required DioClient dioClient,
     Map<String, String>? mapUrlDownloadCID,
   }) async {
+    try {
       final imageElements = document.querySelectorAll('img[style*="max-width"]');
+
+      if (imageElements.isEmpty) return;
+
       await Future.wait(imageElements.map((imageElement) async {
         var exStyle = imageElement.attributes['style'];
         if (exStyle != null) {
@@ -20,5 +25,8 @@ class RemoveMaxWidthInImageStyleTransformer extends DomTransformer {
           imageElement.attributes['style'] = exStyle;
         }
       }));
+    } catch (e) {
+      logError('$runtimeType::process:Exception = $e');
+    }
   }
 }

core/lib/presentation/utils/html_transformer/dom/remove_style_tag_outside_transformers.dart

@@ -1,5 +1,6 @@
 import 'package:core/data/network/dio_client.dart';
 import 'package:core/presentation/utils/html_transformer/base/dom_transformer.dart';
+import 'package:core/utils/app_logger.dart';
 import 'package:html/dom.dart';
 
 class RemoveStyleTagOutsideTransformer extends DomTransformer {
@@ -12,9 +13,16 @@ class RemoveStyleTagOutsideTransformer extends DomTransformer {
     required DioClient dioClient,
     Map<String, String>? mapUrlDownloadCID,
   }) async {
+    try {
       final styleElements = document.querySelectorAll('style');
+
+      if (styleElements.isEmpty) return;
+
       await Future.wait(styleElements.map((element) async {
         element.remove();
       }));
+    } catch (e) {
+      logError('$runtimeType::process:Exception = $e');
+    }
   }
 }

core/lib/presentation/utils/html_transformer/dom/remove_tooltip_link_transformers.dart

@@ -1,6 +1,7 @@
 
 import 'package:core/data/network/dio_client.dart';
 import 'package:core/presentation/utils/html_transformer/base/dom_transformer.dart';
+import 'package:core/utils/app_logger.dart';
 import 'package:core/utils/html/html_template.dart';
 import 'package:html/dom.dart';
 
@@ -14,7 +15,11 @@ class RemoveTooltipLinkTransformer extends DomTransformer {
     required DioClient dioClient,
     Map<String, String>? mapUrlDownloadCID,
   }) async {
+    try {
       final linkElements = document.querySelectorAll('a.${HtmlTemplate.nameClassToolTip}');
+
+      if (linkElements.isEmpty) return;
+
       await Future.wait(linkElements.map((linkElement) async {
         final classAttribute = linkElement.attributes['class'];
         if (classAttribute != null) {
@@ -28,6 +33,9 @@ class RemoveTooltipLinkTransformer extends DomTransformer {
           }
         }
       }));
+    } catch (e) {
+      logError('$runtimeType::process:Exception = $e');
+    }
   }
 
 }

core/lib/presentation/utils/html_transformer/dom/sanitize_hyper_link_tag_in_html_transformers.dart

@@ -1,6 +1,7 @@
 import 'package:core/data/network/dio_client.dart';
 import 'package:core/presentation/utils/html_transformer/base/dom_transformer.dart';
 import 'package:core/presentation/utils/html_transformer/sanitize_url.dart';
+import 'package:core/utils/app_logger.dart';
 import 'package:core/utils/html/html_template.dart';
 import 'package:html/dom.dart';
 
@@ -16,7 +17,11 @@ class SanitizeHyperLinkTagInHtmlTransformer extends DomTransformer {
     required DioClient dioClient,
     Map<String, String>? mapUrlDownloadCID,
   }) async {
+    try {
       final elements = document.querySelectorAll('a');
+
+      if (elements.isEmpty) return;
+
       await Future.wait(elements.map((element) async {
         _sanitizeUrlResource(element);
         if (useTooltip) {
@@ -25,6 +30,9 @@ class SanitizeHyperLinkTagInHtmlTransformer extends DomTransformer {
         _addBlankForTargetProperty(element);
         _addNoReferrerForRelProperty(element);
       }));
+    } catch (e) {
+      logError('$runtimeType::process:Exception = $e');
+    }
   }
 
   void _sanitizeUrlResource(Element element) {

core/lib/presentation/utils/html_transformer/dom/script_transformers.dart

@@ -1,5 +1,6 @@
 
 import 'package:core/data/network/dio_client.dart';
+import 'package:core/utils/app_logger.dart';
 import 'package:html/dom.dart';
 import 'package:core/presentation/utils/html_transformer/base/dom_transformer.dart';
 
@@ -13,9 +14,16 @@ class RemoveScriptTransformer extends DomTransformer {
     required DioClient dioClient,
     Map<String, String>? mapUrlDownloadCID,
   }) async {
+    try {
       final scriptElements = document.getElementsByTagName('script');
+
+      if (scriptElements.isEmpty) return;
+
       await Future.wait(scriptElements.map((scriptElement) async {
         scriptElement.remove();
       }));
+    } catch (e) {
+      logError('$runtimeType::process:Exception = $e');
+    }
   }
 }

core/lib/presentation/utils/html_transformer/dom/signature_transformers.dart

@@ -1,6 +1,7 @@
 
 import 'package:core/data/network/dio_client.dart';
 import 'package:core/presentation/utils/html_transformer/base/dom_transformer.dart';
+import 'package:core/utils/app_logger.dart';
 import 'package:html/dom.dart';
 
 class SignatureTransformer extends DomTransformer {
@@ -13,9 +14,16 @@ class SignatureTransformer extends DomTransformer {
     required DioClient dioClient,
     Map<String, String>? mapUrlDownloadCID,
   }) async {
+    try {
       final signatureElements = document.querySelectorAll('div.tmail-signature');
+
+      if (signatureElements.isEmpty) return;
+
       await Future.wait(signatureElements.map((element) async {
         element.attributes['class'] = 'tmail-signature-blocked';
       }));
+    } catch (e) {
+      logError('$runtimeType::process:Exception = $e');
+    }
   }
 }

core/lib/presentation/utils/html_transformer/sanitize_autolink_filter.dart

@@ -21,6 +21,7 @@ class SanitizeAutolinkFilter {
   SanitizeAutolinkFilter(this.htmlEscape);
 
   String process(String inputText) {
+    try {
       if (inputText.isEmpty) {
         return '';
       }
@@ -28,7 +29,7 @@ class SanitizeAutolinkFilter {
       final elements = linkify(
         inputText,
         options: _linkifyOption,
-      linkifiers: _linkifier
+        linkifiers: _linkifier,
       );
       log('AutolinkFilter::process:elements: $elements');
       final htmlTextBuffer = StringBuffer();
@@ -53,6 +54,10 @@ class SanitizeAutolinkFilter {
       }
 
       return htmlTextBuffer.toString();
+    } catch (e) {
+      logError('$runtimeType::process:Exception = $e');
+      return inputText;
+    }
   }
 
   String _buildUrlLinkTag({required String urlLink, required String value}) {

core/lib/presentation/utils/html_transformer/sanitize_url.dart

@@ -1,3 +1,4 @@
+import 'package:core/utils/app_logger.dart';
 import 'package:get/get.dart';
 
 class SanitizeUrl {
@@ -12,17 +13,20 @@ class SanitizeUrl {
   );
 
   String process(String inputText) {
-    var originalUrl = inputText;
+    try {
-
+      log('SanitizeUrl::process:inputText = $inputText');
-    originalUrl = Uri.decodeFull(originalUrl);
+      var originalUrl = Uri.decodeFull(inputText);
-
       if (GetUtils.isURL(originalUrl)) {
-      if (!originalUrl.startsWith(_protocolIdentifierRegex)) {
+        originalUrl = !originalUrl.startsWith(_protocolIdentifierRegex)
-        originalUrl = (defaultToHttps ? "https://" : "http://") + originalUrl;
+          ? (defaultToHttps ? "https://" : "http://") + originalUrl
-      }
+          : originalUrl;
       } else {
         originalUrl = '';
       }
       return originalUrl;
+    } catch (e) {
+      logError('SanitizeUrl::process:Exception = $e');
+      return inputText;
+    }
   }
 }

core/test/sanitize_url_test.dart

@@ -108,5 +108,17 @@ void main() {
         equals('http://linagora.com')
       );
     });
+
+    test('SHOULD returns original input WHEN get an exception', () {
+      // Arrange
+      const inputText = "%E0%A4%A";
+      const expectedOutput = inputText;
+
+      // Act
+      final result = sanitizeUrl.process(inputText);
+
+      // Assert
+      expect(result, expectedOutput);
+    });
   });
 }