mirror of
				https://github.com/grocy/grocy.git
				synced 2025-10-31 02:36:54 +00:00 
			
		
		
		
	Optimize and refactor latest changes
This commit is contained in:
		| @@ -19,7 +19,6 @@ class BaseController | ||||
| 		$versionInfo = $applicationService->GetInstalledVersion(); | ||||
| 		$container->view->set('version', $versionInfo->Version); | ||||
| 		$container->view->set('releaseDate', $versionInfo->ReleaseDate); | ||||
| 		$container->view->set('isEmbeddedInstallation', $applicationService->IsEmbeddedInstallation()); | ||||
|  | ||||
| 		$container->view->set('localizationStrings', $localizationService->GetCurrentCultureLocalizations()); | ||||
| 		$container->view->set('L', function($text, ...$placeholderValues) use($localizationService) | ||||
|   | ||||
| @@ -1,19 +0,0 @@ | ||||
| <?php | ||||
|  | ||||
| namespace Grocy\Controllers; | ||||
|  | ||||
| use \Grocy\Services\ApplicationService; | ||||
| use \Grocy\Services\DatabaseMigrationService; | ||||
|  | ||||
| class CliController extends BaseController | ||||
| { | ||||
| 	public function RecreateDemo(\Slim\Http\Request $request, \Slim\Http\Response $response, array $args) | ||||
| 	{ | ||||
| 		$applicationService = new ApplicationService(); | ||||
| 		if ($applicationService->IsDemoInstallation()) | ||||
| 		{ | ||||
| 			$databaseMigrationService = new DatabaseMigrationService(); | ||||
| 			$databaseMigrationService->RecreateDemo(); | ||||
| 		} | ||||
| 	} | ||||
| } | ||||
| @@ -3,7 +3,6 @@ | ||||
| namespace Grocy\Controllers; | ||||
|  | ||||
| use \Grocy\Services\SessionService; | ||||
| use \Grocy\Services\ApplicationService; | ||||
| use \Grocy\Services\DatabaseMigrationService; | ||||
| use \Grocy\Services\DemoDataGeneratorService; | ||||
|  | ||||
| @@ -31,8 +30,6 @@ class LoginController extends BaseController | ||||
| 			{ | ||||
| 				$sessionKey = $this->SessionService->CreateSession($user->id); | ||||
| 				setcookie($this->SessionCookieName, $sessionKey, time() + 31536000); // Cookie expires in 1 year, but session validity is up to SessionService | ||||
| 				define('GROCY_USER_USERNAME', $user->username); | ||||
| 				define('GROCY_USER_ID', $user->id); | ||||
|  | ||||
| 				if (password_needs_rehash($user->password, PASSWORD_DEFAULT)) | ||||
| 				{ | ||||
| @@ -71,8 +68,7 @@ class LoginController extends BaseController | ||||
| 		$databaseMigrationService = new DatabaseMigrationService(); | ||||
| 		$databaseMigrationService->MigrateDatabase(); | ||||
|  | ||||
| 		$applicationService = new ApplicationService(); | ||||
| 		if ($applicationService->IsDemoInstallation()) | ||||
| 		if (GROCY_IS_DEMO_INSTALL) | ||||
| 		{ | ||||
| 			$demoDataGeneratorService = new DemoDataGeneratorService(); | ||||
| 			$demoDataGeneratorService->PopulateDemoData(); | ||||
| @@ -81,30 +77,6 @@ class LoginController extends BaseController | ||||
| 		return $response->withRedirect($this->AppContainer->UrlManager->ConstructUrl('/stockoverview')); | ||||
| 	} | ||||
|  | ||||
| 	public function UsersList(\Slim\Http\Request $request, \Slim\Http\Response $response, array $args) | ||||
| 	{ | ||||
| 		return $this->AppContainer->view->render($response, 'users', [ | ||||
| 			'users' => $this->Database->users()->orderBy('username') | ||||
| 		]); | ||||
| 	} | ||||
|  | ||||
| 	public function UserEditForm(\Slim\Http\Request $request, \Slim\Http\Response $response, array $args) | ||||
| 	{ | ||||
| 		if ($args['userId'] == 'new') | ||||
| 		{ | ||||
| 			return $this->AppContainer->view->render($response, 'userform', [ | ||||
| 				'mode' => 'create' | ||||
| 			]); | ||||
| 		} | ||||
| 		else | ||||
| 		{ | ||||
| 			return $this->AppContainer->view->render($response, 'userform', [ | ||||
| 				'user' =>  $this->Database->users($args['userId']), | ||||
| 				'mode' => 'edit' | ||||
| 			]); | ||||
| 		} | ||||
| 	} | ||||
|  | ||||
| 	public function GetSessionCookieName() | ||||
| 	{ | ||||
| 		return $this->SessionCookieName; | ||||
|   | ||||
| @@ -35,7 +35,8 @@ class OpenApiController extends BaseApiController | ||||
| 	public function ApiKeysList(\Slim\Http\Request $request, \Slim\Http\Response $response, array $args) | ||||
| 	{ | ||||
| 		return $this->AppContainer->view->render($response, 'manageapikeys', [ | ||||
| 			'apiKeys' => $this->Database->api_keys() | ||||
| 			'apiKeys' => $this->Database->api_keys(), | ||||
| 			'users' => $this->Database->users() | ||||
| 		]); | ||||
| 	} | ||||
|  | ||||
|   | ||||
| @@ -14,6 +14,18 @@ class UsersApiController extends BaseApiController | ||||
|  | ||||
| 	protected $UsersService; | ||||
|  | ||||
| 	public function GetUsers(\Slim\Http\Request $request, \Slim\Http\Response $response, array $args) | ||||
| 	{ | ||||
| 		try | ||||
| 		{ | ||||
| 			return $this->ApiResponse($this->UsersService->GetUsersAsDto()); | ||||
| 		} | ||||
| 		catch (\Exception $ex) | ||||
| 		{ | ||||
| 			return $this->VoidApiActionResponse($response, false, 400, $ex->getMessage()); | ||||
| 		} | ||||
| 	} | ||||
|  | ||||
| 	public function CreateUser(\Slim\Http\Request $request, \Slim\Http\Response $response, array $args) | ||||
| 	{ | ||||
| 		$requestBody = $request->getParsedBody(); | ||||
| @@ -33,8 +45,8 @@ class UsersApiController extends BaseApiController | ||||
| 	{ | ||||
| 		try | ||||
| 		{ | ||||
| 			$success = $this->UsersService->DeleteUser($args['userId']); | ||||
| 			return $this->ApiResponse(array('success' => $success)); | ||||
| 			$this->UsersService->DeleteUser($args['userId']); | ||||
| 			return $this->ApiResponse(array('success' => true)); | ||||
| 		} | ||||
| 		catch (\Exception $ex) | ||||
| 		{ | ||||
|   | ||||
							
								
								
									
										30
									
								
								controllers/UsersController.php
									
									
									
									
									
										Normal file
									
								
							
							
						
						
									
										30
									
								
								controllers/UsersController.php
									
									
									
									
									
										Normal file
									
								
							| @@ -0,0 +1,30 @@ | ||||
| <?php | ||||
|  | ||||
| namespace Grocy\Controllers; | ||||
|  | ||||
| class UsersController extends BaseController | ||||
| { | ||||
| 	public function UsersList(\Slim\Http\Request $request, \Slim\Http\Response $response, array $args) | ||||
| 	{ | ||||
| 		return $this->AppContainer->view->render($response, 'users', [ | ||||
| 			'users' => $this->Database->users()->orderBy('username') | ||||
| 		]); | ||||
| 	} | ||||
|  | ||||
| 	public function UserEditForm(\Slim\Http\Request $request, \Slim\Http\Response $response, array $args) | ||||
| 	{ | ||||
| 		if ($args['userId'] == 'new') | ||||
| 		{ | ||||
| 			return $this->AppContainer->view->render($response, 'userform', [ | ||||
| 				'mode' => 'create' | ||||
| 			]); | ||||
| 		} | ||||
| 		else | ||||
| 		{ | ||||
| 			return $this->AppContainer->view->render($response, 'userform', [ | ||||
| 				'user' =>  $this->Database->users($args['userId']), | ||||
| 				'mode' => 'edit' | ||||
| 			]); | ||||
| 		} | ||||
| 	} | ||||
| } | ||||
		Reference in New Issue
	
	Block a user