From c0d02153ab2b44ceb103ae35c0cc40799b7c31a4 Mon Sep 17 00:00:00 2001
From: Yassine Guedidi <yassine@guedidi.com>
Date: Tue, 19 Nov 2024 23:30:37 +0100
Subject: [PATCH] Use DomCrawler in PocketHtmlImport

---
 src/Import/PocketHtmlImport.php | 26 ++++++++++++--------------
 1 file changed, 12 insertions(+), 14 deletions(-)

diff --git a/src/Import/PocketHtmlImport.php b/src/Import/PocketHtmlImport.php
index d9ed7be21..b1c7e3edb 100644
--- a/src/Import/PocketHtmlImport.php
+++ b/src/Import/PocketHtmlImport.php
@@ -2,6 +2,8 @@
 
 namespace Wallabag\Import;
 
+use Symfony\Component\DomCrawler\Crawler;
+
 class PocketHtmlImport extends HtmlImport
 {
     protected $filepath;
@@ -44,27 +46,23 @@ class PocketHtmlImport extends HtmlImport
             return false;
         }
 
-        $html = new \DOMDocument();
+        $crawler = new Crawler(file_get_contents($this->filepath));
 
-        libxml_use_internal_errors(true);
-        $html->loadHTMLFile($this->filepath);
-        $hrefs = $html->getElementsByTagName('a');
-        libxml_use_internal_errors(false);
+        $hrefs = $crawler->filterXPath('//a');
 
-        if (0 === $hrefs->length) {
+        if (0 === $hrefs->count()) {
             $this->logger->error('Pocket HTML: no entries in imported file');
 
             return false;
         }
 
-        $entries = [];
-        foreach ($hrefs as $href) {
-            $entry = [];
-            $entry['url'] = $href->getAttribute('href');
-            $entry['tags'] = $href->getAttribute('tags');
-            $entry['created_at'] = $href->getAttribute('time_added');
-            $entries[] = $entry;
-        }
+        $entries = $hrefs->each(function (Crawler $node) {
+            return [
+                'url' => $node->attr('href'),
+                'tags' => $node->attr('tags'),
+                'created_at' => $node->attr('time_added'),
+            ];
+        });
 
         if ($this->producer) {
             $this->parseEntriesForProducer($entries);