From 150989d3ddc37f98b96c9c360041a3a112df0cfb Mon Sep 17 00:00:00 2001
From: CatHood0 <santiagowmar@gmail.com>
Date: Fri, 5 Jul 2024 21:01:36 -0400
Subject: [PATCH] fix: common link is detected as a video link

---
 lib/src/utils/delta_x_utils.dart |  8 ++++----
 test/utils/delta_x_test.dart     | 31 +++++++++++++++++++++----------
 2 files changed, 25 insertions(+), 14 deletions(-)

diff --git a/lib/src/utils/delta_x_utils.dart b/lib/src/utils/delta_x_utils.dart
index b7767b1e..497fa29b 100644
--- a/lib/src/utils/delta_x_utils.dart
+++ b/lib/src/utils/delta_x_utils.dart
@@ -20,7 +20,7 @@ class UnderlineSyntax extends md.DelimiterSyntax {
 class VideoSyntax extends md.LinkSyntax {
   VideoSyntax({super.linkResolver})
       : super(
-          pattern: r'\[',
+          pattern: r'\[!\[',
           startCharacter: _$lbracket,
         );
 
@@ -60,13 +60,13 @@ final videoRule = hmd.Rule('video', filters: ['iframe', 'video'],
       if (!_youtubeVideoUrlValidator.hasMatch(src ?? '')) {
         return '<video>${child.outerHTML}</video>';
       }
-      return '[$content]($src)';
+      return '[![$content]($src)';
     }
     final src = node.getAttribute('src');
     if (src == null || !_youtubeVideoUrlValidator.hasMatch(src)) {
       return node.outerHTML;
     }
-    return '[$content]($src)';
+    return '[![$content]($src)';
   }
   //by now, we can only access to src
   final src = node.getAttribute('src');
@@ -76,5 +76,5 @@ final videoRule = hmd.Rule('video', filters: ['iframe', 'video'],
     return node.outerHTML;
   }
   final title = node.getAttribute('title');
-  return '[$title]($src)';
+  return '[![$title]($src)';
 });
diff --git a/test/utils/delta_x_test.dart b/test/utils/delta_x_test.dart
index 489a7493..cccb9612 100644
--- a/test/utils/delta_x_test.dart
+++ b/test/utils/delta_x_test.dart
@@ -3,28 +3,27 @@ import 'package:flutter_quill/src/models/documents/delta_x.dart';
 import 'package:test/test.dart';
 
 void main() {
-  const htmlWithEmp =
-      '<p>This is a normal sentence, and this section has greater emp<em>hasis.</em></p>';
+  const htmlWithEmp = '<p>This is a normal sentence, and this section has greater emp<em>hasis.</em></p>';
 
-  const htmlWithUnderline =
-      '<p>This is a normal sentence, and this section has greater <u>underline</u>';
+  const htmlWithUnderline = '<p>This is a normal sentence, and this section has greater <u>underline</u>';
 
   const htmlWithIframeVideo =
       '<iframe src="https://www.youtube.com/embed/dQw4w9WgXcQ" title="YouTube video player"></iframe>';
 
   const htmlWithVideoTag =
-      '''<video src="https://www.youtube.com/embed/dQw4w9WgXcQ">Your browser does not support the video tag.</video> 
-''';
+      '<video src="https://www.youtube.com/embed/dQw4w9WgXcQ">Your browser does not support the video tag.</video>';
+
+  const htmlWithNormalLinkAndVideo =
+      '<a href="https://www.macrumors.com/" type="text/html">fdsfsd</a><br><video src="https://www.youtube.com/embed/dQw4w9WgXcQ">Your browser does not support the video tag.</video>';
+
   final expectedDeltaEmp = Delta.fromOperations([
-    Operation.insert(
-        'This is a normal sentence, and this section has greater emp'),
+    Operation.insert('This is a normal sentence, and this section has greater emp'),
     Operation.insert('hasis.', {'italic': true}),
     Operation.insert('\n'),
   ]);
 
   final expectedDeltaUnderline = Delta.fromOperations([
-    Operation.insert(
-        'This is a normal sentence, and this section has greater '),
+    Operation.insert('This is a normal sentence, and this section has greater '),
     Operation.insert('underline', {'underline': true}),
     Operation.insert('\n'),
   ]);
@@ -34,6 +33,13 @@ void main() {
     Operation.insert('\n'),
   ]);
 
+  final expectedDeltaLinkAndVideoLink = Delta.fromOperations([
+    Operation.insert('fdsfsd', {'link': 'https://www.macrumors.com/'}),
+    Operation.insert('\n\n'),
+    Operation.insert({'video': 'https://www.youtube.com/embed/dQw4w9WgXcQ'}),
+    Operation.insert('\n'),
+  ]);
+
   test('should detect emphasis and parse correctly', () {
     final delta = DeltaX.fromHtml(htmlWithEmp);
     expect(delta, expectedDeltaEmp);
@@ -53,4 +59,9 @@ void main() {
     final delta = DeltaX.fromHtml(htmlWithVideoTag);
     expect(delta, expectedDeltaVideo);
   });
+
+  test('should detect by different way normal link and video link', () {
+    final delta = DeltaX.fromHtml(htmlWithNormalLinkAndVideo);
+    expect(delta, expectedDeltaLinkAndVideoLink);
+  });
 }