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

Bulk creation: ignore blank lines

Browse source
This commit is contained in:
ArthurHoaro 2020-10-23 13:28:02 +02:00
parent 6a71675887
commit 34c8f558e5
2 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
application/front/controller/admin/ShaarePublishController.php
View file

@ -46,6 +46,9 @@ public function displayCreateBatchForms(Request $request, Response $response): R
$links = []; $links = [];
foreach ($urls as $url) { foreach ($urls as $url) {
if (empty($url)) {
continue;
}
$link = $this->buildLinkDataFromUrl($request, $url); $link = $this->buildLinkDataFromUrl($request, $url);
$data = $this->buildFormData($link, $link['linkIsNew'], $request); $data = $this->buildFormData($link, $link['linkIsNew'], $request);
$data['token'] = $this->container->sessionManager->generateToken(); $data['token'] = $this->container->sessionManager->generateToken();

3
tests/front/controller/admin/ShaarePublishControllerTest/DisplayCreateBatchFormTest.php
View file

@ -36,6 +36,7 @@ public function testDisplayCreateFormBatch(): void
$urls = [ $urls = [
'https://domain1.tld/url1', 'https://domain1.tld/url1',
'https://domain2.tld/url2', 'https://domain2.tld/url2',
' ',
'https://domain3.tld/url3', 'https://domain3.tld/url3',
]; ];
@ -57,6 +58,6 @@ public function testDisplayCreateFormBatch(): void
static::assertCount(3, $assignedVariables['links']); static::assertCount(3, $assignedVariables['links']);
static::assertSame($urls[0], $assignedVariables['links'][0]['link']['url']); static::assertSame($urls[0], $assignedVariables['links'][0]['link']['url']);
static::assertSame($urls[1], $assignedVariables['links'][1]['link']['url']); static::assertSame($urls[1], $assignedVariables['links'][1]['link']['url']);
static::assertSame($urls[2], $assignedVariables['links'][2]['link']['url']); static::assertSame($urls[3], $assignedVariables['links'][2]['link']['url']);
} }
} }