mirrors/wallabag
1
0
Fork 0
mirror of https://github.com/wallabag/wallabag.git synced 2024-11-26 19:11:07 +00:00
Code Issues Projects Releases Wiki Activity

return id of article instead of true if entry exists

Browse source
This commit is contained in:
Thomas Citharel 2017-02-24 11:31:31 +01:00
parent abba55cdd6
commit ca9a83ee51
2 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/Wallabag/ApiBundle/Controller/EntryRestController.php
View file

@ -41,7 +41,7 @@ class EntryRestController extends WallabagRestController
->getRepository('WallabagCoreBundle:Entry') ->getRepository('WallabagCoreBundle:Entry')
->findByUrlAndUserId($url, $this->getUser()->getId()); ->findByUrlAndUserId($url, $this->getUser()->getId());
$results[$url] = false === $res ? false : true; $results[$url] = $res instanceof Entry ? $res->getId() : false;
} }
$json = $this->get('serializer')->serialize($results, 'json'); $json = $this->get('serializer')->serialize($results, 'json');
@ -60,7 +60,7 @@ class EntryRestController extends WallabagRestController
->getRepository('WallabagCoreBundle:Entry') ->getRepository('WallabagCoreBundle:Entry')
->findByUrlAndUserId($url, $this->getUser()->getId()); ->findByUrlAndUserId($url, $this->getUser()->getId());
$exists = false === $res ? false : true; $exists = $res instanceof Entry ? $res->getId() : false;
$json = $this->get('serializer')->serialize(['exists' => $exists], 'json'); $json = $this->get('serializer')->serialize(['exists' => $exists], 'json');

4
tests/Wallabag/ApiBundle/Controller/EntryRestControllerTest.php
View file

@ -642,7 +642,7 @@ class EntryRestControllerTest extends WallabagApiTestCase
$content = json_decode($this->client->getResponse()->getContent(), true); $content = json_decode($this->client->getResponse()->getContent(), true);
$this->assertEquals(true, $content['exists']); $this->assertEquals(2, $content['exists']);
} }
public function testGetEntriesExistsWithManyUrls() public function testGetEntriesExistsWithManyUrls()
@ -657,7 +657,7 @@ class EntryRestControllerTest extends WallabagApiTestCase
$this->assertArrayHasKey($url1, $content); $this->assertArrayHasKey($url1, $content);
$this->assertArrayHasKey($url2, $content); $this->assertArrayHasKey($url2, $content);
$this->assertEquals(true, $content[$url1]); $this->assertEquals(2, $content[$url1]);
$this->assertEquals(false, $content[$url2]); $this->assertEquals(false, $content[$url2]);
} }