oss/wallabag
oss/wallabag
1
0
Fork 0
mirror of https://github.com/wallabag/wallabag.git synced 2025-09-15 18:57:05 +00:00
Code Wiki Activity

Add a test for updatePublishedAt

Browse source 
To avoid error when a content is re-submitted and it previously add a
published date.

Also, fix the `testPostSameEntry`
This commit is contained in:
Jeremy Benoist 2017-07-24 16:39:29 +02:00
parent b236d3f627
commit ff9f89fd23
No known key found for this signature in database
GPG key ID: BCA73962457ACC3C
2 changed files with 43 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

12
src/Wallabag/CoreBundle/Helper/ContentProxy.php
View file

@ -122,15 +122,21 @@ class ContentProxy
*/
public function updatePublishedAt(Entry $entry, $value)
{
$date = $value instanceof \DateTime ? $value->date : $value;
$date = $value;
// is it a timestamp?
if (filter_var($date, FILTER_VALIDATE_INT) !== false) {
$date = '@' . $value;
$date = '@' . $date;
}
try {
$entry->setPublishedAt(new \DateTime($date));
// is it already a DateTime?
// (it's inside the try/catch in case of fail to be parse time string)
if (!$date instanceof \DateTime) {
$date = new \DateTime($date);
}
$entry->setPublishedAt($date);
} catch (\Exception $e) {
$this->logger->warning('Error while defining date', ['e' => $e, 'url' => $entry->getUrl(), 'date' => $value]);
}