Knah-Tsaeb/MyShaarli
Knah-Tsaeb/MyShaarli
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Merge pull request #1525 from ArthurHoaro/feature/rest-api-bookmark-dates

Browse source 
REST API: allow override of creation and update dates
This commit is contained in:
ArthurHoaro 2020-10-13 12:26:01 +02:00 committed by GitHub
parent 8f269b49d7 b06fc28aa3
commit 543b16b4f4
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
4 changed files with 18 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
application/api/ApiUtils.php
View file

@ -94,7 +94,7 @@ class ApiUtils
* *
* @return Bookmark instance. * @return Bookmark instance.
*/ */
public static function buildLinkFromRequest($input, $defaultPrivate) public static function buildBookmarkFromRequest($input, $defaultPrivate): Bookmark
{ {
$bookmark = new Bookmark(); $bookmark = new Bookmark();
$url = ! empty($input['url']) ? cleanup_url($input['url']) : ''; $url = ! empty($input['url']) ? cleanup_url($input['url']) : '';
@ -110,6 +110,15 @@ class ApiUtils
$bookmark->setTags(! empty($input['tags']) ? $input['tags'] : []); $bookmark->setTags(! empty($input['tags']) ? $input['tags'] : []);
$bookmark->setPrivate($private); $bookmark->setPrivate($private);
$created = \DateTime::createFromFormat(\DateTime::ATOM, $input['created'] ?? '');
if ($created instanceof \DateTimeInterface) {
$bookmark->setCreated($created);
}
$updated = \DateTime::createFromFormat(\DateTime::ATOM, $input['updated'] ?? '');
if ($updated instanceof \DateTimeInterface) {
$bookmark->setUpdated($updated);
}
return $bookmark; return $bookmark;
} }

3
application/api/controllers/ApiController.php
View file

@ -4,6 +4,7 @@ namespace Shaarli\Api\Controllers;
use Shaarli\Bookmark\BookmarkServiceInterface; use Shaarli\Bookmark\BookmarkServiceInterface;
use Shaarli\Config\ConfigManager; use Shaarli\Config\ConfigManager;
use Shaarli\History;
use Slim\Container; use Slim\Container;
/** /**
@ -31,7 +32,7 @@ abstract class ApiController
protected $bookmarkService; protected $bookmarkService;
/** /**
* @var HistoryController * @var History
*/ */
protected $history; protected $history;

4
application/api/controllers/Links.php
View file

@ -116,7 +116,7 @@ class Links extends ApiController
public function postLink($request, $response) public function postLink($request, $response)
{ {
$data = $request->getParsedBody(); $data = $request->getParsedBody();
$bookmark = ApiUtils::buildLinkFromRequest($data, $this->conf->get('privacy.default_private_links')); $bookmark = ApiUtils::buildBookmarkFromRequest($data, $this->conf->get('privacy.default_private_links'));
// duplicate by URL, return 409 Conflict // duplicate by URL, return 409 Conflict
if (! empty($bookmark->getUrl()) if (! empty($bookmark->getUrl())
&& ! empty($dup = $this->bookmarkService->findByUrl($bookmark->getUrl())) && ! empty($dup = $this->bookmarkService->findByUrl($bookmark->getUrl()))
@ -155,7 +155,7 @@ class Links extends ApiController
$index = index_url($this->ci['environment']); $index = index_url($this->ci['environment']);
$data = $request->getParsedBody(); $data = $request->getParsedBody();
$requestBookmark = ApiUtils::buildLinkFromRequest($data, $this->conf->get('privacy.default_private_links')); $requestBookmark = ApiUtils::buildBookmarkFromRequest($data, $this->conf->get('privacy.default_private_links'));
// duplicate URL on a different link, return 409 Conflict // duplicate URL on a different link, return 409 Conflict
if (! empty($requestBookmark->getUrl()) if (! empty($requestBookmark->getUrl())
&& ! empty($dup = $this->bookmarkService->findByUrl($requestBookmark->getUrl())) && ! empty($dup = $this->bookmarkService->findByUrl($requestBookmark->getUrl()))

8
tests/api/controllers/links/PostLinkTest.php
View file

@ -160,6 +160,8 @@ class PostLinkTest extends TestCase
'description' => 'shaare description', 'description' => 'shaare description',
'tags' => ['one', 'two'], 'tags' => ['one', 'two'],
'private' => true, 'private' => true,
'created' => '2015-05-05T12:30:00+03:00',
'updated' => '2016-06-05T14:32:10+03:00',
]; ];
$env = Environment::mock([ $env = Environment::mock([
'REQUEST_METHOD' => 'POST', 'REQUEST_METHOD' => 'POST',
@ -181,10 +183,8 @@ class PostLinkTest extends TestCase
$this->assertEquals($link['description'], $data['description']); $this->assertEquals($link['description'], $data['description']);
$this->assertEquals($link['tags'], $data['tags']); $this->assertEquals($link['tags'], $data['tags']);
$this->assertEquals(true, $data['private']); $this->assertEquals(true, $data['private']);
$this->assertTrue( $this->assertSame($link['created'], $data['created']);
new \DateTime('2 seconds ago') < \DateTime::createFromFormat(\DateTime::ATOM, $data['created']) $this->assertSame($link['updated'], $data['updated']);
);
$this->assertEquals('', $data['updated']);
} }
/** /**