diff options
author | Nicolas LÅ“uillet <nicolas@loeuillet.org> | 2016-05-02 12:49:23 +0200 |
---|---|---|
committer | Nicolas LÅ“uillet <nicolas@loeuillet.org> | 2016-05-02 12:50:42 +0200 |
commit | 51a15609b3bb0635e5cbea8511d983e2c5a97fe7 (patch) | |
tree | 4f57a45ec23b0ea869465fbbea3dc41b0ebef265 | |
parent | f0de35d2ee28031ff6a6a9406b07ed1aafc8ea4b (diff) | |
download | wallabag-51a15609b3bb0635e5cbea8511d983e2c5a97fe7.tar.gz wallabag-51a15609b3bb0635e5cbea8511d983e2c5a97fe7.tar.zst wallabag-51a15609b3bb0635e5cbea8511d983e2c5a97fe7.zip |
Set the title via POST /api/entries
Fix #2009
-rw-r--r-- | src/Wallabag/ApiBundle/Controller/WallabagRestController.php | 5 | ||||
-rw-r--r-- | src/Wallabag/ApiBundle/Tests/Controller/WallabagRestControllerTest.php | 2 |
2 files changed, 7 insertions, 0 deletions
diff --git a/src/Wallabag/ApiBundle/Controller/WallabagRestController.php b/src/Wallabag/ApiBundle/Controller/WallabagRestController.php index 316d45be..5202c524 100644 --- a/src/Wallabag/ApiBundle/Controller/WallabagRestController.php +++ b/src/Wallabag/ApiBundle/Controller/WallabagRestController.php | |||
@@ -109,6 +109,7 @@ class WallabagRestController extends FOSRestController | |||
109 | $this->validateAuthentication(); | 109 | $this->validateAuthentication(); |
110 | 110 | ||
111 | $url = $request->request->get('url'); | 111 | $url = $request->request->get('url'); |
112 | $title = $request->request->get('title'); | ||
112 | $isArchived = (int) $request->request->get('archive'); | 113 | $isArchived = (int) $request->request->get('archive'); |
113 | $isStarred = (int) $request->request->get('starred'); | 114 | $isStarred = (int) $request->request->get('starred'); |
114 | 115 | ||
@@ -121,6 +122,10 @@ class WallabagRestController extends FOSRestController | |||
121 | ); | 122 | ); |
122 | } | 123 | } |
123 | 124 | ||
125 | if (!is_null($title)) { | ||
126 | $entry->setTitle($title); | ||
127 | } | ||
128 | |||
124 | $tags = $request->request->get('tags', ''); | 129 | $tags = $request->request->get('tags', ''); |
125 | if (!empty($tags)) { | 130 | if (!empty($tags)) { |
126 | $this->get('wallabag_core.content_proxy')->assignTagsToEntry($entry, $tags); | 131 | $this->get('wallabag_core.content_proxy')->assignTagsToEntry($entry, $tags); |
diff --git a/src/Wallabag/ApiBundle/Tests/Controller/WallabagRestControllerTest.php b/src/Wallabag/ApiBundle/Tests/Controller/WallabagRestControllerTest.php index 839ff386..2f2d92ee 100644 --- a/src/Wallabag/ApiBundle/Tests/Controller/WallabagRestControllerTest.php +++ b/src/Wallabag/ApiBundle/Tests/Controller/WallabagRestControllerTest.php | |||
@@ -152,6 +152,7 @@ class WallabagRestControllerTest extends WallabagApiTestCase | |||
152 | $this->client->request('POST', '/api/entries.json', [ | 152 | $this->client->request('POST', '/api/entries.json', [ |
153 | 'url' => 'http://www.lemonde.fr/pixels/article/2015/03/28/plongee-dans-l-univers-d-ingress-le-jeu-de-google-aux-frontieres-du-reel_4601155_4408996.html', | 153 | 'url' => 'http://www.lemonde.fr/pixels/article/2015/03/28/plongee-dans-l-univers-d-ingress-le-jeu-de-google-aux-frontieres-du-reel_4601155_4408996.html', |
154 | 'tags' => 'google', | 154 | 'tags' => 'google', |
155 | 'title' => 'New title for my article', | ||
155 | ]); | 156 | ]); |
156 | 157 | ||
157 | $this->assertEquals(200, $this->client->getResponse()->getStatusCode()); | 158 | $this->assertEquals(200, $this->client->getResponse()->getStatusCode()); |
@@ -162,6 +163,7 @@ class WallabagRestControllerTest extends WallabagApiTestCase | |||
162 | $this->assertEquals('http://www.lemonde.fr/pixels/article/2015/03/28/plongee-dans-l-univers-d-ingress-le-jeu-de-google-aux-frontieres-du-reel_4601155_4408996.html', $content['url']); | 163 | $this->assertEquals('http://www.lemonde.fr/pixels/article/2015/03/28/plongee-dans-l-univers-d-ingress-le-jeu-de-google-aux-frontieres-du-reel_4601155_4408996.html', $content['url']); |
163 | $this->assertEquals(false, $content['is_archived']); | 164 | $this->assertEquals(false, $content['is_archived']); |
164 | $this->assertEquals(false, $content['is_starred']); | 165 | $this->assertEquals(false, $content['is_starred']); |
166 | $this->assertEquals('New title for my article', $content['title']); | ||
165 | $this->assertEquals(1, $content['user_id']); | 167 | $this->assertEquals(1, $content['user_id']); |
166 | $this->assertCount(1, $content['tags']); | 168 | $this->assertCount(1, $content['tags']); |
167 | } | 169 | } |