Skip to content
Snippets Groups Projects
Commit 9d6ea89e authored by damjan's avatar damjan
Browse files

[BUGFIX] Check if 'name' parameter is set before reading it

Example when it doesn't need to be set is PATCH method
parent c1f36bc3
No related branches found
No related tags found
No related merge requests found
......@@ -216,8 +216,8 @@ abstract class AbstractPropertyController extends FOSRestController {
private function processForm(Property $property, array $parameters, $method = 'PUT') {
$form = $this->createForm(new PropertyType(), $property, ['method' => $method]);
if (!isset($parameters['urlName'])) {
// If urlName is not already calculated and set in request parameters, then set it now.
if (!isset($parameters['urlName']) && isset($parameters['name'])) {
// If urlName is not already calculated and set in request parameters and if name parameter is set, then set urlName.
$property->setName($parameters['name']);
$parameters['urlName'] = $property->getUrlName();
}
......
......@@ -220,8 +220,8 @@ abstract class AbstractTypeController extends FOSRestController {
private function processForm(Type $type, array $parameters, $method = 'PUT') {
$form = $this->createForm(new TypeType(), $type, ['method' => $method]);
if (!isset($parameters['urlName'])) {
// If urlName is not already calculated and set in request parameters, then set it now.
if (!isset($parameters['urlName']) && isset($parameters['name'])) {
// If urlName is not already calculated and set in request parameters and if name parameter is set, then set urlName.
$type->setName($parameters['name']);
$parameters['urlName'] = $type->getUrlName();
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment