search

verbb / postie

A Craft Commerce shipping calculator plugin.
Other
12 stars 18 forks source link

`$packedBoxes` serialized twice #89

Closed jaydensmith closed 2 years ago

jaydensmith commented 2 years ago

Description $packedBoxes is serialized twice, causing the second invocation to throw an exception. https://github.com/verbb/postie/blob/9b358fd1137713b071e518ad3f2dbf4b0b160419/src/base/SinglePackageProvider.php#L33 https://github.com/verbb/postie/blob/9b358fd1137713b071e518ad3f2dbf4b0b160419/src/base/Provider.php#L633

engram-design commented 2 years ago

Hey buddy! Thanks for that, fixed this for the next version. To get the fix early, change your verbb/postie requirement in composer.json to:

"require": {
  "verbb/postie": "dev-craft-3 as 2.4.16",
  "...": "..."
}

Then run composer update.

jaydensmith commented 2 years ago

Hey buddy! Thanks for that, fixed this for the next version. To get the fix early, change your verbb/postie requirement in composer.json to:

"require": {
  "verbb/postie": "dev-craft-3 as 2.4.16",
  "...": "..."
}

Then run composer update.

Thanks man!

engram-design commented 2 years ago

Fixed in 2.4.17