diff options
-rw-r--r-- | application/front/controller/admin/ShaarePublishController.php | 3 | ||||
-rw-r--r-- | tests/front/controller/admin/ShaarePublishControllerTest/DisplayCreateBatchFormTest.php | 3 |
2 files changed, 5 insertions, 1 deletions
diff --git a/application/front/controller/admin/ShaarePublishController.php b/application/front/controller/admin/ShaarePublishController.php index 65fdcdee..ddcffdc7 100644 --- a/application/front/controller/admin/ShaarePublishController.php +++ b/application/front/controller/admin/ShaarePublishController.php | |||
@@ -46,6 +46,9 @@ class ShaarePublishController extends ShaarliAdminController | |||
46 | 46 | ||
47 | $links = []; | 47 | $links = []; |
48 | foreach ($urls as $url) { | 48 | foreach ($urls as $url) { |
49 | if (empty($url)) { | ||
50 | continue; | ||
51 | } | ||
49 | $link = $this->buildLinkDataFromUrl($request, $url); | 52 | $link = $this->buildLinkDataFromUrl($request, $url); |
50 | $data = $this->buildFormData($link, $link['linkIsNew'], $request); | 53 | $data = $this->buildFormData($link, $link['linkIsNew'], $request); |
51 | $data['token'] = $this->container->sessionManager->generateToken(); | 54 | $data['token'] = $this->container->sessionManager->generateToken(); |
diff --git a/tests/front/controller/admin/ShaarePublishControllerTest/DisplayCreateBatchFormTest.php b/tests/front/controller/admin/ShaarePublishControllerTest/DisplayCreateBatchFormTest.php index 34547120..ce8e112b 100644 --- a/tests/front/controller/admin/ShaarePublishControllerTest/DisplayCreateBatchFormTest.php +++ b/tests/front/controller/admin/ShaarePublishControllerTest/DisplayCreateBatchFormTest.php | |||
@@ -36,6 +36,7 @@ class DisplayCreateBatchFormTest extends TestCase | |||
36 | $urls = [ | 36 | $urls = [ |
37 | 'https://domain1.tld/url1', | 37 | 'https://domain1.tld/url1', |
38 | 'https://domain2.tld/url2', | 38 | 'https://domain2.tld/url2', |
39 | ' ', | ||
39 | 'https://domain3.tld/url3', | 40 | 'https://domain3.tld/url3', |
40 | ]; | 41 | ]; |
41 | 42 | ||
@@ -57,6 +58,6 @@ class DisplayCreateBatchFormTest extends TestCase | |||
57 | static::assertCount(3, $assignedVariables['links']); | 58 | static::assertCount(3, $assignedVariables['links']); |
58 | static::assertSame($urls[0], $assignedVariables['links'][0]['link']['url']); | 59 | static::assertSame($urls[0], $assignedVariables['links'][0]['link']['url']); |
59 | static::assertSame($urls[1], $assignedVariables['links'][1]['link']['url']); | 60 | static::assertSame($urls[1], $assignedVariables['links'][1]['link']['url']); |
60 | static::assertSame($urls[2], $assignedVariables['links'][2]['link']['url']); | 61 | static::assertSame($urls[3], $assignedVariables['links'][2]['link']['url']); |
61 | } | 62 | } |
62 | } | 63 | } |