like title.

CommentFileSizeAuthor
#1 commerce-multiple-order-type.diff1021 bytesthehong

Comments

thehong’s picture

StatusFileSize
new1021 bytes

Status: Needs review » Needs work

The last submitted patch, commerce-multiple-order-type.diff, failed testing.

rszrama’s picture

Title: Support multi-order type - commerce_order_ui.module » Support multiple order types in core
Category: task » feature
Status: Needs work » Postponed

The code you've provided isn't sufficient, because there's much more that would need to be done to support multiple order types in core. Not only would we need to take care of the field interfaces, but we'd also need to take care of it in the Cart module specifically and potentially in other modules like Checkout. I'm making this a feature request but postponing it, as this type of change won't get in before 2.x. I've added it to #1467180: Open 2.x Drupal Commerce branch as a reminder that this is a topic needing consideration.

PatchRanger’s picture

Initial implementation of it (let's say, a fundament for it) is done in #1509994: Missing entity property info results in notices / WSOD / fatal errors during installation.
There you could find hook descriptions and implementations for hook_commerce_order_type_info and hook_commerce_order_type_info_alter that are (obviously) necessary to support multiple order types.

Rosamunda’s picture

Is there any way to implement this in core?
Ther´s already Commerce a contrib Order types module, maybe it can be merged?

rszrama’s picture

Not in the 1.x branch; we intentionally left it out, but see my comment above about 2.x.

Rosamunda’s picture

Thanks!!

rszrama’s picture

Version: 7.x-1.x-dev » 8.x-2.x-dev
Issue summary: View changes
Status: Postponed » Postponed (maintainer needs more info)

I think this may already be taken care of?

bojanz’s picture

Status: Postponed (maintainer needs more info) » Fixed

Correct. There's a full UI, now we just need to add checkout into the mix.

Status: Fixed » Closed (fixed)

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