From 9ddae473484d601e4d789237525b2717df35fdc0 Mon Sep 17 00:00:00 2001
From: nsantacruz <noahssantacruz@gmail.com>
Date: Thu, 10 Aug 2023 09:01:25 +0300
Subject: [PATCH] fix: use isFullSegmentImage instead of isImage for
 AddToSourceSheet because we only want to prohibit full segment images.

---
 static/js/AddToSourceSheet.jsx | 2 +-
 static/js/sefaria/sefaria.js   | 4 ----
 2 files changed, 1 insertion(+), 5 deletions(-)

diff --git a/static/js/AddToSourceSheet.jsx b/static/js/AddToSourceSheet.jsx
index fb308296e3..cd4440f6b0 100644
--- a/static/js/AddToSourceSheet.jsx
+++ b/static/js/AddToSourceSheet.jsx
@@ -132,7 +132,7 @@ class AddToSourceSheetBox extends Component {
     // validate texts corresponding to refs have no images before posting them to sheet
     for (let i = 0; i < refs.length; i++) {
       let ref = Sefaria.getRefFromCache(refs[i]);
-      if (ref && (Sefaria.isImage(ref.he) || Sefaria.isImage(ref.text))) {
+      if (ref && (Sefaria.isFullSegmentImage(ref.he) || Sefaria.isFullSegmentImage(ref.text))) {
         alert("We do not currently support adding images to source sheets.");
         return false;
       }
diff --git a/static/js/sefaria/sefaria.js b/static/js/sefaria/sefaria.js
index 25d5f10300..9698478df8 100644
--- a/static/js/sefaria/sefaria.js
+++ b/static/js/sefaria/sefaria.js
@@ -1013,10 +1013,6 @@ Sefaria = extend(Sefaria, {
     const pattern = /^\s*<img\b[^>]*>\s*$/i;
     return pattern.test(text);
   },
-  isImage: function(textChunk) {
-    const pattern = /<img\b[^>]*>/i;
-    return pattern.test(textChunk);
-  },
   getRefFromCache: function(ref) {
     if (!ref) return null;
     const versionedKey = this._refmap[this._refKey(ref)] || this._refmap[this._refKey(ref, {context:1})];