From fb436e8ca0c7468b9698050df0b78447e2d0854f Mon Sep 17 00:00:00 2001
From: Jeremy Benoist <jeremy.benoist@gmail.com>
Date: Thu, 11 May 2017 20:10:22 +0200
Subject: [PATCH] Add support for authors

---
 src/Wallabag/ApiBundle/Controller/EntryRestController.php   | 6 +++++-
 .../ApiBundle/Controller/EntryRestControllerTest.php        | 4 ++++
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/src/Wallabag/ApiBundle/Controller/EntryRestController.php b/src/Wallabag/ApiBundle/Controller/EntryRestController.php
index dfd04fb4..e6bbe552 100644
--- a/src/Wallabag/ApiBundle/Controller/EntryRestController.php
+++ b/src/Wallabag/ApiBundle/Controller/EntryRestController.php
@@ -284,6 +284,7 @@ class EntryRestController extends WallabagRestController
      *          {"name"="language", "dataType"="string", "required"=false, "description"="Language of the entry"},
      *          {"name"="preview_picture", "dataType"="string", "required"=false, "description"="Preview picture of the entry"},
      *          {"name"="published_at", "dataType"="datetime", "format"="YYYY-MM-DDTHH:II:SS+TZ", "required"=false, "description"="Published date of the entry"},
+     *          {"name"="authors", "dataType"="string", "format"="Name Firstname,author2,author3", "required"=false, "description"="Authors of the entry"},
      *       }
      * )
      *
@@ -295,12 +296,14 @@ class EntryRestController extends WallabagRestController
 
         $url = $request->request->get('url');
         $title = $request->request->get('title');
+        $tags = $request->request->get('tags', []);
         $isArchived = $request->request->get('archive');
         $isStarred = $request->request->get('starred');
         $content = $request->request->get('content');
         $language = $request->request->get('language');
         $picture = $request->request->get('preview_picture');
         $publishedAt = $request->request->get('published_at');
+        $authors = $request->request->get('authors', '');
 
         $entry = $this->get('wallabag_core.entry_repository')->findByUrlAndUserId($url, $this->getUser()->getId());
 
@@ -322,6 +325,7 @@ class EntryRestController extends WallabagRestController
                     'open_graph' => [
                         'og_image' => $picture,
                     ],
+                    'authors' => explode(',', $authors),
                 ]
             );
         } catch (\Exception $e) {
@@ -332,7 +336,7 @@ class EntryRestController extends WallabagRestController
             $entry->setUrl($url);
         }
 
-        $tags = $request->request->get('tags', []);
+
         if (!empty($tags)) {
             $this->get('wallabag_core.tags_assigner')->assignTagsToEntry($entry, $tags);
         }
diff --git a/tests/Wallabag/ApiBundle/Controller/EntryRestControllerTest.php b/tests/Wallabag/ApiBundle/Controller/EntryRestControllerTest.php
index 1b0c06d2..e9ba4634 100644
--- a/tests/Wallabag/ApiBundle/Controller/EntryRestControllerTest.php
+++ b/tests/Wallabag/ApiBundle/Controller/EntryRestControllerTest.php
@@ -345,6 +345,7 @@ class EntryRestControllerTest extends WallabagApiTestCase
             'content' => 'my content',
             'language' => 'de_DE',
             'published_at' => '2016-09-08T11:55:58+0200',
+            'authors' => 'bob,helen',
         ]);
 
         $this->assertEquals(200, $this->client->getResponse()->getStatusCode());
@@ -361,6 +362,9 @@ class EntryRestControllerTest extends WallabagApiTestCase
         $this->assertSame('my content', $content['content']);
         $this->assertSame('de_DE', $content['language']);
         $this->assertSame('2016-09-08T11:55:58+0200', $content['published_at']);
+        $this->assertCount(2, $content['published_by']);
+        $this->assertContains('bob', $content['published_by']);
+        $this->assertContains('helen', $content['published_by']);
     }
 
     public function testPostSameEntry()
-- 
2.41.0