Describe your bug or feature request.

The price fields created via the field API can only allow zero and above prices. It would be much more flexible to open to negative prices via the field settings to cover more use cases.

CommentFileSizeAuthor
#2 commerce_price-widget_allow_negative-3243298-2.patch2.72 KBAerzas
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Aerzas created an issue. See original summary.

Aerzas’s picture

Aerzas CreditAttribution: Aerzas at Watch Dreamer SA commented
FileSize
commerce_price-widget_allow_negative-3243298-2.patch2.72 KB

Here is the related patch

Aerzas’s picture

Aerzas CreditAttribution: Aerzas at Watch Dreamer SA commented
Status: Active » Needs review
jsacksick’s picture

jsacksick CreditAttribution: jsacksick at Centarro commented
Status: Needs review » Reviewed & tested by the community

Looks good to me, wonder if we should say "Allow negative prices" instead, other than that looks ready to commit.

jsacksick’s picture

jsacksick CreditAttribution: jsacksick at Centarro commented
Status: Reviewed & tested by the community » Fixed

  • jsacksick committed e39cc4d on 8.x-2.x authored by Aerzas 
    Issue #3243298 by Aerzas: Allow negative price in price fields handled...

Status: Fixed » Closed (fixed)

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