Fix ExecIds type in container inspect in swagger file

[Rid]

Fixes docker-php/docker-php#304 where ExecIds expects a string but gets an array

[jcrombez]

Anyone can merge this ?

[Rid]

I had this merged upstream moby/moby#36962 and made a fork using 1.37, you're welcome to use it until this is updated, just put this in your composer.json:

...
    "repositories": [
      {
        "type": "package",
        "package": {
          "name": "docker-php/docker-php-api",
          "version": "4.1.37",
          "source": {
            "url": "https://github.com/Rid/docker-php-api.git",
            "type": "git",
            "reference": "4.1.37"
          },
          "autoload": {
            "classmap": [""]
          }
        }
      },
    ],
...
    "require": {
        ...
        "docker-php/docker-php-api": "4.1.37",
    },

[jcrombez]

Thank you ! @Rid

[jcrombez]

I tried using your fork but i get this error :

PHP Fatal error:  Class 'Jane\OpenApiRuntime\Client\Psr7HttplugClient' not found in /app/vendor/docker-php/docker-php-api/src/Client.php on line 13

So i added the deps manually, but then i get the invalid supplied for foreach warning i had initially so i guess my issue isn't exactly the same as this PR aim to fix.

PHP Warning:  Invalid argument supplied for foreach() in /app/vendor/docker-php/docker-php-api/src/Normalizer/NetworkSettingsNormalizer.php on line 60

[jcrombez]

my bad it was the other PR :$

[rodrigoaguilera]

Can we have this simple change merged?

[Chakratos]

please merge this, a big project of mine is depending on this merge.

[Rid]

I've updated my fork to docker api 1.41, you can use it by adding the following to your composer.json:

...
    "repositories": [
      {
        "type": "package",
        "package": {
          "name": "docker-php/docker-php-api",
          "version": "4.1.41",
          "source": {
            "url": "https://github.com/Rid/docker-php-api.git",
            "type": "git",
            "reference": "4.1.41"
          },
          "autoload": {
            "classmap": [""]
          }
        }
      },
    ],
...
    "require": {
        ...
        "docker-php/docker-php-api": "4.1.41",
    },

[rodrigoaguilera]

Have you proposed this changes upstream?

This project seems pretty abandoned so maybe it needs anew maintainer

[Rid]

@rodrigoaguilera this is the upstream repo, it generates the php methods and classes for https://github.com/docker-php/docker-php

I would be happy to maintain this repo if @joelwurtz gives me the required permissions.