spryker/product-image-storage 1.15.0 causing exceptions

U03GRNCV1B4

Hello,

4 days ago this commit was introduced with a new tag 1.15.0 in spryker/product-image-storage .
https://github.com/spryker/product-image-storage/commit/ab4e157b84b07a2006d17cfbc3b0145e230b6e62#diff-126f52f35440e9105c3[…]35f7fc5a4e68702d9e719b81

The bridge now has an array as fixed type, however, the underlying storage-client has multiple return types null|array|....
This causes exceptions whenever the underlying storage-client return something else than array.

I did not know where else to contact than here, as the issue board on github is deactivated. ❤

fsmeier

Hey, good catch! I reported it internally. If you want to push priority you can do this by creating a bug issue through either the partner portal (if you work for an official partner agency) or the customer portal (if you work for a customer right now). There is normally a business person on project level who has access to it.

If there was a ticket submitted please let me know so i can let it be linked to the one i reported 🙂

U03GRNCV1B4

There is no urgency as 1.14.0 is also available and working well so far.

We just pinned that version.

Ty for reaction ❤

U02MH0VAFFT

Hey everyone, we just updated spryker/product-image-storage to version 1.15.0 and with this change https://github.com/spryker/product-image-storage/compare/1.14.0...1.15.0#diff-69f3fb1747c034c9e847e1bc861258e9dcdd9581a0a12e76ab8f943fa744a488R17 it broke our implementation due to this error ERROR - Declaration of Spryker\Client\Storage\StorageClientInterface::get($key) must be compatible with Spryker\Client\ProductImageStorage\Dependency\Client\ProductImageStorageToStorageInterface::get($key): array this happen because we have a Pyz override of Spryker\Client\Storage\StorageClientInterface which is extending ProductImageStorageToStorageInterface for us to be able to have access to \Spryker\Client\Storage\StorageClientInterface::getMulti() in the places where this bridge is used. Since the get() method return type is now different in the bridge it gives this error. Just FYI in case this needs to be changed on your side

U02MH0VAFFT

And I looked a bit deeper and it seems that the underlying StorageClient could potentially return null which would then break due to the type not being an array

fsmeier

The more “official” reports through the dedicated support channels the higher prio 🙂 - Here in slack is no official support. Here we help each other - from dev to dev 💪

U02MH0VAFFT

Makes sense, thank you for the info 🙏

fsmeier

Heyhey, should be fixed now: https://github.com/spryker/product-image-storage/releases/tag/1.15.1