Currently, the description for the "list_price" field on the price list import says: "If left empty, no list price will be set.".

This is actually not true, since the field itself is required on the form.

Additionally, when exporting the price list items, the "list_price" column is present in the exported CSV file, but it is empty if no list prices were specified.

As a result, when importing price list items using the exported file, price list items are skipped because an exception is thrown by the instantiation of the list price, we should fix that.

CommentFileSizeAuthor
#4 commerce_pricelist-3096018-4.patch2.13 KBjsacksick
#3 commerce_pricelist-3096018-3.patch2.11 KBjsacksick
#2 commerce_pricelist-3096018-2.patch2.18 KBjsacksick
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jsacksick created an issue. See original summary.

jsacksick’s picture

jsacksick CreditAttribution: jsacksick at Centarro commented
Status: Active » Needs review
FileSize
commerce_pricelist-3096018-2.patch2.18 KB
jsacksick’s picture

jsacksick CreditAttribution: jsacksick at Centarro commented
FileSize
commerce_pricelist-3096018-3.patch2.11 KB

This one shouldn't fail (hopefully :)).

jsacksick’s picture

jsacksick CreditAttribution: jsacksick at Centarro commented
FileSize
commerce_pricelist-3096018-4.patch2.13 KB

  • jsacksick committed acda446 on 8.x-2.x 
    Issue #3096018 by jsacksick: Allow importing price list items without...
jsacksick’s picture

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

Committed!

Status: Fixed » Closed (fixed)

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