From afceefa24d310e3e9ab716e353ffd5c3e5688c16 Mon Sep 17 00:00:00 2001
From: Mike Decker <pookmish@gmail.com>
Date: Fri, 2 Aug 2024 05:28:35 -0700
Subject: [PATCH] Use "includes" instead of "indexOf" for strings

---
 src/components/elements/link.tsx      | 8 +++++++-
 src/components/elements/slideshow.tsx | 4 ++--
 src/components/elements/wysiwyg.tsx   | 4 ++--
 3 files changed, 11 insertions(+), 5 deletions(-)

diff --git a/src/components/elements/link.tsx b/src/components/elements/link.tsx
index 100abdba..35e3fc49 100644
--- a/src/components/elements/link.tsx
+++ b/src/components/elements/link.tsx
@@ -18,7 +18,13 @@ const DrupalLink = ({href, className, children, ...props}: Props) => {
   href = href || "#"
   const drupalBase: string = (process.env.NEXT_PUBLIC_DRUPAL_BASE_URL || "").replace(/\/$/, "")
 
-  if (!href.indexOf("/files/")) {
+  // Make sure links to documents or images go to the Drupal origin.
+  if (href.startsWith("/") && href.includes("/files/")) {
+    href = `${drupalBase}${href}`
+  }
+
+  // For links not to the file system, make them relative and replace <front>.
+  if (!href.includes("/files/")) {
     href = href.replace(drupalBase, "").replace("<front>", "/")
   }
 
diff --git a/src/components/elements/slideshow.tsx b/src/components/elements/slideshow.tsx
index fd495b8d..b8cd1c47 100644
--- a/src/components/elements/slideshow.tsx
+++ b/src/components/elements/slideshow.tsx
@@ -7,7 +7,7 @@ import {twMerge} from "tailwind-merge"
 import {clsx} from "clsx"
 
 const NextArrow = ({className, onClick}: CustomArrowProps) => {
-  const slickDisabled = !!(className && className?.indexOf("slick-disabled") > 0)
+  const slickDisabled = className?.includes("slick-disabled")
   return (
     <button className="absolute right-1 top-1/3 z-50" onClick={onClick} aria-label="Next" disabled={slickDisabled}>
       <ArrowRightIcon
@@ -18,7 +18,7 @@ const NextArrow = ({className, onClick}: CustomArrowProps) => {
 }
 
 const PrevArrow = ({className, onClick}: CustomArrowProps) => {
-  const slickDisabled = !!(className && className?.indexOf("slick-disabled") > 0)
+  const slickDisabled = className?.includes("slick-disabled")
   return (
     <button className="absolute left-1 top-1/3 z-50" onClick={onClick} aria-label="Previous" disabled={slickDisabled}>
       <ArrowLeftIcon
diff --git a/src/components/elements/wysiwyg.tsx b/src/components/elements/wysiwyg.tsx
index 397b1643..136fccfd 100644
--- a/src/components/elements/wysiwyg.tsx
+++ b/src/components/elements/wysiwyg.tsx
@@ -51,7 +51,7 @@ const options: HTMLReactParserOptions = {
 
         case "div":
           delete nodeProps.role
-          if (nodeProps.className && !!nodeProps.className.indexOf("media-entity-wrapper")) {
+          if (nodeProps.className?.includes("media-entity-wrapper")) {
             return cleanMediaMarkup(domNode)
           }
           return <NodeName {...nodeProps}>{domToReact(children, options)}</NodeName>
@@ -216,7 +216,7 @@ const cleanMediaMarkup = (node: Element) => {
 
     if (figCaption) {
       nodeProps.className = twMerge("table", nodeProps.className)
-      if (!!nodeProps.className?.indexOf("mx-auto")) nodeProps.className += " w-full"
+      if (nodeProps.className?.includes("mx-auto")) nodeProps.className += " w-full"
       delete nodeProps.role
       return (
         <figure {...nodeProps}>