Create User API Not passing in function

Problem

When using Create User API, error received, does not appear to parse body.

Version

Version 1.22.0

Environment

Docker on Ubuntu 18.10 Host

Command;

curl -X POST "https://grocy.fish.local/api/users/create" -H "GROCY-API-KEY: SecretKey" -H "accept: application/json" -H "Content-Type: application/json" -d "{"id":5,"username":"string12","first_name":"string12","last_name":"string12","password":"string2","row_created_timestamp":"2018-12-30T21:30:39.906Z"}"

Error:

\/www\/controllers\/UsersApiController.php(35): Grocy\\Services\\UsersService->CreateUser(NULL, NULL, NULL, NULL)"

If I change the function to;

public function CreateUser(\Slim\Http\Request $request, \Slim\Http\Response $response, array $args)
        {
                $requestBody = $request->getParsedBody();

                try
                {
                        $this->UsersService->CreateUser('username','first','last','pass');
                        return $this->ApiResponse(array('success' => true));
                }
                catch (\Exception $ex)
                {
                        return $this->VoidApiActionResponse($response, false, 400, $ex->getMessage());
                }
        }

It creates the user matching the test variables, however I couldn’t get it to work trying to parse $request. I tried to match from the ChoreTrackExecution with

$request->getQueryParams()['username']

But it seems the $request variable isn’t populating correctly.

[gen2fish]

So the Swagger worked when I fixed the trailing comma that you pointed out.

I also believe I fixed my issue. Powershell was not passing the body of the request as JSON and so it wasn’t getting accepted. Once I adjusted my script to convert to JSON as so;

$params = @{
    body =  @{
      "id" = 0
      "username" = $UserName
      "first_name" = $FirstName
      "last_name" = $LastName
      "password" = $Passwd
      "row_created_timestamp"= $Execute
    } | ConvertTo-Json -Depth 5
      uri = "$URI/api/users/create"
      headers = @{
        "accept" = "application/json"
        "Content-Type" = "application/json"
        "GROCY-API-KEY" = $global:grocyGlobal.APIKEY
      }
      method = "Post"
  }

It worked as expected.

Thank you for the responses and the great app.

[berrnd]

Great, and thanks!

Maybe you’re the first who uses the API with gen2fish/grocy-posh, at least the first project that I saw - so hope the API covers everything needed…

If you think this is something “everyone” could need, feel free to request it to be added under the github.com/grocy organization.

Happy new year! 🎆 🥂