diff --git a/Models/AccountMapper.php b/Models/AccountMapper.php index 76eb47e..4a5d3d6 100644 --- a/Models/AccountMapper.php +++ b/Models/AccountMapper.php @@ -37,7 +37,7 @@ final class AccountMapper extends DataMapperAbstract * @since 1.0.0 */ protected static array $columns = [ - 'account_id' => ['name' => 'account_id', 'type' => 'int', 'internal' => 'id', 'autocomplete' => true], + 'account_id' => ['name' => 'account_id', 'type' => 'int', 'internal' => 'id'], 'account_status' => ['name' => 'account_status', 'type' => 'int', 'internal' => 'status'], 'account_type' => ['name' => 'account_type', 'type' => 'int', 'internal' => 'type'], 'account_login' => ['name' => 'account_login', 'type' => 'string', 'internal' => 'login', 'autocomplete' => true], diff --git a/Models/GroupMapper.php b/Models/GroupMapper.php index 3b764c5..ba2d721 100644 --- a/Models/GroupMapper.php +++ b/Models/GroupMapper.php @@ -15,6 +15,7 @@ declare(strict_types=1); namespace Modules\Admin\Models; use phpOMS\DataStorage\Database\DataMapperAbstract; +use phpOMS\DataStorage\Database\RelationType; /** * Group mapper class. @@ -99,11 +100,12 @@ final class GroupMapper extends DataMapperAbstract */ public static function getPermissionForModule(string $module) : array { + $depth = 3; $query = self::getQuery(); $query->innerJoin(GroupPermissionMapper::getTable()) - ->on(self::$table . '.group_id', '=', GroupPermissionMapper::getTable() . '.group_permission_group') + ->on(self::$table . '_' . $depth . '.group_id', '=', GroupPermissionMapper::getTable() . '.group_permission_group') ->where(GroupPermissionMapper::getTable() . '.group_permission_module', '=', $module); - return self::getAllByQuery($query); + return self::getAllByQuery($query, RelationType::ALL, $depth); } } diff --git a/tests/Controller/ApiControllerTest.php b/tests/Controller/ApiControllerTest.php index 782588b..9c93421 100644 --- a/tests/Controller/ApiControllerTest.php +++ b/tests/Controller/ApiControllerTest.php @@ -14,17 +14,15 @@ declare(strict_types=1); namespace Modules\Admin\tests\Controller; -require_once __DIR__ . '/../../tests/../Autoloader.php'; - use Model\CoreSettings; use Modules\Admin\Models\AccountPermission; -use Modules\tests\Admin\Controller\Api\ApiControllerAccountTrait; -use Modules\tests\Admin\Controller\Api\ApiControllerGroupTrait; -use Modules\tests\Admin\Controller\Api\ApiControllerModuleTrait; -use Modules\tests\Admin\Controller\Api\ApiControllerPermissionTrait; -use Modules\tests\Admin\Controller\Api\ApiControllerSettingsTrait; +use Modules\Admin\tests\Controller\Api\ApiControllerAccountTrait; +use Modules\Admin\tests\Controller\Api\ApiControllerGroupTrait; +use Modules\Admin\tests\Controller\Api\ApiControllerModuleTrait; +use Modules\Admin\tests\Controller\Api\ApiControllerPermissionTrait; +use Modules\Admin\tests\Controller\Api\ApiControllerSettingsTrait; use phpOMS\Account\Account; use phpOMS\Account\AccountManager; @@ -38,7 +36,7 @@ use phpOMS\Router\WebRouter; use phpOMS\Utils\TestUtils; /** - * @testdox Modules\tests\Admin\Controller\ApiControllerTest: Admin api controller + * @testdox Modules\Admin\tests\Controller\ApiControllerTest: Admin api controller * * @internal */ diff --git a/tests/Models/AccountMapperTest.php b/tests/Models/AccountMapperTest.php index 5310b42..c3519ff 100644 --- a/tests/Models/AccountMapperTest.php +++ b/tests/Models/AccountMapperTest.php @@ -21,7 +21,7 @@ use phpOMS\Account\AccountType; use phpOMS\Auth\LoginReturnType; /** - * @testdox Modules\tests\Admin\Models\LocalizationMapperTest: Account database mapper + * @testdox Modules\Admin\tests\Models\LocalizationMapperTest: Account database mapper * * @internal */ diff --git a/tests/Models/AccountPermissionTest.php b/tests/Models/AccountPermissionTest.php index b5df0bf..2a35f00 100644 --- a/tests/Models/AccountPermissionTest.php +++ b/tests/Models/AccountPermissionTest.php @@ -17,7 +17,7 @@ namespace Modules\Admin\tests\Models; use Modules\Admin\Models\AccountPermission; /** - * @testdox Modules\tests\Admin\Models\AccountPermissionTest: Account permission + * @testdox Modules\Admin\tests\Models\AccountPermissionTest: Account permission * * @internal */ diff --git a/tests/Models/AccountTest.php b/tests/Models/AccountTest.php index c4e56da..f66677e 100644 --- a/tests/Models/AccountTest.php +++ b/tests/Models/AccountTest.php @@ -17,7 +17,7 @@ namespace Modules\Admin\tests\Models; use Modules\Admin\Models\Account; /** - * @testdox Modules\tests\Admin\Models\AccountTest: Account model + * @testdox Modules\Admin\tests\Models\AccountTest: Account model * * @internal */ diff --git a/tests/Models/GroupMapperTest.php b/tests/Models/GroupMapperTest.php index a80b239..e5e0beb 100644 --- a/tests/Models/GroupMapperTest.php +++ b/tests/Models/GroupMapperTest.php @@ -20,7 +20,7 @@ use Modules\Admin\Models\GroupPermission; use Modules\Admin\Models\GroupPermissionMapper; /** - * @testdox Modules\tests\Admin\Models\GroupMapperTest: Group mapper + * @testdox Modules\Admin\tests\Models\GroupMapperTest: Group mapper * * @internal */ diff --git a/tests/Models/GroupPermissionTest.php b/tests/Models/GroupPermissionTest.php index e6a5caa..95f9562 100644 --- a/tests/Models/GroupPermissionTest.php +++ b/tests/Models/GroupPermissionTest.php @@ -17,7 +17,7 @@ namespace Modules\Admin\tests\Models; use Modules\Admin\Models\GroupPermission; /** - * @testdox Modules\tests\Admin\Models\GroupPermissionTest: Group permission + * @testdox Modules\Admin\tests\Models\GroupPermissionTest: Group permission * * @internal */ diff --git a/tests/Models/GroupTest.php b/tests/Models/GroupTest.php index 437c439..d17b70d 100644 --- a/tests/Models/GroupTest.php +++ b/tests/Models/GroupTest.php @@ -18,7 +18,7 @@ use Modules\Admin\Models\Group; use Modules\Admin\Models\NullAccount; /** - * @testdox Modules\tests\Admin\Models\GroupTest: Group model + * @testdox Modules\Admin\tests\Models\GroupTest: Group model * * @internal */ diff --git a/tests/Models/LocalizationMapperTest.php b/tests/Models/LocalizationMapperTest.php index 69781bc..2ad0841 100644 --- a/tests/Models/LocalizationMapperTest.php +++ b/tests/Models/LocalizationMapperTest.php @@ -18,7 +18,7 @@ use Modules\Admin\Models\LocalizationMapper; use phpOMS\Localization\Localization; /** - * @testdox Modules\tests\Admin\Models\LocalizationMapperTest: Localization database mapper + * @testdox Modules\Admin\tests\Models\LocalizationMapperTest: Localization database mapper * * @internal */ diff --git a/tests/Models/ModuleTest.php b/tests/Models/ModuleTest.php index 3fd5267..6c0ef18 100644 --- a/tests/Models/ModuleTest.php +++ b/tests/Models/ModuleTest.php @@ -18,7 +18,7 @@ use Modules\Admin\Models\Module; use phpOMS\Module\ModuleStatus; /** - * @testdox Modules\tests\Admin\Models\ModuleTest: Module container + * @testdox Modules\Admin\tests\Models\ModuleTest: Module container * * @internal */