Skip to content

Commit fd27ffd

Browse files
committed
PB-366: Templates Lower Level Test Coverage
- Resolve issue with valid name check
1 parent 0795f1a commit fd27ffd

File tree

1 file changed

+2
-2
lines changed
  • app/code/Magento/PageBuilder/Controller/Adminhtml/Template

1 file changed

+2
-2
lines changed

app/code/Magento/PageBuilder/Controller/Adminhtml/Template/Save.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,11 +19,11 @@
1919
use Magento\Framework\Controller\ResultFactory;
2020
use Magento\Framework\Exception\LocalizedException;
2121
use Magento\Framework\Filesystem;
22+
use Magento\MediaStorage\Helper\File\Storage\Database;
2223
use Magento\PageBuilder\Api\Data\TemplateInterface;
2324
use Magento\PageBuilder\Api\TemplateRepositoryInterface;
2425
use Magento\PageBuilder\Model\TemplateFactory;
2526
use Psr\Log\LoggerInterface;
26-
use Magento\MediaStorage\Helper\File\Storage\Database;
2727

2828
/**
2929
* Save a template within template manager
@@ -186,7 +186,7 @@ public function execute()
186186
private function validate(RequestInterface $request)
187187
{
188188
// If we're missing required data return an error
189-
if (!$request->getParam(TemplateInterface::KEY_NAME)
189+
if (trim($request->getParam(TemplateInterface::KEY_NAME, "")) === ""
190190
|| !$request->getParam(TemplateInterface::KEY_TEMPLATE)
191191
) {
192192
throw new LocalizedException(__('A required field is missing.'));

0 commit comments

Comments
 (0)