diff --git a/Controller/ApiController.php b/Controller/ApiController.php index 8a39c6b..596e42f 100755 --- a/Controller/ApiController.php +++ b/Controller/ApiController.php @@ -29,6 +29,7 @@ use phpOMS\Message\ResponseAbstract; use phpOMS\Model\Message\FormValidation; use phpOMS\Module\NullModule; use phpOMS\Utils\Parser\Markdown\Markdown; +use phpOMS\Message\Http\RequestStatusCode; /** * News controller class. @@ -135,6 +136,7 @@ final class ApiController extends Controller { if (!empty($val = $this->validateNewsCreate($request))) { $response->set('news_create', new FormValidation($val)); + $response->getHeader()->setStatusCode(RequestStatusCode::R_400); return; } diff --git a/tests/Controller/Api/ApiControllerNewsArticleTrait.php b/tests/Controller/Api/ApiControllerNewsArticleTrait.php index 0384880..0b48779 100755 --- a/tests/Controller/Api/ApiControllerNewsArticleTrait.php +++ b/tests/Controller/Api/ApiControllerNewsArticleTrait.php @@ -14,8 +14,10 @@ declare(strict_types=1); namespace Modules\News\tests\Controller\Api; +use Modules\News\Models\NewsArticleMapper; use Modules\News\Models\NewsStatus; use Modules\News\Models\NewsType; +use Modules\News\Models\NullNewsArticle; use phpOMS\Message\Http\HttpRequest; use phpOMS\Message\Http\HttpResponse; use phpOMS\Uri\HttpUri; @@ -99,6 +101,7 @@ trait ApiControllerNewsArticleTrait $request->setData('id', 1); $this->module->apiNewsDelete($request, $response); - self::assertGreaterThan(0, $response->get('')['response']->getId()); + self::assertEquals(1, $response->get('')['response']->getId()); + self::assertInstanceOf(NullNewsArticle::class, NewsArticleMapper::get(1)); } }