Merge pull request #2919 from wallabag/api-exists-returns-id

Api exists returns
This commit is contained in:
Nicolas Lœuillet 2017-02-28 21:12:30 +01:00 committed by GitHub
commit e1cc8fd799
2 changed files with 4 additions and 4 deletions

View file

@ -41,7 +41,7 @@ class EntryRestController extends WallabagRestController
->getRepository('WallabagCoreBundle:Entry')
->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');
@ -60,7 +60,7 @@ class EntryRestController extends WallabagRestController
->getRepository('WallabagCoreBundle:Entry')
->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');

View file

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