diff options
author | Jeremy Benoist <jeremy.benoist@gmail.com> | 2017-05-11 08:14:29 +0200 |
---|---|---|
committer | Jeremy Benoist <jeremy.benoist@gmail.com> | 2017-05-31 13:59:45 +0200 |
commit | e668a8124c46d47add4248963d77f3b29b37b3ce (patch) | |
tree | b9b56d19b06fe268d025c3591119470162efc99a /tests/Wallabag/ApiBundle/Controller/EntryRestControllerTest.php | |
parent | 4423b88c5b2c2d530b0a83a822f521a61ca4d4b8 (diff) | |
download | wallabag-e668a8124c46d47add4248963d77f3b29b37b3ce.tar.gz wallabag-e668a8124c46d47add4248963d77f3b29b37b3ce.tar.zst wallabag-e668a8124c46d47add4248963d77f3b29b37b3ce.zip |
Allow other fields to be send using API
Entry API can now have these new fields:
- content
- language
- preview_picture
- published_at
Re-use the ContentProxy to be able to do the same using the web UI (in the future).
htmLawed is used to clean stuff from content, I hope it’ll be enough to avoid security breach.
Lower content validation when we want to update an entry with content already defined. Before, language & content_type were required. If there weren’t provided, we re-fetched the content using graby. I think these fields aren’t required for an entry to be created. So I removed them.
Which means some import from the v1 export won’t be re-fetched since they provide content, url & title.
Also, remove liberation link from Readability import to avoid overlaping import (from wallabag v1, which had the same link)
Diffstat (limited to 'tests/Wallabag/ApiBundle/Controller/EntryRestControllerTest.php')
-rw-r--r-- | tests/Wallabag/ApiBundle/Controller/EntryRestControllerTest.php | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/tests/Wallabag/ApiBundle/Controller/EntryRestControllerTest.php b/tests/Wallabag/ApiBundle/Controller/EntryRestControllerTest.php index bf7d373a..1b0c06d2 100644 --- a/tests/Wallabag/ApiBundle/Controller/EntryRestControllerTest.php +++ b/tests/Wallabag/ApiBundle/Controller/EntryRestControllerTest.php | |||
@@ -342,6 +342,9 @@ class EntryRestControllerTest extends WallabagApiTestCase | |||
342 | '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', | 342 | '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', |
343 | 'tags' => 'google', | 343 | 'tags' => 'google', |
344 | 'title' => 'New title for my article', | 344 | 'title' => 'New title for my article', |
345 | 'content' => 'my content', | ||
346 | 'language' => 'de_DE', | ||
347 | 'published_at' => '2016-09-08T11:55:58+0200', | ||
345 | ]); | 348 | ]); |
346 | 349 | ||
347 | $this->assertEquals(200, $this->client->getResponse()->getStatusCode()); | 350 | $this->assertEquals(200, $this->client->getResponse()->getStatusCode()); |
@@ -355,6 +358,9 @@ class EntryRestControllerTest extends WallabagApiTestCase | |||
355 | $this->assertEquals('New title for my article', $content['title']); | 358 | $this->assertEquals('New title for my article', $content['title']); |
356 | $this->assertEquals(1, $content['user_id']); | 359 | $this->assertEquals(1, $content['user_id']); |
357 | $this->assertCount(2, $content['tags']); | 360 | $this->assertCount(2, $content['tags']); |
361 | $this->assertSame('my content', $content['content']); | ||
362 | $this->assertSame('de_DE', $content['language']); | ||
363 | $this->assertSame('2016-09-08T11:55:58+0200', $content['published_at']); | ||
358 | } | 364 | } |
359 | 365 | ||
360 | public function testPostSameEntry() | 366 | public function testPostSameEntry() |