Commerce: saveShippingMethod does not call _memoizeShippingMethod

Created on 26 Aug 2020  路  3Comments  路  Source: craftcms/commerce

When calling getAllShippingMethods(), then saveShippingMethod() and finally another getAllShippingMethods(), it does not show the saved one.

Two obvious choices here:

  • Maybe updating _fetchedAllShippingMethods to false,
  • Or probably better, just calling _memoizeShippingMethod() with the saved one.
馃悶 bug
Source
picture cgivord

Most helpful comment

Thanks this has been fixed for the next release.

picture lukeholder on 31 Aug 2020
🚀1 ❤1

All 3 comments

Thanks for reporting. Marked as a bug, and will fix shortly.

lukeholder picture lukeholder on 27 Aug 2020
❤1 👍1

Fixed in b07702ed88daadaa67716ef197e4af7c5e7f64ed

lukeholder picture lukeholder on 31 Aug 2020

Thanks this has been fixed for the next release.

lukeholder picture lukeholder on 31 Aug 2020
🚀1 ❤1
Was this page helpful?
0 / 5 - 0 ratings

Related issues

nfourtythree picture nfourtythree  路  6Comments
Tam picture Tam  路  7Comments
engram-design picture engram-design  路  4Comments
alexanderloewe picture alexanderloewe  路  4Comments
Anubarak picture Anubarak  路  3Comments