From: Jeremy Benoist <jbenoist@20minutes.fr>
Date: Thu, 1 Jun 2017 09:45:02 +0000 (+0200)
Subject: No need to catch that Exception
X-Git-Tag: 2.3.0~31^2~73^2~2
X-Git-Url: https://git.immae.eu/?a=commitdiff_plain;h=ec970721525d9a4761a30ee417406cf8ad8bb171;p=github%2Fwallabag%2Fwallabag.git

No need to catch that Exception
---

diff --git a/src/Wallabag/CoreBundle/Helper/ContentProxy.php b/src/Wallabag/CoreBundle/Helper/ContentProxy.php
index 3024fdc5..bfaa1976 100644
--- a/src/Wallabag/CoreBundle/Helper/ContentProxy.php
+++ b/src/Wallabag/CoreBundle/Helper/ContentProxy.php
@@ -45,14 +45,7 @@ class ContentProxy
         }
 
         if ((empty($content) || false === $this->validateContent($content)) && false === $disableContentUpdate) {
-            try {
-                $fetchedContent = $this->graby->fetchContent($url);
-            } catch (\Exception $e) {
-                $this->logger->error('Error while trying to fetch content from URL.', [
-                    'entry_url' => $url,
-                    'error_msg' => $e->getMessage(),
-                ]);
-            }
+            $fetchedContent = $this->graby->fetchContent($url);
 
             // when content is imported, we have information in $content
             // in case fetching content goes bad, we'll keep the imported information instead of overriding them
@@ -73,7 +66,7 @@ class ContentProxy
      * Will fall back to OpenGraph data if available.
      *
      * @param Entry $entry   Entry to stock
-     * @param array $content Array with at least title and URL
+     * @param array $content Array with at least title, url & html
      */
     private function stockEntry(Entry $entry, array $content)
     {