We need to reimplement commerce_cart_order_convert() as a service, invoke it from hook_user_login().

Comments

bojanz created an issue. See original summary.

bojanz’s picture

bojanz CreditAttribution: bojanz at Centarro commented
Title: Reimplement order coversion » Reimplement order conversion

So, we need \Drupal\commerce_cart\CartOrderConverter and \Drupal\commerce_cart\CartOrderConverterInterface.

rszrama’s picture

rszrama CreditAttribution: rszrama at Centarro commented

It may be worth considering as a default or optional behavior the ability to delete previous cart orders as well, as this always comes up on 1.x.

bojanz’s picture

bojanz CreditAttribution: bojanz at Centarro commented
Title: Reimplement order conversion » Reimplement cart order assignment

Changing to match new language (convert -> assign), after discussing with rszrama and mglaman.

I'll leave #3 to a followup, commit the critical part first.

  • bojanz committed 3c4273f on 8.x-2.x 
    Issue #2713501: Reimplement cart order assignment
bojanz’s picture

bojanz CreditAttribution: bojanz at Centarro commented
Status: Active » Fixed

Boom!

johnstorey’s picture

johnstorey CreditAttribution: johnstorey commented
Assigned: Unassigned » johnstorey
agoradesign’s picture

agoradesign CreditAttribution: agoradesign commented

Why do you assign yourself @johnstorey?? This is already committed and fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.