Defines the inline entity form for line items.
The same as it's done for Product Variations.

CommentFileSizeAuthor
#5 line_item_ief_class-2619238-4.patch1.83 KBvasike
#3 defines_the_inline-2619238-3.patch1.43 KBmglaman
#2 line_item_ief_class-2619238-2.patch1.33 KBvasike
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

vasike created an issue. See original summary.

vasike’s picture

vasike
Location Ramnicu Valcea
CreditAttribution: vasike commented
Status: Active » Needs review
FileSize
line_item_ief_class-2619238-2.patch1.33 KB

there is patch for this
Build LineItemInlineForm for Order line items.

mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman at Centarro commented
FileSize
defines_the_inline-2619238-3.patch1.43 KB

Update to use \Drupal\Core\StringTranslation\StringTranslationTrait for translation.

vasike’s picture

vasike
Location Ramnicu Valcea
CreditAttribution: vasike commented

There is a new patch
Update for previous patch (#2) : Add submission handler for LineItem, to set the LineItem title from purchased entity selected.

@mglaman : for now we'll keep the t(), StringTranslationTrait probably it should go in IEF, and then maybe used here.
but thanks a lot.

vasike’s picture

vasike
Location Ramnicu Valcea
CreditAttribution: vasike commented
FileSize
line_item_ief_class-2619238-4.patch1.83 KB

forgot the patch file

mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman at Centarro commented

vasike makes sense!

bojanz’s picture

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

Moved the logic to an #entity_builder so that it integrates more cleanly with IEF, and committed.

  • bojanz committed bc9ed14 on 8.x-2.x authored by vasike 
    Issue #2619238 by vasike, mglaman: Defines the inline entity form for...

Status: Fixed » Closed (fixed)

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