From fba9e7d44d5987a603add50dc450f50079340dcc Mon Sep 17 00:00:00 2001
From: Jeremy Benoist <jeremy.benoist@gmail.com>
Date: Fri, 4 Mar 2016 11:40:48 +0100
Subject: [PATCH] Remove 'content' from API

Waiting to find a good solution to avoid side problem since user can no define the content
---
 .../Controller/WallabagRestController.php        |  6 ------
 .../Controller/WallabagRestControllerTest.php    | 16 ----------------
 2 files changed, 22 deletions(-)

diff --git a/src/Wallabag/ApiBundle/Controller/WallabagRestController.php b/src/Wallabag/ApiBundle/Controller/WallabagRestController.php
index 28145e45..35a90edd 100644
--- a/src/Wallabag/ApiBundle/Controller/WallabagRestController.php
+++ b/src/Wallabag/ApiBundle/Controller/WallabagRestController.php
@@ -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);
 
diff --git a/src/Wallabag/ApiBundle/Tests/Controller/WallabagRestControllerTest.php b/src/Wallabag/ApiBundle/Tests/Controller/WallabagRestControllerTest.php
index 6e64d84f..630b75bf 100644
--- a/src/Wallabag/ApiBundle/Tests/Controller/WallabagRestControllerTest.php
+++ b/src/Wallabag/ApiBundle/Tests/Controller/WallabagRestControllerTest.php
@@ -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()
-- 
2.41.0