Merge pull request #3095 from aaa2000/api-error-on-fail-fetch-content

Create a new entry via API even when its content can't be retrieved

diff --combined src/Wallabag/ApiBundle/Controller/EntryRestController.php
index c544815ebc7dee9bb308787940559ee376565af3,e9e1aca3b2c2d6ea2f08453cc615db659aee1b33..54c1747c5a9fb9412f3101892df651b12e34d07d
--- 1/src/Wallabag/ApiBundle/Controller/EntryRestController.php
--- 2/src/Wallabag/ApiBundle/Controller/EntryRestController.php
+++ b/src/Wallabag/ApiBundle/Controller/EntryRestController.php
@@@ -98,13 -98,12 +98,13 @@@ class EntryRestController extends Walla
          $tags = $request->query->get('tags', '');
          $since = $request->query->get('since', 0);
  
 +        /** @var \Pagerfanta\Pagerfanta $pager */
          $pager = $this->getDoctrine()
              ->getRepository('WallabagCoreBundle:Entry')
              ->findEntries($this->getUser()->getId(), $isArchived, $isStarred, $sort, $order, $since, $tags);
  
 -        $pager->setCurrentPage($page);
          $pager->setMaxPerPage($perPage);
 +        $pager->setCurrentPage($page);
  
          $pagerfantaFactory = new PagerfantaFactory('page', 'perPage');
          $paginatedCollection = $pagerfantaFactory->createRepresentation(
@@@ -200,10 -199,19 +200,19 @@@
          $entry = $this->get('wallabag_core.entry_repository')->findByUrlAndUserId($url, $this->getUser()->getId());
  
          if (false === $entry) {
-             $entry = $this->get('wallabag_core.content_proxy')->updateEntry(
-                 new Entry($this->getUser()),
-                 $url
-             );
+             $entry = new Entry($this->getUser());
+             try {
+                 $entry = $this->get('wallabag_core.content_proxy')->updateEntry(
+                     $entry,
+                     $url
+                 );
+             } catch (\Exception $e) {
+                 $this->get('logger')->error('Error while saving an entry', [
+                     'exception' => $e,
+                     'entry' => $entry,
+                 ]);
+                 $entry->setUrl($url);
+             }
          }
  
          if (!is_null($title)) {

diff --combined tests/Wallabag/ApiBundle/Controller/EntryRestControllerTest.php
index 63d70bd9bb3b1c2ef51f59a6e1c05921e0cd4c88,deafd1fab70513dfad35c06a2ca8819897431646..0a65f9cee0628a78a38ab6b2fd7ce259c1383371
--- 1/tests/Wallabag/ApiBundle/Controller/EntryRestControllerTest.php
--- 2/tests/Wallabag/ApiBundle/Controller/EntryRestControllerTest.php
+++ b/tests/Wallabag/ApiBundle/Controller/EntryRestControllerTest.php
@@@ -4,6 -4,7 +4,7 @@@ namespace Tests\Wallabag\ApiBundle\Cont
  
  use Tests\Wallabag\ApiBundle\WallabagApiTestCase;
  use Wallabag\CoreBundle\Entity\Tag;
+ use Wallabag\CoreBundle\Helper\ContentProxy;
  
  class EntryRestControllerTest extends WallabagApiTestCase
  {
@@@ -156,22 -157,6 +157,22 @@@
          $this->assertEquals('application/json', $this->client->getResponse()->headers->get('Content-Type'));
      }
  
 +    public function testGetEntriesOnPageTwo()
 +    {
 +        $this->client->request('GET', '/api/entries', [
 +            'page' => 2,
 +            'perPage' => 2,
 +        ]);
 +
 +        $this->assertEquals(200, $this->client->getResponse()->getStatusCode());
 +
 +        $content = json_decode($this->client->getResponse()->getContent(), true);
 +
 +        $this->assertGreaterThanOrEqual(0, $content['total']);
 +        $this->assertEquals(2, $content['page']);
 +        $this->assertEquals(2, $content['limit']);
 +    }
 +
      public function testGetStarredEntries()
      {
          $this->client->request('GET', '/api/entries', ['starred' => 1, 'sort' => 'updated']);
@@@ -375,6 -360,39 +376,39 @@@
          $this->assertCount(2, $content['tags']);
      }
  
+     public function testPostEntryWhenFetchContentFails()
+     {
+         /** @var \Symfony\Component\DependencyInjection\Container $container */
+         $container = $this->client->getContainer();
+         $contentProxy = $this->getMockBuilder(ContentProxy::class)
+             ->disableOriginalConstructor()
+             ->setMethods(['updateEntry'])
+             ->getMock();
+         $contentProxy->expects($this->any())
+             ->method('updateEntry')
+             ->willThrowException(new \Exception('Test Fetch content fails'));
+         $container->set('wallabag_core.content_proxy', $contentProxy);
+ 
+         try {
+             $this->client->request('POST', '/api/entries.json', [
+                 'url' => 'http://www.example.com/',
+             ]);
+ 
+             $this->assertEquals(200, $this->client->getResponse()->getStatusCode());
+             $content = json_decode($this->client->getResponse()->getContent(), true);
+             $this->assertGreaterThan(0, $content['id']);
+             $this->assertEquals('http://www.example.com/', $content['url']);
+         } finally {
+             // Remove the created entry to avoid side effects on other tests
+             if (isset($content['id'])) {
+                 $em = $this->client->getContainer()->get('doctrine.orm.entity_manager');
+                 $entry = $em->getReference('WallabagCoreBundle:Entry', $content['id']);
+                 $em->remove($entry);
+                 $em->flush();
+             }
+         }
+     }
+ 
      public function testPostArchivedAndStarredEntry()
      {
          $this->client->request('POST', '/api/entries.json', [