diff options
author | Jeremy Benoist <jbenoist@20minutes.fr> | 2017-06-01 11:45:02 +0200 |
---|---|---|
committer | Jeremy Benoist <jbenoist@20minutes.fr> | 2017-06-01 11:45:02 +0200 |
commit | ec970721525d9a4761a30ee417406cf8ad8bb171 (patch) | |
tree | ee48efb44e7cc79d5fe73f94560c35f03d097d6d /src/Wallabag | |
parent | 6acadf8e98cf6021a9019773df75bdb151865687 (diff) | |
download | wallabag-ec970721525d9a4761a30ee417406cf8ad8bb171.tar.gz wallabag-ec970721525d9a4761a30ee417406cf8ad8bb171.tar.zst wallabag-ec970721525d9a4761a30ee417406cf8ad8bb171.zip |
No need to catch that Exception
Diffstat (limited to 'src/Wallabag')
-rw-r--r-- | src/Wallabag/CoreBundle/Helper/ContentProxy.php | 11 |
1 files changed, 2 insertions, 9 deletions
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 | |||
45 | } | 45 | } |
46 | 46 | ||
47 | if ((empty($content) || false === $this->validateContent($content)) && false === $disableContentUpdate) { | 47 | if ((empty($content) || false === $this->validateContent($content)) && false === $disableContentUpdate) { |
48 | try { | 48 | $fetchedContent = $this->graby->fetchContent($url); |
49 | $fetchedContent = $this->graby->fetchContent($url); | ||
50 | } catch (\Exception $e) { | ||
51 | $this->logger->error('Error while trying to fetch content from URL.', [ | ||
52 | 'entry_url' => $url, | ||
53 | 'error_msg' => $e->getMessage(), | ||
54 | ]); | ||
55 | } | ||
56 | 49 | ||
57 | // when content is imported, we have information in $content | 50 | // when content is imported, we have information in $content |
58 | // in case fetching content goes bad, we'll keep the imported information instead of overriding them | 51 | // in case fetching content goes bad, we'll keep the imported information instead of overriding them |
@@ -73,7 +66,7 @@ class ContentProxy | |||
73 | * Will fall back to OpenGraph data if available. | 66 | * Will fall back to OpenGraph data if available. |
74 | * | 67 | * |
75 | * @param Entry $entry Entry to stock | 68 | * @param Entry $entry Entry to stock |
76 | * @param array $content Array with at least title and URL | 69 | * @param array $content Array with at least title, url & html |
77 | */ | 70 | */ |
78 | private function stockEntry(Entry $entry, array $content) | 71 | private function stockEntry(Entry $entry, array $content) |
79 | { | 72 | { |