hotfix(link-resolver): Update to new interface definition

CHANGELOG.md

@@ -5,6 +5,10 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 
 ## [Unreleased](https://github.com/contentful/contentful.php/compare/4.0.0...HEAD)
 
+### Fixed
+
+* Second parameter in `LinkResolver::resolveLinkCollection()` was changed from `string $locale = null` to `array $parameters = []`.
+
 ## [4.0.0](https://github.com/contentful/contentful.php/tree/4.0.0) (2018-11-08)
 
 **ATTENTION**: This release contains breaking changes. Please take extra care when updating to this version. See [the upgrade guide](UPGRADE-4.0.md) for more.

composer.json

@@ -5,7 +5,7 @@
     "license": "MIT",
     "require": {
         "php": "^7.0",
-        "contentful/core": "^2.1",
+        "contentful/core": "^2.1.1",
         "contentful/rich-text": "^1.2",
         "psr/cache": "^1.0",
         "symfony/console": "~2.7|~3.0|~4.0",

src/Client.php

@@ -414,7 +414,9 @@ public function resolveLink(Link $link, string $locale = \null): ResourceInterfa
      */
     public function resolveLinkCollection(array $links, string $locale = \null): array
     {
-        return $this->linkResolver->resolveLinkCollection($links, $locale);
+        return $this->linkResolver->resolveLinkCollection($links, [
+            'locale' => (string) $locale,
+        ]);
     }
 
     /**

src/LinkResolver.php

@@ -70,8 +70,10 @@ public function resolveLink(Link $link, array $parameters = []): ResourceInterfa
     /**
      * {@inheritdoc}
      */
-    public function resolveLinkCollection(array $links, string $locale = \null): array
+    public function resolveLinkCollection(array $links, array $parameters = []): array
     {
+        $locale = $parameters['locale'] ?? \null;
+
         // We load all resources for the given resource types
         $types = \array_unique(\array_map(function (Link $link) {
             return $link->getLinkType();

tests/Implementation/LinkResolver.php

@@ -28,10 +28,10 @@ public function resolveLink(Link $link, array $parameters = []): ResourceInterfa
     /**
      * {@inheritdoc}
      */
-    public function resolveLinkCollection(array $links, string $locale = \null): array
+    public function resolveLinkCollection(array $links, array $parameters = []): array
     {
-        return \array_map(function (Link $link) use ($locale): ResourceInterface {
-            return $this->resolveLink($link, $locale);
+        return \array_map(function (Link $link) use ($parameters): ResourceInterface {
+            return $this->resolveLink($link, $parameters);
         }, $links);
     }
 }

tests/Unit/LinkResolverTest.php

@@ -104,6 +104,6 @@ public function testInvalidLinkCollection()
     {
         $linkResolver = new LinkResolver(new MockClient(), new MockResourcePool());
 
-        $linkResolver->resolveLinkCollection([new Link('irrelevant', 'InvalidType')]);
+        $linkResolver->resolveLinkCollection([new Link('irrelevant', 'InvalidType')], []);
     }
 }