Minor functioinality improvements

This commit is contained in:
Dennis Eichhorn 2018-11-23 11:37:30 +01:00
parent b0d5839ba6
commit 67bca98fb0
2 changed files with 17 additions and 12 deletions

View File

@ -39,6 +39,7 @@ use phpOMS\Utils\Parser\Markdown\Markdown;
use phpOMS\Module\InfoManager;
use phpOMS\Account\PermissionAbstract;
use phpOMS\Account\PermissionOwner;
use phpOMS\Validation\Network\Email;
/**
* Admin controller class.
@ -305,6 +306,7 @@ final class ApiController extends Controller
*/
public function apiGroupFind(RequestAbstract $request, ResponseAbstract $response, $data = null) : void
{
$response->getHeader()->set('Content-Type', MimeType::M_JSON, true);
$response->set(
$request->getUri()->__toString(),
\array_values(
@ -354,6 +356,7 @@ final class ApiController extends Controller
*/
public function apiAccountFind(RequestAbstract $request, ResponseAbstract $response, $data = null) : void
{
$response->getHeader()->set('Content-Type', MimeType::M_JSON, true);
$response->set(
$request->getUri()->__toString(),
\array_values(
@ -373,12 +376,12 @@ final class ApiController extends Controller
*/
private function validateAccountCreate(RequestAbstract $request) : array
{
// todo: validate email correctness
$val = [];
if (($val['login'] = empty($request->getData('login')))
|| ($val['name1'] = empty($request->getData('name1')))
|| ($val['type'] = !AccountType::isValidValue((int) $request->getData('type')))
|| ($val['status'] = !AccountStatus::isValidValue((int) $request->getData('status')))
|| ($val['email'] = !empty($request->getData('email')) && !Email::isValid((string) $request->getData('email')))
) {
return $val;
}
@ -443,6 +446,8 @@ final class ApiController extends Controller
$account->setEmail((string) ($request->getData('email') ?? ''));
$account->generatePassword((string) ($request->getData('password') ?? ''));
// todo: set remaining login tries based on global default config here. but before you have to add it to the model and mapper
return $account;
}

View File

@ -78,7 +78,7 @@ class Module
}
/**
* Get group id.
* Get module id.
*
* @return int
*
@ -102,7 +102,7 @@ class Module
}
/**
* Get group name.
* Get module name.
*
* @return string
*
@ -114,9 +114,9 @@ class Module
}
/**
* Set group name.
* Set module name.
*
* @param string $name Group name
* @param string $name module name
*
* @return void
*
@ -128,7 +128,7 @@ class Module
}
/**
* Get group description.
* Get module description.
*
* @return string
*
@ -140,9 +140,9 @@ class Module
}
/**
* Set group description.
* Set module description.
*
* @param string $description Group description
* @param string $description Module description
*
* @return void
*
@ -154,9 +154,9 @@ class Module
}
/**
* Get group status.
* Get module status.
*
* @return int Group status
* @return int Module status
*
* @since 1.0.0
*/
@ -166,9 +166,9 @@ class Module
}
/**
* Set group status.
* Set module status.
*
* @param int $status Group status
* @param int $status Module status
*
* @return void
*