Followup to #2730131: Add a commerce_number_pattern submodule for sequential (order/invoice) number generation.

Now that the module has landed, commerce_order needs to depend on it, and enable it in an update hook. The order type needs a number pattern reference. And the OrderNumberSubscriber needs to use the number pattern to generate an order number.

CommentFileSizeAuthor
#4 3080229-4-enable-number-pattern.patch19.83 KBbojanz
#4 3080229-4-enable-number-pattern.patch19.83 KBbojanz
#3 3080229-3-enable-number-pattern.patch17.15 KBbojanz
#2 3080229-2-enable-number-pattern.patch8.03 KBbojanz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bojanz created an issue. See original summary.

bojanz’s picture

bojanz CreditAttribution: bojanz at Centarro commented
Status: Active » Needs review
FileSize
3080229-2-enable-number-pattern.patch8.03 KB

Here's some initial work.

bojanz’s picture

bojanz CreditAttribution: bojanz at Centarro commented
FileSize
3080229-3-enable-number-pattern.patch17.15 KB

Added the needed code to OrderNumberSubscriber and updated OrderNumberTest.
Fixed the ConfigUpdaterTest, which was enabling commerce_order for no reasons.

EDIT: We're green. Now we just need to pretty up the OrderTypeForm.

bojanz’s picture

bojanz CreditAttribution: bojanz at Centarro commented
FileSize
3080229-4-enable-number-pattern.patch19.83 KB
3080229-4-enable-number-pattern.patch19.83 KB

Final patch. Improved order type UI, expanded UI tests.

  • bojanz committed 3b4ef56 on 8.x-2.x 
    Issue #3080229 by bojanz: Update commerce_order to use...
bojanz’s picture

bojanz CreditAttribution: bojanz at Centarro commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

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