]> git.immae.eu Git - github/wallabag/wallabag.git/blame - src/Wallabag/CoreBundle/Helper/ContentProxy.php
ContentProxy: fix a corner case when entry.url is empty in updateEntry
[github/wallabag/wallabag.git] / src / Wallabag / CoreBundle / Helper / ContentProxy.php
CommitLineData
558d9aab
JB
1<?php
2
3namespace Wallabag\CoreBundle\Helper;
4
5use Graby\Graby;
45fd7e09 6use Psr\Log\LoggerInterface;
8d7b4f0e 7use Symfony\Component\HttpFoundation\File\MimeType\MimeTypeExtensionGuesser;
42f3bb2c 8use Symfony\Component\Validator\Constraints\Locale as LocaleConstraint;
0d349ea6
JB
9use Symfony\Component\Validator\Constraints\Url as UrlConstraint;
10use Symfony\Component\Validator\Validator\ValidatorInterface;
f808b016
JB
11use Wallabag\CoreBundle\Entity\Entry;
12use Wallabag\CoreBundle\Tools\Utils;
558d9aab
JB
13
14/**
15 * This kind of proxy class take care of getting the content from an url
f1e29e69 16 * and update the entry with what it found.
558d9aab
JB
17 */
18class ContentProxy
19{
20 protected $graby;
c3510620 21 protected $tagger;
be54dfe4 22 protected $validator;
1c9cd2a7 23 protected $logger;
8d7b4f0e 24 protected $mimeGuesser;
29dca432 25 protected $fetchingErrorMessage;
6bc6fb1f 26 protected $eventDispatcher;
8a219854 27 protected $storeArticleHeaders;
558d9aab 28
709e21a3 29 public function __construct(Graby $graby, RuleBasedTagger $tagger, ValidatorInterface $validator, LoggerInterface $logger, $fetchingErrorMessage, $storeArticleHeaders = false)
558d9aab 30 {
347fa6be 31 $this->graby = $graby;
c3510620 32 $this->tagger = $tagger;
0d349ea6 33 $this->validator = $validator;
1c9cd2a7 34 $this->logger = $logger;
8d7b4f0e 35 $this->mimeGuesser = new MimeTypeExtensionGuesser();
29dca432 36 $this->fetchingErrorMessage = $fetchingErrorMessage;
8a219854 37 $this->storeArticleHeaders = $storeArticleHeaders;
558d9aab
JB
38 }
39
40 /**
6acadf8e 41 * Update entry using either fetched or provided content.
4d0ec0e7 42 *
6acadf8e
JB
43 * @param Entry $entry Entry to update
44 * @param string $url Url of the content
45 * @param array $content Array with content provided for import with AT LEAST keys title, html, url to skip the fetchContent from the url
46 * @param bool $disableContentUpdate Whether to skip trying to fetch content using Graby
558d9aab 47 */
6acadf8e 48 public function updateEntry(Entry $entry, $url, array $content = [], $disableContentUpdate = false)
558d9aab 49 {
6acadf8e
JB
50 if (!empty($content['html'])) {
51 $content['html'] = $this->graby->cleanupHtml($content['html'], $url);
d5c2cc54 52 }
e668a812 53
6acadf8e 54 if ((empty($content) || false === $this->validateContent($content)) && false === $disableContentUpdate) {
ec970721 55 $fetchedContent = $this->graby->fetchContent($url);
106bdbcd
JB
56
57 // 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
6acadf8e 59 if (empty($content) || $fetchedContent['html'] !== $this->fetchingErrorMessage) {
29dca432
JC
60 $content = $fetchedContent;
61 }
4d0ec0e7 62 }
558d9aab 63
6acadf8e
JB
64 // be sure to keep the url in case of error
65 // so we'll be able to refetch it in the future
66 $content['url'] = !empty($content['url']) ? $content['url'] : $url;
67
3362b3e3
KD
68 // In one case (at least in tests), url is empty here
69 // so we set it using $url provided in the updateEntry call.
70 // Not sure what are the other possible cases where this property is empty
71 if (empty($entry->getUrl()) && !empty($url))
72 {
73 $entry->setUrl($url);
74 }
75
d0e9b3d6
JC
76 $this->stockEntry($entry, $content);
77 }
78
c18a2476
JB
79 /**
80 * Use a Symfony validator to ensure the language is well formatted.
81 *
82 * @param Entry $entry
83 * @param string $value Language to validate and save
84 */
85 public function updateLanguage(Entry $entry, $value)
86 {
87 // some lang are defined as fr-FR, es-ES.
88 // replacing - by _ might increase language support
89 $value = str_replace('-', '_', $value);
90
91 $errors = $this->validator->validate(
92 $value,
93 (new LocaleConstraint())
94 );
95
96 if (0 === count($errors)) {
97 $entry->setLanguage($value);
98
99 return;
100 }
101
102 $this->logger->warning('Language validation failed. ' . (string) $errors);
103 }
104
105 /**
106 * Use a Symfony validator to ensure the preview picture is a real url.
107 *
108 * @param Entry $entry
109 * @param string $value URL to validate and save
110 */
111 public function updatePreviewPicture(Entry $entry, $value)
112 {
113 $errors = $this->validator->validate(
114 $value,
115 (new UrlConstraint())
116 );
117
118 if (0 === count($errors)) {
119 $entry->setPreviewPicture($value);
120
121 return;
122 }
123
124 $this->logger->warning('PreviewPicture validation failed. ' . (string) $errors);
125 }
126
127 /**
128 * Update date.
129 *
130 * @param Entry $entry
131 * @param string $value Date to validate and save
132 */
133 public function updatePublishedAt(Entry $entry, $value)
134 {
ff9f89fd 135 $date = $value;
c18a2476
JB
136
137 // is it a timestamp?
3ef055ce 138 if (false !== filter_var($date, FILTER_VALIDATE_INT)) {
ff9f89fd 139 $date = '@' . $date;
c18a2476
JB
140 }
141
142 try {
ff9f89fd
JB
143 // is it already a DateTime?
144 // (it's inside the try/catch in case of fail to be parse time string)
145 if (!$date instanceof \DateTime) {
146 $date = new \DateTime($date);
147 }
148
149 $entry->setPublishedAt($date);
c18a2476
JB
150 } catch (\Exception $e) {
151 $this->logger->warning('Error while defining date', ['e' => $e, 'url' => $entry->getUrl(), 'date' => $value]);
152 }
153 }
154
af29e1bf
KD
155 /**
156 * Helper to extract and save host from entry url.
157 *
158 * @param Entry $entry
159 */
160 public function setEntryDomainName(Entry $entry)
161 {
162 $domainName = parse_url($entry->getUrl(), PHP_URL_HOST);
163 if (false !== $domainName) {
164 $entry->setDomainName($domainName);
165 }
166 }
167
168 /**
169 * Helper to set a default title using:
170 * - url basename, if applicable
171 * - hostname.
172 *
173 * @param Entry $entry
174 */
175 public function setDefaultEntryTitle(Entry $entry)
176 {
177 $url = parse_url($entry->getUrl());
178 $path = pathinfo($url['path'], PATHINFO_BASENAME);
179
180 if (empty($path)) {
181 $path = $url['host'];
182 }
183
184 $entry->setTitle($path);
185 }
186
d0e9b3d6
JC
187 /**
188 * Stock entry with fetched or imported content.
189 * Will fall back to OpenGraph data if available.
190 *
d5c2cc54 191 * @param Entry $entry Entry to stock
ec970721 192 * @param array $content Array with at least title, url & html
d0e9b3d6
JC
193 */
194 private function stockEntry(Entry $entry, array $content)
195 {
a05b6115
JB
196 $entry->setUrl($content['url']);
197
af29e1bf 198 $this->setEntryDomainName($entry);
a05b6115
JB
199
200 if (!empty($content['title'])) {
201 $entry->setTitle($content['title']);
202 } elseif (!empty($content['open_graph']['og_title'])) {
203 $entry->setTitle($content['open_graph']['og_title']);
558d9aab
JB
204 }
205
206 $html = $content['html'];
207 if (false === $html) {
36e6ef52 208 $html = $this->fetchingErrorMessage;
558d9aab 209
e668a812 210 if (!empty($content['open_graph']['og_description'])) {
558d9aab
JB
211 $html .= '<p><i>But we found a short description: </i></p>';
212 $html .= $content['open_graph']['og_description'];
213 }
214 }
215
48656e0e 216 $entry->setContent($html);
a05b6115 217 $entry->setReadingTime(Utils::getReadingTime($html));
f0378b4d 218
a05b6115
JB
219 if (!empty($content['status'])) {
220 $entry->setHttpStatus($content['status']);
5e9009ce
NL
221 }
222
645291e8 223 if (!empty($content['authors']) && is_array($content['authors'])) {
7b0b3622
NL
224 $entry->setPublishedBy($content['authors']);
225 }
226
8a219854 227 if (!empty($content['all_headers']) && $this->storeArticleHeaders) {
dda6a6ad
NL
228 $entry->setHeaders($content['all_headers']);
229 }
230
a05b6115
JB
231 if (!empty($content['date'])) {
232 $this->updatePublishedAt($entry, $content['date']);
233 }
0d349ea6 234
a05b6115
JB
235 if (!empty($content['language'])) {
236 $this->updateLanguage($entry, $content['language']);
237 }
238
239 if (!empty($content['open_graph']['og_image'])) {
240 $this->updatePreviewPicture($entry, $content['open_graph']['og_image']);
241 }
0d349ea6 242
be54dfe4 243 // if content is an image, define it as a preview too
0d349ea6 244 if (!empty($content['content_type']) && in_array($this->mimeGuesser->guess($content['content_type']), ['jpeg', 'jpg', 'gif', 'png'], true)) {
d0ec2ddd 245 $this->updatePreviewPicture($entry, $content['url']);
0d349ea6
JB
246 }
247
a05b6115
JB
248 if (!empty($content['content_type'])) {
249 $entry->setMimetype($content['content_type']);
4d0ec0e7 250 }
558d9aab 251
1c9cd2a7
KG
252 try {
253 $this->tagger->tag($entry);
254 } catch (\Exception $e) {
4094ea47 255 $this->logger->error('Error while trying to automatically tag an entry.', [
d0e9b3d6 256 'entry_url' => $content['url'],
1c9cd2a7 257 'error_msg' => $e->getMessage(),
4094ea47 258 ]);
1c9cd2a7 259 }
558d9aab 260 }
c2656f96 261
4d0ec0e7 262 /**
d0e9b3d6 263 * Validate that the given content has at least a title, an html and a url.
4d0ec0e7
JB
264 *
265 * @param array $content
6acadf8e
JB
266 *
267 * @return bool true if valid otherwise false
4d0ec0e7
JB
268 */
269 private function validateContent(array $content)
270 {
6acadf8e 271 return !empty($content['title']) && !empty($content['html']) && !empty($content['url']);
4d0ec0e7 272 }
558d9aab 273}