From 96864f737fc144c365c4132a4c5fae9b0c6c5254 Mon Sep 17 00:00:00 2001
From: brandonkelly <brandon@pixelandtonic.com>
Date: Fri, 28 Feb 2025 10:23:00 -0800
Subject: [PATCH] Cleanup

---
 src/blocktransformers/Blockquote.php   | 6 +++---
 src/blocktransformers/Button.php       | 8 ++++----
 src/blocktransformers/Code.php         | 6 +++---
 src/blocktransformers/Cover.php        | 6 +++---
 src/blocktransformers/DsbDetails.php   | 6 +++---
 src/blocktransformers/GBHeadline.php   | 2 +-
 src/blocktransformers/GBImage.php      | 8 ++++----
 src/blocktransformers/Heading.php      | 6 +++---
 src/blocktransformers/Preformatted.php | 6 +++---
 src/blocktransformers/PullQuote.php    | 6 +++---
 src/blocktransformers/Table.php        | 6 +++---
 11 files changed, 33 insertions(+), 33 deletions(-)

diff --git a/src/blocktransformers/Blockquote.php b/src/blocktransformers/Blockquote.php
index 70b4350..2a0f1c6 100644
--- a/src/blocktransformers/Blockquote.php
+++ b/src/blocktransformers/Blockquote.php
@@ -27,9 +27,9 @@ public function render(array $data, Entry $entry): string
         $content = $this->command->renderBlocks($data['innerBlocks'], $entry);
 
         // is there a <cite> tag in innerHTML?
-        $nodes = (new Crawler($data['innerHTML']))->filter('cite');
-        if ($nodes->count()) {
-            $content .= $nodes->outerHtml();
+        $node = (new Crawler($data['innerHTML']))->filter('cite');
+        if ($node->count()) {
+            $content .= $node->outerHtml();
         }
 
         return Html::tag('blockquote', $content);
diff --git a/src/blocktransformers/Button.php b/src/blocktransformers/Button.php
index 4b8108a..5bc3c47 100644
--- a/src/blocktransformers/Button.php
+++ b/src/blocktransformers/Button.php
@@ -24,12 +24,12 @@ public static function blockName(): string
 
     public function render(array $data, Entry $entry): string
     {
-        $nodes = (new Crawler($data['innerHTML']))->filter('a');
-        if (!$nodes->count()) {
+        $node = (new Crawler($data['innerHTML']))->filter('a');
+        if (!$node->count()) {
             return '';
         }
-        $label = $nodes->html();
-        $url = $nodes->first()->attr('href');
+        $label = $node->text();
+        $url = $node->attr('href');
 
         return $this->command->createNestedEntry($entry, function(Entry $nestedEntry) use ($label, $url) {
             $nestedEntry->setTypeId(ButtonEntryType::get()->id);
diff --git a/src/blocktransformers/Code.php b/src/blocktransformers/Code.php
index e7d187b..e579473 100644
--- a/src/blocktransformers/Code.php
+++ b/src/blocktransformers/Code.php
@@ -25,11 +25,11 @@ public static function blockName(): string
     public function render(array $data, Entry $entry): string
     {
         // `<pre class="wp-block-code"><code>` → `<pre><code>`
-        $nodes = (new Crawler($data['innerHTML']))->filter('code');
-        if (!$nodes->count()) {
+        $node = (new Crawler($data['innerHTML']))->filter('code');
+        if (!$node->count()) {
             return '';
         }
-        $code = $nodes->html();
+        $code = $node->html();
         return Html::beginTag('pre') .
             Html::tag('code', $code, [
                 'class' => sprintf('language-%s', $this->detectLanguage($code)),
diff --git a/src/blocktransformers/Cover.php b/src/blocktransformers/Cover.php
index 0c3a813..ec830ab 100644
--- a/src/blocktransformers/Cover.php
+++ b/src/blocktransformers/Cover.php
@@ -30,9 +30,9 @@ public function render(array $data, Entry $entry): string
         }
 
         if (isset($data['innerBlocks'][0]) && $data['innerBlocks'][0]['blockName'] === 'core/paragraph') {
-            $nodes = (new Crawler($data['innerBlocks'][0]['innerHTML']))->filter('p');
-            if ($nodes->count()) {
-                $entry->setFieldValue('coverText', $nodes->html());
+            $node = (new Crawler($data['innerBlocks'][0]['innerHTML']))->filter('p');
+            if ($node->count()) {
+                $entry->setFieldValue('coverText', $node->html());
             }
         }
 
diff --git a/src/blocktransformers/DsbDetails.php b/src/blocktransformers/DsbDetails.php
index d99f999..b33232f 100644
--- a/src/blocktransformers/DsbDetails.php
+++ b/src/blocktransformers/DsbDetails.php
@@ -28,9 +28,9 @@ public function render(array $data, Entry $entry): string
         return $this->command->createNestedEntry($entry, function(Entry $nestedEntry) use ($data) {
             $nestedEntry->setTypeId(DetailsEntryType::get()->id);
 
-            $summaryNodes = (new Crawler($data['innerHTML']))->filter('summary');
-            if ($summaryNodes->count()) {
-                $nestedEntry->setFieldValue(Summary::get()->handle, $summaryNodes->html());
+            $node = (new Crawler($data['innerHTML']))->filter('summary');
+            if ($node->count()) {
+                $nestedEntry->setFieldValue(Summary::get()->handle, $node->html());
             }
 
             // save it so we get an ID, before parsing the nested blocks
diff --git a/src/blocktransformers/GBHeadline.php b/src/blocktransformers/GBHeadline.php
index 81d51c8..05a9343 100644
--- a/src/blocktransformers/GBHeadline.php
+++ b/src/blocktransformers/GBHeadline.php
@@ -34,6 +34,6 @@ public function render(array $data, Entry $entry): string
             $tag = 'p';
         }
 
-        return HtmlHelper::tag($tag, $node->text());
+        return HtmlHelper::tag($tag, $node->html());
     }
 }
diff --git a/src/blocktransformers/GBImage.php b/src/blocktransformers/GBImage.php
index 0ece2e4..1b98ab6 100644
--- a/src/blocktransformers/GBImage.php
+++ b/src/blocktransformers/GBImage.php
@@ -30,16 +30,16 @@ public function render(array $data, Entry $entry): string
             return '';
         }
 
-        $img = (new Crawler($data['innerHTML']))->filter('img');
-        $src = $img->attr('src');
+        $node = (new Crawler($data['innerHTML']))->filter('img');
+        $src = $node->attr('src');
 
         try {
             if ($src) {
                 $assetId = $this->command->import(Media::SLUG, [
                     'id' => $id,
                     'source_url' => $src,
-                    'title' => $img->attr('title'),
-                    'alt_text' => $img->attr('alt'),
+                    'title' => $node->attr('title'),
+                    'alt_text' => $node->attr('alt'),
                 ]);
             } else {
                 $assetId = $this->command->import(Media::SLUG, $id);
diff --git a/src/blocktransformers/Heading.php b/src/blocktransformers/Heading.php
index bc98379..0639822 100644
--- a/src/blocktransformers/Heading.php
+++ b/src/blocktransformers/Heading.php
@@ -26,10 +26,10 @@ public function render(array $data, Entry $entry): string
     {
         // render a new heading tag without `class="wp-block-heading"`
         // (can't rely on `$data['attrs']['level']` unfortunately)
-        $nodes = (new Crawler($data['innerHTML']))->filter('h1,h2,h3,h4,h5,h6');
-        if (!$nodes->count()) {
+        $node = (new Crawler($data['innerHTML']))->filter('h1,h2,h3,h4,h5,h6');
+        if (!$node->count()) {
             return '';
         }
-        return Html::tag($nodes->first()->nodeName(), $nodes->html());
+        return Html::tag($node->nodeName(), $node->html());
     }
 }
diff --git a/src/blocktransformers/Preformatted.php b/src/blocktransformers/Preformatted.php
index 1015e9d..35f549e 100644
--- a/src/blocktransformers/Preformatted.php
+++ b/src/blocktransformers/Preformatted.php
@@ -25,12 +25,12 @@ public static function blockName(): string
     public function render(array $data, Entry $entry): string
     {
         // `<pre class="wp-block-preformatted">` → `<pre><code>`
-        $nodes = (new Crawler($data['innerHTML']))->filter('pre');
-        if (!$nodes->count()) {
+        $node = (new Crawler($data['innerHTML']))->filter('pre');
+        if (!$node->count()) {
             return '';
         }
         return Html::beginTag('pre') .
-            Html::tag('code', $nodes->html(), ['class' => 'language-plaintext']) .
+            Html::tag('code', $node->html(), ['class' => 'language-plaintext']) .
             Html::endTag('pre');
     }
 }
diff --git a/src/blocktransformers/PullQuote.php b/src/blocktransformers/PullQuote.php
index 4ed7d27..65f9ca1 100644
--- a/src/blocktransformers/PullQuote.php
+++ b/src/blocktransformers/PullQuote.php
@@ -25,15 +25,15 @@ public static function blockName(): string
     public function render(array $data, Entry $entry): string
     {
         $crawler = new Crawler($data['innerHTML']);
-        $paragraphNodes = $crawler->filter('p');
+        $pNodes = $crawler->filter('p');
         $citeNodes = $crawler->filter('cite');
 
-        if (!$paragraphNodes->count() && !$citeNodes->count()) {
+        if (!$pNodes->count() && !$citeNodes->count()) {
             return '';
         }
 
         $html = '';
-        $paragraphNodes->each(function(Crawler $node) use (&$html) {
+        $pNodes->each(function(Crawler $node) use (&$html) {
             $html .= $node->outerHtml();
         });
 
diff --git a/src/blocktransformers/Table.php b/src/blocktransformers/Table.php
index f82ba25..21cf521 100644
--- a/src/blocktransformers/Table.php
+++ b/src/blocktransformers/Table.php
@@ -25,10 +25,10 @@ public static function blockName(): string
     public function render(array $data, Entry $entry): string
     {
         // render a new <figure> with `class="table"`
-        $nodes = (new Crawler($data['innerHTML']))->filter('table');
-        if (!$nodes->count()) {
+        $node = (new Crawler($data['innerHTML']))->filter('table');
+        if (!$node->count()) {
             return '';
         }
-        return Html::tag('figure', $nodes->outerHtml(), ['class' => 'table']);
+        return Html::tag('figure', $node->outerHtml(), ['class' => 'table']);
     }
 }