devol/wallabag
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove 'content' from API

Browse Source 
Waiting to find a good solution to avoid side problem since user can no define the content
This commit is contained in:
Jeremy Benoist 2016-03-04 11:40:48 +01:00
parent 11a452813c
commit fba9e7d44d
2 changed files with 0 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/Wallabag/ApiBundle/Controller/WallabagRestController.php
View File

@ -99,7 +99,6 @@ class WallabagRestController extends FOSRestController
* {"name"="tags", "dataType"="string", "required"=false, "format"="tag1,tag2,tag3", "description"="a comma-separated list of tags."},
* {"name"="starred", "dataType"="boolean", "required"=false, "format"="true or false", "description"="entry already starred"},
* {"name"="archive", "dataType"="boolean", "required"=false, "format"="true or false", "description"="entry already archived"},
* {"name"="content", "dataType"="string", "required"=false, "format"="content", "description"="content you want to pass directly"},
* }
* )
*
@ -110,7 +109,6 @@ class WallabagRestController extends FOSRestController
$this->validateAuthentication();
$url = $request->request->get('url');
$content = $request->request->get('content');
$isArchived = $request->request->get('archive');
$isStarred = $request->request->get('starred');
@ -132,10 +130,6 @@ class WallabagRestController extends FOSRestController
$entry->setArchived(true);
}
if (!empty($content)) {
$entry->setContent($content);
}
$em = $this->getDoctrine()->getManager();
$em->persist($entry);

16
src/Wallabag/ApiBundle/Tests/Controller/WallabagRestControllerTest.php
View File

@ -180,22 +180,6 @@ class WallabagRestControllerTest extends WallabagApiTestCase
$this->assertEquals(false, $content['is_starred']);
}
public function testPostEntryWithContent()
{
$this->client->request('POST', '/api/entries.json', array(
'url' => 'http://www.lemonde.fr/idees/article/2016/02/08/preserver-la-liberte-d-expression-sur-les-reseaux-sociaux_4861503_3232.html',
'content' => 'This is a new content for my entry',
));
$this->assertEquals(200, $this->client->getResponse()->getStatusCode());
$content = json_decode($this->client->getResponse()->getContent(), true);
$this->assertGreaterThan(0, $content['id']);
$this->assertEquals('http://www.lemonde.fr/idees/article/2016/02/08/preserver-la-liberte-d-expression-sur-les-reseaux-sociaux_4861503_3232.html', $content['url']);
$this->assertEquals('This is a new content for my entry', $content['content']);
}
public function testPatchEntry()
{
$entry = $this->client->getContainer()